Fix ApiClient constructor mocks in tests

Author: mattinannt

packages/react-native/src/lib/environment/tests/state.test.ts

@@ -22,7 +22,7 @@ import type { TEnvironmentState } from "@/types/config";
 
 // Mock the FormbricksAPI so we can control environment.getState
 vi.mock("@/lib/common/api", () => ({
-  ApiClient: vi.fn().mockImplementation(() => {
+  ApiClient: vi.fn().mockImplementation(function MockApiClient() {
     return { getEnvironmentState: vi.fn() };
   }),
 }));
@@ -70,7 +70,7 @@ describe("environment/state.ts", () => {
   describe("fetchEnvironmentState()", () => {
     test("returns ok(...) with environment state", async () => {
       // Setup mock
-      (ApiClient as unknown as Mock).mockImplementationOnce(() => {
+      (ApiClient as unknown as Mock).mockImplementationOnce(function MockApiClient() {
         return {
           getEnvironmentState: vi.fn().mockResolvedValue({
             ok: true,
@@ -103,7 +103,7 @@ describe("environment/state.ts", () => {
         message: "Access denied",
       };
 
-      (ApiClient as unknown as Mock).mockImplementationOnce(() => {
+      (ApiClient as unknown as Mock).mockImplementationOnce(function MockApiClient() {
         return {
           getEnvironmentState: vi.fn().mockResolvedValue({
             ok: false,
@@ -131,7 +131,7 @@ describe("environment/state.ts", () => {
         responseMessage: "Network fail",
       };
 
-      (ApiClient as unknown as Mock).mockImplementationOnce(() => {
+      (ApiClient as unknown as Mock).mockImplementationOnce(function MockApiClient() {
         return {
           getEnvironmentState: vi.fn().mockRejectedValue(mockNetworkError),
         };
@@ -208,7 +208,7 @@ describe("environment/state.ts", () => {
 
       mockRNConfig.mockReturnValue(mockConfig as unknown as Promise<RNConfig>);
 
-      (ApiClient as Mock).mockImplementation(() => {
+      (ApiClient as Mock).mockImplementation(function MockApiClient() {
         return {
           getEnvironmentState: vi.fn().mockResolvedValue({
             ok: true,
@@ -244,7 +244,7 @@ describe("environment/state.ts", () => {
       mockRNConfig.mockReturnValue(mockConfig as unknown as Promise<RNConfig>);
 
       // Mock API to throw an error
-      (ApiClient as Mock).mockImplementation(() => {
+      (ApiClient as Mock).mockImplementation(function MockApiClient() {
         return {
           getEnvironmentState: vi
             .fn()
@@ -276,7 +276,7 @@ describe("environment/state.ts", () => {
 
       mockRNConfig.mockReturnValue(mockConfig as unknown as Promise<RNConfig>);
 
-      const apiMock = vi.fn().mockImplementation(() => {
+      const apiMock = vi.fn().mockImplementation(function MockApiClient() {
         return { getEnvironmentState: vi.fn() };
       });
 

packages/react-native/src/lib/user/tests/update.test.ts

@@ -34,7 +34,7 @@ vi.mock("@/lib/common/utils", () => ({
 }));
 
 vi.mock("@/lib/common/api", () => ({
-  ApiClient: vi.fn().mockImplementation(() => {
+  ApiClient: vi.fn().mockImplementation(function MockApiClient() {
     return { createOrUpdateUser: vi.fn() };
   }),
 }));
@@ -57,7 +57,7 @@ describe("sendUpdatesToBackend", () => {
       },
     };
 
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return { createOrUpdateUser: vi.fn().mockResolvedValue(mockResponse) };
     });
 
@@ -82,7 +82,7 @@ describe("sendUpdatesToBackend", () => {
       attributes: mockAttributes,
     };
 
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return {
         createOrUpdateUser: vi.fn().mockResolvedValue({
           ok: false,
@@ -116,7 +116,7 @@ describe("sendUpdatesToBackend", () => {
       attributes: { plan: "premium" },
     };
 
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return {
         createOrUpdateUser: vi
           .fn()
@@ -169,7 +169,7 @@ describe("sendUpdates", () => {
       },
     };
 
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return { createOrUpdateUser: vi.fn().mockResolvedValue(mockResponse) };
     });
 
@@ -193,7 +193,7 @@ describe("sendUpdates", () => {
       },
     };
 
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return {
         createOrUpdateUser: vi.fn().mockResolvedValue(mockErrorResponse),
       };
@@ -224,7 +224,7 @@ describe("sendUpdates", () => {
       },
     };
 
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return { createOrUpdateUser: vi.fn().mockResolvedValue(mockResponse) };
     });
 
@@ -261,7 +261,7 @@ describe("sendUpdates", () => {
       },
     };
 
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return { createOrUpdateUser: vi.fn().mockResolvedValue(mockResponse) };
     });
 
@@ -278,7 +278,7 @@ describe("sendUpdates", () => {
   });
 
   test("handles unexpected errors", async () => {
-    (ApiClient as Mock).mockImplementation(() => {
+    (ApiClient as Mock).mockImplementation(function MockApiClient() {
       return {
         createOrUpdateUser: vi
           .fn()