basedosdados / basedosdados/pipelines

fix: teste órfão de check_if_data_is_outdated_by_size quebra a suíte

Open
#2,045 0 comments 0 reactions 1 assignee Claimed by @Winzen View on GitHub
bug
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.