Skip to content

Commit d5d218e

Browse files
🩹 [Patch]: Simplify environment variable names in Action-Test workflow and related tests for consistency
1 parent 846a33f commit d5d218e

File tree

2 files changed

+20
-20
lines changed

2 files changed

+20
-20
lines changed

‎.github/workflows/Action-Test.yml‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -157,16 +157,16 @@ jobs:
157157
if: always()
158158
runs-on: ubuntu-latest
159159
env:
160-
ActionTestSrcSourceCodeOutcome: ${{ needs.ActionTestSrcSourceCode.outputs.Outcome }}
161-
ActionTestSrcSourceCodeConclusion: ${{ needs.ActionTestSrcSourceCode.outputs.Conclusion }}
162-
ActionTestSrcCustomOutcome: ${{ needs.ActionTestSrcCustom.outputs.Outcome }}
163-
ActionTestSrcCustomConclusion: ${{ needs.ActionTestSrcCustom.outputs.Conclusion }}
164-
ActionTestSrcWithManifestOutcome: ${{ needs.ActionTestSrcWithManifest.outputs.Outcome }}
165-
ActionTestSrcWithManifestConclusion: ${{ needs.ActionTestSrcWithManifest.outputs.Conclusion }}
166-
ActionTestSrcWithManifestDefaultOutcome: ${{ needs.ActionTestSrcWithManifestDefault.outputs.Outcome }}
167-
ActionTestSrcWithManifestDefaultConclusion: ${{ needs.ActionTestSrcWithManifestDefault.outputs.Conclusion }}
168-
ActionTestOutputsOutcome: ${{ needs.ActionTestOutputs.outputs.Outcome }}
169-
ActionTestOutputsConclusion: ${{ needs.ActionTestOutputs.outputs.Conclusion }}
160+
SourceCodeOutcome: ${{ needs.ActionTestSrcSourceCode.outputs.Outcome }}
161+
SourceCodeConclusion: ${{ needs.ActionTestSrcSourceCode.outputs.Conclusion }}
162+
CustomOutcome: ${{ needs.ActionTestSrcCustom.outputs.Outcome }}
163+
CustomConclusion: ${{ needs.ActionTestSrcCustom.outputs.Conclusion }}
164+
WithManifestOutcome: ${{ needs.ActionTestSrcWithManifest.outputs.Outcome }}
165+
WithManifestConclusion: ${{ needs.ActionTestSrcWithManifest.outputs.Conclusion }}
166+
WithManifestDefaultOutcome: ${{ needs.ActionTestSrcWithManifestDefault.outputs.Outcome }}
167+
WithManifestDefaultConclusion: ${{ needs.ActionTestSrcWithManifestDefault.outputs.Conclusion }}
168+
OutputsOutcome: ${{ needs.ActionTestOutputs.outputs.Outcome }}
169+
OutputsConclusion: ${{ needs.ActionTestOutputs.outputs.Conclusion }}
170170
steps:
171171
- name: Checkout repo
172172
uses: actions/checkout@v5

‎tests/Get-AggregatedStatus.ps1‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,10 @@ $WithManifestDefaultOutcomeResult = $env:WithManifestDefaultOutcome -eq $WithMan
4040
$WithManifestDefaultExpectedConclusion = 'success'
4141
$WithManifestDefaultConclusionResult = $env:WithManifestDefaultConclusion -eq $WithManifestDefaultExpectedConclusion
4242

43-
$ActionTestOutputsExpectedOutcome = 'success'
44-
$ActionTestOutputsOutcomeResult = $env:ActionTestOutputsOutcome -eq $ActionTestOutputsExpectedOutcome
45-
$ActionTestOutputsExpectedConclusion = 'success'
46-
$ActionTestOutputsConclusionResult = $env:ActionTestOutputsConclusion -eq $ActionTestOutputsExpectedConclusion
43+
$OutputsExpectedOutcome = 'success'
44+
$OutputsOutcomeResult = $env:OutputsOutcome -eq $OutputsExpectedOutcome
45+
$OutputsExpectedConclusion = 'success'
46+
$OutputsConclusionResult = $env:OutputsConclusion -eq $OutputsExpectedConclusion
4747

4848
$jobs = @(
4949
[PSCustomObject]@{
@@ -84,12 +84,12 @@ $jobs = @(
8484
},
8585
[PSCustomObject]@{
8686
Name = 'Action-Test - [outputs]'
87-
Outcome = $env:ActionTestOutputsOutcome
88-
ExpectedOutcome = $ActionTestOutputsExpectedOutcome
89-
PassedOutcome = $ActionTestOutputsOutcomeResult
90-
Conclusion = $env:ActionTestOutputsConclusion
91-
ExpectedConclusion = $ActionTestOutputsExpectedConclusion
92-
PassedConclusion = $ActionTestOutputsConclusionResult
87+
Outcome = $env:OutputsOutcome
88+
ExpectedOutcome = $OutputsExpectedOutcome
89+
PassedOutcome = $OutputsOutcomeResult
90+
Conclusion = $env:OutputsConclusion
91+
ExpectedConclusion = $OutputsExpectedConclusion
92+
PassedConclusion = $OutputsConclusionResult
9393
}
9494
)
9595

0 commit comments

Comments
 (0)