Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: validação do CRUD de usuário #45

Merged
merged 21 commits into from
Mar 1, 2024

Conversation

JonasFortes12
Copy link
Collaborator

@JonasFortes12 JonasFortes12 commented Mar 1, 2024

Issue

Como especificado na Issue #39, as rotas de usuário precisam de validações para garantir a integridade dos dados e a segurança da aplicação.

Proposta deste PR

  • Validar os dados enviados pelas requisições POST e PUT à API MandacaruBroker.
  • Garantir que apenas informações seguras e relevantes devem ser enviadas como resposta ao cliente.
  • Implementar e atualizar testes de integração para garantir o funcionamento das features adicionadas.

Impactos deste PR

  • Adição de validações aos dados transferidos pelo Request DTO (Data Transfer Object) de usuário.
  • Implementação de validação personalizada para idade mínima de usuário.
  • O campo username agora é definido como UNIQUE no banco de dados. Dando suporte a implementação de login usando nome de usuário e senha.
  • Implementação de uma Response DTO para selecionar dados seguros como respostas às rotas da API.
  • Adição dos seguintes testes na suíte de testes
    • itShouldHandlePostInvalidUserEmail e itShouldHandlePutInvalidUserEmail
    • itShouldReturnConflictStatusWhenPostDuplicatedUsername
    • itShouldHandlePostInvalidUserPassword e itShouldHandlePutInvalidUserPassword
    • itShouldHandlePostInvalidUserAge e itShouldHandlePutInvalidUserAge
    • itShouldHandlePostInvalidUserBalance e itShouldHandlePutInvalidUserBalance

Evidências de teste

Requisição de cadastro de usuário com dados inválidos
image

Rota para obter usuário pelo ID
image

Testes de integração
image

@JonasFortes12 JonasFortes12 linked an issue Mar 1, 2024 that may be closed by this pull request
@JonasFortes12 JonasFortes12 marked this pull request as ready for review March 1, 2024 20:14
@JonasFortes12 JonasFortes12 changed the title Feature/validacao crud usuario Feat: validação do CRUD de usuário Mar 1, 2024
Copy link
Owner

@izaiasmachado izaiasmachado left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Sobre o username e email no response, depois adicionamos se for necessário

@JonasFortes12 JonasFortes12 merged commit 4180ede into main Mar 1, 2024
4 checks passed
@JonasFortes12 JonasFortes12 deleted the feature/validacao-crud-usuario branch March 1, 2024 22:13
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.

BDD Feature: validação de parâmetros do CRUD de usuários
3 participants