Skip to content

Cobre o health check, os filtros por data e o Basic auth - #57

Merged
ThomasTDS merged 1 commit into
masterfrom
test/lacunas-de-cobertura
Oct 24, 2026
Merged

ThomasTDS merged 1 commit into
masterfrom
test/lacunas-de-cobertura

Conversation

@ThomasTDS

@ThomasTDS ThomasTDS commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner

Resumo

Fecha as três lacunas listadas na estratégia de testes, com 9 casos novos (TC-026 a TC-034):

TC Cenário Resultado
TC-026 Health check (GET /ping) 201, como documentado
TC-027 Filtro checkin com data anterior à do booking Booking encontrado
TC-028 Filtro checkin com a mesma data do booking Bug #54: o booking fica de fora ($gt em vez de $gte)
TC-029 Filtro checkout com data posterior à do booking Bug #55: o filtro funciona no sentido contrário ao documentado ($lt)
TC-030 Filtro com data inválida Bug #56: 500 em vez de 400
TC-031 PUT com Basic auth 200
TC-032 PATCH com Basic auth 200
TC-033 DELETE com Basic auth 201
TC-034 PUT com Basic auth inválido 403
  • As três issues novas têm a causa raiz confirmada no código-fonte da restful-booker, com link para a linha
  • BookingApiClient: PUT, PATCH e DELETE aceitam basic_auth=(usuário, senha), repassado ao auth= do requests. Sem ele, nada muda. Com Basic auth e sem token, o cookie não é enviado
  • Novo HealthApiClient com ping(), que vai ser reaproveitado na verificação de saúde antes da suíte (próximo item)
  • Steps de Basic auth e o de erro 500 passam para common_steps.py
  • Matriz, estratégia de testes (PT e EN) e READMEs (PT e EN) atualizados: 34 casos de teste e 10 bugs documentados, sem lacunas conhecidas

Test plan

  • Comportamentos explorados na API pública antes de escrever os cenários (booking de teste criado e removido)
  • mypy .
  • ruff check .
  • ruff format --check .
  • pytest: 79/79 passando contra a API pública (35 cenários BDD e 44 unitários), cobertura de api/ e models/ em 100%, inclusive os desvios de _credentials
  • CI deste PR: 79 testes passando contra a API local, nas duas versões do Python, e codecov/patch passando

Adiciona os casos TC-026 a TC-034: o GET /ping, os filtros checkin e checkout do GET /booking (com análise de valor-limite) e o Basic auth em PUT, PATCH e DELETE. O BookingApiClient passa a aceitar basic_auth como opção, e o novo HealthApiClient faz o ping. Os três bugs encontrados nos filtros estão documentados nas issues #54, #55 e #56.
@codecov

codecov Bot commented Oct 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ThomasTDS
ThomasTDS merged commit b7bdebf into master Oct 24, 2026
7 checks passed
@ThomasTDS
ThomasTDS deleted the test/lacunas-de-cobertura branch October 24, 2026 19:41
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