next-exp/IC

Reorganize, simplify and clean up fixtures

Open

#913 aberto em 30 de out. de 2024

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)Python (77 forks)auto 404
easygood first issuehelp wantedtesting

Métricas do repositório

Stars
 (10 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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.

Guia do colaborador

Direção de pesquisa
Identifique fixtures usadas apenas em um arquivo e mova as para esse arquivo; identifique fixtures usadas apenas em uma pasta e mova as para o conftest.py dessa pasta; caso contrário, mantenha as no conftest.py raiz. Para fixtures monstruosas, considere simplificá las ou substituí las por arquivos de teste.
Pilha de tecnologia
python
Domain
testing qa
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythonpytest
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70