search

tapjs / tap-parser

parse the test anything protocol
121 stars 35 forks source link

Fix parsing of directives #71

Closed juergba closed 4 years ago

juergba commented 4 years ago

related issue node-tap

The tap specification states:

Skipping tests If the directive starts with # SKIP, the test is counted as having been skipped.

The directive is case-insensitive, but includes a space between # and the keyword. This parser does not consider that space.

As a result tests as eg. tap.test('LazyRange#skip', async (suite) => { } will run and be reported as skipped. This is valid for both keywords SKIP and TODO.

closes #66