Merge pull request #1241 from firebase/@invertase/bb-23

Author: Ehesp

packages/react/src/auth/forms/mfa/totp-multi-factor-enrollment-form.test.tsx

@@ -123,7 +123,7 @@ describe("useMultiFactorEnrollmentVerifyTotpFormAction", () => {
     });
 
     expect(TotpMultiFactorGeneratorAssertionMock).toHaveBeenCalledWith(mockSecret, "123456");
-    expect(enrollWithMultiFactorAssertionMock).toHaveBeenCalledWith(expect.any(Object), mockAssertion, "123456");
+    expect(enrollWithMultiFactorAssertionMock).toHaveBeenCalledWith(expect.any(Object), mockAssertion, "Test User");
   });
 
   it("should throw an unknown error when its not a FirebaseUIError", async () => {

packages/react/src/auth/forms/mfa/totp-multi-factor-enrollment-form.tsx

@@ -82,9 +82,17 @@ function TotpMultiFactorSecretGenerationForm(props: TotpMultiFactorSecretGenerat
 export function useMultiFactorEnrollmentVerifyTotpFormAction() {
   const ui = useUI();
   return useCallback(
-    async ({ secret, verificationCode }: { secret: TotpSecret; verificationCode: string; displayName: string }) => {
+    async ({
+      secret,
+      verificationCode,
+      displayName,
+    }: {
+      secret: TotpSecret;
+      verificationCode: string;
+      displayName: string;
+    }) => {
       const assertion = TotpMultiFactorGenerator.assertionForEnrollment(secret, verificationCode);
-      return await enrollWithMultiFactorAssertion(ui, assertion, verificationCode);
+      return await enrollWithMultiFactorAssertion(ui, assertion, displayName);
     },
     [ui]
   );