Skip to content

Commit

Permalink
Test: Document why cli-watch.js fails on Linux with Node.js 20.12 exa…
Browse files Browse the repository at this point in the history
…ctly

This doesn't affect CI which uses a newer Node, but wikimedia/fresh
is currently pinned to Node.js 20.12 and others may run into this
as well.
  • Loading branch information
Krinkle committed Jun 1, 2024
1 parent 01b222e commit cdab9e9
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/cli/cli-watch.js
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,9 @@ QUnit.module('CLI Watch', function (hooks) {
assert.equal(result.stdout, expectedWatchOutput['add-file']);
});

// NOTE: This is known to fail on Linux with Node.js 20.12.
// The regression was fixed in Node.js 20.13.
// https://github.com/nodejs/node/issues/52018
QUnit.test('re-runs tests on file removed', async assert => {
fixturify.writeSync(fixturePath, {
'foo.js': "QUnit.test('foo', function(assert) { assert.true(true); });",
Expand Down

0 comments on commit cdab9e9

Please sign in to comment.