Skip to content

Migrate Spring Batch Infrastructure to JUnit Jupiter #4125

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

Closed
wants to merge 1 commit into from
Closed

Migrate Spring Batch Infrastructure to JUnit Jupiter #4125

wants to merge 1 commit into from

Conversation

hpoettker
Copy link
Contributor

@hpoettker hpoettker commented Jun 7, 2022

This is a follow-up on #4124. It's quite a high number of touched files but I don't think there is a single change that's in itself interesting. If it helps, I can also split this up e.g. in tests from JUnit 3 and from JUnit 4 or something similar.

JUnit 4 is no longer on the classpath of the module.

Together with #4124 and #4112, it feels like the biggest part of migrating all tests to JUnit 5 is already done. The biggest open part is Spring Batch Core which shouldn't take long either. So far this migration is not on any roadmap but I think it would be nice to do it. With the release of Spring Batch 5, there will also be 5 years of JUnit 5. Perfect moment! 🥳

@fmbenhassine fmbenhassine added this to the 5.0.0-M5 milestone Jun 8, 2022
fmbenhassine pushed a commit that referenced this pull request Jul 27, 2022
fmbenhassine added a commit that referenced this pull request Jul 27, 2022
* Fix assertions
* Update RecordFieldExtractorTests to use JUnit Jupiter
@fmbenhassine fmbenhassine mentioned this pull request Jul 27, 2022
5 tasks
@fmbenhassine
Copy link
Contributor

Rebased and merged as 83d01fb. I noticed that some assertion updates were missed, I fixed them in 932e934.

Other PRs will follow. I created #4166 to track the progress of the migration of all modules.

Thank you very much for this huge update!! Really appreciated 👍

@hpoettker hpoettker deleted the jupiter-integ branch August 17, 2022 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants