Refetch token if attempt was invalidated

[schmidt-sebastian]

Port of firebase/firebase-js-sdk#3184

[google-oss-bot]

Binary Size Report

Affected SDKs

• firebase-common

  Type	Base (3da23a6)	Head (060ff43d)	Diff
  apk (aggressive)	82.5 kB	82.5 kB	-17 B (-0.0%)
  apk (debug)	770 kB	770 kB	-3 B (-0.0%)

• firebase-components

  Type	Base (3da23a6)	Head (060ff43d)	Diff
  apk (aggressive)	10.9 kB	11.0 kB	+11 B (+0.1%)
  apk (debug)	35.8 kB	35.9 kB	+26 B (+0.1%)

• firebase-database-collection

  Type	Base (3da23a6)	Head (060ff43d)	Diff
  apk (aggressive)	314 kB	314 kB	+2 B (+0.0%)
  apk (debug)	1.07 MB	1.07 MB	+100 B (+0.0%)

• firebase-firestore

  Type	Base (3da23a6)	Head (060ff43d)	Diff
  aar	1.07 MB	1.07 MB	+103 B (+0.0%)
  apk (debug)	3.74 MB	3.74 MB	+22 B (+0.0%)
  apk (release)	3.13 MB	3.13 MB	+167 B (+0.0%)

• protolite-well-known-types

  Type	Base (3da23a6)	Head (060ff43d)	Diff
  apk (debug)	643 kB	643 kB	-16 B (-0.0%)

Test Logs

• Base (3da23a65): https://android-ci.firebaseopensource.com/view/gcs/android-ci/logs/postsubmit-binary-size/1270037008814706692
• Head (060ff43d): https://android-ci.firebaseopensource.com/view/gcs/android-ci/pr-logs/pull/firebase_firebase-android-sdk/1628/binary-size/1270064570253185027

Notes

Head commit (060ff43d) is created by Prow via merging commits: 3da23a6 d041342.

[google-oss-bot]

Coverage Report

Affected SDKs

• firebase-firestore

  SDK overall coverage changed from 43.11% (3da23a6) to 43.11% (060ff43d) by -0.00%.

  Filename	Base (3da23a6)	Head (060ff43d)	Diff

Test Logs

• Base (3da23a65): https://android-ci.firebaseopensource.com/view/gcs/android-ci/logs/postsubmit-check-coverage/1270037008814706690
• Head (060ff43d): https://android-ci.firebaseopensource.com/view/gcs/android-ci/pr-logs/pull/firebase_firebase-android-sdk/1628/check-coverage-changed/1270064570253185026

Notes

HTML coverage reports can be produced locally with ./gradlew <product>:checkCoverage.
Report files are located at <product-build-dir>/reports/jacoco/.

Head commit (060ff43d) is created by Prow via merging commits: 3da23a6 d041342.

[google-oss-bot]

@schmidt-sebastian: The following test failed, say /retest to rerun them all:

Test name	Commit	Details	Rerun command
smoke-tests	d041342	link	/test smoke-tests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[var-const]

General observation: it seems a little unfortunate that getToken and getResult().getToken() look so similar while having a very different effect. In general, I'd expect a get-prefixed function to be a simple getter, so it might be better to rename getToken to something like fetchToken to better convey its semantics. Of course, this is absolutely out of scope for this PR.

[schmidt-sebastian]

Ack :)