-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix #5141: remove junit5 junit-jupiter-migrationsupport dependency from codebase #5187
Conversation
@manusa @rohanKanojia @shawkins please take a look when you get a chance. There are references to |
...src/test/java/io/fabric8/kubernetes/client/utils/internal/OptionalDependencyWrapperTest.java
Outdated
Show resolved
Hide resolved
kubernetes-client/src/test/java/io/fabric8/kubernetes/client/impl/HandlersTest.java
Outdated
Show resolved
Hide resolved
@degant: Thanks a lot for your PR 👍
Scope of this issue is just to remove junit-jupiter-migrationsupport dependency. Your changes look okay to me. |
@degant : There are some minor code smells reported by Sonar. Could you please address them whenever you revisit this? |
@rohanKanojia I assumed that the sonar report was unrelated to my changes. Addressed those as well |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Description
Fixes #5141
Fixing issue #5141, removing
junit-jupiter-migrationsupport
and migrating various junit4 references to junit5junit-jupiter-migrationsupport
from variouspom.xml
files (total 36)UberJarTest
to use@TempDir
instead of rules.org.junit.Assert
references to the newerorg.junit.jupiter.api.Assertions
org.junit.After
/ Before / Test to the newerorg.junit.jupiter.api.AfterEach
/ BeforeEach / TestThe only thing pending is the
org.junit.Test
andorg.junit.Assert
refernces remaining in karaf\itests which I couldn't complete.Type of change
test, version modification, documentation, etc.)
Checklist