test: avoid test_runner watch restart in spec snapshot by trivikr · Pull Request #63392 · nodejs/node · GitHub
Skip to content

test: avoid test_runner watch restart in spec snapshot#63392

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
trivikr:flaky-test-output-test-runner-watch-spec
May 24, 2026
Merged

test: avoid test_runner watch restart in spec snapshot#63392
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
trivikr:flaky-test-output-test-runner-watch-spec

Conversation

@trivikr

@trivikr trivikr commented May 17, 2026

Copy link
Copy Markdown
Member

This updates the test_runner watch-mode spec reporter fixture to pass
explicit test files to run().

The fixture no longer watches the whole temporary directory, so setup writes
cannot race with watcher initialization and emit an unexpected restart banner.

Refs:


Assisted-by: openai:gpt-5.5

Pass explicit test files to the watch-mode spec reporter fixture.
This prevents setup writes from triggering a watch restart.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: openai:gpt-5.5
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels May 17, 2026
@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

@trivikr trivikr added the request-ci Add this label to start a Jenkins CI on a PR. label May 18, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 18, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@trivikr trivikr added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 18, 2026
@aduh95 aduh95 added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label May 19, 2026
@trivikr trivikr added the commit-queue Add this label to land a pull request using GitHub Actions. label May 24, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 24, 2026
@nodejs-github-bot nodejs-github-bot merged commit 1dbea7a into nodejs:main May 24, 2026
100 of 102 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

araujogui pushed a commit to araujogui/node that referenced this pull request May 26, 2026
Pass explicit test files to the watch-mode spec reporter fixture.
This prevents setup writes from triggering a watch restart.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: openai:gpt-5.5
PR-URL: nodejs#63392
Refs: https://github.com/nodejs/node/actions/runs/25983032903/job/76375266688
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@trivikr trivikr deleted the flaky-test-output-test-runner-watch-spec branch May 27, 2026 07:49
aduh95 pushed a commit that referenced this pull request May 27, 2026
Pass explicit test files to the watch-mode spec reporter fixture.
This prevents setup writes from triggering a watch restart.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: openai:gpt-5.5
PR-URL: #63392
Refs: https://github.com/nodejs/node/actions/runs/25983032903/job/76375266688
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
sxa pushed a commit to sxa/node that referenced this pull request Jun 18, 2026
Pass explicit test files to the watch-mode spec reporter fixture.
This prevents setup writes from triggering a watch restart.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: openai:gpt-5.5
PR-URL: nodejs#63392
Refs: https://github.com/nodejs/node/actions/runs/25983032903/job/76375266688
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
sxa pushed a commit that referenced this pull request Jun 18, 2026
Pass explicit test files to the watch-mode spec reporter fixture.
This prevents setup writes from triggering a watch restart.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: openai:gpt-5.5
PR-URL: #63392
Refs: https://github.com/nodejs/node/actions/runs/25983032903/job/76375266688
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. flaky-test Issues and PRs related to the tests with unstable failures on the CI. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants