strangelookingnerd opened a new pull request, #555: URL: https://github.com/apache/incubator-wayang/pull/555
This PR aims to migrate all tests to JUnit5. Changes include: * Migrate annotations and imports * Migrate assertions * Remove public visibility for test classes and methods * Minor clean up I am well aware that this is a huge changeset however I hope that there is still interest in this PR and it will be reviewed. If there are any questions, please do not hesitate to ping me. Why this is a good change: * JUnit 5 is the modern standard: It offers a cleaner and more powerful programming model, better extensibility, and improved support for modern Java features (like lambdas, streams, and optional parameters). * Improved test maintainability: JUnit 5’s more expressive annotations and lifecycle management make tests easier to read, write, and debug. * Enables use of modern extensions: Migrating paves the way to leverage powerful third-party extensions and tooling (e.g., parameterized tests, dynamic tests, conditional execution). It is important to notice that this change should not alter the test logic, but bring the project in line with modern best practices and help keeping it future-proof. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
