Skip to content

ci(github): add integration test support to build pipelines#18

Merged
ncipollina merged 2 commits intomainfrom
ci/github-add-integration-test-support
Mar 18, 2026
Merged

ci(github): add integration test support to build pipelines#18
ncipollina merged 2 commits intomainfrom
ci/github-add-integration-test-support

Conversation

@ncipollina
Copy link
Copy Markdown
Collaborator

Summary

  • Added integrationTestSolution input to build-deploy.yaml to support running integration tests after the CDK deploy step
  • Updated all dotnet test steps in pr-build.yaml to conditionally pass the solution input, enabling scoped test runs when multiple solution files exist at the root
  • Updated the Test Code step in build-deploy.yaml with the same solution-aware pattern

Changes

pr-build.yaml

  • All three test steps (Test Code, Test Code with Legacy Coverage, Test Code with MTP Coverage) now pass $solution to dotnet test when provided, falling back to the default discovery behavior when omitted

build-deploy.yaml

  • New integrationTestSolution input (optional)
  • Compute Build Flags now sets NEEDS_DOTNET=true when integrationTestSolution is provided
  • Test Code step updated to conditionally pass solution
  • Added Restore Integration Test Dependencies and Run Integration Tests steps after CDK deploy, both gated on integrationTestSolution != ''

Reviewer Notes

  • Integration tests only run in build-deploy (post-deploy), not in pr-build — intentional, as integration tests require a live environment
  • The ${SOLUTION:+"$SOLUTION"} pattern expands to the solution path when set, and to nothing when empty — preserves existing behavior for repos with a single solution at the root

Allows callers to pass a solution file to dotnet test in pr-build, and adds an integrationTestSolution input to build-deploy to run integration tests after CDK deploy.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

…l arg

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ncipollina ncipollina merged commit d352969 into main Mar 18, 2026
@ncipollina ncipollina deleted the ci/github-add-integration-test-support branch March 18, 2026 20:00
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.

1 participant