Skip to content

Commit

Permalink
merge: #9965
Browse files Browse the repository at this point in the history
9965: chore(ci): remove system-out from test output xml r=menski a=menski

## Description

Mitigate to large test output files by deleting the system-out element using sed.

Note: we cannot use XML tools like xmlstarlet as they all suffer the same libxml2 limitation

Also replaces the deprecated usage of `files` with `junit_files` in the publish test result action.
 
## Related issues

related #9959



Co-authored-by: Sebastian Menski <sebastian.menski@camunda.com>
  • Loading branch information
zeebe-bors-camunda[bot] and menski committed Aug 3, 2022
2 parents 77114d6 + 4b45d6f commit 0e6c55d
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/publish-test-results.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,19 @@ jobs:
unzip -d "$name" "$name.zip"
done
# Mitigate to large test output files by deleting the system-out element using sed
# Note: we cannot use XML tools like xmlstarlet as they all suffer the same libxml2 limitation
# Related to https://github.com/camunda/zeebe/issues/9959
- name: Remove system-out from test xml files
run: |
find . -iname TEST-*.xml -print0 | xargs -0 sed '/<system-out>/,/<\/system-out>/d' -i
- name: Publish Test Results
uses: EnricoMi/publish-unit-test-result-action@v2
with:
commit: ${{ github.event.workflow_run.head_sha }}
event_file: artifacts/Event File/event.json
event_name: ${{ github.event.workflow_run.event }}
files: |
junit_files: |
artifacts/**/surefire-reports/*.xml
artifacts/**/failsafe-reports/TEST-*.xml

0 comments on commit 0e6c55d

Please sign in to comment.