Skip to content

test: deflake WASI poll timing checks - #65672

Open
panva wants to merge 1 commit into
nodejs:mainfrom
panva:fix-wasi-poll-timeout-flake
Open

test: deflake WASI poll timing checks#65672
panva wants to merge 1 commit into
nodejs:mainfrom
panva:fix-wasi-poll-timeout-flake

Conversation

@panva

@panva panva commented Aug 30, 2026

Copy link
Copy Markdown
Member

Use a monotonic high-resolution clock for elapsed-time checks and allow a small tolerance for timer granularity. The previous whole-second clock could report an interval one second short when a slightly early timer coincided with a wall-clock boundary.

@panva panva added the flaky-test Issues and PRs involving tests that fail intermittently in CI. label Aug 30, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/wasi

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure. labels Aug 30, 2026
Use a monotonic high-resolution clock for elapsed-time checks and allow
a small tolerance for timer granularity. The previous whole-second clock
could report an interval one second short when a slightly early timer
coincided with a wall-clock boundary.

Assisted-by: Codex
Signed-off-by: Filip Skokan <panva.ip@gmail.com>
@panva
panva force-pushed the fix-wasi-poll-timeout-flake branch from 349cac4 to 7c49367 Compare August 30, 2026 18:09
@codecov

codecov Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.06%. Comparing base (f7e2c14) to head (7c49367).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #65672      +/-   ##
==========================================
- Coverage   91.72%   90.06%   -1.67%     
==========================================
  Files         402      754     +352     
  Lines      181167   255747   +74580     
  Branches    27812    48321   +20509     
==========================================
+ Hits       166173   230330   +64157     
- Misses      14653    16546    +1893     
- Partials      341     8871    +8530     

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

@panva

panva commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

cc @mhdawson @cjihrig @addaleax

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flaky-test Issues and PRs involving tests that fail intermittently in CI. needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants