fix: console.error

Author: FredrikOseberg

src/useFlagContext.test.ts

@@ -1,17 +1,24 @@
-import { renderHook } from '@testing-library/react-hooks/native';
+import { renderHook } from '@testing-library/react';
+import { vi, test, expect } from 'vitest';
 import FlagProvider from "./FlagProvider";
 import { useFlagContext } from "./useFlagContext";
 
-test("throws an error if used outside of a FlagProvider", () => {
+test("logs an error if used outside of a FlagProvider", () => {
+    const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+    
     const { result } = renderHook(() => useFlagContext());
+    expect(consoleSpy).toHaveBeenCalledWith("This hook must be used within a FlagProvider");
+    expect(result.current).toBeNull();
 
-    expect(result.error).toEqual(
-        Error("This hook must be used within a FlagProvider")
-    );
+    consoleSpy.mockRestore();
 });
 
-test("does not throw an error if used inside of a FlagProvider", () => {
+test("does not log an error if used inside of a FlagProvider", () => {
+    const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+    
     const { result } = renderHook(() => useFlagContext(), { wrapper: FlagProvider });
+    expect(consoleSpy).not.toHaveBeenCalled();
+    expect(result.current).not.toBeNull();
 
-    expect(result.error).toBeUndefined();
+    consoleSpy.mockRestore();
 });

src/useFlagContext.ts

@@ -4,7 +4,8 @@ import FlagContext from './FlagContext';
 export function useFlagContext() {
     const context = useContext(FlagContext);
     if (!context) {
-      throw new Error('This hook must be used within a FlagProvider');
+      console.error('This hook must be used within a FlagProvider');
+      return null;
     }
     return context;
 }