fix: Resolve bulk action handler test mocking issue

Author: ivo-toby

test/integration/bulk-action-handler.test.ts

@@ -2,6 +2,45 @@ import { expect, vi } from "vitest"
 import { bulkActionHandlers } from "../../src/handlers/bulk-action-handlers.js"
 import { server } from "../msw-setup.js"
 
+// Define constants before vi.mock since it's hoisted to the top
+const TEST_ENTRY_ID = "test-entry-id"
+const TEST_ASSET_ID = "test-asset-id"
+const TEST_BULK_ACTION_ID = "test-bulk-action-id"
+
+// Mock the getContentfulClient function to avoid actual API calls
+vi.mock("../../src/config/client.js", () => ({
+  getContentfulClient: vi.fn().mockResolvedValue({
+    entry: {
+      get: vi.fn().mockResolvedValue({
+        sys: { id: TEST_ENTRY_ID, version: 1 },
+      }),
+    },
+    asset: {
+      get: vi.fn().mockResolvedValue({
+        sys: { id: TEST_ASSET_ID, version: 1 },
+      }),
+    },
+    bulkAction: {
+      publish: vi.fn().mockResolvedValue({
+        sys: { id: TEST_BULK_ACTION_ID, status: "created" },
+      }),
+      unpublish: vi.fn().mockResolvedValue({
+        sys: { id: TEST_BULK_ACTION_ID, status: "created" },
+      }),
+      validate: vi.fn().mockResolvedValue({
+        sys: { id: TEST_BULK_ACTION_ID, status: "created" },
+      }),
+      get: vi.fn().mockResolvedValue({
+        sys: { id: TEST_BULK_ACTION_ID, status: "succeeded" },
+        succeeded: [
+          { sys: { id: TEST_ENTRY_ID, type: "Entry" } },
+          { sys: { id: TEST_ASSET_ID, type: "Asset" } },
+        ],
+      }),
+    },
+  }),
+}))
+
 describe("Bulk Action Handlers Integration Tests", () => {
   // Start MSW Server before tests
   beforeAll(() => server.listen())
@@ -10,42 +49,8 @@ describe("Bulk Action Handlers Integration Tests", () => {
 
   const testSpaceId = "test-space-id"
   const testEnvironmentId = "master"
-  const testEntryId = "test-entry-id"
-  const testAssetId = "test-asset-id"
-
-  // Mock the getContentfulClient function to avoid actual API calls
-  vi.mock("../../src/config/client.js", () => ({
-    getContentfulClient: vi.fn().mockResolvedValue({
-      entry: {
-        get: vi.fn().mockResolvedValue({
-          sys: { id: testEntryId, version: 1 },
-        }),
-      },
-      asset: {
-        get: vi.fn().mockResolvedValue({
-          sys: { id: testAssetId, version: 1 },
-        }),
-      },
-      bulkAction: {
-        publish: vi.fn().mockResolvedValue({
-          sys: { id: "test-bulk-action-id", status: "created" },
-        }),
-        unpublish: vi.fn().mockResolvedValue({
-          sys: { id: "test-bulk-action-id", status: "created" },
-        }),
-        validate: vi.fn().mockResolvedValue({
-          sys: { id: "test-bulk-action-id", status: "created" },
-        }),
-        get: vi.fn().mockResolvedValue({
-          sys: { id: "test-bulk-action-id", status: "succeeded" },
-          succeeded: [
-            { sys: { id: testEntryId, type: "Entry" } },
-            { sys: { id: testAssetId, type: "Asset" } },
-          ],
-        }),
-      },
-    }),
-  }))
+  const testEntryId = TEST_ENTRY_ID
+  const testAssetId = TEST_ASSET_ID
 
   describe("bulkPublish", () => {
     it("should publish multiple entries and assets", async () => {