Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

[DevOps] Merged HTML test report of a fork PR includes runner files named in its blob reports

Đã đóng
  • Có gì không ổn?
#43,702 4 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@yury-s đang làm issue này rồi.

Từ ngày 27/10/2026.

  • #43693 của @yury-s — đang mở

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
35/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
github-actions, playwright, typescript
Lĩnh vực
ci-cd, security, testing

Hướng nghiên cứu

Read .github/workflows/create_test_report.yml and .github/workflows/merge.config.ts to trace how fork PR blob reports reach the HTML reporter, then inspect packages/playwright/src/reporters/html.ts, especially createSnippets and createErrorCodeframe. Reproduce with blob data whose location.file points outside the extracted blob folder. Done means the merged HTML report includes only blob report data and does not read runner files; linked PR #43693 is already open.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Version

main (1.65.0-next)

Steps to reproduce
  1. In a pull request from a fork, make the test run upload a blob report in which a test step or an error has a location.file outside the checkout, for example an absolute path on the runner.
  2. When tests 1 completes, the "Publish Test Results" workflow (.github/workflows/create_test_report.yml) downloads the blob reports, runs merge-reports with the html reporter (.github/workflows/merge.config.ts), and uploads playwright-report/index.html as an artifact.
Expected behavior

The published report contains only data from the blob reports.

Actual behavior

While it builds the report, the html reporter reads files on the runner at the locations from the blob: createSnippets for step locations and createErrorCodeframe for error locations (packages/playwright/src/reporters/html.ts). The text of those files goes into the uploaded report. A fork PR controls its blob reports, so it chooses which runner files are read.

Attachment paths are not affected: merge limits them to the extracted blob folder. The markdown report reads no files.

Ngôn ngữ chính
TypeScript
Star
98.5k
Fork
6.7k
Merge trung bình
16 giờ 10 phút
Pull request đã merge (30 ngày)
305

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của microsoft/playwright

Tất cả issue của microsoft/playwright

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.