Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(tests): Introduce junit5 vintage engine for running junit4 test cases over junit5 in halyard #2022

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

j-sandy
Copy link
Contributor

@j-sandy j-sandy commented Mar 3, 2023

Spring boot 2.4.x removed JUnit5 vintage engine from spring-boot-starter-test. It is required for executing junit4 based test cases in halyard. So, introducing junit-vintage-engine dependency in build.gradle, using testRuntimeOnly() as suggested in section 3.1 of https://junit.org/junit5/docs/5.6.2/user-guide/index.pdf

After applying this fix, coverage increased from 54 to 227 test case executions.

…ases over junit5 in halyard

Spring boot 2.4.x removed JUnit5 vintage engine from spring-boot-starter-test.
[https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.4-Release-Notes#junit-5s-vintage-engine-removed-from-spring-boot-starter-test]
It is required for executing junit4 based test cases in halyard.
So, introducing junit-vintage-engine dependency in build.gradle, using testRuntimeOnly() as suggested in section 3.1 of https://junit.org/junit5/docs/5.6.2/user-guide/index.pdf

After applying this fix, coverage increased from 54 to 227 test case executions.
@dbyron-sf dbyron-sf added the ready to merge Approved and ready for a merge label Mar 3, 2023
@mergify mergify bot added the auto merged label Mar 3, 2023
@mergify mergify bot merged commit 511da27 into spinnaker:master Mar 3, 2023
@j-sandy j-sandy deleted the junit5-vintage-engine branch March 6, 2023 05:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants