added notification alert

Author: alessioalex

index.js

@@ -9,6 +9,7 @@ var PrettyError = require('pretty-error');
 var ejs = require('ejs');
 var sep = require('path').sep;
 var hljs = require('highlight.js');
+var notifier = require('node-notifier');
 
 if (process.env.NODE_ENV === 'production') {
   throw new Error('dev-error-handler shouldn\'t be used in production');
@@ -74,8 +75,18 @@ module.exports = function(err, req, res, next) {
     // remove empty data from the array (coming from the excluded lines)
     lines = lines.filter(function(line) { return !!line; });
 
+    // pretty print in the terminal
     console.error((new PrettyError).render(err) || err.stack);
 
+    // bypass for e2e testing
+    if (!process.env.NODE_ENV || process.env.NODE_ENV === 'development') {
+      // send OS notification
+      notifier.notify({
+        title: 'Error in ' + req.method + ' ' + req.url,
+        message: err.message
+      });
+    }
+
     res.writeHead(500, { 'Content-Type': 'text/html' });
 
     res.end(render({

package.json

@@ -7,6 +7,7 @@
     "ejs": "^2.3.4",
     "errto": "^0.2.1",
     "highlight.js": "https://github.com/alessioalex/highlight.js/releases/download/npm-v0.1.1/highlight.js.tar.gz.tgz",
+    "node-notifier": "^4.4.0",
     "pretty-error": "^1.2.0",
     "stack-trace": "^0.0.9",
     "tiny-map-async": "^1.0.0"
@@ -27,7 +28,7 @@
     "errorhandler"
   ],
   "scripts": {
-    "test": "node test/test",
+    "test": "NODE_ENV=test node test/test",
     "example": "node examples/http.js",
     "lint": "alessioalex-standard",
     "precommit": "npm run lint && npm test"