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

check_ui.sh: refuse an unknown test module before building anything

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

I maintainer di solito rispondono entro 1 giorno

Una pull request collegata è già stata integrata.

  • #543 di @PelikanFix16 — integrata

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
85/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
bash, docker, python
Ambito
developer-experience, testing

Direzione di ricerca

Inizia da scripts/check_ui.sh: il commento sugli argomenti si trova alle righe 6-7, e il controllo anticipato esistente di FLUX_UI_BROWSER alle righe 11-17 mostra il modello da seguire prima di qualsiasi comando docker. Gli argomenti arrivano a python3 -m unittest alle righe 64-68, che vengono eseguiti in app/tests/ui/. È completato quando un modulo sconosciuto come test_doc termina con stato 2 e il messaggio su stderr prima che Docker si avvii, test_docs e Module.Class.test continuano a funzionare e il commento di intestazione menziona il rifiuto anticipato.

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

Descrizione

good first issue

./scripts/check_ui.sh test_doc (a typo for test_docs) builds the images and starts the Compose stack before unittest fails with a missing-module error, several minutes later. The script already rejects a wrong FLUX_UI_BROWSER before Docker starts; module names should get the same early check.

Where to look

  • scripts/check_ui.sh (lines 6–7): the comment that describes the arguments (test_docs, or Module.Class.test).
  • scripts/check_ui.sh (lines 11–17): the existing early FLUX_UI_BROWSER check, which runs before Docker or temporary secrets.
  • scripts/check_ui.sh (lines 64–68): where the arguments reach python3 -m unittest in the ui-test container (working directory /work/tests/ui, which is app/tests/ui/ in the repository).

Acceptance criteria

  • For each argument, the module part (the text before the first .) must be a file app/tests/ui/<module>.py; otherwise the script prints unknown UI test module: <module> to stderr and exits with status 2 before any docker command runs.
  • Module.Class.test arguments and running with no arguments keep working as today.
  • The header comment says that unknown modules are refused early.

How to check
./scripts/check_ui.sh test_doc exits at once with the message and no Docker activity (docker ps shows no new flux-ui-* project). ./scripts/check_ui.sh test_docs still runs that module. Then the repository checks: python3 scripts/check_agent_setup.py, python3 -m unittest discover -s tests -p 'test_*.py', git diff --check.

New to Flux? Start with CONTRIBUTING. Comment here to say you are taking it.

Lingua principale
TypeScript
Stelle
3
Fork
0
Merge medio
19h 44m
PR unite (30g)
367

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 ColdPhase/flux

Tutte le issue di ColdPhase/flux

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.