Summary
The Acquisition test project (tests/Aspire.Acquisition.Tests) includes outerloop integration tests (PRScriptIntegrationTests) that require GH_TOKEN to call real GitHub APIs. Currently, the token is not provided to these tests when run in the outerloop workflow.
In PR #15995, we removed the || inputs.testShortName == 'Acquisition' special-case from run-tests.yml to avoid unnecessarily exposing GH_TOKEN to the normal (non-outerloop) Acquisition tests, which use mock gh scripts and don't need the token.
What's Needed
Once #16056 (generic properties bag for test pipeline) is merged, add a requiresGhToken MSBuild property to the Acquisition test project so the outerloop workflow can provide GH_TOKEN without hardcoding test names in the workflow YAML.
Blocked By
Context
Summary
The Acquisition test project (
tests/Aspire.Acquisition.Tests) includes outerloop integration tests (PRScriptIntegrationTests) that requireGH_TOKENto call real GitHub APIs. Currently, the token is not provided to these tests when run in the outerloop workflow.In PR #15995, we removed the
|| inputs.testShortName == 'Acquisition'special-case fromrun-tests.ymlto avoid unnecessarily exposingGH_TOKENto the normal (non-outerloop) Acquisition tests, which use mockghscripts and don't need the token.What's Needed
Once #16056 (generic properties bag for test pipeline) is merged, add a
requiresGhTokenMSBuild property to the Acquisition test project so the outerloop workflow can provideGH_TOKENwithout hardcoding test names in the workflow YAML.Blocked By
Context
GH_TOKENis absent (Assert.Skip), so this is not blocking any CI runs.