basedosdados / basedosdados/pipelines
fix: teste órfão de check_if_data_is_outdated_by_size quebra a suíte
- Dominant language
- Python
- Stars
- 49
- Forks
- 22
- Avg merge
- 19h 31m
- Merged PRs (30d)
- 165
Description
# Contexto
`pipelines/utils/tests/test_check_if_data_is_outdated_by_size_task.py`
falha na coleta do pytest com `ImportError`:
```
ImportError while importing test module '.../test_check_if_data_is_outdated_by_size_task.py'.
pipelines/utils/tests/test_check_if_data_is_outdated_by_size_task.py:7: in
from pipelines.utils.metadata.tasks import check_if_data_is_outdated_by_size
E ImportError: cannot import name 'check_if_data_is_outdated_by_size' from 'pipelines.utils.metadata.tasks'
```
Isso interrompe a coleta de **toda** a suíte (`pytest pipelines/utils/tests/`
já para em "1 error during collection", nenhum outro teste do diretório
roda).
# Causa
`check_if_data_is_outdated_by_size` foi renomeada pra
`register_source_poll_by_size_task` no commit `727806f3` ("feat:
introduz camada register_* para atualização de cobertura temporal"),
que também mudou a assinatura (ganhou `env: str = "dev"`) e o
comportamento (grava Poll/Update via Redis, não é mais só uma
checagem). O teste antigo nunca foi atualizado — ficou órfão desde
então.
# O que fazer
Reescrever `test_check_if_data_is_outdated_by_size_task.py` contra
`register_source_poll_by_size_task` (nova assinatura/comportamento —
ver `pipelines/utils/metadata/tasks.py:176`), ou removê-lo se a
cobertura já existir em outro lugar (ex.
`pipelines/utils/tests/metadata/`, que parece ser onde os testes da
camada `register_*` moraram depois do refactor).
Encontrado enquanto testava as mudanças da #1971 — rodei a suíte
localmente pra confirmar que não quebrei nada e esbarrei nisso, sem
relação com aquela mudança.
Contributor guide
Assessment
This issue has not been assessed yet.