Skip to content

Commit

Permalink
Fix for workflow Upload Playwright report in ci/cd (#809)
Browse files Browse the repository at this point in the history
* Fix for workflow Upload Playwright report in ci/cd

* updated the build-rc.yml post-comment
  • Loading branch information
DuskaT021 authored Dec 5, 2024
1 parent bb74e6a commit b2d4adb
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
13 changes: 10 additions & 3 deletions .github/workflows/build-rc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ jobs:
path: ./build
retention-days: 5
if-no-files-found: error
overwrite: true
UItest:
needs: [build]
name: E2E Test ${{ matrix.shardIndex }} of ${{ matrix.shardTotal }}
Expand Down Expand Up @@ -90,9 +91,13 @@ jobs:
if: ${{ !cancelled() }}
uses: actions/upload-artifact@v4
with:
name: all-blob-reports
name: blob-report-${{ matrix.shardIndex }}-of-${{ matrix.shardTotal }}
path: blob-report
retention-days: 1
if-no-files-found: warn
compression-level: 6
overwrite: true
include-hidden-files: false

merge-reports:
# Merge reports after playwright-tests, even if some shards have failed but not if the job was skipped
Expand All @@ -116,8 +121,9 @@ jobs:
- name: Download blob reports from GitHub Actions Artifacts
uses: actions/download-artifact@v4
with:
name: all-blob-reports
pattern: blob-report-*
path: all-blob-reports
merge-multiple: true

- name: Merge into HTML Report
run: npx playwright merge-reports --reporter html ./all-blob-reports
Expand All @@ -128,6 +134,7 @@ jobs:
name: html-report--attempt-${{ github.run_attempt }}
path: playwright-report
retention-days: 4
overwrite: true

publish-rc:
# TODO also keep the develop PR description up to date
Expand Down Expand Up @@ -206,7 +213,7 @@ jobs:
- name: zip
run: zip -rj build.zip ./build
- name: Upload release asset
uses: actions/upload-release-asset@v1
uses: actions/upload-release-asset@v1 # TODO: use softprops/action-gh-release@v1
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
UPLOAD_URL: ${{needs.publish-rc.outputs.upload_url}}
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/playwright.yml
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ jobs:
if: ${{ !cancelled() }}
uses: actions/upload-artifact@v4
with:
name: all-blob-reports
name: blob-report-${{ matrix.shardIndex }}-of-${{ matrix.shardTotal }}
path: blob-report
retention-days: 1
overwrite: true
Expand All @@ -111,8 +111,9 @@ jobs:
- name: Download blob reports from GitHub Actions Artifacts
uses: actions/download-artifact@v4
with:
name: all-blob-reports
pattern: blob-report-*
path: all-blob-reports
merge-multiple: true

- name: Merge into HTML Report
run: npx playwright merge-reports --reporter html ./all-blob-reports
Expand Down

0 comments on commit b2d4adb

Please sign in to comment.