t.comment() = fail exit code

[timoxley]

Is this correct behaviour?

Including any test comments seems to fail a suite.

// test.js
var test = require('tape')

test('comment test', function(t) {
  t.comment('this is a comment')
  t.ok(true)
  t.end()
})

> tape test.js
TAP version 13
# comment test
# this is a comment
ok 1 (unnamed assert)

1..1
# tests 1
# pass  1

# ok


> echo $?
1

[ghost]

Nope! That should pass. Bug.

[tmcw]

👍 should be closed by a22fdd1

[davidtheclark]

I'm still having this problem. Addition of a single comment is causing exit code 1 despite all tests passing; remove the comment and everything's fine.

[geraintwhite]

This is still a bug in v4.0.1

[soundyogi]

+1
getting exit code 1