House Keeping - Update Dependencies and Unit Test (#105)

* Fix unit tests from mocha update
* Adding babelrc with config that supports Node 4

Author: louisgv

.babelrc

@@ -0,0 +1,17 @@
+{
+  "presets": [
+    [
+      "env",
+      {
+        "targets": {
+          "node": 4
+        },
+        "exclude": [
+          "transform-es2015-classes",
+          "transform-es2015-block-scoping",
+          "transform-es2015-arrow-functions"
+        ]
+      }
+    ]
+  ]
+}

package.json

@@ -7,12 +7,12 @@
     "ros"
   ],
   "scripts": {
-    "test": "mocha test/directory.js",
+    "test": "mocha --exit test/directory.js",
     "gennodejsTest": "mocha test/gennodejsTest.js test/onTheFlyMessages.js",
     "stressTest": "mocha test/stress.js",
     "flatten": "node tools/flatten.js",
     "generate": "node dist/tools/generateMessages.js",
-    "compile": "babel --presets es2015 src/ -d dist/",
+    "compile": "babel src/ -d dist/",
     "prepublish": "npm run compile"
   },
   "author": "chris smith",
@@ -22,19 +22,19 @@
     "url": "git://github.com/RethinkRobotics-opensource/rosnodejs"
   },
   "devDependencies": {
-    "chai": "^3.5.0",
-    "mocha": "^2.4.5",
     "babel-cli": "^6.18.0",
-    "babel-preset-es2015": "^6.18.0"
+    "babel-preset-env": "^1.7.0",
+    "chai": "^4.1.2",
+    "mocha": "^5.2.0"
   },
   "dependencies": {
-    "argparse": "1.0.7",
-    "async": "2.0.1",
+    "argparse": "1.0.10",
+    "async": "2.6.1",
     "bn.js": "^4.11.6",
-    "bunyan": "1.8.1",
-    "md5": "2.1.0",
-    "moment": "2.19.3",
-    "ultron": "1.1.0",
+    "bunyan": "1.8.12",
+    "md5": "2.2.1",
+    "moment": "2.22.2",
+    "ultron": "1.1.1",
     "walker": "1.0.7",
     "xmlrpc": "chfritz/node-xmlrpc"
   }

test/onTheFly.js

@@ -11,7 +11,7 @@ const MASTER_PORT = 11234;
 describe('OnTheFly', function () {
   let master;
 
-  before(function (done) {
+  before(() => {
     this.timeout(0);
 
     master = new Master('localhost', MASTER_PORT);
@@ -22,9 +22,6 @@ describe('OnTheFly', function () {
       onTheFly: true,
       notime: true,
       logging: {skipRosLogging: true}})
-      .then(() => {
-        done();
-      });
   });
 
   after(() => {

test/xmlrpcTest.js

@@ -994,7 +994,7 @@ describe('Protocol Test', () => {
 
   describe('Shutdown', () => {
     let masterStub;
-    before((done) => {
+    before(() => {
       stopMasterStub(() => {
         masterStub = new MasterStub('localhost', MASTER_PORT);
         masterStub.provideAll();
@@ -1003,7 +1003,6 @@ describe('Protocol Test', () => {
       return rosnodejs.shutdown()
       .then(() => {
         rosnodejs.reset();
-        done();
       });
     });
 
@@ -1077,16 +1076,17 @@ describe('Protocol Test', () => {
 
       rosnodejs.initNode(nodeName, initArgs)
       .then((nh) => {
+        
         const pub = nh.advertise('/chatter', 'std_msgs/String');
         const sub = nh.subscribe('/chatter', 'std_msgs/String');
         pub.publish({data: 'hi'});
+      
 
         rosnodejs.shutdown()
         .then(() => {
           rosnodejs.reset();
           expect(gotEvent).to.be.true;
-          expect(nh._node._spinner._spinTimer.clientCallQueue).to.be.empty;
-          expect
+          expect(nh._node._spinner._spinTimer.clientCallQueue).to.be.undefined;
           done();
         });
       });
@@ -1100,7 +1100,7 @@ describe('Protocol Test', () => {
   describe('Parameters', function() {
     let masterStub;
 
-    before((done) => {
+    before(() => {
       masterStub = new MasterStub('localhost', MASTER_PORT);
       masterStub.provideAll();
 
@@ -1109,15 +1109,13 @@ describe('Protocol Test', () => {
         rosnodejs.reset();
         return rosnodejs.initNode(nodeName, initArgs);
       })
-      .then(() => done());
     });
 
-    after((done) => {
+    after(() => {
       masterStub.shutdown();
       return rosnodejs.shutdown()
       .then(() => {
         rosnodejs.reset();
-        done();
       });
     });