test(NODE-3738): fix uri spec runner (#3117)

Author: dariakp

test/integration/uri-options/uri_options.spec.test.js

@@ -3,6 +3,12 @@ const path = require('path');
 const fs = require('fs');
 const { EJSON } = require('bson');
 
+function hasDuplicates(testArray) {
+  const testNames = testArray.map(test => test.description);
+  const testNameSet = new Set(testNames);
+  return testNameSet.size !== testNames.length;
+}
+
 /**
  * Given spec test folder names, loads the corresponding JSON
  *
@@ -12,13 +18,23 @@ const { EJSON } = require('bson');
 function loadSpecTests(...args) {
   const specPath = path.resolve(...[__dirname].concat(args));
 
-  return fs
+  const suites = fs
     .readdirSync(specPath)
     .filter(x => x.includes('.json'))
     .map(x => ({
       ...EJSON.parse(fs.readFileSync(path.join(specPath, x)), { relaxed: true }),
       name: path.basename(x, '.json')
     }));
+
+  for (const suite of suites) {
+    if (suite.tests && hasDuplicates(suite.tests)) {
+      throw new Error(
+        `Failed to load suite ${suite.name} because it contains duplicate test cases`
+      );
+    }
+  }
+
+  return suites;
 }
 
 module.exports = {

test/spec/index.js

@@ -44,15 +44,6 @@
         "tlsAllowInvalidCertificates": true
       }
     },
-    {
-      "description": "Invalid tlsAllowInvalidCertificates causes a warning",
-      "uri": "mongodb://example.com/?tlsAllowInvalidCertificates=invalid",
-      "valid": true,
-      "warning": true,
-      "hosts": null,
-      "auth": null,
-      "options": {}
-    },
     {
       "description": "tlsAllowInvalidHostnames is parsed correctly",
       "uri": "mongodb://example.com/?tlsAllowInvalidHostnames=true",

test/spec/uri-options/tls-options.json

@@ -36,14 +36,6 @@ tests:
         auth: ~
         options:
             tlsAllowInvalidCertificates: true
-    -
-        description: "Invalid tlsAllowInvalidCertificates causes a warning"
-        uri: "mongodb://example.com/?tlsAllowInvalidCertificates=invalid"
-        valid: true
-        warning: true
-        hosts: ~
-        auth: ~
-        options: {}
     -
         description: "tlsAllowInvalidHostnames is parsed correctly"
         uri: "mongodb://example.com/?tlsAllowInvalidHostnames=true"