Skip to content

Lê as credenciais da API de variáveis de ambiente - #31

Merged
ThomasTDS merged 1 commit into
masterfrom
chore/credenciais-por-variavel-de-ambiente
Oct 2, 2026
Merged

ThomasTDS merged 1 commit into
masterfrom
chore/credenciais-por-variavel-de-ambiente

Conversation

@ThomasTDS

Copy link
Copy Markdown
Owner

Resumo

  • AuthApiClient recebe username e password pelo construtor. O get_valid_token usa essas credenciais em vez de admin/password123 fixos no código
  • O conftest.py lê API_USERNAME e API_PASSWORD, como já fazia com BASE_URL. O padrão são as credenciais públicas da documentação da restful-booker, então pytest continua rodando sem configuração e o CI não precisa de secrets
  • TC-001: o passo passa a ser "ele solicita um token com credenciais válidas", usando as credenciais configuradas, para continuar válido em outro ambiente. O TC-002 mantém admin/senha_errada no Gherkin, porque a senha errada é o objeto do teste
  • README: a seção "Rodar contra outra URL" vira "Rodar contra outro ambiente", com uma tabela das três variáveis

Test plan

  • mypy .
  • ruff check .
  • ruff format --check .
  • pytest: 52/52 testes passando
  • API_PASSWORD=errada pytest -m TC-001: o cenário falha, confirmando que a variável é usada
  • Testes unitários do AuthApiClient usam credenciais próprias, diferentes do padrão, para provar que o client não depende de valores fixos

O AuthApiClient recebe usuário e senha pelo construtor, e o conftest os lê de API_USERNAME e API_PASSWORD, com as credenciais públicas da restful-booker como padrão. O TC-001 passa a usar as credenciais configuradas, para continuar válido em outro ambiente.
@codecov

codecov Bot commented Oct 2, 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 330c92b into master Oct 2, 2026
5 checks passed
@ThomasTDS
ThomasTDS deleted the chore/credenciais-por-variavel-de-ambiente branch October 2, 2026 19:20
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