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

test: add campusnotexists and exists by initial registration pattern #121

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

GusttavoSoares
Copy link
Member

@GusttavoSoares GusttavoSoares commented Jun 13, 2022

Resolve as issues #127 e #120 e se relaciona com o pull request #136.

Esse pull request adiciona os testes para os métodos existsByInternshipSectorEmail e existsByInitialRegistrationPattern, que estão localizados na classe CampusRepository. Além disso, foi adicionado no CampusServiceImpl o toUpperCase nos métodos getAbbreviation e getInitialRegistrationPattern, quando cria e atualiza algum campus, dessa forma, os valores dos atributos abbreviation e initialRegistrationPattern serão sempre salvos em letra maiúscula.

As modificações relacionadas aos testes podem ser verificadas ao rodar a classe CampusRepositoryTest. Já a alteração feita no CampusService, pode ser vista por meio do Postman.

  • POST /campuses colocando initial registration pattern com valores em minúsculo

  • POST /campuses colocando abbreviation com valores em minúsculo

  • PUT /campuses colocando initial registration pattern com valores em minúsculo

  • PUT /campuses colocando abbreviation com valores em minúsculo

  • GET / campuses e GET/ campuses/ id

@GusttavoSoares GusttavoSoares self-assigned this Jun 13, 2022
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.

2 participants