test(auth): add MFA e2e #9034

Lyokone · 2022-07-05T08:57:06Z

Description

Add MFA tests

Related Issues

Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

I read the Contributor Guide and followed the process outlined there for submitting PRs.
My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
All existing and new tests are passing.
I updated/added relevant documentation (doc comments with ///).
The analyzer (melos run analyze) does not report any problems on my PR.
I read and followed the Flutter Style Guide.
I signed the CLA.
I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

Yes, this is a breaking change.
No, this is not a breaking change.

* chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth): add documentation * feat(auth, android): fix typings * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(ui, android): use mocktail for mocking dependencies * feat(auth, android): fix analyze * feat(auth): fix melos generate command * Update packages/firebase_auth/firebase_auth_platform_interface/lib/src/method_channel/method_channel_multi_factor.dart Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com>

* chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(auth, android): fix analyze * feat(auth, ios): implement enroll * feat(auth, ios): implement MFA for signin * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): fix ios build * Update packages/firebase_auth/firebase_auth/ios/Classes/FLTFirebaseAuthPlugin.m Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * fix: fix rebase Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com>

tests/test_driver/firebase_auth/firebase_auth_e2e.dart

Salakar · 2022-07-06T15:00:14Z

tests/test_driver/firebase_auth/test_utils.dart

@@ -105,6 +105,16 @@ Future<String?> emulatorPhoneVerificationCode(String phoneNumber) async {
  )['code'];
 }

+/// Retrieve a sms phone authentication code that may have been sent for a specific
+/// phone number.
+Future<void> emulatorVerifyEmail(String oobCode) async {


Nice! Didn't know you could do this

russellwheatley

LGTM when test failing test passes 😄

* feat(auth, android): add phone MFA to Android (#8998) * chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth): add documentation * feat(auth, android): fix typings * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(ui, android): use mocktail for mocking dependencies * feat(auth, android): fix analyze * feat(auth): fix melos generate command * Update packages/firebase_auth/firebase_auth_platform_interface/lib/src/method_channel/method_channel_multi_factor.dart Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * feat(auth, ios): add phone MFA (#9008) * chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(auth, android): fix analyze * feat(auth, ios): implement enroll * feat(auth, ios): implement MFA for signin * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): fix ios build * Update packages/firebase_auth/firebase_auth/ios/Classes/FLTFirebaseAuthPlugin.m Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * fix: fix rebase Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * feat(auth): fix android merge * feat(auth): fix ios merge * feat(auth): add e2e tests * fix(auth): fix reintroduce tests * feat(auth): add documentation * feat(auth): add e2e tests Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com>

* feat(auth, android): add phone MFA to Android (#8998) * chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth): add documentation * feat(auth, android): fix typings * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(ui, android): use mocktail for mocking dependencies * feat(auth, android): fix analyze * feat(auth): fix melos generate command * Update packages/firebase_auth/firebase_auth_platform_interface/lib/src/method_channel/method_channel_multi_factor.dart Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * feat(auth, ios): add phone MFA (#9008) * chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(auth, android): fix analyze * feat(auth, ios): implement enroll * feat(auth, ios): implement MFA for signin * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): fix ios build * Update packages/firebase_auth/firebase_auth/ios/Classes/FLTFirebaseAuthPlugin.m Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * fix: fix rebase Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * feat(auth): fix android merge * feat(auth): fix ios merge * test(auth): add MFA e2e (#9034) * feat(auth, android): add phone MFA to Android (#8998) * chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth): add documentation * feat(auth, android): fix typings * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(ui, android): use mocktail for mocking dependencies * feat(auth, android): fix analyze * feat(auth): fix melos generate command * Update packages/firebase_auth/firebase_auth_platform_interface/lib/src/method_channel/method_channel_multi_factor.dart Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * feat(auth, ios): add phone MFA (#9008) * chores: update format CI step to match local melos format * chores: update format CI step to match local melos format * feat(auth, android): mfa * feat(auth, android): add MFA to phone options * feat(auth, android): add phone MFA * feat(auth, android): add MFA signin * feat(auth, android): fix tests * feat(auth, android): add input for phone number to example app * feat(auth, android): add unenroll and getEnrolledFactors * feat(auth, android): fix formatting * feat(auth, android): fix formatting * feat(auth, android): fix analyze * feat(auth, ios): implement enroll * feat(auth, ios): implement MFA for signin * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): add conditions to allow macos to build * feat(auth, ios): fix ios build * Update packages/firebase_auth/firebase_auth/ios/Classes/FLTFirebaseAuthPlugin.m Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * fix: fix rebase Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * feat(auth): fix android merge * feat(auth): fix ios merge * feat(auth): add e2e tests * fix(auth): fix reintroduce tests * feat(auth): add documentation * feat(auth): add e2e tests Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> * feat(auth, web): add phone mfa (#9031) * feat(auth): add multifactoruser * feat(auth, web): bridge * feat(auth, web): solve bridging errors * feat(auth, web): finish phone web mfa * feat(auth, web): finish phone web mfa * feat(auth, web): fix analyze * feat(auth, web): fix analyze * feat(auth, web): fix wrong argument * feat(auth, android): add early return * feat(auth, android): change dynamic to Object Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com>

Lyokone force-pushed the feat/mfa-ios branch from 470338c to bd37609 Compare July 6, 2022 13:28

Lyokone requested a review from lesnitsky as a code owner July 6, 2022 13:28

Base automatically changed from feat/mfa-ios to feat/mfa July 6, 2022 13:39

Lyokone force-pushed the feat/mfa-e2e branch from 8c0c944 to 2852bef Compare July 6, 2022 13:42

feat(auth): fix android merge

590c5bc

Lyokone force-pushed the feat/mfa-e2e branch from 2852bef to d73be8c Compare July 6, 2022 13:56

Lyokone added 2 commits July 6, 2022 16:10

feat(auth): fix ios merge

d29f201

feat(auth): add e2e tests

28c0b08

Lyokone force-pushed the feat/mfa-e2e branch from d73be8c to 28c0b08 Compare July 6, 2022 14:20

Salakar reviewed Jul 6, 2022

View reviewed changes

tests/test_driver/firebase_auth/firebase_auth_e2e.dart Outdated Show resolved Hide resolved

Salakar changed the title ~~feat(auth): add MFA e2e~~ test(auth): add MFA e2e Jul 6, 2022

Salakar approved these changes Jul 6, 2022

View reviewed changes

Salakar reviewed Jul 6, 2022

View reviewed changes

Lyokone added 2 commits July 6, 2022 17:16

fix(auth): fix reintroduce tests

eebf995

feat(auth): add documentation

43e35d8

russellwheatley approved these changes Jul 6, 2022

View reviewed changes

feat(auth): add e2e tests

36edcca

Lyokone force-pushed the feat/mfa branch from d29f201 to eb29b91 Compare July 7, 2022 07:16

Lyokone merged commit 0af57e3 into feat/mfa Jul 7, 2022

Lyokone deleted the feat/mfa-e2e branch July 7, 2022 07:20

firebase locked and limited conversation to collaborators Aug 7, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test(auth): add MFA e2e #9034

test(auth): add MFA e2e #9034

Lyokone commented Jul 5, 2022

Salakar Jul 6, 2022

russellwheatley left a comment

test(auth): add MFA e2e #9034

test(auth): add MFA e2e #9034

Conversation

Lyokone commented Jul 5, 2022

Description

Related Issues

Checklist

Breaking Change

Salakar Jul 6, 2022

Choose a reason for hiding this comment

russellwheatley left a comment

Choose a reason for hiding this comment