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

Subject test controller #194

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

RamonJales
Copy link
Collaborator

Por favor, informe se a PR segue os requisitos obrigatórios:

  • Mesmo padrão de código do projeto
  • Arquivos alterados/adicionados seguem o padrão Camel Case
  • A PR está relacionada a uma ou mais issue
  • Relacionei todas as issues na seção de "Development" da PR
  • O código foi revisado uma vez ou mais

Motivação para a criação da PR

O que foi feito

  • Teste do controller com Mock e WebMvcTest.

@RamonJales RamonJales added the enhancement New feature or request label Jan 27, 2024
@RamonJales RamonJales linked an issue Jan 27, 2024 that may be closed by this pull request
@RamonJales
Copy link
Collaborator Author

Quando adicionarmos o test com o WebMvcTest, o github também irá realizar esses tests?

@RamonJales
Copy link
Collaborator Author

RamonJales commented Jan 27, 2024

No commit shouldFindAll controller tested há uma linha comentando, aquela linha deveria testar se o método retorna um json. Porém, pelo o que parece, o método não retorna um json, deve ter algo a ver com a paginação.

@isaacmsl
Copy link
Owner

Quando adicionarmos o test com o WebMvcTest, o github também irá realizar esses tests?

Você consegue verificar isso rodando localmente na sua máquina ./mvnw verify

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

(feat) Testes em nível de service e controller
2 participants