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

Migrate tests from JUnit 3 to JUnit 5 [SWF-1739] #914

Closed
spring-operator opened this issue Aug 26, 2019 · 5 comments
Closed

Migrate tests from JUnit 3 to JUnit 5 [SWF-1739] #914

spring-operator opened this issue Aug 26, 2019 · 5 comments
Assignees
Milestone

Comments

@spring-operator
Copy link
Contributor

spring-operator commented Aug 26, 2019

Rossen Stoyanchev opened SWF-1739 and commented

Currently Web Flow's own tests still depend on the JUnit 3 class hierarchy and could be migrated to JUnit 5.


Affects: 2.5.1

Issue Links:

1 votes, 3 watchers

@spring-operator
Copy link
Contributor Author

Rossen Stoyanchev commented

Related PR #97 with migration to JUnit 4.

@spring-operator
Copy link
Contributor Author

@spring-operator
Copy link
Contributor Author

spring-operator commented Aug 26, 2019

Sam Brannen commented

Related to #915

@spring-operator
Copy link
Contributor Author

Etienne Dysli Metref commented

I used sed to migrate. ;)
What makes jumping from 3 to 5 tricky is that the message argument of assertion methods moved from the first to the last position. Instances of assertEquals still compile, but are incorrect.

@spring-operator
Copy link
Contributor Author

Etienne Dysli Metref commented

Changes to migrate to JUnit 5 are ready for review in PR#99. I'm looking forward to your comments. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants