Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

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

Chiusa
  • Qualcosa non va?
#43,702 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@yury-s ci sta già lavorando.

Dal 27/10/2026.

  • #43693 di @yury-s — aperta

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
github-actions, playwright, typescript
Ambito
ci-cd, security, testing

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
TypeScript
Stelle
98.5k
Fork
6.7k
Merge medio
16h 10m
PR unite (30g)
305

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di microsoft/playwright

Tutte le issue di microsoft/playwright

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.