Check for duplicate test names

Author: Marcono1234

build.js

@@ -92,6 +92,8 @@ function buildTestSuite(testsFolder, tags, all, exclude) {
         : list => tags.some(t => list.includes(t))
     const tests = readTestsFromDir(testsFolder)
         .filter(test => tags.length === 0 || (tagFilter(getTags(test)) !== exclude));
+    checkDuplicateNames(tests);
+
     tests.forEach(test => {
         if ('tags' in test) test.tags.sort();
     });
@@ -194,3 +196,28 @@ function prependToTestName(prefix) {
     };
 }
 
+/**
+ * Verifies that no tests have duplicate names.
+ * @param allTests - The list of all tests.
+ */
+function checkDuplicateNames(allTests) {
+    const names = new Set();
+    const duplicates = [];
+
+    for (const test of allTests) {
+        const oldSize = names.size;
+        const name = test.name;
+        names.add(name);
+
+        if (oldSize === names.size) {
+            duplicates.push(name);
+        }
+    }
+
+    duplicates.sort();
+    if (duplicates.length > 0) {
+        const prefix = "\n  ";
+        console.error("Error: Duplicate test names:" + prefix + duplicates.join(prefix));
+        process.exit(1);
+    }
+}

tests/whitespace/selectors.json

@@ -36,21 +36,21 @@
       "tags": ["whitespace"]
     },
     {
-      "name": "newline between root and bracket",
+      "name": "newline between bracket and bracket",
       "selector" : "$['a'] \n['b']",
       "document" : {"a": {"b": "ab"} },
       "result": [ "ab" ],
       "tags": ["whitespace"]
     },
     {
-      "name": "tab between root and bracket",
+      "name": "tab between bracket and bracket",
       "selector" : "$['a'] \t['b']",
       "document" : {"a": {"b": "ab"} },
       "result": [ "ab" ],
       "tags": ["whitespace"]
     },
     {
-      "name": "return between root and bracket",
+      "name": "return between bracket and bracket",
       "selector" : "$['a'] \r['b']",
       "document" : {"a": {"b": "ab"} },
       "result": [ "ab" ],