Skip to content

test: deflake test-debugger-probe-timeout#63547

Open
joyeecheung wants to merge 1 commit into
nodejs:mainfrom
joyeecheung:probe-timeout-flake
Open

test: deflake test-debugger-probe-timeout#63547
joyeecheung wants to merge 1 commit into
nodejs:mainfrom
joyeecheung:probe-timeout-flake

Conversation

@joyeecheung
Copy link
Copy Markdown
Member

@joyeecheung joyeecheung commented May 24, 2026

On slow machines in the CI, the 200ms timeout may fire before the probed process finishes evaluation. Increase the timeout to common.platformTimeout(1000) to accomondate.

Refs: https://github.com/nodejs/reliability/blob/main/reports/2026-05-24.md

On slow machines in the CI, the 200ms timeout may fire before
the probed process finishes evaluation. Increase the timeout
to common.platformTimeout(1000) to accomondate.

Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>
@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 24, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@codecov
Copy link
Copy Markdown

codecov Bot commented May 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.33%. Comparing base (fa0aadc) to head (e5c3949).
⚠️ Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #63547      +/-   ##
==========================================
- Coverage   91.77%   90.33%   -1.44%     
==========================================
  Files         365      730     +365     
  Lines      158387   234161   +75774     
  Branches    24099    43916   +19817     
==========================================
+ Hits       145357   211526   +66169     
- Misses      12757    14378    +1621     
- Partials      273     8257    +7984     

see 511 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.

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. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants