Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Captain1653
Copy link
Contributor

It belongs to #756 . Second iteration for oauth2_http module.

@Captain1653 Captain1653 requested a review from a team as a code owner October 9, 2021 16:18
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 9, 2021
@Captain1653
Copy link
Contributor Author

Captain1653 commented Oct 9, 2021

Hmm... Single test failed on java 11. I didn't touch it. Is it an accidental failure? Synchronized section as I can see.

Error: Tests run: 23, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.057 s <<< FAILURE! - in com.google.auth.oauth2.OAuth2CredentialsTest
Error: com.google.auth.oauth2.OAuth2CredentialsTest.getRequestMetadata_staleTemporaryToken_expirationWaits Time elapsed: 0.108 s <<< FAILURE!
java.lang.AssertionError: expected null, but was:<com.google.common.util.concurrent.ListenableFutureTask@66b62ebd[Completed normally]>
at com.google.auth.oauth2.OAuth2CredentialsTest.getRequestMetadata_staleTemporaryToken_expirationWaits(OAuth2CredentialsTest.java:625)

@chanseokoh chanseokoh merged commit d045247 into googleapis:main Oct 11, 2021
@Captain1653 Captain1653 deleted the oauth-http-junit-second-iteration branch October 11, 2021 15:03
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