Corrige tests de researcher#1452
Merged
robertatakenaka merged 2 commits intoJul 22, 2026
Merged
Conversation
Propósito test_researcher_id_create_with_wrong_lattes estava sem a instrução 'with self.assertRaises(...):' antes da chamada de ResearcherIds.get_or_create, deixando o bloco indentado sem nenhum with/if/for correspondente. Isso causa IndentationError ao importar o módulo, derrubando toda a suíte de testes de researcher (e não apenas este teste). Solução técnica Adicionada a linha 'with self.assertRaises(ValidationError):' antes da chamada de ResearcherIds.get_or_create(identifier='invalid_lattes_format', source_name='LATTES'), seguindo o mesmo padrão já usado em test_researcher_id_create_with_wrong_email e nos testes de ORCID inválido deste mesmo arquivo.
robertatakenaka
force-pushed
the
fix_researcher_and_editorialm_tests
branch
from
July 22, 2026 18:05
f0c5991 to
e2f28e0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O que esse PR faz?
Corrige
test_researcher_id_create_with_wrong_lattes, que testava porengano um identifier de e-mail (
"user.teste",source_name="EMAIL") —o mesmo cenário já coberto por
test_researcher_id_create_with_wrong_email— em vez de um identifier de Lattes em formato inválido. Corrigido para
identifier="invalid_lattes_format",source_name="LATTES", dando aoteste a cobertura real que seu nome promete.
Além disso, limpa problemas de estilo acumulados no arquivo: typo em
subTest, comentários redundantes,assertEqualduplicados, espaços embranco desnecessários, e corrige o
setUpdeAffiliationMixinTest, queusava
Country.acron2eLocation.objects.create— nenhum dos doiscorresponde à interface atual dos models — substituídos por
Country.acronymeLocation.create_or_update. Fixtures deacronymque ultrapassavam
max_length=20também foram corrigidas.Onde a revisão poderia começar?
test_researcher_id_create_with_wrong_lattes, para confirmar que o novoidentifier (
"invalid_lattes_format") realmente é um formato de Lattesinválido e que a exceção esperada (
ValidationError) é a levantada pelavalidação real. Em seguida,
AffiliationMixinTest.setUp, para confirmarque os parâmetros usados batem com a assinatura atual de
CountryeLocation.Como este poderia ser testado manualmente?
python manage.py test researchere confirmar que todos ostestes passam, incluindo
test_researcher_id_create_with_wrong_lattese
test_researcher_id_create_with_wrong_emailseparadamente.AffiliationMixinTestpassa com os novos parâmetros deCountry/Location.Algum cenário de contexto que queira dar?
Esse bug (teste de Lattes testando e-mail) só ficou claro ao comparar a
posição do trecho alterado com o restante do arquivo — o nome do método
não deixava dúvida sobre a intenção, mas o corpo testava outra coisa.
Aproveitou-se a limpeza para resolver, no mesmo PR, os pequenos problemas
de estilo e a desatualização do
setUpdeAffiliationMixinTestemrelação aos models atuais.
Screenshots
Não aplicável.
Quais são os tickets relevantes?
Relacionado com #1450, #1451
Referências
Comparação com
test_researcher_id_create_with_wrong_email, no mesmoarquivo.
Segurança da informação (NSI.04)
Este PR manipula dados sensíveis ou pessoais (LGPD)?
Este PR altera autenticação, autorização, controle de acesso ou gerenciamento de sessão?
Este PR introduz, atualiza ou remove dependências de terceiros?
Este PR foi validado pelo pipeline de segurança (SonarQube / Trivy)?
Este PR concatena, monta ou executa comandos SQL, HTML ou JavaScript a partir de entrada externa?
Este PR expõe novos endpoints, telas ou serviços?
Algum segredo, senha, chave ou token está sendo adicionado ao código-fonte?