Skip to content

test: fix skipping behavior for test-runner-run-files-undefined#62026

Open
aduh95 wants to merge 1 commit intonodejs:mainfrom
aduh95:test-runner-run-files-undefined
Open

test: fix skipping behavior for test-runner-run-files-undefined#62026
aduh95 wants to merge 1 commit intonodejs:mainfrom
aduh95:test-runner-run-files-undefined

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Feb 27, 2026

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/test_runner

@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 Feb 27, 2026
@nodejs-github-bot
Copy link
Collaborator

@codecov
Copy link

codecov bot commented Feb 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.73%. Comparing base (7cf2dad) to head (76c4ee9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #62026      +/-   ##
==========================================
+ Coverage   89.70%   89.73%   +0.02%     
==========================================
  Files         676      676              
  Lines      206069   206069              
  Branches    39520    39517       -3     
==========================================
+ Hits       184860   184921      +61     
+ Misses      13358    13324      -34     
+ Partials     7851     7824      -27     

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

@Renegade334
Copy link
Member

Renegade334 commented Feb 27, 2026

Believe this also needs to be skipped in worker threads (separate from the question mark over shared OpenSSL).

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