Skip to content

[release/v7.6.7] Fix the PATH test for MSIX installed PowerShell - #28108

Merged
Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/28068-5f9fcddc2
Sep 25, 2026
Merged

Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/28068-5f9fcddc2

Conversation

@daxian-dbw

Copy link
Copy Markdown
Member

Backport of #28068 to release/v7.6.7

Triggered by Dongbo Wang (@daxian-dbw) on behalf of Dongbo Wang (@daxian-dbw)

Original CL Label: CL-Test

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Fixes a test that was failing/misaligned after the MSIX PATH handling change was backported, ensuring the test suite correctly validates the stable execution-alias PATH behavior.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Original PR updated the PATH test to align with the MSIX execution-alias PATH handling fix. Backport verified by confirming the cherry-pick applied cleanly with no conflicts on release/v7.6.7, which already includes the corresponding product fix from PR #27782 (backport PR #28091).

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Low risk: test-only change that updates the PATH test to expect the stable MSIX execution alias path instead of $PSHOME, matching the behavior introduced by PR #27782 (already backported to this release). No product code is modified.

Copilot AI lite review requested due to automatic review settings September 25, 2026 18:07
@daxian-dbw Dongbo Wang (daxian-dbw) added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Sep 25, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

No unresolved review issues; the change is low-risk and test-only.

Review effort: Lite
Findings: None

What changed in this PR

Backports a test-only fix for validating the stable MSIX execution-alias PATH behavior.

Changes:

  • Renames the PATH startup test.
  • Computes expected PATH entries for MSIX and non-MSIX installations.
File Summary
test/​powershell/​Host/​ConsoleHost.Tests.ps1 Adjusts PATH expectations for MSIX installations.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit c8961c7 into PowerShell:release/v7.6.7 Sep 25, 2026
36 checks passed
@daxian-dbw
Dongbo Wang (daxian-dbw) deleted the backport/release/v7.6.7/28068-5f9fcddc2 branch September 25, 2026 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Test Indicates that a PR should be marked as a test change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants