Skip to content

Commit

Permalink
Setup Windows test result artifact.
Browse files Browse the repository at this point in the history
  • Loading branch information
jsirois committed Feb 9, 2025
1 parent a78408d commit dba1f50
Showing 1 changed file with 16 additions and 3 deletions.
19 changes: 16 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -231,16 +231,19 @@ jobs:
tox-env: typecheck,package,docs
- python-version: [ 3, 13 ]
tox-env: py313-pip25_0
artifact-name: unit-tests
cache: true
pex-test-pos-args: ${{ needs.setup.outputs.pex_test_posargs }}
pex-test-pos-args: ${{ needs.setup.outputs.pex_test_posargs }} --junit-xml dist/reports/tests-unit.xml
- python-version: [ 3, 13 ]
tox-env: py313-pip25_0-integration
artifact-name: integration-tests-1
cache: true
pex-test-pos-args: --shard 1/2 ${{ needs.setup.outputs.pex_test_posargs }}
pex-test-pos-args: --shard 1/2 ${{ needs.setup.outputs.pex_test_posargs }} --junit-xml dist/reports/tests-integration-1.xml
- python-version: [ 3, 13 ]
tox-env: py313-pip25_0-integration
artifact-name: integration-tests-2
cache: true
pex-test-pos-args: --shard 2/2 ${{ needs.setup.outputs.pex_test_posargs }}
pex-test-pos-args: --shard 2/2 ${{ needs.setup.outputs.pex_test_posargs }} --junit-xml dist/reports/tests-integration-2.xml
steps:
- name: Checkout Pex
uses: actions/checkout@v4
Expand Down Expand Up @@ -276,6 +279,11 @@ jobs:
path: repo/tox.ini
tox-env: >-
${{ matrix.tox-env }} -- ${{ matrix.pex-test-pos-args }}
- uses: actions/upload-artifact@v4
if: always() && matrix.artifact-name
with:
name: test-results-${{ matrix.artifact-name }}
path: 'dist/reports/tests-*.xml'
- name: Cache Devpi Server
uses: actions/cache/save@v4
if: matrix.cache && github.ref == 'refs/heads/main'
Expand All @@ -291,6 +299,11 @@ jobs:
- windows-tests
runs-on: ubuntu-24.04
steps:
- uses: actions/download-artifact@v4
- uses: actions/upload-artifact@v4
with:
name: test-results
path: 'dist/reports/tests-*.xml'
- name: Check Non-Success
if: |
contains(needs.*.result, 'cancelled') ||
Expand Down

0 comments on commit dba1f50

Please sign in to comment.