Skip to content

[release/v7.6.7] Mark DSC Profile tests pending on MSIX test runs - #28092

Merged
Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/27884-9e734952a
Sep 24, 2026
Merged

Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/27884-9e734952a

Conversation

@daxian-dbw

Copy link
Copy Markdown
Member

Backport of #27884 to release/v7.6.7

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

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)

Prevents false test failures in CI/test runs against MSIX-packaged PowerShell builds by marking known-incompatible DSC Profile tests as pending rather than failing.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Original PR was verified by running the DSC profile resource test suite in both MSIX and non-MSIX test contexts, confirming tests are marked pending only under MSIX. Backport verified by confirming the cherry-pick applied cleanly with no conflicts on release/v7.6.7.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Low risk: test-only change that detects MSIX-packaged test runs and marks known-incompatible DSC profile tests as pending instead of failing. No product code is modified.

Copilot AI lite review requested due to automatic review settings September 24, 2026 18:28
@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 24, 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 were identified.

Review effort: Lite
Findings: None

What changed in this PR

Updates DSC profile tests to prevent false failures in MSIX-packaged PowerShell environments.

Changes:

  • Detects MSIX execution contexts.
  • Marks affected tests as pending.
  • Skips incompatible setup and cleanup operations.
File Description
test/​powershell/​dsc/​dsc.profileresource.Tests.ps1 Adds MSIX detection and pending-test handling.

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

@daxian-dbw Dongbo Wang (daxian-dbw) added CL-Test Indicates that a PR should be marked as a test change in the Change Log and removed CL-Test Indicates that a PR should be marked as a test change in the Change Log labels Sep 24, 2026
@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 2c3e458 into PowerShell:release/v7.6.7 Sep 24, 2026
37 of 38 checks passed
@daxian-dbw
Dongbo Wang (daxian-dbw) deleted the backport/release/v7.6.7/27884-9e734952a branch September 24, 2026 22:09
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.

3 participants