Fix tests

tutao · Feb 5, 2025 · 98faf02 · 98faf02
1 parent 6ec2a49
commit 98faf02
Showing 1 changed file with 56 additions and 54 deletions.
diff --git a/packages/tutanota-utils/test/ArrayUtilsTest.ts b/packages/tutanota-utils/test/ArrayUtilsTest.ts
@@ -891,59 +891,61 @@ o.spec("array utils", function () {
 		o(compare(new Uint8Array([1, 1, 2]), new Uint8Array([1, 1, 3]))).equals(-1)
 	})
 
-	o("moveElement move right from left border", function () {
-		let a = [1, 2, 3]
-		moveElement(a, 0, "right")
-		o(arrayEquals(a, [2, 1, 3])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1, 2, 3]
-		moveElement(a, 0, "left")
-		o(arrayEquals(a, [1, 2, 3])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1, 2, 3]
-		moveElement(a, 2, "left")
-		o(arrayEquals(a, [1, 3, 2])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1, 2, 3]
-		moveElement(a, 2, "right")
-		o(arrayEquals(a, [1, 2, 3])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1, 2, 3]
-		moveElement(a, 1, "right")
-		o(arrayEquals(a, [1, 3, 2])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1, 2, 3]
-		moveElement(a, 1, "left")
-		o(arrayEquals(a, [2, 1, 3])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1]
-		moveElement(a, 0, "right")
-		o(arrayEquals(a, [1])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1]
-		moveElement(a, 0, "left")
-		o(arrayEquals(a, [1])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1]
-		moveElement(a, 1, "right")
-		o(arrayEquals(a, [1])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = [1]
-		moveElement(a, 1, "left")
-		o(arrayEquals(a, [1])).equals(true)
-	})
-	o("moveElement move right from left border", function () {
-		a = []
-		moveElement(a, 0, "right")
-		o(arrayEquals(a, [])).equals(true)
+	o.spec("moveElement", () => {
+		o("move right from left border", function () {
+			const a = [1, 2, 3]
+			moveElement(a, 0, "right")
+			o(arrayEquals(a, [2, 1, 3])).equals(true)
+		})
+		o("move left from left border does not modify array", function () {
+			const a = [1, 2, 3]
+			moveElement(a, 0, "left")
+			o(arrayEquals(a, [1, 2, 3])).equals(true)
+		})
+		o("move left from right border", function () {
+			const a = [1, 2, 3]
+			moveElement(a, 2, "left")
+			o(arrayEquals(a, [1, 3, 2])).equals(true)
+		})
+		o("move right from right border does not modify array", function () {
+			const a = [1, 2, 3]
+			moveElement(a, 2, "right")
+			o(arrayEquals(a, [1, 2, 3])).equals(true)
+		})
+		o("move right from non-border index", function () {
+			const a = [1, 2, 3]
+			moveElement(a, 1, "right")
+			o(arrayEquals(a, [1, 3, 2])).equals(true)
+		})
+		o("move left from non-border index", function () {
+			const a = [1, 2, 3]
+			moveElement(a, 1, "left")
+			o(arrayEquals(a, [2, 1, 3])).equals(true)
+		})
+		o("move right from array with one element does not modify array", function () {
+			const a = [1]
+			moveElement(a, 0, "right")
+			o(arrayEquals(a, [1])).equals(true)
+		})
+		o("move left from array with one element does not modify array", function () {
+			const a = [1]
+			moveElement(a, 0, "left")
+			o(arrayEquals(a, [1])).equals(true)
+		})
+		o("move right from invalid index does not modify array", function () {
+			const a = [1]
+			moveElement(a, 1, "right")
+			o(arrayEquals(a, [1])).equals(true)
+		})
+		o("move left from invalid index does not modify array", function () {
+			const a = [1]
+			moveElement(a, 1, "left")
+			o(arrayEquals(a, [1])).equals(true)
+		})
+		o("move right in empty array does not modify array", function () {
+			const a = []
+			moveElement(a, 0, "right")
+			o(arrayEquals(a, [])).equals(true)
+		})
 	})
 })