Print tested data on assertion failure (#21825)

jbajic · web-flow · commit 09220c42cf29 · 2025-06-30T10:49:21.000+02:00
diff --git a/tests/js/client/aql/vector/aql-vector.js b/tests/js/client/aql/vector/aql-vector.js
@@ -275,13 +275,13 @@ function VectorIndexL2TestSuite() {
                 // Assert that results are deterministic
                 if (i !== 0) {
                     for (let j = 0; j < previousResult.length; ++j) {
-                        assertEqual(previousResult[j].key, results[j].key);
+                        assertEqual(previousResult[j].key, results[j].key, "Results are not deterministic: " + JSON.stringify(results));
                     }
                 }
 
                 // For l2 metric the results must be ordered in descending order
                 for (let j = 1; j < results.length; ++j) {
-                    assertTrue(results[j - 1].dist <= results[j].dist);
+                    assertTrue(results[j - 1].dist <= results[j].dist, "Results are not in ascending order: " + JSON.stringify(results));
                 }
             }
         },
@@ -420,7 +420,7 @@ function VectorIndexL2TestSuite() {
             const skipKeys = new Set(resultsWithSkip.map(r => r.k));
             const withoutSkipKeys = new Set(resultsWithoutSkip.map(r => r.k));
             
-            assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)));
+            assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)), "Skip results are not contained within without skip results: " + JSON.stringify(resultsWithSkip) + " " + JSON.stringify(resultsWithoutSkip));
         },
 
         testApproxL2Subquery: function() {
@@ -631,11 +631,11 @@ function VectorIndexCosineTestSuite() {
 
                 // For cosine similarity the results must be ordered in descending order
                 for (let j = 1; j < results.length; ++j) {
-                    assertTrue(results[j - 1].sim >= results[j].sim);
+                    assertTrue(results[j - 1].sim >= results[j].sim, "Results are not in descending order: " + JSON.stringify(results));
                 }
                 // Assert that distances are in [-1, 1] range
                 for (let j = 0; j < results.length; ++j) {
-                  assertTrue(Math.abs(results[j].sim) <= 1.01);
+                  assertTrue(Math.abs(results[j].sim) <= 1.01, "Cosine similarity is not in [-1, 1] range: " + JSON.stringify(results));
                 }
             }
         },
@@ -787,7 +787,7 @@ function VectorIndexInnerProductTestSuite() {
 
                 // For inner product metric the results must be ordered in descending order
                 for (let j = 1; j < results.length; ++j) {
-                    assertTrue(results[j - 1].sim >= results[j].sim);
+                    assertTrue(results[j - 1].sim >= results[j].sim, "Results are not in descending order: " + JSON.stringify(results));
                 }
             }
         },
@@ -835,7 +835,7 @@ function VectorIndexInnerProductTestSuite() {
             const skipKeys = new Set(resultsWithSkip.map(r => r.k));
             const withoutSkipKeys = new Set(resultsWithoutSkip.map(r => r.k));
             
-            assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)));
+            assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)), "Skip results are not contained within without skip results: " + JSON.stringify(resultsWithSkip) + " " + JSON.stringify(resultsWithoutSkip));
         },
     };
 }

Original file line number	Diff line number	Diff line change
`@@ -275,13 +275,13 @@ function VectorIndexL2TestSuite() {`
`275`	`275`	`// Assert that results are deterministic`
`276`	`276`	`if (i !== 0) {`
`277`	`277`	`for (let j = 0; j < previousResult.length; ++j) {`
`278`		`- assertEqual(previousResult[j].key, results[j].key);`
	`278`	`+ assertEqual(previousResult[j].key, results[j].key, "Results are not deterministic: " + JSON.stringify(results));`
`279`	`279`	`}`
`280`	`280`	`}`
`281`	`281`
`282`	`282`	`// For l2 metric the results must be ordered in descending order`
`283`	`283`	`for (let j = 1; j < results.length; ++j) {`
`284`		`- assertTrue(results[j - 1].dist <= results[j].dist);`
	`284`	`+ assertTrue(results[j - 1].dist <= results[j].dist, "Results are not in ascending order: " + JSON.stringify(results));`
`285`	`285`	`}`
`286`	`286`	`}`
`287`	`287`	`},`
`@@ -420,7 +420,7 @@ function VectorIndexL2TestSuite() {`
`420`	`420`	`const skipKeys = new Set(resultsWithSkip.map(r => r.k));`
`421`	`421`	`const withoutSkipKeys = new Set(resultsWithoutSkip.map(r => r.k));`
`422`	`422`
`423`		`- assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)));`
	`423`	`+ assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)), "Skip results are not contained within without skip results: " + JSON.stringify(resultsWithSkip) + " " + JSON.stringify(resultsWithoutSkip));`
`424`	`424`	`},`
`425`	`425`
`426`	`426`	`testApproxL2Subquery: function() {`
`@@ -631,11 +631,11 @@ function VectorIndexCosineTestSuite() {`
`631`	`631`
`632`	`632`	`// For cosine similarity the results must be ordered in descending order`
`633`	`633`	`for (let j = 1; j < results.length; ++j) {`
`634`		`- assertTrue(results[j - 1].sim >= results[j].sim);`
	`634`	`+ assertTrue(results[j - 1].sim >= results[j].sim, "Results are not in descending order: " + JSON.stringify(results));`
`635`	`635`	`}`
`636`	`636`	`// Assert that distances are in [-1, 1] range`
`637`	`637`	`for (let j = 0; j < results.length; ++j) {`
`638`		`- assertTrue(Math.abs(results[j].sim) <= 1.01);`
	`638`	`+ assertTrue(Math.abs(results[j].sim) <= 1.01, "Cosine similarity is not in [-1, 1] range: " + JSON.stringify(results));`
`639`	`639`	`}`
`640`	`640`	`}`
`641`	`641`	`},`
`@@ -787,7 +787,7 @@ function VectorIndexInnerProductTestSuite() {`
`787`	`787`
`788`	`788`	`// For inner product metric the results must be ordered in descending order`
`789`	`789`	`for (let j = 1; j < results.length; ++j) {`
`790`		`- assertTrue(results[j - 1].sim >= results[j].sim);`
	`790`	`+ assertTrue(results[j - 1].sim >= results[j].sim, "Results are not in descending order: " + JSON.stringify(results));`
`791`	`791`	`}`
`792`	`792`	`}`
`793`	`793`	`},`
`@@ -835,7 +835,7 @@ function VectorIndexInnerProductTestSuite() {`
`835`	`835`	`const skipKeys = new Set(resultsWithSkip.map(r => r.k));`
`836`	`836`	`const withoutSkipKeys = new Set(resultsWithoutSkip.map(r => r.k));`
`837`	`837`
`838`		`- assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)));`
	`838`	`+ assertTrue([...skipKeys].every(key => withoutSkipKeys.has(key)), "Skip results are not contained within without skip results: " + JSON.stringify(resultsWithSkip) + " " + JSON.stringify(resultsWithoutSkip));`
`839`	`839`	`},`
`840`	`840`	`};`
`841`	`841`	`}`