Gruntfile: Add "phantom" task; avoid displaying Saucelabs or adding t…

…estJobs when not needed (e.g., for "dev"); document
kounta · Mar 26, 2016 · ed4071a · ed4071a
1 parent 429a282
commit ed4071a
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 9 deletions.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -136,20 +136,23 @@ module.exports = function (grunt) {
     });
 
     var devJobs = ['eslint', 'babel', 'browserify', 'uglify', 'clean', 'jade'];
-    var testJobs = devJobs.concat('connect');
-    if (saucekey && !process.env.TRAVIS_PULL_REQUEST) {
-        console.info('adding Saucelabs integration');
-        testJobs.push('saucelabs-mocha');
-    }
-
+    var karmaJobs = devJobs.slice();
     if (process.env.TRAVIS_JOB_ID) {
-        testJobs.push('karma:ci');
+        karmaJobs.push('karma:ci');
     } else {
-        testJobs.push('karma:dev-single');
+        karmaJobs.push('karma:dev-single');
     }
 
     grunt.registerTask('dev', devJobs);
-    grunt.registerTask('test', testJobs);
+    grunt.registerTask('phantom', karmaJobs);
+    grunt.registerTask('test', function () {
+        var testJobs = karmaJobs.concat('connect');
+        if (saucekey && !process.env.TRAVIS_PULL_REQUEST) {
+            console.info('adding Saucelabs integration');
+            testJobs.push('saucelabs-mocha');
+        }
+        grunt.task.run(testJobs);
+    });
     grunt.registerTask('default', 'test');
     grunt.registerTask('test:local', function () {
         grunt.task.run(devJobs);

diff --git a/README.md b/README.md
@@ -572,9 +572,24 @@ library.
 # Contributor notes
 
 - `npm install` to install all the dependencies
+
+In browser:
+
 - `npm run grunt test:local` to run the mocha server
 - Open (`http://localhost:9999/tests`)[] to run the mocha tests
 
+In Node.js:
+
+- `npm test`
+
+or to avoid Saucelabs if set up:
+
+- `npm run grunt phantom`
+
+or to also avoid PhantomJS:
+
+- `npm run grunt dev`
+
 # License
 
 The MIT License