fix: correctly use template string for connection string error message

mongodb · Mar 10, 2020 · b4d89ea · b4d89ea
1 parent eed1131
commit b4d89ea
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/connection_string.js b/lib/connection_string.js
@@ -523,7 +523,7 @@ function assertTlsOptionsAreEqual(optionName, queryString, queryStringKeys) {
       const firstValue = queryString[optionName][0];
       queryString[optionName].forEach(tlsValue => {
         if (tlsValue !== firstValue) {
-          throw new MongoParseError('All values of ${optionName} must be the same.');
+          throw new MongoParseError(`All values of ${optionName} must be the same.`);
         }
       });
     }

diff --git a/test/unit/core/connection_string.test.js b/test/unit/core/connection_string.test.js
@@ -203,6 +203,13 @@ describe('Connection String', function() {
         done();
       });
     });
+
+    it('should validate non-equal tls values', function(done) {
+      parseConnectionString('mongodb://localhost/?tls=true&tls=false', err => {
+        expect(err).to.have.property('message', 'All values of tls must be the same.');
+        done();
+      });
+    });
   });
 
   describe('spec tests', function() {