Skip to content

chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - fifth iteration (#756)#767

Merged
chanseokoh merged 1 commit intogoogleapis:mainfrom
Captain1653:oauth-http-fifth-iteration
Oct 13, 2021
Merged

chore: migrate tests in oauth2_http module from JUnit4 to JUnit5 - fifth iteration (#756)#767
chanseokoh merged 1 commit intogoogleapis:mainfrom
Captain1653:oauth-http-fifth-iteration

Conversation

@Captain1653
Copy link
Copy Markdown
Contributor

It belongs to #756

@Captain1653 Captain1653 requested a review from a team October 13, 2021 06:10
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 13, 2021
Comment on lines +790 to +792
OAuth2Credentials credentials =
OAuth2Credentials.newBuilder()
.setAccessToken(new AccessToken(ACCESS_TOKEN, null))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This portion should better be taken out of the scope. However, since this doesn't change the semantics of the old code, this is fine.

@chanseokoh chanseokoh merged commit 59dfb35 into googleapis:main Oct 13, 2021
@Captain1653 Captain1653 deleted the oauth-http-fifth-iteration branch October 13, 2021 16:18
clundin25 added a commit to clundin25/google-auth-library-java that referenced this pull request Aug 9, 2022
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.
@clundin25 clundin25 mentioned this pull request Aug 9, 2022
4 tasks
clundin25 added a commit to clundin25/google-auth-library-java that referenced this pull request Aug 9, 2022
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.
clundin25 added a commit to clundin25/google-auth-library-java that referenced this pull request Aug 9, 2022
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.
clundin25 added a commit that referenced this pull request Aug 11, 2022
* refactor: Revert JUnit5 to JUnit4

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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants