⚙️ [Maintenance]: Hard-lock Pester test dependency to 5.8.0 by GUID#119
Conversation
No Significant Changes DetectedThis PR does not contain changes to files that would trigger a new release:
Build, test, and publish stages will be skipped for this PR. If you believe this is incorrect, please verify that your changes are in the correct locations. |
There was a problem hiding this comment.
Pull request overview
This PR updates the Pester #Requires -Modules directive at the top of the test suites to align with the Pester version currently being installed by the CI runner, addressing discovery failures caused by version conflicts during Pester discovery/import.
Changes:
- Update
#Requires -Modulesin test entrypoints fromRequiredVersion = '5.7.1'toRequiredVersion = '5.8.0'. - Add the Pester module
GUIDto the#Requireshashtable to pin module identity.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tests/Context.Tests.ps1 | Updates the Pester #Requires pin (version + GUID) used during test discovery. |
| tests/ContextVaults.Tests.ps1 | Updates the Pester #Requires pin (version + GUID) used during test discovery. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -1,4 +1,4 @@ | |||
| #Requires -Modules @{ ModuleName = 'Pester'; RequiredVersion = '5.7.1' } | |||
| #Requires -Modules @{ ModuleName = 'Pester'; RequiredVersion = '5.8.0'; GUID = 'a699dea5-2c73-4616-a270-1f7abb777e71' } | |||
| @@ -1,4 +1,4 @@ | |||
| #Requires -Modules @{ ModuleName = 'Pester'; RequiredVersion = '5.7.1' } | |||
| #Requires -Modules @{ ModuleName = 'Pester'; RequiredVersion = '5.8.0'; GUID = 'a699dea5-2c73-4616-a270-1f7abb777e71' } | |||
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Hard-locks the Pester test dependency to an exact version pinned by module GUID, replacing the
RequiredVersion = '5.7.1'pin that broke test discovery when the runner installed Pester 5.8.0.#Requires -Modules @{ ModuleName = 'Pester'; RequiredVersion = '5.8.0'; GUID = 'a699dea5-2c73-4616-a270-1f7abb777e71' }Why 5.8.0 + GUID
@<sha>,RequiredVersionalready states the version in plain text, and#Requiresrejects a GUID without a version key.Files updated: 2 test file(s).
Fixes #118.
Note
This pin fully holds once
Invoke-Pesterinstalls the pinned version instead of the latest (today it installs newest, so a future Pester release would re-break exact pins). Tracked in PSModule/Invoke-Pester#68. Part of the dependency-management epic PSModule/Process-PSModule#356.