Skip to content

Commit

Permalink
Fix report merging
Browse files Browse the repository at this point in the history
  • Loading branch information
RXminuS committed Oct 21, 2024
1 parent 7984af7 commit f34d719
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
8 changes: 7 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -276,9 +276,11 @@ jobs:
pattern: e2e-blob-report-*
merge-multiple: true
- name: Merge into HTML Report
run: pnpm -C vscode exec playwright merge-reports --reporter html .test/e2e/reports/blob
run: pnpm -C vscode exec playwright merge-reports --reporter html,json -c .test/e2e/reports/blob .test/e2e/reports/blob
env:
PLAYWRIGHT_HTML_OUTPUT_DIR: .test/e2e/reports/html
PLAYWRIGHT_JSON_OUTPUT_NAME: report.json
PLAYWRIGHT_JSON_OUTPUT_DIR: .test/e2e/reports/json
- name: Upload HTML report
uses: actions/upload-artifact@v4
with:
Expand All @@ -293,6 +295,10 @@ jobs:
# comes to infer the branch name on GitHub action.
BRANCH_NAME: ${{ github.head_ref || github.ref_name }}
GITHUB_REF: ${{ github.head_ref || github.ref_name }}
- name: Comment on PR
uses: daun/playwright-report-summary@v3
with:
report-file: vscode/.test/e2e/reports/json/report.json

# Sets a variable that is used to determine the matrix to run slow tests (e2e) on.
# Everything runs on ubuntu, only commits to main run on macos and windows.
Expand Down
3 changes: 3 additions & 0 deletions vscode/e2e/utils/merge.config.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
export default {
testDir: '/cody/vscode',
}

0 comments on commit f34d719

Please sign in to comment.