Skip to content

🪲 [Fix]: Built modules with resolved versions can be tested#119

Merged
Marius Storhaug (MariusStorhaug) merged 5 commits into
mainfrom
fix/bump-install-psmodulehelpers
Jul 8, 2026
Merged

🪲 [Fix]: Built modules with resolved versions can be tested#119
Marius Storhaug (MariusStorhaug) merged 5 commits into
mainfrom
fix/bump-install-psmodulehelpers

Conversation

@MariusStorhaug

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Jul 8, 2026

Copy link
Copy Markdown
Member

Built modules whose manifests carry a real, resolved version can now be tested by Test-PSModule. Previously the action installed the built module under the historical 999.0.0 placeholder folder, so module tests failed once the build flow started stamping manifests with the version that will actually be shipped.

Fixed: Built modules with resolved versions can be tested

Test-PSModule now installs and validates built module outputs whose manifest ModuleVersion has already been resolved to the real build version. The module is installed under a matching version folder before module tests run, so PowerShell accepts the manifest and Pester validates the artifact from the expected path.

This is delivered by bumping the pinned PSModule/Install-PSModuleHelpers to v1.0.8, which installs built modules at their real version instead of the 999.0.0 placeholder.

Technical Details

  • action.yml: PSModule/Install-PSModuleHelpers pin bumped v1.0.7 -> v1.0.8 (the release from PSModule/Install-PSModuleHelpers#19).
  • src/tests/Module/PSModule/PSModule.Tests.ps1: new test asserting the installed version folder matches the manifest ModuleVersion, guarding against a regression to the 999.0.0 behavior.
  • tests/outputTestRepo/outputs/module/PSModuleTest/PSModuleTest.psd1: fixture ModuleVersion updated 999.0.0 -> 3.0.11 to model the current build output.
  • The branch is merged up to date with main, which already carries the actions/checkout v7.0.0, super-linter v8.7.0, and PSModule/Invoke-Pester v4.2.6 pins, so this PR no longer changes those.

…ersion

Install-PSModuleHelpers now installs the built module into a folder matching the manifest ModuleVersion instead of the 999.0.0 placeholder, which is required now that the build pipeline stamps the real version (PSModule/Process-PSModule#326). Temporarily pinned to the fix branch (PSModule/Install-PSModuleHelpers#19); will repin to the released tag before merge.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Test-PSModule composite action to install Install-PSModuleHelpers from a revision that installs built modules into a version folder matching the manifest ModuleVersion, avoiding Import-Module failures now that builds stamp the real resolved version.

Changes:

  • Bump PSModule/Install-PSModuleHelpers from v1.0.7 to a temporary fix-branch commit SHA that installs modules at their real versioned path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread action.yml Outdated
Copilot AI review requested due to automatic review settings July 8, 2026 12:09
@MariusStorhaug Marius Storhaug (MariusStorhaug) changed the title 🩹 [Patch]: Install built modules at their real version (bump Install-PSModuleHelpers) 🪲 [Fix]: Built modules with resolved versions can be tested Jul 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 8, 2026 12:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

Marius Storhaug (MariusStorhaug) added a commit that referenced this pull request Jul 8, 2026
Bumps
[PSModule/Invoke-Pester](https://github.com/PSModule/Invoke-Pester) from
**v4.2.4** to **v4.2.6** (latest).

Supersedes #115, which only bumped to v4.2.5 — going straight to the
latest avoids chaining two dependency PRs.

## What this PR changes

A single pin in [action.yml](action.yml):

```
uses: PSModule/Invoke-Pester@9cf262a # v4.2.6
```

Upstream changes inside Invoke-Pester between v4.2.4 and v4.2.6 are
captured in its own [release
notes](https://github.com/PSModule/Invoke-Pester/releases). Those (e.g.
its internal upload-artifact / GitHub-Script bumps) are internal to the
Invoke-Pester action, not separate dependencies of this repo.

## Status of all pinned actions in this repo

| Action | Pinned | Latest | Status |
| --- | --- | --- | --- |
| `actions/checkout` | v7.0.0 | v7.0.0 | up to date |
| `super-linter/super-linter` | v8.7.0 | v8.7.0 | up to date |
| `PSModule/Release-GHRepository` | v2.0.2 | v2.0.2 | up to date |
| `PSModule/Invoke-Pester` | v4.2.6 | v4.2.6 | **this PR** |
| `PSModule/Install-PSModuleHelpers` | v1.0.7 | v1.0.8 | handled by #119
|
Copilot AI review requested due to automatic review settings July 8, 2026 15:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread action.yml

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit 25c9cd8 into main Jul 8, 2026
31 checks passed
@MariusStorhaug Marius Storhaug (MariusStorhaug) deleted the fix/bump-install-psmodulehelpers branch July 8, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants