Merge pull request #922 from lirantal/feature/travis-add-node-v4-gcc-update

NodeJS v4 support - fixing the failed build

.travis.yml

@@ -4,13 +4,23 @@ node_js:
   - 0.10
   - 0.12
   - 4
+# NodeJS v4 requires gcc 4.8
 env:
-  - NODE_ENV=travis
+  - NODE_ENV=travis CXX="g++-4.8" CC="gcc-4.8"
 matrix:
   allow_failures:
     - node_js: 4
 services:
   - mongodb
+# gcc 4.8 requires ubuntu-toolchain-r-test
+addons:
+  apt:
+    sources:
+    - ubuntu-toolchain-r-test
+    packages:
+    - g++-4.8
+    - gcc-4.8
+    - clang
 before_install:
   - gem update --system
   - gem install sass --version "=3.3.7"

modules/users/tests/server/user.server.routes.tests.js

@@ -97,7 +97,15 @@ describe('User CRUD tests', function () {
             }
 
             signoutRes.redirect.should.equal(true);
-            signoutRes.text.should.equal('Moved Temporarily. Redirecting to /');
+
+            // NodeJS v4 changed the status code representation so we must check
+            // before asserting, to be comptabile with all node versions.
+            if (process.version.indexOf('v4') === 0) {
+              signoutRes.text.should.equal('Found. Redirecting to /');
+            } else {
+              signoutRes.text.should.equal('Moved Temporarily. Redirecting to /');
+            }
+
             return done();
           });
       });

package.json

@@ -60,7 +60,7 @@
     "passport-twitter": "^1.0.2",
     "phantomjs": ">=1.9.0",
     "serve-favicon": "^2.3.0",
-    "socket.io": "^1.3.5",
+    "socket.io": "^1.3.7",
     "swig": "^1.4.2",
     "validator": "^3.41.2"
   },
@@ -80,7 +80,7 @@
     "grunt-mocha-istanbul": "^2.4.0",
     "grunt-mocha-test": "~0.12.7",
     "grunt-ng-annotate": "^1.0.1",
-    "grunt-node-inspector": "~0.2.0",
+    "grunt-node-inspector": "~0.3.0",
     "grunt-nodemon": "~0.4.0",
     "grunt-protractor-runner": "^2.1.0",
     "gulp": "^3.9.0",