ICE-6737 Improved gulp message if python not in PATH

lidongdongsx · Sep 18, 2015 · 1147986 · 1147986
1 parent fd91507
commit 1147986
Showing 1 changed file with 25 additions and 1 deletion.
diff --git a/js/gulpfile.js b/js/gulpfile.js
@@ -218,7 +218,7 @@ tests.forEach(
                 return gulp.src(path.join(name, ".depend"))
                     .pipe(paths(del));
             });
-        
+
         gulp.task(testCleanTask(name), [testCleanDependTask(name)],
             function(){
                 return gulp.src(path.join(name, "*.ice"))
@@ -385,6 +385,18 @@ gulp.task("test:run-with-browser", ["watch"].concat(useBinDist ? ["test"] : ["bu
         require("./bin/HttpServer")();
 
         var p  = require("child_process").spawn("python", ["../scripts/TestController.py"], {stdio: "inherit"});
+        p.on("error", function(err)
+            {
+                if(err.message == "spawn python ENOENT")
+                {
+                    console.log("Error: python is required in PATH to run tests")
+                    process.exit(1)
+                }
+                else
+                {
+                    throw err;
+                }
+            });
         process.on(process.platform == "win32" ? "SIGBREAK" : "SIGINT",
             function()
             {
@@ -401,6 +413,18 @@ gulp.task("test:run-with-browser", ["watch"].concat(useBinDist ? ["test"] : ["bu
 gulp.task("test:run-with-node", (useBinDist ? ["test"] : ["build"]),
     function(){
         var p  = require("child_process").spawn("python", ["allTests.py", "--all"], {stdio: "inherit"});
+        p.on("error", function(err)
+            {
+                if(err.message == "spawn python ENOENT")
+                {
+                    console.log("Error: python is required in PATH to run tests")
+                    process.exit(1)
+                }
+                else
+                {
+                    throw err;
+                }
+            });
         process.on(process.platform == "win32" ? "SIGBREAK" : "SIGINT",
             function()
             {