qunitjs / qunit / 3169

89%

main: 90%

push

travis-ci-com

Core: Fix QUnit.module.only logic for unscoped modules

This bug affected use of QUnit.module.only when used in a "flat" way,
e.g. without scoping. For the following, it resulted in running test 1A and
2A, instead of only 1A:

```
QUnit.module.only( "Module one" );
QUnit.test( "1A", assert => { assert.ok( true ); } );

QUnit.module( "Module two" );
QUnit.test( "2A", assert => { assert.ok( false ); } );
```

Also move the "only" tests to CLI suite to verify TAP output
(more stable than `QUnit.done` post-run hack in HTML).

Fixes https://github.com/qunitjs/qunit/issues/1272.

1180 of 1438 branches covered (82.06%)

2084 of 2334 relevant lines covered (89.29%)

349.05 hits per line