aio-libs/multidict

[TODO] Replace `tests/gen_pickles.py` with a pytest plugin

Aberta

#922 aberto em 16 de jan. de 2024

Ver no GitHubGuia do colaborador
 (13 comentários) (2 reações) (1 responsável)Python (130 forks)auto 404
Hacktoberfestenhancementgood first issuehelp wanted

Métricas do repositório

Stars
 (491 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

The idea is simple:

  • Make a session-scoped fixture in tests/conftest.py
  • Move the logic from gen_pickles.py there and generate the pickles in a tmppath-provided dir
  • Integrate that into the pickling tests
  • Delete tests/gen_pickles.py and the pickle files in the tests/ dir
  • Bonus points: keep pickles in-memory, so they don't hit the disk
  • Bonus 2: check if a session fixture is needed and switch to the normal function scope, if not

Guia do colaborador

Direção de pesquisa
Leia `tests/gen pickles.py` e entenda a lógica de geração de pickles. Aprenda sobre fixtures do pytest, especialmente as de escopo de sessão e conftest.py. Em seguida, mova a lógica para uma fixture no conftest.py, integre com os testes existentes e, finalmente, remova o script antigo e os arquivos pickle.
Pilha de tecnologia
python
Domain
testing
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-3 horas
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.
75