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 test to curriculumrepositorytest findbycourseidandstatus #142

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

Conversation

lucasg-c
Copy link
Member

@lucasg-c lucasg-c commented Jun 29, 2022

Resolves #140, resolves #141

Neste Pull Request, para solucionar as Issues #140 e #141, foram criadadas três classes e modificadas quatro. A primeira modificação na classe Application foi necessária para executar os testes codificados, visto que a linha 17 @Import(RequestExpiresVerification.class) causa um erro que impossibilita essas execuções. A modificação na classe CampusRepositoryTest, embora alheia aos conteúdos das Issues, ocorreu para testar inicialmente o ambiente de testes dado o erro mencionado anteriormente.

Para resolver o item "Implementar o teste do método existsByAbbreviationAndDepartmentIdExcludedId" (#141) foi necessária a criação da classe CourseRepositoryTest e nela implementar três métodos, sendo dois deles o teste em si. O primeiro método public void setUp(), com assinatura @BeforeEach, contém a criação de objetos sample que são usados durante as execuções dos testes. O método existsByAbbreviationAndDepartmentIdExcludedId() testa o método considerando um retorno boolean true, enquanto o método existsByAbbreviationAndDepartmentIdExcludedIdReturnsFalseWhenAbbreviationDiffers() testa o retorno false. Ambos os métodos foram assinados com @Test.

Na resolução dos itens da Issue #140 foram criadas as classes CurriculumFactoryUtils e CurriculumRepositoryTest. Na primeira classe há um construtor sampleCurriculum que retorna um objeto sample de tipo Curriculum a ser utilizado nos testes efetuados em CurriculumRepositoryTest, que além de também contar com um método public void setUp() contém os testes findByCourseIdAndStatusReturnsOnlyEnabledCourses(), existsByCourseId() e disableAllByCourseId(). Todos os testes foram assinados com @Test e são métodos public.

Referências:
Este Pull Request — e também a resolução integral das Issues #140 e #141 — relaciona-se com os seguintes PRs:

@lucasg-c lucasg-c self-assigned this Jun 29, 2022
@lucasg-c lucasg-c linked an issue Jun 29, 2022 that may be closed by this pull request
8 tasks
@lucasg-c lucasg-c requested a review from domingoslatorre July 8, 2022 20:35
@lucasg-c lucasg-c marked this pull request as ready for review July 8, 2022 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant