PRs now pass travis build, because they don't execute sauce tests

armendarabyan · Dec 31, 2013 · 32c1c34 · 32c1c34
1 parent f6c3d03
commit 32c1c34
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -204,7 +204,8 @@ module.exports = function (grunt) {
 
     // travis build task
     grunt.registerTask('build:travis', [
-        'jshint', 'test:node', 'test:travis-sauce-browser'
+        'jshint', 'test:node', 'check-sauce-creds',
+        'test:travis-sauce-browser'
     ]);
 
     // Task to be run when releasing a new version

diff --git a/tasks/check_sauce_creds.js b/tasks/check_sauce_creds.js
@@ -0,0 +1,14 @@
+module.exports = function (grunt) {
+
+    // Pull Requests do not have secure variables enabled for security reasons.
+    // Use this task before launching travis-sauce-browser task, so it would
+    // exit early and won't try connecting to saucelabs without credentials.
+    grunt.registerTask('check-sauce-creds', function () {
+        if (process.env.SAUCE_USERNAME === undefined) {
+            grunt.log.writeln("No sauce credentials found");
+            grunt.task.clearQueue();
+        } else {
+            grunt.log.writeln("Sauce credentials found");
+        }
+    });
+};