Skip to content

fix: remove testes órfãos e corrige incompatibilidade Flask-WTF/Werkzeug - #1277

Merged
robertatakenaka merged 1 commit into
scieloorg:masterfrom
Rossi-Luciano:fix/1274-remove-orphan-tests-and-webapp-extras
Aug 11, 2026
Merged

fix: remove testes órfãos e corrige incompatibilidade Flask-WTF/Werkzeug#1277
robertatakenaka merged 1 commit into
scieloorg:masterfrom
Rossi-Luciano:fix/1274-remove-orphan-tests-and-webapp-extras

Conversation

@Rossi-Luciano

Copy link
Copy Markdown
Contributor

O que esse PR faz?

Resolve #1274 e cobre a subtarefa "Garantir que todos os testes rodam antes de começar" de #1251: até agora, pytest não rodava nenhum teste — a coleção era interrompida por 4 erros de import antes de executar qualquer coisa.

  1. Remove 3 arquivos de teste órfãos que importavam módulos removidos em refactors anteriores (Refactor: Remove módulo obsoleto #839, Modifica utilitario xml_content_validation e dependências e renomeia para data_checker #870, Aprimora geração de PDFs (tabelas e figuras) #1033) — investiguei cada um antes de remover; detalhes e conclusões em Testes órfãos bloqueiam a coleção completa do pytest (3 módulos importam código removido em #839/#870/#1033) #1274.
  2. Corrige os pins de Flask-WTF/Werkzeug em setup.py (extras webapp/testes) e optional-requirements.txt, que estavam incompatíveis entre si assim que os extras eram de fato instalados: Werkzeug>=3.0 remove o atributo __version__ usado internamente pelo Flask-Testing, e Flask-WTF==1.1.0 usa werkzeug.urls.url_encode, também removido nas versões novas do Werkzeug.

Com isso, a suíte completa passa a coletar e rodar de ponta a ponta: 5973 passed, 40 failed, 30 skipped. As 40 falhas são pré-existentes e não relacionadas a este PR — confirmei rodando os mesmos testes via git stash direto em master, sem nenhuma mudança deste PR aplicada, e reproduzem de forma idêntica. Documentadas separadamente em #1276 (não bloqueante).

Onde a revisão poderia começar?

  • setup.pyEXTRAS_REQUIRE['webapp'] e TESTS_REQUIRE.
  • optional-requirements.txt.
  • Os 3 arquivos removidos (o diff já mostra o que cada um tentava importar).

Como este poderia ser testado manualmente?

pip install -r requirements.txt
pip install -e .[webapp] "Flask-Testing>=0.6.2"
pytest -q

Antes deste PR: Interrupted: 4 errors during collection, zero testes executados.
Depois: 5973 passed, 40 failed, 30 skipped (falhas pré-existentes, ver #1276).

Algum cenário de contexto que queira dar?

Trabalho de investigação/estratégia para #1251 (atualização de dependências). Antes de bumpar qualquer pacote, era necessário ter uma baseline confiável de "os testes rodam" — sem isso, não dá pra saber se uma falha futura foi causada por um bump ou já existia.

Quais são tickets relevantes?

Closes #1274
Relacionado a #1251, #1276

Referências

Achados detalhados em #1274 (cobertura preservada vs. perdida por arquivo) e #1276 (lista completa das 40 falhas pré-existentes).

…eug (scieloorg#1274)

Remove 3 arquivos de teste que importavam módulos removidos em refactors
anteriores (scieloorg#839, scieloorg#870, scieloorg#1033) e bloqueavam a coleção completa do pytest.
Corrige também os pins de Flask-WTF/Werkzeug em setup.py e
optional-requirements.txt: Werkzeug>=3.0 remove `__version__`, quebrando
Flask-Testing; Flask-WTF==1.1.0 usa `werkzeug.urls.url_encode`, removido
nas versões mais novas do Werkzeug.
@robertatakenaka
robertatakenaka merged commit 4cbc3ff into scieloorg:master Aug 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Testes órfãos bloqueiam a coleção completa do pytest (3 módulos importam código removido em #839/#870/#1033)

2 participants