Merge pull request #1188 from firebase/@invertase/align-react-tests

Author: Ehesp

packages/core/src/auth.ts

@@ -213,7 +213,7 @@ export async function signInAnonymously(ui: FirebaseUIConfiguration): Promise<Us
   }
 }
 
-export async function signInWithOAuth(ui: FirebaseUIConfiguration, provider: AuthProvider): Promise<void> {
+export async function signInWithProvider(ui: FirebaseUIConfiguration, provider: AuthProvider): Promise<void> {
   try {
     if (hasBehavior(ui, "autoUpgradeAnonymousProvider")) {
       await getBehavior(ui, "autoUpgradeAnonymousProvider")(ui, provider);
@@ -222,6 +222,8 @@ export async function signInWithOAuth(ui: FirebaseUIConfiguration, provider: Aut
     }
 
     ui.setState("pending");
+
+    // TODO(ehesp): Handle popup or redirect based on behavior
     await signInWithRedirect(ui.auth, provider);
     // We don't modify state here since the user is redirected.
     // If we support popups, we'd need to modify state here.

packages/core/src/config.ts

@@ -21,7 +21,7 @@ import { deepMap, DeepMapStore, map } from "nanostores";
 import { Behavior, type BehaviorHandlers, type BehaviorKey, getBehavior, hasBehavior } from "./behaviors";
 import { FirebaseUIState } from "./state";
 
-type FirebaseUIConfigurationOptions = {
+export type FirebaseUIConfigurationOptions = {
   app: FirebaseApp;
   auth?: Auth;
   locale?: RegisteredLocale;

packages/core/tests/unit/auth.test.ts

@@ -41,7 +41,7 @@ import {
   sendSignInLinkToEmail,
   signInWithEmailLink,
   signInAnonymously,
-  signInWithOAuth,
+  signInWithProvider,
   completeEmailLinkSignIn,
 } from "../../src/auth";
 import { FirebaseUIConfiguration } from "../../src/config";
@@ -302,11 +302,11 @@ describe("Firebase UI Auth", () => {
     });
   });
 
-  describe("signInWithOAuth", () => {
+  describe("signInWithProvider", () => {
     it("should sign in with OAuth provider", async () => {
       (signInWithRedirect as any).mockResolvedValue(undefined);
 
-      await signInWithOAuth(mockUi, mockProvider as any);
+      await signInWithProvider(mockUi, mockProvider as any);
 
       expect(signInWithRedirect).toHaveBeenCalledWith(mockAuth, mockProvider);
     });
@@ -318,7 +318,7 @@ describe("Firebase UI Auth", () => {
 
       mockUi.behaviors.autoUpgradeAnonymousProvider = vi.fn();
 
-      await signInWithOAuth(mockUi, mockProvider as any);
+      await signInWithProvider(mockUi, mockProvider as any);
 
       expect(mockUi.behaviors.autoUpgradeAnonymousProvider).toHaveBeenCalledWith(mockUi, mockProvider);
     });

packages/react/package.json

@@ -43,7 +43,7 @@
     "@firebase-ui/styles": "workspace:*",
     "@nanostores/react": "^0.8.4",
     "@radix-ui/react-slot": "^1.2.3",
-    "@tanstack/react-form": "^0.41.3",
+    "@tanstack/react-form": "^1.20.0",
     "clsx": "^2.1.1",
     "tailwind-merge": "^3.0.1",
     "zod": "catalog:"

packages/react/setup-test.ts

@@ -14,8 +14,4 @@
  * limitations under the License.
  */
 
-import { expect } from "vitest";
-import * as matchers from "@testing-library/jest-dom/matchers";
-
-// Extend Vitest's expect with jest-dom matchers
-expect.extend(matchers);
+import '@testing-library/jest-dom/vitest';