next-exp/IC

Reorganize, simplify and clean up fixtures

Open

#913 aperta il 30 ott 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Python (77 fork)auto 404
easygood first issuehelp wantedtesting

Metriche repository

Star
 (10 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

We use many fixtures. Most of them are thrown into invisible_cities/conftest.py without thinking much. Let's reorganize them following the following logic:

  • If they are used only in one file, move them to that file
  • If they are used only in one folder, move them to the conftest.py file in that folder
  • Otherwise, keep them in invisible_cities/conftest.py

Moreover, some of those fixtures are monstrous. Let's try to simplify them or replace them with test files whenever possible to keep things readable.

Guida contributor

Direzione di ricerca
Identifica le fixture utilizzate solo in un file e spostale in quel file; identifica le fixture utilizzate solo in una cartella e spostale nel conftest.py di quella cartella; altrimenti mantienile nel conftest.py radice. Per le fixture mostruose, considera di semplificarle o sostituirle con file di test.
Tech stack
python
Dominio
testing qa
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
Pythonpytest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70