Add Firebase auth emulator#510
Conversation
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
src/main/java/com/google/firebase/auth/FirebaseUserManager.java
Outdated
Show resolved
Hide resolved
src/main/java/com/google/firebase/auth/multitenancy/FirebaseTenantClient.java
Outdated
Show resolved
Hide resolved
src/main/java/com/google/firebase/auth/multitenancy/FirebaseTenantClient.java
Outdated
Show resolved
Hide resolved
src/test/java/com/google/firebase/auth/multitenancy/FirebaseTenantClientTest.java
Show resolved
Hide resolved
ssbushi
left a comment
There was a problem hiding this comment.
Hi @7agustibm!
Thank you for contributing to this feature request. I've reviewed your code and left a few comments for you to address.
src/test/java/com/google/firebase/auth/FirebaseUserManagerTest.java
Outdated
Show resolved
Hide resolved
ssbushi
left a comment
There was a problem hiding this comment.
Hi @7agustibm,
This looks good. I left a few comments and I can pull this PR once they are addressed. I'll also add one more reviewer before submission.
Thanks a lot!
* Add Firebase Auth emulator support - UserManager & TenantClient(#510) * Adds Firebase Auth emulator support for FirebaseUserManager and FirebaseTenantClient classes. * Adds tests and minor fixes * Remove new dependency from tests * Test integrations on CI * Add unit tests for Auth emulator support * Lint fixes * Add more test * Fix and clean tests * Address review comments * Minor fixes * More review comment changes * Final fixes * Fix typo Co-authored-by: Agustí Becerra Milà <contact@agustibm.com>
Add support for Auth emulator.
More info: