-
Notifications
You must be signed in to change notification settings - Fork 248
refactor: Revert JUnit5 to JUnit4 #969
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
Conversation
Refactored tests added since 928dd04 to use JUnit4 instead of JUnit5. Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - sixth iteration (googleapis#756) (googleapis#769)" This reverts commit 928dd04. Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - fifth iteration (googleapis#756) (googleapis#767)" This reverts commit 59dfb35. Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - fourth iteration (googleapis#756) (googleapis#766)" This reverts commit d7edc45. Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - third iteration (googleapis#756) (googleapis#764)" This reverts commit e9dfaf9. Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - second iteration (googleapis#756) (googleapis#763)" This reverts commit d045247. Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - first iteration (googleapis#756) (googleapis#762)" This reverts commit 4e6576f. Revert "refactor: migration tests in credentials module from JUnit4 to JUnit5 (googleapis#756) (googleapis#761)" This reverts commit a64d35c. Revert "refactor: migration tests in appengine module from JUnit4 to JUnit5 (googleapis#756) (googleapis#760)" This reverts commit cccaa44.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
partial review, some nits in comments
@TimurSadykov Addressed your comments in 25c4df5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mostly missing fail() statements
oauth2_http/javatests/com/google/auth/oauth2/ExternalAccountCredentialsTest.java
Show resolved
Hide resolved
oauth2_http/javatests/com/google/auth/oauth2/ExternalAccountCredentialsTest.java
Show resolved
Hide resolved
oauth2_http/javatests/com/google/auth/oauth2/ExternalAccountCredentialsTest.java
Show resolved
Hide resolved
oauth2_http/javatests/com/google/auth/oauth2/IdentityPoolCredentialsTest.java
Show resolved
Hide resolved
oauth2_http/javatests/com/google/auth/oauth2/ImpersonatedCredentialsTest.java
Show resolved
Hide resolved
oauth2_http/javatests/com/google/auth/oauth2/TokenVerifierTest.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Refactored tests added since 928dd04 to
use JUnit4 instead of JUnit5.
Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - sixth iteration (#756) (#769)"
This reverts commit 928dd04.
Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - fifth iteration (#756) (#767)"
This reverts commit 59dfb35.
Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - fourth iteration (#756) (#766)"
This reverts commit d7edc45.
Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - third iteration (#756) (#764)"
This reverts commit e9dfaf9.
Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - second iteration (#756) (#763)"
This reverts commit d045247.
Revert "chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - first iteration (#756) (#762)"
This reverts commit 4e6576f.
Revert "refactor: migration tests in credentials module from JUnit4 to JUnit5 (#756) (#761)"
This reverts commit a64d35c.
Revert "refactor: migration tests in appengine module from JUnit4 to JUnit5 (#756) (#760)"
This reverts commit cccaa44.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
Fixes #933 ☕️
If you write sample code, please follow the samples format.