If a test is skipped at runtime (with this.skip();), the test result gets overwritten, and all the data (links, labels, etc) is lost.
In such cases, onTest is called before onPending so there is no need to create a new test result.
Also, we might want to implement a countermeasure of some sort (a warning or an error) if we dismiss the currently running test result.
Other changes
Issue template was added for allure-mocha.
The runner parameter is now added to allure-mocha test results (we test against three types of runners: the CLI runner and two code runners, in the CommonJS and ESM formats).
Context
If a test is skipped at runtime (with
this.skip();), the test result gets overwritten, and all the data (links, labels, etc) is lost. In such cases,onTestis called beforeonPendingso there is no need to create a new test result.Also, we might want to implement a countermeasure of some sort (a warning or an error) if we dismiss the currently running test result.
Other changes
Fixes #457
Checklist