Notebook updates

Makefile

@@ -1,8 +1,8 @@
 test:
-	@node ./bin/_lab -f -m 800
+	@node ./bin/_lab -f -m 1000
 test-cov:
-	@node ./bin/_lab -f -t 100 -m 800
+	@node ./bin/_lab -f -t 100 -m 1000
 test-cov-html:
-	@node ./bin/_lab -f -r html -m 800 -o coverage.html
+	@node ./bin/_lab -f -r html -m 1000 -o coverage.html
 
 .PHONY: test test-cov test-cov-html

lib/reporters/console.js

@@ -102,15 +102,44 @@ internals.Reporter.prototype.end = function (notebook) {
         return !!test.err;
     });
 
+    var errors = notebook.errors || [];
+
     var output = '';
+    var message;
+    var stack;
+    var index;
+
+
+    if (errors.length) {
+        output += 'Session errors:\n\n';
+        for (var i = 0, il = errors.length; i < il; ++i) {
+            var error = errors[i];
+            message = error.message || error;
+            stack = error.stack || message;
+            index = stack.indexOf(message) + message.length;
+
+            stack = stack.slice(index + 1).replace(/^/gm, '  ');
+
+            // remove node_modules folders and only show the first 5 lines of the stack
+            stack = stack.split('\n').filter(function (line) {
+
+                return !(/\/node_modules\//.test(line));
+            }).slice(0,5).join('\n');
+
+            output += red(message) + '\n';
+            output += gray(stack) + '\n\n';
+        }
+        output += red('There were ' + errors.length + ' session error(s).') + '\n\n'
+    }
+
     if (failures.length) {
         output += 'Failed tests:\n\n';
 
         for (var i = 0, il = failures.length; i < il; ++i) {
             var test = failures[i];
-            var message = test.err.message || '';
-            var stack = test.err.stack || message;
-            var index = stack.indexOf(message) + message.length;
+            message = test.err.message || '';
+            stack = test.err.stack || message;
+            index = stack.indexOf(message) + message.length;
 
             // Actual vs Expected
 
@@ -154,7 +183,7 @@ internals.Reporter.prototype.end = function (notebook) {
 
             if (!test.timeout) {
                 var isChai = stack.indexOf('chai') !== -1;
-                stack = stack.slice(index ? index + 1 : index).replace(/^/gm, '  ');
+                stack = stack.slice(index + 1).replace(/^/gm, '  ');
                 if (isChai) {
                     stack = stack.split('\n').filter(function (line) { return !(/\/chai\//.test(line)); })[0];
                 }

lib/runner.js

@@ -406,4 +406,4 @@ internals.failTest = function (test, state, skip, err) {
     test.duration = 0;
     state.report.tests.push(test);
     state.reporter.test(test);
-};
+};

test/reporters.js

@@ -478,6 +478,61 @@ describe('Reporter', function () {
                 done();
             });
         });
+
+        it('displays session errors if there in an error in "before"', function (done) {
+
+            var script = Lab.script();
+            script.experiment('test', function () {
+
+                script.before(function (done) {
+
+                    done(new Error('there was an error in the before function'));
+                });
+
+                script.test('works', function (finished) {
+                    Lab.expect(true).to.equal(true, 'Working right');
+                    finished();
+                });
+            });
+
+            Lab.report(script, { reporter: 'console', colors: false }, function (err, code, output) {
+
+                var result = output.replace(/\/[\/\w]+\.js\:\d+\:\d+/g, '<trace>');
+
+                expect(code).to.equal(1);
+                expect(result).to.contain('There were 1 session error(s).');
+                expect(result).to.contain('there was an error in the before function');
+                done();
+            });
+        });
+
+        it('displays session errors if there in an error in "afterEach"', function (done) {
+
+            var script = Lab.script();
+            script.experiment('test', function () {
+
+                script.afterEach(function (done) {
+
+                    done('there was an error in the afterEach function');
+                });
+
+                script.test('works', function (finished) {
+                    Lab.expect(true).to.equal(true, 'Working right');
+                    finished();
+                });
+            });
+
+            Lab.report(script, { reporter: 'console', colors: false }, function (err, code, output) {
+
+                var result = output.replace(/\/[\/\w]+\.js\:\d+\:\d+/g, '<trace>');
+
+                expect(code).to.equal(1);
+                expect(result).to.contain('There were 1 session error(s).');
+                expect(result).to.contain('there was an error in the afterEach function');
+                done();
+            });
+        });
+
     });
 
     describe('json', function () {