Skip to content

Commit

Permalink
fix: merge playwright reports with assets
Browse files Browse the repository at this point in the history
Signed-off-by: Mason Hu <[email protected]>
  • Loading branch information
mas-who authored and edlerd committed Feb 20, 2024
1 parent 2b4ba8f commit 09f5662
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ jobs:
- uses: actions/setup-node@v4
with:
node-version: 18
node-version: 20

- name: Install Playwright Browsers
run: npx playwright install --with-deps ${{ matrix.browser }}
Expand Down Expand Up @@ -165,18 +165,18 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 18
node-version: 20

- name: Download blob reports from GitHub Actions Artifacts
uses: actions/download-artifact@v4
with:
path: playwright-report
path: blob-reports
pattern: blob-report-*
merge-multiple: true

# NOTE: there is no need to install playwright dependencies since we only need to merge reports here
- name: Merge into HTML Report
run: npx playwright merge-reports --reporter html ./playwright-report
run: npx playwright merge-reports --reporter html ./blob-reports

- name: Save additional test information
if: always()
Expand Down

0 comments on commit 09f5662

Please sign in to comment.