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 servicetalk-grpc-* tests to junit5 #1660

Merged
merged 1 commit into from
Jul 9, 2021

Conversation

danfaer
Copy link
Contributor

@danfaer danfaer commented Jul 6, 2021

JUnit 5 leverages features from Java 8 or later, such as lambda functions, making tests more powerful and easier to maintain.
JUnit 5 has added some very useful new features for describing, organizing, and executing tests. For instance, tests get better display names and can be organized hierarchically.
JUnit 5 is organized into multiple libraries, so only the features you need are imported into your project. With build systems such as Maven and Gradle, including the right libraries is easy.
JUnit 5 can use more than one extension at a time, which JUnit 4 could not (only one runner could be used at a time). This means you can easily combine the Spring extension with other extensions (such as your own custom extension).
Modifications:

Unit tests have been migrated from JUnit 4 to JUnit 5

Result:

Modules servicetalk-grpc-* now run tests and test suits using JUnit 5

#1568

@tkountis tkountis mentioned this pull request Jul 6, 2021
44 tasks
@bondolo bondolo added the technical debt Reduces technical debt label Jul 8, 2021
@tkountis tkountis self-requested a review July 9, 2021 15:50
@tkountis
Copy link
Contributor

tkountis commented Jul 9, 2021

@danfaer massive thanks, I will have a look at the failing tests before I merge both of your PRs.
Code LGTM.

@idelpivnitskiy
Copy link
Member

@tkountis this is a new flaky one. Saw it before this PR, but decided not to open an issue until it became more visible. Flakiness is related to observability and does not impact general control flow: #1663

Copy link
Member

@idelpivnitskiy idelpivnitskiy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @danfaer! ❤️

@idelpivnitskiy idelpivnitskiy merged commit 46354d4 into apple:main Jul 9, 2021
chemicL pushed a commit that referenced this pull request Jul 21, 2021
Motivation:

JUnit 5 leverages features from Java 8 or later, such as lambda functions, making tests more powerful and easier to maintain.
JUnit 5 has added some very useful new features for describing, organizing, and executing tests. For instance, tests get better display names and can be organized hierarchically.
JUnit 5 is organized into multiple libraries, so only the features you need are imported into your project. With build systems such as Maven and Gradle, including the right libraries is easy.
JUnit 5 can use more than one extension at a time, which JUnit 4 could not (only one runner could be used at a time). This means you can easily combine the Spring extension with other extensions (such as your own custom extension).

Modifications:

Unit tests have been migrated from JUnit 4 to JUnit 5

Result:

Modules servicetalk-grpc-* now run tests and test suits using JUnit 5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical debt Reduces technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants