Skip to content

test: fix cp.exec-any-shells test on windows with wsl - #63118

Open
nadalaba wants to merge 1 commit into
nodejs:mainfrom
nadalaba:fix-win-shell-test
Open

test: fix cp.exec-any-shells test on windows with wsl#63118
nadalaba wants to merge 1 commit into
nodejs:mainfrom
nadalaba:fix-win-shell-test

Conversation

@nadalaba

@nadalaba nadalaba commented May 4, 2026

Copy link
Copy Markdown
Contributor

Avoid symlink failures when WSL bash executables are present on PATH.
WSL bash files (from WindowsApps and System32) either cannot be symlinked
or don't work as expected from symlinks. This change detects WSL bash and:

  • Skips testing it if no WSL distro is installed.
  • Skips symlink tests for all WSL bash executables.

@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 May 4, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nadalaba

Copy link
Copy Markdown
Contributor Author

hey @aduh95 can we run CI again since #63209 might've solved the issue?

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 90.14%. Comparing base (1e9fd95) to head (975db92).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #63118      +/-   ##
==========================================
- Coverage   90.17%   90.14%   -0.03%     
==========================================
  Files         769      769              
  Lines      261448   261448              
  Branches    49674    49637      -37     
==========================================
- Hits       235759   235684      -75     
- Misses      16736    16788      +52     
- Partials     8953     8976      +23     

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

@nadalaba
nadalaba force-pushed the fix-win-shell-test branch 2 times, most recently from b4a0e7e to fd3e639 Compare August 5, 2026 20:38
@MikeMcC399

MikeMcC399 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

@nadalaba

You asked about rebasing in Slack. Your branch is now more than 500 commits behind due to the passage of time. I would suggest to rebase now in any case.

The workflow Test macOS has been re-run twice and it still fails.

@MikeMcC399 MikeMcC399 added the wsl Issues and PRs related to the Windows Subsystem for Linux. label Sep 4, 2026
WSL bash files (from WindowsApps and System32) either cannot be
symlinked or don't work as expected from symlinks. This change detects
WSL bash and:
- Skips testing it if no WSL distro is installed.
- Skips symlink tests for all WSL bash executables.

Signed-off-by: Nad Alaba <[email protected]>
@MikeMcC399

Copy link
Copy Markdown
Contributor

Good to see that you rebased and that GitHub Actions CI is now green πŸŽ‰ !

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 Node.js core tests and test infrastructure. wsl Issues and PRs related to the Windows Subsystem for Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants