TypeError using interface ui: 'exports'

[juergba]

Description of the Change

Cannot run ui: 'exports' tests using API. The call to mocha.run() throws an exception "TypeError: Cannot read property 'skip' of undefined".

/home/rpaterson/programming/signpost/toshokan/node_modules/mocha/lib/mocha.js:207
    exports.xit = context.xit || context.test.skip;
                                             ^

TypeError: Cannot read property 'skip' of undefined

The ui: 'exports' interface is implemented in a basic manner. Only suites, tests and before-/beforeAll- and after-/afterAll-hooks are supported. Neither it.skip nor it.only are accepted.

This bug was introduced by PR #2997.
The correction is very simple:

exports.xit = context.xit || (context.test && context.test.skip);

Benefits

Bug fixing

Applicable issues

issue #3004

[coveralls]

Coverage increased (+0.09%) to 90.663% when pulling ca6492e on juergba:issue/3004 into d60b333 on mochajs:master.

[juergba]

@boneskull please close, since solved in b5c0fb0