Make the onComplete callback for the CLI utility optional (See #105)

jestjs · Aug 5, 2014 · 8b8fbfc · 8b8fbfc
1 parent ab3d966
commit 8b8fbfc
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/bin/jest.js b/bin/jest.js
@@ -84,7 +84,7 @@ function _onRunComplete(completionData) {
 	  );
 	results += ', ';
   }
-  results += 
+  results +=
     colors.colorize(
 		[numPassedTests, (numPassedTests > 1 ? 'tests' : 'test'), 'passed'].join(' '),
 		colors.GREEN + colors.BOLD
@@ -134,7 +134,8 @@ function runCLI(argv, packageRoot, onComplete) {
 
   if (argv.version) {
     console.log('v' + _getJestVersion());
-    onComplete(true);
+    onComplete && onComplete(true);
+    return;
   }
 
   var config;
@@ -201,7 +202,7 @@ function runCLI(argv, packageRoot, onComplete) {
         })
         .then(function(completionData) {
           _onRunComplete(completionData);
-          onComplete(completionData.numFailedTests === 0);
+          onComplete && onComplete(completionData.numFailedTests === 0);
         });
     }