Make initializeAuth() idempotent

[hsubox76]

Making all product initialize() functions idempotent.
See #5272

[changeset-bot]

⚠️ No Changeset found

Latest commit: 7970f50

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[github-actions]

Changeset File Check ⚠️

• Changeset formatting error in following file:

  Package "firebase-compat-typings-test" must depend on the current version of "firebase-exp": "9.0.0-beta.8" vs "file:../../packages-exp/firebase-exp"
  Package "firebase-compat-typings-test" must depend on the current version of "firebase-exp": "9.0.0-beta.8" vs "file:../../packages-exp/firebase-exp"
  Package "firebase-compat-typings-test" must depend on the current version of "firebase-exp": "9.0.0-beta.8" vs "file:../../packages-exp/firebase-exp"
  Some packages have been changed but no changesets were found. Run `changeset add` to resolve this error.
  If this change doesn't need a release, run `changeset add --empty`.
  

[google-oss-bot]

Binary Size Report

Affected SDKs

No changes between base commit (d46d84b) and head commit (598ec31).

Test Logs

• Base (d46d84b3): https://github.com/firebase/firebase-js-sdk/actions/runs/1121934258
• Head (598ec318): https://github.com/firebase/firebase-js-sdk/actions/runs/1122081466

[google-oss-bot]

Size Analysis Report

Affected Products

• @firebase/auth-exp

  • getAuth

    Size Table

    Type	Base (d46d84b)	Head (598ec31)	Diff
    size	61.1 kB	61.2 kB	+90 B (+0.1%)
    size-with-ext-deps	73.4 kB	73.8 kB	+421 B (+0.6%)

    External Dependency Table

    Module	Base (d46d84b)	Head (598ec31)	Diff
    @firebase/util	12 dependencies ErrorFactory FirebaseError base64Decode createSubscribe getModularInstance getUA isBrowserExtension isEmpty isIE isMobileCordova isReactNative querystring	13 dependencies ErrorFactory FirebaseError base64Decode createSubscribe deepEqual getModularInstance getUA isBrowserExtension isEmpty isIE isMobileCordova isReactNative querystring	+ deepEqual

  • initializeAuth

    Size Table

    Type	Base (d46d84b)	Head (598ec31)	Diff
    size	29.8 kB	29.9 kB	+92 B (+0.3%)
    size-with-ext-deps	41.7 kB	42.1 kB	+421 B (+1.0%)

    External Dependency Table

    Module	Base (d46d84b)	Head (598ec31)	Diff
    @firebase/util	ErrorFactory FirebaseError base64Decode createSubscribe getModularInstance getUA isBrowserExtension isMobileCordova isReactNative querystring	11 dependencies ErrorFactory FirebaseError base64Decode createSubscribe deepEqual getModularInstance getUA isBrowserExtension isMobileCordova isReactNative querystring	+ deepEqual

  • debugErrorMap

    Size Table

    Type	Base (d46d84b)	Head (598ec31)	Diff
    size	40.5 kB	40.7 kB	+192 B (+0.5%)
    size-with-ext-deps	52.4 kB	52.6 kB	+192 B (+0.4%)

Test Logs

• Base (d46d84b): https://github.com/firebase/firebase-js-sdk/actions/runs/1121934258
• Head (598ec31): https://github.com/firebase/firebase-js-sdk/actions/runs/1122081466

[Feiyang1]

I thought we are going to use reference equality for comparing builtin objects like popupRedirectResolvers and persistences. Using deepEqual feels unnecessary.

[hsubox76]

I was mainly using deepEqual to avoid extra lines of code, but also, while 2 of the 3 options (popupRedirectResolver and errorMap) can be compared using a simple ===, persistence can be an array, and on top of that, an array where order matters.