Skip to content

Fix integration test to install built package with explicit version#17

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

Fix integration test to install built package with explicit version#17
Malcolmnixon merged 3 commits intomainfrom
copilot/fix-integration-test-package-installation

Conversation

Copy link
Contributor

Copilot AI commented Jan 8, 2026

Fix Integration Test Package Installation

Completed Tasks:

  • Understand the problem: test job doesn't specify version when installing, may pull from nuget.org
  • Modify build.yaml to use inputs.version directly (no outputs needed)
  • Update test job to use the specific version when installing the tool
  • Separate version and help tests into distinct steps with explicit error messages
  • Run quality checks (spelling and markdown linting)
  • Address PR review feedback

Changes Made:

  1. ✅ Removed unnecessary outputs section - test job now uses ${{ inputs.version }} directly
  2. ✅ Updated test job to use --version ${{ inputs.version }} in install command
  3. ✅ Split tests into separate steps: "Test Pandoc Version Command" and "Test Pandoc Help Command"
  4. ✅ Added explicit error handling with informative messages using || and exit codes
  5. ✅ All quality checks passed (spelling, markdown linting, YAML validation)

How This Solves the Issue:

  • Version Specification: Explicitly passing --version ensures dotnet tool only looks in the local build-artifacts directory for that exact version, preventing fallback to nuget.org
  • Exit Code Validation: Explicit || { echo "✗ ... failed"; exit 1; } ensures any failures are clearly reported
  • Better Testing: Separated test steps provide clearer output and easier debugging
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>### Feature Description

The actual package built by the build job should be the one tested.

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.

Additionally when executing the tool it would be nice to use the exitcode to confirm whether the pandoc tool failed or succeeded.

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 instead of NuGet Fix integration test to install built package with explicit version Jan 8, 2026
Copilot AI requested a review from Malcolmnixon January 8, 2026 20:26
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI requested a review from Malcolmnixon January 8, 2026 20:31
@Malcolmnixon Malcolmnixon marked this pull request as ready for review January 9, 2026 00:41
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