useAuth should not throw

authts · Aug 23, 2023 · 1afc47e · 1afc47e
1 parent 4f0d303
commit 1afc47e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 13 deletions.
diff --git a/src/useAuth.ts b/src/useAuth.ts
@@ -9,8 +9,8 @@ export const useAuth = (): AuthContextProps => {
     const context = React.useContext(AuthContext);
 
     if (!context) {
-        throw new Error("AuthProvider context is undefined, please verify you are calling useAuth() as child of a <AuthProvider> component.");
+        console.warn("AuthProvider context is undefined, please verify you are calling useAuth() as child of a <AuthProvider> component.");
     }
 
-    return context;
+    return context as AuthContextProps;
 };
diff --git a/test/useAuth.test.tsx b/test/useAuth.test.tsx
@@ -18,16 +18,8 @@ describe("useAuth", () => {
         await waitFor(() => expect(result.current).toBeDefined());
     });
 
-    it("should throw with no provider", async () => {
-        // act
-        try {
-            renderHook(() => useAuth());
-        } catch (err) {
-            //assert
-            expect(err).toBeInstanceOf(Error);
-            expect((err as Error).message).toContain(
-                "AuthProvider context is undefined, please verify you are calling useAuth() as child of a <AuthProvider> component.",
-            );
-        }
+    it("should return undefined with no provider", async () => {
+        const { result } = renderHook(() => useAuth());
+        expect(result.current).toBeUndefined();
     });
 });