Skip to content

test: reduce watch mode restart flakiness#63390

Open
trivikr wants to merge 1 commit into
nodejs:mainfrom
trivikr:flaky-test-watch-mode-restart-esm-loading-error
Open

test: reduce watch mode restart flakiness#63390
trivikr wants to merge 1 commit into
nodejs:mainfrom
trivikr:flaky-test-watch-mode-restart-esm-loading-error

Conversation

@trivikr
Copy link
Copy Markdown
Member

@trivikr trivikr commented May 17, 2026

Fixes intermittent failures in
test-watch-mode-restart-esm-loading-error.mjs.

The test now starts waiting for the watch restart before writing the file and
waits for a platform-scaled settling period before the next mutation. This
reduces races with filesystem watch event delivery and dependency reporting
from the watched child process on slower CI machines.

Refs: https://github.com/nodejs/reliability/blob/main/reports/2026-05-16.md#jstest-failure


Assisted-by: openai:gpt-5.5

Start waiting for each watch restart before writing the file, then wait
for a platform-scaled settling period before mutating it again.

This gives watch mode time to process filesystem events and dependency
reporting from the child process on slower CI machines.

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
Copy link
Copy Markdown

codecov Bot commented May 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.04%. Comparing base (550f195) to head (1a627f9).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #63390      +/-   ##
==========================================
- Coverage   90.06%   90.04%   -0.02%     
==========================================
  Files         714      714              
  Lines      225648   225648              
  Branches    42711    42698      -13     
==========================================
- Hits       203237   203192      -45     
- Misses      14200    14257      +57     
+ Partials     8211     8199      -12     

see 54 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@trivikr trivikr added the request-ci Add this label to start a Jenkins CI on a PR. label May 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run. request-ci Add this label to start a Jenkins CI on a PR. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants