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.

Sign up for GitHub

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

Jump to bottom

test: add tests for existsbyabbreviation and findallbystatus methods #136

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

test: add tests for existsbyabbreviation and findallbystatus methods #136

wants to merge 16 commits into from

Conversation

AlexsandroCampos
Copy link
Member

@AlexsandroCampos AlexsandroCampos commented Jun 14, 2022 •
edited
Loading

Relacionado a issue #120

Este pull request adiciona testes aos métodos existsByAbbreviation e findAllByStatus da classe CampusRepository e aos métodos create, findAll, findById, findByCampusIdAndId, update, setStatus, delete, existsByCampusId, disableAllByCampusId e enable da classe DepartmentServiceImpl.

Para verificar se essas modificações funcionam, basta rodar os testes criados no IDE.

@AlexsandroCampos
fix: error with RequestExpiresVerification bean
75c9450 
The @DataJpaTeste annotation disables Spring's automatic configuration, as a result, it is unable to @import the RequestExpiresVerification class in the tests, generating the error of not being able to create its bean. Since the @import annotation is not needed, so the solution was to remove it.
@AlexsandroCampos
test: add tests for findallbystatus method
570ed27
@AlexsandroCampos
test: add tests for existsbyabbreviation method
b620888
@AlexsandroCampos AlexsandroCampos self-assigned this Jun 14, 2022
@AlexsandroCampos AlexsandroCampos marked this pull request as ready for review June 15, 2022 19:18
@GusttavoSoares GusttavoSoares mentioned this pull request Jun 15, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@domingoslatorre domingoslatorre domingoslatorre approved these changes

Assignees

@AlexsandroCampos AlexsandroCampos

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AlexsandroCampos @domingoslatorre