Merge pull request #518 from strongloop/update-travis-platforms

Add Node.js 12 to Travis CI + fix failing tests

Author: bajtos

.travis.yml

@@ -1,8 +1,8 @@
 language: node_js
 node_js:
-  - "6"
-  - "8"
+  - "12"
   - "10"
+  - "8"
 env:
   - CXX=g++-4.8
 addons:

package.json

@@ -31,7 +31,7 @@
     "bson": "^1.0.6",
     "debug": "^3.1.0",
     "loopback-connector": "^4.5.0",
-    "mongodb": "^3.1.4",
+    "mongodb": "^3.2.4",
     "strong-globalize": "^4.1.1"
   },
   "devDependencies": {

test/mongodb.test.js

@@ -57,7 +57,9 @@ describe('lazyConnect', function() {
     });
 
     ds.on('error', function(err) {
-      err.code.should.eql('ECONNREFUSED');
+      should.exist(err);
+      err.name.should.equal('MongoNetworkError');
+      err.message.should.match(/ECONNREFUSED/);
       done();
     });
   });
@@ -361,8 +363,9 @@ describe('mongodb connector', function() {
         port: 4, // unassigned by IANA
       });
       ds.ping(function(err) {
-        (!!err).should.be.True();
-        err.code.should.eql('ECONNREFUSED');
+        should.exist(err);
+        err.name.should.equal('MongoNetworkError');
+        err.message.should.match(/ECONNREFUSED/);
         done();
       });
     });