Skip to content

Fix integration test to install built package instead of nuget.org version#16

Merged
Malcolmnixon merged 3 commits intomainfrom
copilot/fix-integration-test-installation
Jan 8, 2026
Merged

Fix integration test to install built package instead of nuget.org version#16
Malcolmnixon merged 3 commits intomainfrom
copilot/fix-integration-test-installation

Conversation

Copy link
Contributor

Copilot AI commented Jan 8, 2026

Plan: Fix Integration Test Package Installation

  • Understand the issue: The test job installs from nuget.org instead of the built package
  • Modify the test job in build.yaml to explicitly specify the version when installing
  • Verify the fix ensures the locally built package is installed
  • Run quality checks (spelling and markdown linting)
  • Address PR feedback: Remove unnecessary nupkg search and simplify shell
  • Complete the task

Summary

Fixed the integration test to install the locally built package instead of potentially fetching from nuget.org by adding --version ${{ inputs.version }} to the dotnet tool install command in the test job.

Additionally simplified the install step by removing the unused $nupkg variable search and changing from PowerShell to bash shell since no PowerShell-specific features are needed.

Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature]: Integration test must install the built package</issue_title>
<issue_description>### Problem Statement

The integration testing ends up installing the package from nuget.org rather than the package just built by the build job.

Proposed Solution

The dotnet tool install may have to explicitly specify the version.

Alternatives Considered

No response

Additional Context

No response</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix integration test to install built package Fix integration test to install built package instead of nuget.org version Jan 8, 2026
Copilot AI requested a review from Malcolmnixon January 8, 2026 19:27
@@ -101,7 +101,7 @@ jobs:
shell: pwsh
run: |
$nupkg = Get-ChildItem -Path artifacts -Filter "*.nupkg" | Select-Object -First 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason for running this search for the nupkg? If not then this probably also doesn't need to be a pwsh shell step?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right - the nupkg search wasn't being used. I've simplified the step to a single-line command without the pwsh shell. Fixed in 1d667ce.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
@Malcolmnixon Malcolmnixon marked this pull request as ready for review January 8, 2026 19:32
@Malcolmnixon Malcolmnixon merged commit 69f0bb3 into main Jan 8, 2026
5 checks passed
@Malcolmnixon Malcolmnixon deleted the copilot/fix-integration-test-installation branch January 8, 2026 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Integration test must install the built package

2 participants