test: add tls spec tests skip unsupported

Adds spec tests from commit 85343c63dbde030a7e31df0ca63278cdf126b91e of "mongodb/specifications" and skips tests that have "tlsDisableOCSPEndpointCheck" or "tlsDisableCertificateRevocationCheck" in the connection string as there's are not supported by the Node.js driver.

NODE-2528

Author: Thomas Reggi

test/functional/uri_options_spec.test.js

@@ -1,5 +1,7 @@
 'use strict';
 
+const qs = require('qs');
+const url = require('url');
 const chai = require('chai');
 const expect = chai.expect;
 chai.use(require('chai-subset'));
@@ -18,6 +20,10 @@ describe('URI Options (spec)', function() {
         itFn(test.description, {
           metadata: { requires: { topology: 'single' } },
           test: function(done) {
+            const query = qs.parse(url.parse(test.uri).query);
+            if (query.hasOwnProperty('tlsDisableOCSPEndpointCheck')) return this.skip();
+            if (query.hasOwnProperty('tlsDisableCertificateRevocationCheck')) return this.skip();
+
             parse(test.uri, {}, (err, result) => {
               if (test.valid === true) {
                 expect(err).to.not.exist;

test/spec/uri-options/README.rst

@@ -21,10 +21,7 @@ array of test case objects, each of which have the following keys:
 
 - ``description``: A string describing the test.
 - ``uri``: A string containing the URI to be parsed.
-- ``valid``: A boolean indicating if the URI should be considered valid. 
-  This will always be true, as the Connection String spec tests the validity of the structure, but 
-  it's still included to make it easier to reuse the connection string spec test runners that 
-  drivers already have.
+- ``valid``: A boolean indicating if the URI should be considered valid.
 - ``warning``: A boolean indicating whether URI parsing should emit a warning.
 - ``hosts``: Included for compatibility with the Connection String spec tests. This will always be ``~``.
 - ``auth``: Included for compatibility with the Connection String spec tests. This will always be ``~``.

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

@@ -1,7 +1,7 @@
 {
   "tests": [
     {
-      "description": "Valid auth options are parsed correctly",
+      "description": "Valid auth options are parsed correctly (GSSAPI)",
       "uri": "mongodb://foo:[email protected]/?authMechanism=GSSAPI&authMechanismProperties=SERVICE_NAME:other,CANONICALIZE_HOST_NAME:true&authSource=$external",
       "valid": true,
       "warning": false,
@@ -15,6 +15,18 @@
         },
         "authSource": "$external"
       }
+    },
+    {
+      "description": "Valid auth options are parsed correctly (SCRAM-SHA-1)",
+      "uri": "mongodb://foo:[email protected]/?authMechanism=SCRAM-SHA-1&authSource=authSourceDB",
+      "valid": true,
+      "warning": false,
+      "hosts": null,
+      "auth": null,
+      "options": {
+        "authMechanism": "SCRAM-SHA-1",
+        "authSource": "authSourceDB"
+      }
     }
   ]
 }

test/spec/uri-options/auth-options.yml

@@ -1,6 +1,6 @@
 tests:
     -
-        description: "Valid auth options are parsed correctly"
+        description: "Valid auth options are parsed correctly (GSSAPI)"
         uri: "mongodb://foo:[email protected]/?authMechanism=GSSAPI&authMechanismProperties=SERVICE_NAME:other,CANONICALIZE_HOST_NAME:true&authSource=$external"
         valid: true
         warning: false
@@ -12,3 +12,13 @@ tests:
                 SERVICE_NAME: "other"
                 CANONICALIZE_HOST_NAME: true
             authSource: "$external"
+    -
+        description: "Valid auth options are parsed correctly (SCRAM-SHA-1)"
+        uri: "mongodb://foo:[email protected]/?authMechanism=SCRAM-SHA-1&authSource=authSourceDB"
+        valid: true
+        warning: false
+        hosts: ~
+        auth: ~
+        options:
+            authMechanism: "SCRAM-SHA-1"
+            authSource: "authSourceDB"