Skip to content

Reorganiza os testes do app location#1450

Merged
robertatakenaka merged 1 commit into
scieloorg:mainfrom
robertatakenaka:fix_location_tests
Jul 22, 2026
Merged

Reorganiza os testes do app location#1450
robertatakenaka merged 1 commit into
scieloorg:mainfrom
robertatakenaka:fix_location_tests

Conversation

@robertatakenaka

Copy link
Copy Markdown
Member

O que esse PR faz?

Reorganiza os testes do app location em um pacote (location/tests/).
O arquivo original é movido, sem alteração de conteúdo, para
location/tests/tests.py, e um novo location/tests/test_mixins.py é
criado com fixtures/mixins reutilizáveis para Country, State, City
e Location.

A motivação principal: location é dependência comum de outros apps, que
hoje recriam suas próprias fixtures de Country/State/City/Location
de forma duplicada e potencialmente inconsistente. Centralizar essa lógica
em location/tests/test_mixins.py permite que outros apps (ex.:
organization) a reutilizem diretamente.

Onde a revisão poderia começar?

location/tests/test_mixins.py, para conferir as fixtures/mixins
disponibilizados. Em seguida, location/tests/__init__.py, apenas para
confirmar que o pacote foi formalizado corretamente.

Como este poderia ser testado manualmente?

  1. Rodar python manage.py test location.tests e confirmar que os testes
    movidos continuam passando sem alteração de comportamento.
  2. Confirmar que location/tests.py não existe mais na raiz do app
    (apenas location/tests/tests.py).

Algum cenário de contexto que queira dar?

Este é o primeiro de uma série de PRs menores que reorganizam a suíte de
testes de vários apps do projeto. location vem primeiro porque outros
PRs desta série (ex.: organization) dependem dos mixins criados aqui.

Screenshots

Não aplicável.

Quais são os tickets relevantes?

Relacionado com #1449
Complementar a ...

Referências

Commit b9a077f5 — "Cria location/tests, move location/tests.py para
tests e cria test_mixins".


Segurança da informação (NSI.04)

Este PR manipula dados sensíveis ou pessoais (LGPD)?

  • Não

Este PR altera autenticação, autorização, controle de acesso ou gerenciamento de sessão?

  • Não

Este PR introduz, atualiza ou remove dependências de terceiros?

  • Não

Este PR foi validado pelo pipeline de segurança (SonarQube / Trivy)?

  • Não aplicável a este PR (justifique): reorganização de arquivos de teste, sem alteração de lógica de produção.

Este PR concatena, monta ou executa comandos SQL, HTML ou JavaScript a partir de entrada externa?

  • Não

Este PR expõe novos endpoints, telas ou serviços?

  • Não

Algum segredo, senha, chave ou token está sendo adicionado ao código-fonte?

  • Não, nenhum segredo foi commitado

@robertatakenaka
robertatakenaka merged commit ee67cf3 into scieloorg:main Jul 22, 2026
3 of 5 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.

1 participant