add more tests

Author: Michael Hayes

lib/results.js

@@ -148,8 +148,8 @@ function encodeResult (res, count) {
         var ac = inspect(res.actual);
 
         if (Math.max(ex.length, ac.length) > 65 || invalidYaml(ex) || invalidYaml(ac)) {
-            output += inner + 'expected: |\n' + inner + '  ' + ex + '\n';
-            output += inner + 'actual: |\n' + inner + '  ' + ac + '\n';
+            output += inner + 'expected: |-\n' + inner + '  ' + ex + '\n';
+            output += inner + 'actual: |-\n' + inner + '  ' + ac + '\n';
         }
         else {
             output += inner + 'expected: ' + ex + '\n';
@@ -161,7 +161,7 @@ function encodeResult (res, count) {
     }
     if (res.operator === 'error' && res.actual && res.actual.stack) {
         var lines = String(res.actual.stack).split('\n');
-        output += inner + 'stack: |\n';
+        output += inner + 'stack: |-\n';
         for (var i = 0; i < lines.length; i++) {
             output += inner + '  ' + lines[i] + '\n';
         }

package.json

@@ -18,9 +18,10 @@
     "through": "~2.3.4"
   },
   "devDependencies": {
-    "tap": "~0.7.1",
+    "concat-stream": "~1.4.1",
     "falafel": "~1.0.1",
-    "concat-stream": "~1.4.1"
+    "tap": "~0.7.1",
+    "tap-parser": "^1.1.6"
   },
   "scripts": {
     "test": "tap test/*.js"

test/circular-things.js

@@ -1,43 +1,36 @@
 var tape = require('../');
 var tap = require('tap');
+var concat = require('concat-stream');
 
 tap.test('circular test', function (assert) {
     var test = tape.createHarness({ exit : false });
-    var tc = tap.createConsumer();
+    assert.plan(1);
 
-    var rows = [];
-    tc.on('data', function (r) { rows.push(r) });
-    tc.on('end', function () {
-        // console.log("rs", rows)
-
-        // console.log("deepEqual?")
-
-        assert.same(rows, [
-            "TAP version 13"
-            , "circular"
-            , { id: 1
-                , ok: false
-                , name: " should be equal"
-                , operator: "equal"
-                , expected: "{}"
-                , actual: '{ circular: [Circular] }'
-            }
-            , "tests 1"
-            , "pass  0"
-            , "fail  1"
-        ])
-        assert.end()
-    })
-
-    // tt.equal(10, 10)
-    // tt.end()
-
-    test.createStream().pipe(tc);
+    test.createStream().pipe(concat(function (body) {
+        assert.equal(
+            body.toString('utf8'),
+            'TAP version 13\n'
+            + '# circular\n'
+            + 'not ok 1 should be equal\n'
+            + '  ---\n'
+            + '    operator: equal\n'
+            + '    expected: |-\n'
+            + '      {}\n'
+            + '    actual: |-\n'
+            + '      { circular: [Circular] }\n'
+            + '  ...\n'
+            + '\n'
+            + '1..1\n'
+            + '# tests 1\n'
+            + '# pass  0\n'
+            + '# fail  1\n'
+        );
+    }));
 
     test("circular", function (t) {
-        t.plan(1)
-        var circular = {}
-        circular.circular = circular
-        t.equal(circular, {})
+        t.plan(1);
+        var circular = {};
+        circular.circular = circular;
+        t.equal(circular, {});
     })
 })

test/deep-equal-failure.js

@@ -0,0 +1,51 @@
+var tape = require('../');
+var tap = require('tap');
+var concat = require('concat-stream');
+var tapParser = require('tap-parser')
+
+tap.test('deep equal failure', function (assert) {
+    var test = tape.createHarness({ exit : false });
+    var stream = test.createStream();
+    var parser = tapParser()
+    assert.plan(2);
+
+    stream.pipe(parser)
+    stream.pipe(concat(function (body) {
+        assert.equal(
+            body.toString('utf8'),
+            'TAP version 13\n'
+            + '# deep equal\n'
+            + 'not ok 1 should be equal\n'
+            + '  ---\n'
+            + '    operator: equal\n'
+            + '    expected: |-\n'
+            + '      { b: 2 }\n'
+            + '    actual: |-\n'
+            + '      { a: 1 }\n'
+            + '  ...\n'
+            + '\n'
+            + '1..1\n'
+            + '# tests 1\n'
+            + '# pass  0\n'
+            + '# fail  1\n'
+        );
+    }));
+
+    parser.once('assert', function (data) {
+        assert.deepEqual(data, {
+            ok: false,
+            id: 1,
+            name: 'should be equal',
+            diag: {
+              operator: 'equal',
+              expected: '{ b: 2 }',
+              actual: '{ a: 1 }'
+            }
+        })
+    })
+
+    test("deep equal", function (t) {
+        t.plan(1);
+        t.equal({a: 1}, {b: 2});
+    })
+})

test/stackTrace.js

@@ -1,24 +1,44 @@
 var tape = require('../');
 var tap = require('tap');
 var concat = require('concat-stream');
+var tapParser = require('tap-parser');
 
-tap.test('array test', function (tt) {
-    tt.plan(1);
+tap.test('preserves stack trace with newlines', function (tt) {
+    tt.plan(2);
 
     var test = tape.createHarness();
-    test.createStream().pipe(concat(function (body) {
+    var stream = test.createStream();
+    var parser = stream.pipe(tapParser());
+    var stackTrace = 'foo\n  bar'
+
+    parser.once('assert', function (data) {
+        tt.deepEqual(data, {
+            ok: false,
+            id: 1,
+            name: "Error: Preserve stack",
+            diag: {
+                stack: stackTrace,
+                operator: 'error',
+                expected: 'undefined',
+                actual: '[Error: Preserve stack]'
+            }
+        });
+    });
+
+    stream.pipe(concat(function (body) {
+        var body = body.toString('utf8')
         tt.equal(
-            body.toString('utf8'),
+            body,
             'TAP version 13\n'
-            + '# undef\n'
+            + '# multiline stack trace\n'
             + 'not ok 1 Error: Preserve stack\n'
             + '  ---\n'
             + '    operator: error\n'
-            + '    expected: |\n'
+            + '    expected: |-\n'
             + '      undefined\n'
-            + '    actual: |\n'
+            + '    actual: |-\n'
             + '      [Error: Preserve stack]\n'
-            + '    stack: |\n'
+            + '    stack: |-\n'
             + '      foo\n'
             + '        bar\n'
             + '  ...\n'
@@ -30,10 +50,10 @@ tap.test('array test', function (tt) {
         );
     }));
 
-    test('undef', function (t) {
+    test('multiline stack trace', function (t) {
         t.plan(1);
         var err = new Error('Preserve stack')
-        err.stack = 'foo\n  bar'
+        err.stack = stackTrace
         t.error(err);
     });
 });

test/undef.js

@@ -14,9 +14,9 @@ tap.test('array test', function (tt) {
             + 'not ok 1 should be equivalent\n'
             + '  ---\n'
             + '    operator: deepEqual\n'
-            + '    expected: |\n'
+            + '    expected: |-\n'
             + '      { beep: undefined }\n'
-            + '    actual: |\n'
+            + '    actual: |-\n'
             + '      {}\n'
             + '  ...\n'
             + '\n'