"# E:" lines (etc) in test files are ignored at the beginning of lines
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 20.6k
- Fork
- 3.3k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
As discovered in the unrelated https://github.com/python/mypy/pull/19494#discussion_r2243899624, "# E:" lines (etc) in test files are ignored at the beginning of lines. Having looked at the code, I've discovered this is because it desires the # to be preceded by a space, which is not present when the # is at the beginning of a line. This is bad because attempted error lines should not be silently ignored. (Although it is unlikely for someone to try to E an empty line.)
My PR #19546 will fix this. Edit: that would still be true, but #21172 is a much more minimal fix for this.
Easy repro:
[case testErrorCodeOnEmptyLine]
# E: This isn't a real message, and thus this test could never pass, although somehow it currently does :)
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia da mypy/test/data.py, intorno alla riga 536, e riproduci il comportamento usando il test case mostrato nell’issue. Confronta la correzione minima proposta nell’issue #21172, poiché il corpo indica che l’approccio più ampio di PR #19546 non è più quello preferito. Il lavoro è concluso quando l’assertion non valida “# E:” all’inizio della riga non viene più ignorata silenziosamente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- testing-qa
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 20/100