qunitjs / qunit / 9451007881

90%

push

github

Core: Disalllow passing async/promise-returning function as QUnit.module callback

Throw an Error when a module callback has a promise as a return value.

Error early if module callback is an `async function` to avoid its tests from being defined as part of other modules.

Update tests:
* rename CLI test async-module-warning to async-module-error
* rename CLI test async-module-warning-promise to async-module-error-promise
* add main test to verify at runtime (use then-able syntax instead of Promise/async for IE11 compat).

Ref https://github.com/qunitjs/qunit/issues/1600.
Closes https://github.com/qunitjs/qunit/pull/1761.

3 of 3 new or added lines in 2 files covered. (100.0%)

1203 of 2049 relevant lines covered (58.71%)

91.14 hits per line