Mark flaky Update-Help web tests as pending to unblock CI#26796
Merged
TravisEz13 merged 4 commits intomasterPowerShell/PowerShell:masterfrom Feb 13, 2026
copilot/fix-update-help-testsPowerShell/PowerShell:copilot/fix-update-help-testsCopy head branch name to clipboard
Merged
Mark flaky Update-Help web tests as pending to unblock CI#26796TravisEz13 merged 4 commits intomasterPowerShell/PowerShell:masterfrom copilot/fix-update-help-testsPowerShell/PowerShell:copilot/fix-update-help-testsCopy head branch name to clipboard
TravisEz13 merged 4 commits intomasterPowerShell/PowerShell:masterfrom
copilot/fix-update-help-testsPowerShell/PowerShell:copilot/fix-update-help-testsCopy head branch name to clipboard
Conversation
Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix or mark update-help tests as pending
Mark flaky Update-Help web tests as pending to unblock CI
Feb 12, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR addresses intermittent network failures in Update-Help tests that have been blocking CI builds, as documented in issues #2807 and #6541. The solution marks web-dependent CI tests as pending while preserving Feature-tagged tests and tests using local assets.
Changes:
- Adds a
markAsPendingparameter toRunUpdateHelpTeststhat conditionally marks tests as pending usingSet-ItResult - Implements defensive null checks for
$moduleHelpPath,$saveHelpFolder, and$TestDriveto handle Pester discovery phase gracefully - Creates comprehensive instruction documentation for the
Set-ItResultpattern to guide future test development
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| test/powershell/engine/Help/UpdatableHelpSystem.Tests.ps1 | Adds markAsPending parameter to selectively mark CI web tests as pending; adds defensive null checks for variables used during Pester discovery phase |
| .github/instructions/pester-set-itresult-pattern.instructions.md | New instruction document explaining when and how to use Set-ItResult for pending/skipped tests, with emphasis on the importance of using return after Set-ItResult |
daxian-dbw
approved these changes
Feb 12, 2026
TravisEz13
added a commit
to TravisEz13/PowerShell
that referenced
this pull request
Feb 13, 2026
…#26796) Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
9 tasks
daxian-dbw
pushed a commit
to daxian-dbw/PowerShell
that referenced
this pull request
Feb 15, 2026
…#26796) Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
9 tasks
daxian-dbw
pushed a commit
to daxian-dbw/PowerShell
that referenced
this pull request
Feb 17, 2026
…#26796) Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
12 tasks
daxian-dbw
pushed a commit
to daxian-dbw/PowerShell
that referenced
this pull request
Feb 17, 2026
…#26796) Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
9 tasks
9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update-Help tests downloading from the web have intermittent network failures (timeouts, task cancellations) blocking CI builds. Issues #2807 and #6541 document the pattern.
Changes
Mark web-dependent CI tests as pending using
Set-ItResult -PendinginRunUpdateHelpTests-SourcePathvariants using local assets)Add defensive null checks for
$moduleHelpPath,$saveHelpFolder,$TestDriveto handle Pester discovery phase gracefullyDocument Set-ItResult pattern in
.github/instructions/pester-set-itresult-pattern.instructions.mdreturnafterSet-ItResultPattern
Result
33 of 38 tests now marked as NotRun (pending), unblocking CI while keeping tests visible for future resolution.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.