astral-sh/ruff

INP001 false positive in tests folder

Open

#6474 aperta il 10 ago 2023

Vedi su GitHub
 (11 commenti) (8 reazioni) (0 assegnatari)Rust (2088 fork)batch import
help wanted

Metriche repository

Star
 (47.527 star)
Metriche merge PR
 (Merge medio 3g 8h) (573 PR mergiate in 30 g)

Descrizione

The recommended Python + Pytest layout is

  • src/
    • mypkg/
      • __init__.py
      • subpkg/
        • __init__.py
  • tests/
    • test_thing.py
    • test_more/
      • test_other_thing.py

Notably, tests and all files under it aren’t packages. The recommended import mode enforces that you can’t import from them.

Therefore INP001 shouldn’t trigger when tests is on top level of the project.

In fact, it would be great to have both a rule that enforces the opposite and to ensure that tests/**/test_*.py files don’t import things from each other.

Guida contributor

Direzione di ricerca
Indaga la logica della regola INP001 per determinare perché contrassegna tests/ come un pacchetto e modificala per trattare tests/ come una directory speciale di primo livello.
Tech stack
pythonrust
Dominio
developer experiencebackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonRust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70