Skip to content

test: make Brotli 16GB test wait for backpressure#63389

Merged
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
trivikr:flaky-test-zlib-brotli-16GB
Jun 6, 2026
Merged

test: make Brotli 16GB test wait for backpressure#63389
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
trivikr:flaky-test-zlib-brotli-16GB

Conversation

@trivikr
Copy link
Copy Markdown
Member

@trivikr trivikr commented May 17, 2026

This updates parallel/test-zlib-brotli-16GB to wait until the Brotli
decoder reaches the expected readable-buffer backpressure point
before asserting the buffered chunk count.

The previous test used a fixed timeout and could check _readableState.buffer.length
before libuv worker-pool processing had pushed any output, causing intermittent
failures like 0 !== 4 on loaded CI machines.

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


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.33%. Comparing base (8b88e2c) to head (5e854e7).
⚠️ Report is 56 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #63389      +/-   ##
==========================================
- Coverage   91.95%   90.33%   -1.63%     
==========================================
  Files         379      732     +353     
  Lines      166486   236454   +69968     
  Branches    25452    44531   +19079     
==========================================
+ Hits       153099   213594   +60495     
- Misses      13094    14597    +1503     
- Partials      293     8263    +7970     

see 481 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
Comment thread test/parallel/test-zlib-brotli-16GB.js Outdated
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 22, 2026
@nodejs-github-bot

This comment was marked as outdated.

@trivikr trivikr force-pushed the flaky-test-zlib-brotli-16GB branch from a6f7f6e to 143affe Compare May 28, 2026 05:06
@trivikr trivikr requested a review from aduh95 May 28, 2026 05:07
@trivikr trivikr added the request-ci Add this label to start a Jenkins CI on a PR. label May 28, 2026
@trivikr trivikr force-pushed the flaky-test-zlib-brotli-16GB branch from 143affe to 5e854e7 Compare May 31, 2026 05:22
@trivikr trivikr added the review wanted PRs that need reviews. label Jun 4, 2026
@trivikr trivikr requested a review from jasnell June 4, 2026 05:09
@trivikr trivikr added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 4, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 4, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

trivikr added 2 commits June 5, 2026 11:53
Wait for the Brotli decoder to fill its readable buffer before checking
that decompression stops at the high water mark. This avoids racing the
fixed timeout against libuv worker-pool scheduling.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: openai:gpt-5.5
@aduh95 aduh95 force-pushed the flaky-test-zlib-brotli-16GB branch from 5e854e7 to cf75868 Compare June 5, 2026 09:53
@aduh95 aduh95 added commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. request-ci Add this label to start a Jenkins CI on a PR. and removed review wanted PRs that need reviews. labels Jun 5, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 5, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@trivikr trivikr added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 6, 2026
@trivikr

This comment was marked as outdated.

@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 6, 2026
@nodejs-github-bot nodejs-github-bot merged commit 2adaeee into nodejs:main Jun 6, 2026
75 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 2adaeee

@trivikr trivikr deleted the flaky-test-zlib-brotli-16GB branch June 6, 2026 04:49
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. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. 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.

6 participants