feat: make integration-test workflow reusable via workflow_call - #42
Merged
Elizabeth Maher (EMaher) merged 9 commits intoMay 9, 2026
Merged
Conversation
- Add workflow_call trigger to integration-test.yml so it can be reused as a called workflow - Create publish.yml workflow that gates npm publish on the all-resource-types round-trip integration test passing first Closes #25 Agent-Logs-Url: https://github.com/Azure/apiops-cli/sessions/2d7b027b-8e3f-43e5-828d-fdca8fd0fc7a Co-authored-by: EMaher <9244742+EMaher@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add GitHub action for round trip integration test workflow
feat: gate npm publish on all-resource-types integration test
May 8, 2026
Elizabeth Maher (EMaher)
suggested changes
May 8, 2026
Closes #25 Agent-Logs-Url: https://github.com/Azure/apiops-cli/sessions/8127b66f-239b-4f7d-8cd9-47771d6f0284 Co-authored-by: EMaher <9244742+EMaher@users.noreply.github.com>
Copilot
AI
changed the title
feat: gate npm publish on all-resource-types integration test
feat: make integration-test workflow reusable via workflow_call
May 8, 2026
Agent-Logs-Url: https://github.com/Azure/apiops-cli/sessions/d2a8284f-2302-400f-ad67-d15f32b4a2e0 Co-authored-by: EMaher <9244742+EMaher@users.noreply.github.com>
Elizabeth Maher (EMaher)
had a problem deploying
to
integration-test
May 8, 2026 21:35 — with 
GitHub Actions
Failure
GitHub Actions
Failure
Elizabeth Maher (EMaher)
suggested changes
May 8, 2026
Agent-Logs-Url: https://github.com/Azure/apiops-cli/sessions/42168ff2-9b29-44ed-a16d-48322b6dd431 Co-authored-by: EMaher <9244742+EMaher@users.noreply.github.com>
Elizabeth Maher (EMaher)
temporarily deployed
to
integration-test
May 9, 2026 01:34 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Elizabeth Maher (EMaher)
approved these changes
May 9, 2026
Elizabeth Maher (EMaher)
marked this pull request as ready for review
May 9, 2026 01:51
Elizabeth Maher (EMaher)
requested review from
Copilot and
Peter Hauge (petehauge)
May 9, 2026 01:53
Contributor
There was a problem hiding this comment.
Pull request overview
This PR makes the integration test GitHub Actions workflow reusable via workflow_call, enabling other workflows to invoke the extract→publish round-trip integration test while explicitly declaring required secrets and inputs.
Changes:
- Added
workflow_calltrigger (with required secrets) to.github/workflows/integration-test.ymlalongside the existingworkflow_dispatch. - Updated the round-trip integration test script to default to hard-delete/purge behavior on teardown.
- Added a troubleshooting “skill” document describing OIDC + RBAC prerequisites for running the integration tests reliably.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/integration-test.yml |
Adds workflow_call reusability and enhances emergency teardown with RG wait + APIM purge. |
tests/integration/all-resource-types/run-roundtrip-test.ps1 |
Changes default teardown behavior to hard-delete unless explicitly overridden. |
.github/skills/integration-test-prerequisites/SKILL.md |
Documents setup/troubleshooting steps for OIDC federated creds, RBAC, and required secrets. |
Comment on lines
+63
to
+66
| # Default to hard-delete on teardown unless explicitly disabled. | ||
| if (-not $PSBoundParameters.ContainsKey('HardDelete')) { | ||
| $HardDelete = $true | ||
| } |
| foreach ($apimName in @($sourceApimName, $targetApimName)) { | ||
| if (-not [string]::IsNullOrWhiteSpace($apimName)) { | ||
| Write-Host "🗑️ Purging soft-deleted APIM: $apimName" | ||
| az apim deletedservice purge --service-name $apimName --location $location 2>$null |
Peter Hauge (petehauge)
approved these changes
May 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No reusable trigger existed on the integration test workflow to allow it to be called from other workflows.
Changes
.github/workflows/integration-test.ymlworkflow_calltrigger with explicitsecrets:block alongside the existingworkflow_dispatch, making it a reusable callable workflow for future publish gates or other automationRequired environment secrets (for
workflow_callcallers)AZURE_CLIENT_IDAZURE_TENANT_IDAZURE_SUBSCRIPTION_IDAPIM_PUBLISHER_EMAIL