Skip to content

Commit 850e2aa

Browse files
committed
Integrate build-number test into existing test matrix.
1 parent 780c46a commit 850e2aa

File tree

1 file changed

+4
-26
lines changed

1 file changed

+4
-26
lines changed

Diff for: .github/workflows/test.yml

+4-26
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
strategy:
1515
matrix:
1616
os: [ubuntu-latest, windows-latest, macos-latest]
17-
action: [report, done] # Note: We're also testing 'install' since it's implicitly in each of these two actions
17+
action: [report, done, build-number] # Note: We're also testing 'install' since it's implicit in each action
1818
fail_on_error: [true, false]
1919
steps:
2020
- name: Checkout code
@@ -36,36 +36,14 @@ jobs:
3636
run: |
3737
echo "Running on Windows"
3838
39-
- name: Run Test Action
39+
- name: Test Action
4040
uses: ./
4141
with:
4242
github-token: ${{ secrets.GITHUB_TOKEN }}
4343
fail-on-error: ${{ matrix.fail_on_error }}
4444
debug: true
45-
parallel-finished: ${{ matrix.action == 'done' }} # Only set 'parallel-finished' to true when testing 'done'
45+
build-number: ${{ matrix.action == 'build-number' && github.sha || '' }} # Only set 'build-number' to `${{ github.sha }}` when testing `build-number`
46+
parallel-finished: ${{ matrix.action == 'done' }} # Only set `parallel-finished` to `true` when testing `done`
4647
env:
4748
CI: true
4849
continue-on-error: ${{ matrix.fail_on_error }}
49-
50-
- name: Test report with build-number input option
51-
if: ${{ matrix.action == 'report' }}
52-
uses: ./
53-
with:
54-
github-token: ${{ secrets.GITHUB_TOKEN }}
55-
fail-on-error: ${{ matrix.fail_on_error }}
56-
debug: true
57-
build-number: ${{ github.sha }}
58-
env:
59-
CI: true
60-
61-
- name: Test done with build-number input option
62-
if: ${{ matrix.action == 'done' }}
63-
uses: ./
64-
with:
65-
github-token: ${{ secrets.GITHUB_TOKEN }}
66-
fail-on-error: ${{ matrix.fail_on_error }}
67-
debug: true
68-
parallel-finished: true
69-
build-number: ${{ github.sha }}
70-
env:
71-
CI: true

0 commit comments

Comments
 (0)