Skip to content

[codex] Skip Windows symlink tests without permission#98

Closed
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/fix-windows-symlink-test-baseline
Closed

[codex] Skip Windows symlink tests without permission#98
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/fix-windows-symlink-test-baseline

Conversation

@w287346141

Copy link
Copy Markdown

What changed

  • Reuse the existing file-symlink capability probe across both version detection regression tests.
  • Skip the npm install-prefix symlink test when Windows does not grant symlink creation permission.

Why

The install-source test already handled Windows environments without Developer Mode or elevated symlink privileges, but the related install-prefix test still called symlinkSync directly. That made the full verification suite fail with EPERM even though the runtime behavior was not broken.

Impact

Windows contributors can run the full test suite without enabling symlink privileges. Environments that support symlinks still execute both regression tests normally.

Validation

  • npm run verify
  • 338 test files passed
  • 3910 tests passed, 4 environment-related tests skipped

@w287346141 w287346141 marked this pull request as ready for review June 12, 2026 16:11
@w287346141

Copy link
Copy Markdown
Author

Closing this PR to resubmit from the Star-Star66 account.

@w287346141 w287346141 closed this Jun 12, 2026
@w287346141 w287346141 deleted the codex/fix-windows-symlink-test-baseline branch June 12, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants