Skip to content

Test(cliente_pncp): Adiciona testes unitários para cliente PNCP#404

Open
Joa0V wants to merge 4 commits into
GovHub-br:mainfrom
Joa0V:test/unit_test_cliente_pncp
Open

Test(cliente_pncp): Adiciona testes unitários para cliente PNCP#404
Joa0V wants to merge 4 commits into
GovHub-br:mainfrom
Joa0V:test/unit_test_cliente_pncp

Conversation

@Joa0V

@Joa0V Joa0V commented Jun 23, 2026

Copy link
Copy Markdown

Descrição

Adição de testes unitários para cliente_pncp.py em tests/test_plugins/test_cliente_pncp.py e alterações em cliente_pncp.py em airflow_lappis/plugins/cliente_pncp.py

  1. Obrigatoriedade do argumento codigo_modalidade_contratacao

Segundo a documentação da api pncp consulta o código de modalidade de contratação é um parâmetro obrigatório. O argumento codigo_modalidade_contratacao no método get_contratacoes_publicacao() de ClientePNCP foram modificadas de Optional[int] = None para int.

failing_test_1
  1. Loop em get_contratacoes_publicacao_paginado

Loop em get_contratacoes_publicacao_paginado() sobrescrevia argumento max_paginas passado ao método com retorno do total de páginas da api. A lógica do laço foi modificada para levar em consideração o argumento max_paginas passado ao método.

failing_test_2

Tipo de mudança

  • Nova funcionalidade / pipeline
  • Correção de bug ou inconsistência de dados
  • Refatoração de modelo DBT
  • Documentação
  • Infraestrutura / CI
  • Outro: Testes

Issues relacionadas

Closes #316

Como testar / validar

make lint
make test

Evidências

image

Checklist

  • Título do PR segue Conventional Commits
  • Issue relacionada foi referenciada
  • Testes/lint foram executados ou a ausência foi justificada
  • Testes DBT adicionados/atualizados, se aplicável
  • Documentação atualizada, se aplicável
  • Sem dados sensíveis ou credenciais no código
  • Branch atualizada com upstream/main ou origin/main

- Testes unitários para cliente_pncp.py em tests/test_plugins/test_cliente_pncp.py
- Alterações em cliente_pncp.py em airflow_lappis/plugins/cliente_pncp.py
@Joa0V Joa0V requested a review from a team as a code owner June 23, 2026 19:03
@github-actions github-actions Bot added the team:oss Revisao de contribuicoes OSS, GCES e governanca aberta label Jun 25, 2026
- Remove testes de argumentos obrigatórios apontados pelo SonarQubeCloud
@github-actions github-actions Bot added team:ipea Revisao do dominio IPEA team:mcid Revisao do dominio MCid team:mir Revisao do dominio MIR labels Jun 25, 2026
@arthurpalhares1 arthurpalhares1 removed team:oss Revisao de contribuicoes OSS, GCES e governanca aberta team:ipea Revisao do dominio IPEA team:mcid Revisao do dominio MCid team:mir Revisao do dominio MIR labels Jun 25, 2026
@sonarqubecloud

Copy link
Copy Markdown

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.

test: Testes unitarios para cliente_pncp.py

2 participants