node icon indicating copy to clipboard operation
node copied to clipboard

test: allow recursive testing

Open redyetidev opened this issue 9 months ago • 7 comments

Blocks #52875

This PR gives the test runner support for recursive searching for test files. This is being backported, as in #52875 (and similar future PRs), testing will (hopefully) be moved to subdirectories.

redyetidev avatar May 08 '24 15:05 redyetidev

@nodejs/testing @nodejs/backporters

redyetidev avatar May 08 '24 15:05 redyetidev

By the way, this PR should be backported to 18 and 20, I'm not sure if the labels I applied are correct.

redyetidev avatar May 08 '24 17:05 redyetidev

Thanks @richardlau, I'm new to backporting. For the future, what would be the appropriate labels?

redyetidev avatar May 08 '24 18:05 redyetidev

Possibly lts-watch-*, although strictly speaking it shouldn't be necessary for a release line in active LTS (e.g. 20.x).

Refer to https://github.com/nodejs/node/blob/main/doc/contributing/backporting-to-release-lines.md

richardlau avatar May 08 '24 18:05 richardlau

Possibly lts-watch-*, although strictly speaking it shouldn't be necessary for a release line in active LTS (e.g. 20.x).

Refer to main/doc/contributing/backporting-to-release-lines.md

Okay, thanks. Should I add lts-watch for 18 and 20? This change should be the exact same for the other lines, as I don't think the files that this is editing have changed.

redyetidev avatar May 08 '24 18:05 redyetidev

@targos, is there anything in particular I need to do for the backport?

redyetidev avatar May 09 '24 20:05 redyetidev

As @richardlau suggested: add lts-watch-* labels.

targos avatar May 10 '24 06:05 targos

Unfortunately, the eslint changes have resulting in this PR needing the modification of files that no longer exist. I've reopened this PR at #53309

redyetidev avatar Jun 03 '24 17:06 redyetidev