-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Remove junit vintage engine, no more JUnit 3/4 #2324
Conversation
* @before -> @beforeeach * @after -> @AfterEach * import org.junit.Before; -> import org.junit.jupiter.api.BeforeEach; * import org.junit.After; -> import org.junit.jupiter.api.AfterEach; * import org.junit.Test; -> import org.junit.jupiter.api.Test * commit using 'Reformat Code' and 'Optimize imports'
* import org.junit.BeforeClass -> import org.junit.jupiter.api.BeforeAll * import org.junit.AfterClass -> import org.junit.jupiter.api.AfterAll
* import org.junit.Ignore -> import org.junit.jupiter.api.Disabled
Oh, wow. Really nice - didn't notice this :) 👍 |
I saw the earlier thread but just hadn't had time to refresh my memory on JUnit in order to mount a meaningful response, but it seems like you've got it all covered 👍 |
Thanks a lot for your initial work on this @Anvoker! Can I add you to the contributors list? |
If you think that's enough to warrant it, then sure thing! Don't see why not. |
I just saw you are already on the list :) But anyway I updated it here: d141dbd |
After replacing
@Theory
(#2310) and class-wide parameters (#2312) moving from JUnit 4 to 5 entirely is 'only' a matter of changing the import statements and assertion method calls and I did this here.For a very nice write up and initial work in this direction see #1632.