Merge pull request #2 from paulcpederson/watch-errors

Watch errors

Author: JamesMessinger

.jscsrc

@@ -1,7 +1,8 @@
 {
   "excludeFiles": [
     "node_modules/**",
-    "test-app/dist/**"
+    "test-app/dist/**",
+    "test-app/error"
   ],
   "requireCurlyBraces": [
     "if",

.jshintignore

@@ -1,2 +1,3 @@
 /node_modules
 /test-app/dist
+/test-app/error

bin/simplifyify.js

@@ -66,16 +66,18 @@ program
       })
       .on('error', function(err, fileSet) {
         // Log an error
-        var message = process.env.DEBUG ? err.stack : err.message;
-        if (fileSet && fileSet.entry) {
-          console.error('Error bundling %s\n%s', fileSet.entry, message);
+        if (fileSet && fileSet.entryFile) {
+          console.error('Error bundling %s\n%s', fileSet.entryFile, err);
         }
         else {
+          var message = process.env.DEBUG ? err.stack : err.message;
           console.error(message);
         }
 
         // Exit the app with an error code
-        process.exit(1);
+        if (!program.watch) {
+          process.exit(1);
+        }
       });
   });
 

lib/write-bundles.js

@@ -158,6 +158,7 @@ function addCoverageTransform(b) {
 function bundle(b, fileSet) {
   var stream = b.bundle();
   stream.on('end', b.emit.bind(b, 'end'));
+  stream.on('error', b.emit.bind(b, 'error'));
 
   if (fileSet.mapFile) {
     util.ensureFileExists(fileSet.mapFile);

test-app/error/error.js

@@ -0,0 +1 @@
+console.log('this is an error'))

tests/watch.spec.js

@@ -216,4 +216,26 @@ describe('simplifyify --watch', function() {
       done();
     }
   });
+
+  it('should report errors', function(done) {
+    this.timeout(10000);
+
+    // Run Watchify
+    var watchify = helper.run('test-app/error/error.js --watch --outfile test-app/dist/error.js', onExit);
+
+    // Check the initial outputs after a few seconds
+    setTimeout(firstCheck, 3000);
+
+    function firstCheck() {
+      watchify.kill();
+    }
+
+    // Verify the final results
+    function onExit(err, stdout, stderr) {
+      expect(stderr).to.be.ok;
+      done();
+    }
+
+  });
+
 });