Make various random tests pass when TAP_BUFFER=1

tapjs · Jan 8, 2017 · 750b63d · 750b63d
1 parent a3bb777
commit 750b63d
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 17 deletions.
diff --git a/test/segv.js b/test/segv.js
@@ -52,7 +52,7 @@ test('setup', function (t) {
 })
 
 test('segv', function (t) {
-  var tt = new Test({ bail: false })
+  var tt = new Test({ bail: false, buffered: false })
   tt.spawn('./segv')
   var res = ''
   tt.on('data', function (c) {
@@ -81,7 +81,6 @@ test('segv', function (t) {
 })
 
 test('cleanup', function (t) {
-  return t.end()
   t.plan(2)
   fs.unlink('segv', function (er) {
     t.ifError(er, 'clean up segv')

diff --git a/test/test-test.js b/test/test-test.js
@@ -82,24 +82,24 @@ test('testing the test object', function (t) {
 
 test('plan stuff', function (t) {
   t.throws(function () {
-    var tt = new Test()
+    var tt = new Test({ buffered: false })
     tt.plan(1)
     tt.plan(1)
   }, new Error('Cannot set plan more than once'))
   t.throws(function () {
-    var tt = new Test()
+    var tt = new Test({ buffered: false })
     tt.plan('foo')
   }, new TypeError('plan must be a number'))
   t.throws(function () {
-    var tt = new Test()
+    var tt = new Test({ buffered: false })
     tt.plan(-1)
   }, new TypeError('plan must be a number'))
 
   t.end()
 })
 
 test('do nothing after bailout', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -117,7 +117,7 @@ test('do nothing after bailout', function (t) {
 })
 
 test('subtest without arguments', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -131,7 +131,7 @@ test('subtest without arguments', function (t) {
 })
 
 test('subtest with only a name', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -145,7 +145,7 @@ test('subtest with only a name', function (t) {
 })
 
 test('subtest with only options', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -159,7 +159,7 @@ test('subtest with only options', function (t) {
 })
 
 test('subtest with only a function', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -173,7 +173,7 @@ test('subtest with only a function', function (t) {
 })
 
 test('subtest with name and options', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -187,7 +187,7 @@ test('subtest with name and options', function (t) {
 })
 
 test('subtest with name and function', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -201,7 +201,7 @@ test('subtest with name and function', function (t) {
 })
 
 test('subtest with options and function', function (t) {
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   var out = ''
   tt.on('data', function (c) {
     out += c
@@ -216,7 +216,7 @@ test('subtest with options and function', function (t) {
 
 test('invalid test arguments', function (t) {
   t.throws(function () {
-    var tt = new Test()
+    var tt = new Test({ buffered: false })
     tt.test('name', { skip: false }, 'not a function')
   }, new TypeError('test() callback must be function if provided'))
 
@@ -228,7 +228,7 @@ test('throws type', function (t)  {
     throw new TypeError('some type error');
   }, TypeError, 'should throw a TypeError');
 
-  var tt = new Test()
+  var tt = new Test({ buffered: false })
   t.notOk(tt.throws(function () {
     throw new RangeError('x')
   }, TypeError))

diff --git a/test/timeout.js b/test/timeout.js
@@ -40,7 +40,7 @@ tap.test('t.setTimeout()', function (t) {
     tt.setTimeout('not a number')
   }, {}, { message: 'setTimeout: number > 0 required' })
 
-  var tt = new Test({ timeout: 1000, bail: false })
+  var tt = new Test({ timeout: 1000, bail: false, buffered: false })
   tt.setTimeout(2000)
   tt.setTimeout(Infinity)
   t.notOk(tt._timeout)
@@ -58,7 +58,7 @@ tap.test('t.setTimeout()', function (t) {
     '    # failed 1 of 2 tests\n' +
     'not ok 1 - child test # time=')
 
-  tt = new Test({ name: 'some name', bail: false })
+  tt = new Test({ name: 'some name', bail: false, buffered: false })
   tt.test('child test', { bail: false }, function (tt) {
     tt.pass('this is fine')
   })