Skip to content

test(env): cover PowerShell setup - #2495

Merged
fengmk2 merged 1 commit into
agent/posix-shell-snapshotsfrom
agent/powershell-shell-snapshots
Aug 20, 2026
Merged

test(env): cover PowerShell setup#2495
fengmk2 merged 1 commit into
agent/posix-shell-snapshotsfrom
agent/powershell-shell-snapshots

Conversation

@fengmk2

@fengmk2 fengmk2 commented Aug 19, 2026

Copy link
Copy Markdown
Member

Tests did not run the generated env.ps1 file in PowerShell. They missed errors in PowerShell process-environment changes.

The snapshot runner accepts a pwsh path. Windows CI supplies this path. The fixture starts PowerShell and dot-sources env.ps1. It checks setup, wrapper passthrough, help, version selection, unset, and .node-version selection.

PowerShell users get test coverage for the generated environment file.

Stack created with GitHub Stacks CLIGive Feedback 💬

@fengmk2
fengmk2 force-pushed the agent/powershell-shell-snapshots branch from 82b9eb6 to c96a557 Compare August 19, 2026 02:59
@fengmk2
fengmk2 force-pushed the agent/powershell-shell-snapshots branch from c96a557 to 6c2d414 Compare August 19, 2026 03:28
@fengmk2
fengmk2 force-pushed the agent/powershell-shell-snapshots branch from 6c2d414 to be82b72 Compare August 19, 2026 05:56
@fengmk2
fengmk2 marked this pull request as ready for review August 19, 2026 06:01
@fengmk2
fengmk2 requested a review from wan9chi August 19, 2026 06:01
@fengmk2 fengmk2 self-assigned this Aug 19, 2026
@fengmk2
fengmk2 force-pushed the agent/powershell-shell-snapshots branch from be82b72 to 3dcfa46 Compare August 19, 2026 14:30
@fengmk2
fengmk2 force-pushed the agent/powershell-shell-snapshots branch from 3dcfa46 to 8fac97b Compare August 20, 2026 03:28
@fengmk2
fengmk2 merged commit 11dd8d2 into main Aug 20, 2026
41 of 75 checks passed
@fengmk2
fengmk2 deleted the agent/powershell-shell-snapshots branch August 20, 2026 05:05
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