Skip to content

Commit

Permalink
Fix report merging in E2E tests
Browse files Browse the repository at this point in the history
  • Loading branch information
mmuzikar committed Dec 20, 2023
1 parent ac32dd6 commit cfc6fa2
Showing 1 changed file with 17 additions and 10 deletions.
27 changes: 17 additions & 10 deletions .github/workflows/e2e-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ on:
push:
branches:
- main
- mmuzikar-ci-test-branch
paths-ignore:
- '**.md'

Expand Down Expand Up @@ -142,7 +143,7 @@ jobs:
cp $REPORT_DIR/cucumber-reports/* results/$REPORT_DIR/
ls $REPORT_DIR/cucumber-reports/
- name: Archive test artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: always()
with:
name: 'test-${{ env.REPORT_DIR }}'
Expand All @@ -151,13 +152,13 @@ jobs:
${{ env.REPORT_DIR }}/*.log
${{ env.REPORT_DIR }}/cucumber-reports/*
- name: Archive failed test reports
uses: actions/upload-artifact@v3
if: always()
with:
name: 'test-results'
path: |
results/**/*
# - name: Archive failed test reports
# uses: actions/upload-artifact@v4
# if: always()
# with:
# name: 'test-results'
# path: |
# results/**/*

publish-results:
runs-on: ubuntu-latest
Expand All @@ -169,17 +170,23 @@ jobs:
pull-requests: write
steps:
- name: Download Test Results
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: 'test-results'
pattern: 'test-*'
path: 'test-results'
merge-multiple: true
- name: Publish Test Results
uses: EnricoMi/publish-unit-test-result-action@v2
if: always()
with:
files: |
**/Cucumber.xml
json_file: results.json
- run: |
ls **/Cucumber.xml
cat **/Cucumber.xml
cat results.json
- name: Install xmllint
run: sudo apt-get install -y xmlstarlet jq
- name: Generate summary
Expand Down

0 comments on commit cfc6fa2

Please sign in to comment.