⚙️ [Maintenance]: Adopt Process-PSModule v6.1.4 and require Pester 6.x in tests#639
⚙️ [Maintenance]: Adopt Process-PSModule v6.1.4 and require Pester 6.x in tests#639Marius Storhaug (MariusStorhaug) wants to merge 4 commits into
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. |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
There was a problem hiding this comment.
Pull request overview
This PR updates the test suite’s Pester dependency declaration to allow any Pester 6.x release (rather than pinning to 5.8.0), aligning the tests with the newly published Pester 6 major version while still pinning the module identity via GUID.
Changes:
- Replaced
RequiredVersion = '5.8.0'with a 6.x range lock usingModuleVersion = '6.0.0'andMaximumVersion = '6.999.999'. - Applied the updated
#Requires -Modulesdeclaration consistently across all test files, includingtests/TEMPLATE.ps1.
Reviewed changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/Variables.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Users.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/TEMPLATE.ps1 | Updates the template Pester #Requires to allow Pester 6.x for new tests. |
| tests/Teams.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Secrets.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Repositories.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Releases.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Permissions.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Organizations.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/GitHubFormatter.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/GitHub.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Environments.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Enterprise.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Emojis.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Artifacts.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Apps.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
| tests/Actions.Tests.ps1 | Updates Pester #Requires to allow Pester 6.x. |
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. |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
The GUID pins module identity (precise pinning), a stricter control than the lock-to-major risk appetite. Keep only the version range.
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. |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
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. |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
This module now runs on the latest Process-PSModule automation (v6.1.4) and pins its test suite to Pester 6, so continuous integration uses the current build, test, and publish pipeline and the tests fail fast if Pester 6 is ever missing.
Changed: CI runs on Process-PSModule v6.1.4
The reusable workflow pin moves to v6.1.4 from the module's previously pinned v5 release, adopting the current pipeline, including the Pester 6 test runner and the consolidated
TestDatasecret model. No caller changes are required — the existingsecretswiring keeps passing everything the pipeline needs.Changed: Tests require Pester 6.x
Every test file now declares the Pester 6 requirement, so the suite always resolves Pester 6 and fails with a clear message if only an older Pester is available, instead of silently running on it.
Technical Details
.github/workflows/Process-PSModule.yml: pin bumped toworkflow.yml@da180bac16b13bfbcdf08b2e4e221b5b49e5ff28 # v6.1.4.tests/*.Tests.ps1: added#Requires -Modules @{ ModuleName = 'Pester'; ModuleVersion = '6.0.0'; MaximumVersion = '6.*' }.Shouldassertions are left unchanged. Pester 6 is backward-compatible, so converting to the newShould-*syntax is intentionally out of scope for this maintenance change.