mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 21:19:50 +01:00
7dfa4750af
This commit adds a regression test for the edge case where a test runner test is marked as both todo and skip. Refs: https://github.com/nodejs/node/issues/49013 PR-URL: https://github.com/nodejs/node/pull/52204 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
33 lines
1.0 KiB
JavaScript
33 lines
1.0 KiB
JavaScript
'use strict';
|
|
const common = require('../common');
|
|
const { strictEqual } = require('node:assert');
|
|
const { run, suite, test } = require('node:test');
|
|
|
|
if (!process.env.NODE_TEST_CONTEXT) {
|
|
const stream = run({ files: [__filename] });
|
|
|
|
stream.on('test:fail', common.mustNotCall());
|
|
stream.on('test:pass', common.mustCall((event) => {
|
|
strictEqual(event.skip, true);
|
|
strictEqual(event.todo, undefined);
|
|
}, 4));
|
|
} else {
|
|
test('test options only', { skip: true, todo: true }, common.mustNotCall());
|
|
|
|
test('test context calls only', common.mustCall((t) => {
|
|
t.todo();
|
|
t.skip();
|
|
}));
|
|
|
|
test('todo test with context skip', { todo: true }, common.mustCall((t) => {
|
|
t.skip();
|
|
}));
|
|
|
|
// Note - there is no test for the skip option and t.todo() because the skip
|
|
// option prevents the test from running at all. This is verified by other
|
|
// tests.
|
|
|
|
// Suites don't have the context methods, so only test the options combination.
|
|
suite('suite options only', { skip: true, todo: true }, common.mustNotCall());
|
|
}
|