Merge pull request #168 from grit96/issue-92

Comments should not make exit code incorrect. Fixes #92

Author: Raynos

index.js

@@ -110,7 +110,7 @@ function createHarness (conf_) {
                 inspectCode(st_);
             });
             st.on('result', function (r) {
-                if (!r.ok) test._exitCode = 1
+                if (!r.ok && typeof r !== 'string') test._exitCode = 1
             });
         })(t);
 

test/exit.js

@@ -18,6 +18,7 @@ tap.test('exit ok', function (t) {
         t.same(rs, [
             'TAP version 13',
             'array',
+            'hi',
             { id: 1, ok: true, name: 'should be equivalent' },
             { id: 2, ok: true, name: 'should be equivalent' },
             { id: 3, ok: true, name: 'should be equivalent' },

test/exit/ok.js

@@ -2,6 +2,7 @@ var falafel = require('falafel');
 var test = require('../../');
 
 test('array', function (t) {
+    t.comment('hi');
     t.plan(5);
 
     var src = '(' + function () {