test: limit cache-tests workers on Windows by mcollina · Pull Request #5309 · nodejs/undici (original) (raw)

Skip to content

Navigation Menu

• Pricing

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

mcollina

merged 1 commit into

May 18, 2026

Conversation

Summary

• run cache-test environment workers serially on Windows to avoid intermittent worker termination
• keep existing parallelism on other platforms
• add CACHE_TEST_WORKER_CONCURRENCY for validation/debugging

Fixes: #5199

Testing

• npm run lint -- test/cache-interceptor/cache-tests.mjs
• npm run test:cache-tests
• CACHE_TEST_WORKER_CONCURRENCY=1 npm run test:cache-tests

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.23%. Comparing base (bd98954) to head (5087839).

Additional details and impacted files

@@ Coverage Diff @@ ## main #5309 +/- ##

Coverage 93.23% 93.23%

Files 110 110
Lines 36575 36575

• Hits 34101 34102 +1
• Misses 2474 2473 -1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

mcollina deleted the fix-cache-tests-windows-concurrency branch

May 18, 2026 16:09

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})