Skip to content

[release/v7.7.0] Fix the regex in ConsoleHost.Tests.ps1 used to verify help content - #28365

Merged
Justin Chung (jshigetomi) merged 1 commit into
release/v7.7.0from
backport/release/v7.7.0/28308-01b9a93c3
Nov 6, 2026
Merged

Justin Chung (jshigetomi) merged 1 commit into
release/v7.7.0from
backport/release/v7.7.0/28308-01b9a93c3

Conversation

@jshigetomi

Copy link
Copy Markdown
Collaborator

Backport of #28308 to release/v7.7.0

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

Original CL Label: CL-Test

/cc @PowerShell/powershell-maintainers

Impact

Tooling Impact

  • Required tooling change
  • Optional tooling change

Optional because this is a test-only correctness fix.

Regression

  • Yes
  • No

This is not a regression.

Testing

The original PR corrects the regex used to verify help content, and the cherry-pick applied conflict-free. CI validation is required.

Risk

  • High
  • Medium
  • Low

Low risk because this is a test-only change.

@jshigetomi
Justin Chung (jshigetomi) requested a review from a team as a code owner November 5, 2026 19:45
Copilot AI balanced review requested due to automatic review settings November 5, 2026 19:45
@azure-pipelines

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

@jshigetomi Justin Chung (jshigetomi) added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Nov 5, 2026

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.

🟢 Approval recommended

No active review issues are reported for the test-only correction to the ConsoleHost help-output assertion.

No open issues 🎉

What changed in this PR

Backports #28308 to release/v7.7.0, correcting the ConsoleHost help-output test so unrelated output no longer satisfies its assertion. This is a test-only change.

Changes:

  • Escapes regex metacharacters to match the literal help syntax.
  • Updates the expected usage text to include -h.
File Description
test/​powershell/​Host/​ConsoleHost.Tests.ps1 Corrects the regex used to validate output from help parameters.

Review effort: Balanced

@daxian-dbw Dongbo Wang (daxian-dbw) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copilot CLI reviewed this backport and confirmed that its changes match the original PR, #28308, as merged into master.

auto-merge was automatically disabled November 6, 2026 02:16

Pull request was closed

@jshigetomi Justin Chung (jshigetomi) changed the title [release/v7.7.0] Fix the regex in ConsoleHost.Tests.ps1 used to verify help content. [release/v7.7.0] Fix the regex in ConsoleHost.Tests.ps1 used to verify help content Nov 6, 2026
@jshigetomi
Justin Chung (jshigetomi) merged commit c9b051f into release/v7.7.0 Nov 6, 2026
69 of 70 checks passed
@daxian-dbw
Dongbo Wang (daxian-dbw) deleted the backport/release/v7.7.0/28308-01b9a93c3 branch November 6, 2026 18:00
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