fix(tests): CI error reporting improved

Running karma on ci was not exiting with non zero code, giving the impression the specs passed

Closes #2535.

Author: ThomasBurleson

gulp/tasks/karma.js

@@ -6,14 +6,27 @@ exports.task = function (done) {
     configFile: root + '/config/karma.conf.js'
   };
 
+  var errorCount = 0;
+
+  function captureError(next) {
+    return function(exitCode) {
+      if (exitCode != 0) {
+        gutil.log(gutil.colors.red("Karma exited with the following exit code: " + exitCode));
+        errorCount++;
+      }
+      next();
+    };
+  }
+
+
   gutil.log('Running unit tests on unminified source.');
   buildJs(true);
-  karma.start(karmaConfig, testMinified);
+  karma.start(karmaConfig, captureError(testMinified));
 
   function testMinified() {
     gutil.log('Running unit tests on minified source.');
     process.env.KARMA_TEST_COMPRESSED = true;
-    karma.start(karmaConfig, testMinifiedJquery);
+    karma.start(karmaConfig, captureError(testMinifiedJquery));
   }
 
   function testMinifiedJquery() {
@@ -26,6 +39,8 @@ exports.task = function (done) {
   function clearEnv() {
     process.env.KARMA_TEST_COMPRESSED = undefined;
     process.env.KARMA_TEST_JQUERY = undefined;
+
+    if (errorCount > 0) { process.exit(errorCount); }
     done();
   }
 };