chore(test): added emailmfa attribute to first user sign up in test (#6141)

* changed mfa preference flow for verifying if we can set sms as preferred and forego selection of mfa type upon sign in

* removed duplicate test and changed "because" clause of test

* changed test text to properly reflect the flow of the test

Author: ekjotmultani

packages/auth/amplify_auth_cognito/example/integration_test/mfa_sms_email_required_test.dart

@@ -62,6 +62,7 @@ void main() {
           username: username,
           password: password,
         );
+
         check(
           resignInRes.nextStep.signInStep,
         ).equals(AuthSignInStep.confirmSignInWithOtpCode);
@@ -99,9 +100,18 @@ void main() {
           username: username,
           password: password,
         );
+
         check(
           signInRes.nextStep.signInStep,
-          because: 'MFA is required so Cognito automatically enables SMS MFA',
+          because: 'MFA is required so Cognito will ask for MFA type',
+        ).equals(AuthSignInStep.continueSignInWithMfaSelection);
+
+        final selectMfaRes = await Amplify.Auth.confirmSignIn(
+          confirmationValue: 'SMS',
+        );
+
+        check(
+          selectMfaRes.nextStep.signInStep,
         ).equals(AuthSignInStep.confirmSignInWithSmsMfaCode);
 
         final confirmRes = await Amplify.Auth.confirmSignIn(
@@ -215,9 +225,18 @@ void main() {
           username: username,
           password: password,
         );
+
         check(
           signInRes.nextStep.signInStep,
-          because: 'MFA is required so Cognito automatically enables SMS MFA',
+          because: 'MFA is required so Cognito prompts MFA type selection',
+        ).equals(AuthSignInStep.continueSignInWithMfaSelection);
+
+        final selectMfaRes = await Amplify.Auth.confirmSignIn(
+          confirmationValue: 'SMS',
+        );
+
+        check(
+          selectMfaRes.nextStep.signInStep,
         ).equals(AuthSignInStep.confirmSignInWithSmsMfaCode);
 
         final confirmRes = await Amplify.Auth.confirmSignIn(
@@ -227,44 +246,15 @@ void main() {
 
         check(
           await cognitoPlugin.fetchMfaPreference(),
-          because: 'MFA is required so Cognito automatically enables SMS MFA',
+          because:
+              'SMS MFA has been selected so Cognito fetches SMS as preferred',
         ).equals(
           const UserMfaPreference(
             enabled: {MfaType.sms},
             preferred: MfaType.sms,
           ),
         );
 
-        // Verify we can set SMS as preferred and forego selection.
-
-        {
-          await signOutUser(assertComplete: true);
-
-          final mfaCode = await getOtpCode(UserAttribute.phone(phoneNumber));
-          final signInRes = await Amplify.Auth.signIn(
-            username: username,
-            password: password,
-          );
-          check(
-            signInRes.nextStep.signInStep,
-            because: 'Preference is SMS MFA now',
-          ).equals(AuthSignInStep.confirmSignInWithSmsMfaCode);
-          check(signInRes.nextStep.codeDeliveryDetails).isNotNull()
-            ..has(
-              (d) => d.deliveryMedium,
-              'deliveryMedium',
-            ).equals(DeliveryMedium.sms)
-            ..has(
-              (d) => d.destination,
-              'destination',
-            ).isNotNull().startsWith('+');
-
-          final confirmRes = await Amplify.Auth.confirmSignIn(
-            confirmationValue: await mfaCode.code,
-          );
-          check(confirmRes.nextStep.signInStep).equals(AuthSignInStep.done);
-        }
-
         // Verify we can switch to EMAIL as preferred.
 
         await cognitoPlugin.updateMfaPreference(email: MfaPreference.preferred);