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

No test covers the web profiler's buildHtmlReport()/downloadHtmlReport() output

Chiusa Adatta ai principianti
  • Qualcosa non va?
#765 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript, python
Ambito
frontend, testing

Direzione di ricerca

Inizia da assets/profiler-ui.js, in particolare da buildHtmlReport() e dal percorso di downloadHtmlReport(), quindi leggi i controlli esistenti a livello di codice sorgente in tests/test_js_parity.py. Aggiungi una copertura che verifichi nel generatore di report la presenza di blocchi di dimensioni, di una sezione flags e della struttura di proxy-hints, quindi esegui i test di parità pertinenti per confermare che la lacuna sia coperta.

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

Descrizione

enhancement good first issue

Where: assets/profiler-ui.js - buildHtmlReport() (line 678); tests/test_js_parity.py, tests/test_report.py.

Current gap: faircode/report.py::to_html() has 8+ targeted Python tests (test_to_html_renders_key_figures, test_to_html_renders_proxy_hints, test_to_html_reports_groups_omitted_by_display_cap, etc.), but the web profiler's equivalent - the actual HTML file a browser user downloads via the "⬇ Download report (HTML)" button - has no test at all. scripts/engine-js.js (the Node bridge tests/test_js_parity.py uses) only exposes profile/profile-json/profile-xlsx/compare; it never touches profiler-ui.js, where buildHtmlReport actually lives. This gap is exactly how a bug like "proxy hints silently missing from the downloaded HTML report" (see the companion issue on this) can ship unnoticed.

Repro:

$ grep -rn "downloadHtmlBtn\|buildHtmlReport\|downloadHtmlReport" tests/*.py
(zero matches)

Suggested approach: add a source-level test in tests/test_js_parity.py asserting buildHtmlReport's function body includes the same structural pieces to_html() does (dimension blocks, flags section, and a proxy-hints section once that gap is fixed), following the existing source-level-check precedent already used for DOM-coupled JS in this file (e.g. test_compare_card_renderers_special_case_kind_mismatch, test_proxy_hints_ui_wiring_present_in_profiler_html_and_ui_js).

Lingua principale
HTML
Stelle
47
Fork
45
Merge medio
3h 2m
PR unite (30g)
85

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 yakew7/Fair-Code

Tutte le issue di yakew7/Fair-Code

Issue simili

Altre issue su Web Dev

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.