test: fix most vitests

Author: BCerki

bciers/apps/administration/tests/components/operations/OperationInformationForm.test.tsx

@@ -10,7 +10,11 @@ import {
 import {
   downloadUrl,
   downloadUrl2,
-} from "libs/components/src/form/widgets/FileWidget.test";
+  mock21MBFile,
+  mockFile,
+  mockFile2,
+  mockFileUnaccepted,
+} from "libs/testConfig/src/constants";
 
 import { createAdministrationOperationInformationSchema } from "apps/administration/app/data/jsonSchema/operationInformation/administrationOperationInformation";
 import { Apps, FrontEndRoles, OperationStatus } from "@bciers/utils/src/enums";
@@ -30,10 +34,6 @@ useSearchParams.mockReturnValue({
   get: vi.fn(),
 });
 
-global.URL.createObjectURL = vi.fn(
-  () => "this is the link to download the File",
-);
-
 // Just using a simple schema for testing purposes
 const testSchema: RJSFSchema = {
   type: "object",
@@ -176,6 +176,9 @@ const newEntrantFormData = {
 const operationId = "8be4c7aa-6ab3-4aad-9206-0ef914fea063";
 
 describe("the OperationInformationForm component", () => {
+  global.URL.createObjectURL = vi.fn(
+    () => "this is the link to download the File",
+  );
   beforeEach(() => {
     vi.clearAllMocks();
   });
@@ -346,15 +349,15 @@ describe("the OperationInformationForm component", () => {
     expect(actionHandler).toHaveBeenCalledTimes(1);
     expect(actionHandler).toHaveBeenCalledWith(
       `registration/operations/${operationId}`,
-      "PUT",
+      "POST",
       "",
       {
-        body: JSON.stringify({
-          name: "Operation 4",
-          type: "Single Facility Operation",
-        }),
+        body: expect.any(FormData),
       },
     );
+    const formData = actionHandler.mock.calls[0][3].body;
+    expect(formData.get("name")).toBe("Operation 4");
+    expect(formData.get("type")).toBe("Single Facility Operation");
 
     // Expect the form to be submitted
     expect(screen.getByText(/Operation 4/i)).toBeVisible();
@@ -835,9 +838,7 @@ describe("the OperationInformationForm component", () => {
     );
 
     await userEvent.click(afterAprilRadioButton);
-    const mockFile = new File(["test"], "mock_file.pdf", {
-      type: "application/pdf",
-    });
+
     const newEntrantApplicationDocument = screen.getByLabelText(
       /new entrant application and statutory declaration/i,
     );
@@ -850,19 +851,21 @@ describe("the OperationInformationForm component", () => {
     expect(actionHandler).toHaveBeenCalledTimes(1);
     expect(actionHandler).toHaveBeenCalledWith(
       `registration/operations/${operationId}`,
-      "PUT",
+      "POST",
       "",
       {
-        body: JSON.stringify({
-          name: "Operation 5",
-          type: "Single Facility Operation",
-          registration_purpose: "New Entrant Operation",
-          date_of_first_shipment: "On or after April 1, 2024",
-          new_entrant_application:
-            "data:application/pdf;name=mock_file.pdf;base64,dGVzdA==",
-        }),
+        body: expect.any(FormData),
       },
     );
+    const formData = actionHandler.mock.calls[0][3].body;
+
+    expect(formData.get("name ")).toBe("Operation 5");
+    expect(formData.get("type ")).toBe("Single Facility Operation");
+    expect(formData.get("registration_purpose ")).toBe("New Entrant Operation");
+    expect(formData.get("date_of_first_shipment ")).toBe(
+      "On or after April 1, 2024",
+    );
+    expect(formData.get("new_entrant_application")).toBe(mockFile);
   });
 
   it("should not allow external users to remove their operation rep", async () => {
@@ -903,7 +906,7 @@ describe("the OperationInformationForm component", () => {
     expect(screen.getByText(/Must not have fewer than 1 items/i)).toBeVisible();
   });
 
-  it(
+  it.only(
     "should allow external users to replace their operation rep",
     { timeout: 100000 },
     async () => {
@@ -919,7 +922,7 @@ describe("the OperationInformationForm component", () => {
         opt_in: false,
         operation_representatives: [1],
         boundary_map: downloadUrl,
-        process_flow_diagram: downloadUrl,
+        process_flow_diagram: downloadUrl2,
       };
       useSession.mockReturnValue({
         data: {
@@ -970,23 +973,25 @@ describe("the OperationInformationForm component", () => {
       expect(actionHandler).toHaveBeenCalledTimes(1);
       expect(actionHandler).toHaveBeenCalledWith(
         `registration/operations/${operationId}`,
-        "PUT",
+        "POST",
         "",
         {
-          body: JSON.stringify({
-            name: "Operation 3",
-            type: "Single Facility Operation",
-            naics_code_id: 1,
-            secondary_naics_code_id: 2,
-            process_flow_diagram: downloadUrl,
-            boundary_map: downloadUrl2,
-            operation_has_multiple_operators: false,
-            registration_purpose: "Reporting Operation",
-            operation_representatives: [2],
-            activities: [1, 2],
-          }),
+          body: expect.any(FormData),
         },
       );
+      const formData = actionHandler.mock.calls[0][3].body;
+
+      expect(formData.get("name")).toBe("Operation 3");
+      expect(formData.get("type")).toBe("Single Facility Operation");
+      expect(formData.get("naics_code_id")).toBe("1");
+      expect(formData.get("secondary_naics_code_id")).toBe("2");
+      expect(formData.get("boundary_map")).toBe(downloadUrl);
+      expect(formData.get("process_flow_diagram")).toBe(downloadUrl2);
+      expect(formData.get("operation_has_multiple_operators")).toBe("false");
+      expect(formData.get("registration_purpose")).toBe("Reporting Operation"),
+        expect(formData.get("operation_representatives")).toBe("2");
+      expect(formData.get("activities")).toBe("1");
+      expect(formData.get("activities")).toBe("2");
     },
   );
 

bciers/apps/registration/app/components/operations/registration/OperationInformationForm.tsx

@@ -290,7 +290,7 @@ const OperationInformationForm = ({
           setConfirmedFormState(e.formData);
         }}
         uiSchema={currentUiSchema}
-        // customValidate={customValidate}
+        customValidate={customValidate}
       />
     </>
   );

bciers/apps/registration/tests/components/operations/registration/NewEntrantOperationForm.test.tsx

@@ -15,7 +15,11 @@ import { actionHandler, useRouter, useSession } from "@bciers/testConfig/mocks";
 import {
   downloadUrl,
   downloadUrl2,
-} from "libs/components/src/form/widgets/FileWidget.test";
+  mock21MBFile,
+  mockFile,
+  mockFile2,
+  mockFileUnaccepted,
+} from "libs/testConfig/src/constants";
 
 useSession.mockReturnValue({
   data: {
@@ -31,15 +35,12 @@ useRouter.mockReturnValue({
   push: mockPush,
 });
 
-global.URL.createObjectURL = vi.fn(
-  () => "this is the link to download the File",
-);
-
-const mockFile = new File(["test"], "test.pdf", { type: "application/pdf" });
-
 describe("the NewEntrantOperationForm component", () => {
   beforeEach(() => {
     vi.clearAllMocks();
+    global.URL.createObjectURL = vi.fn(
+      () => "this is the link to download the File",
+    );
   });
 
   it("should render the NewEntrantOperationForm component", () => {
@@ -199,12 +200,17 @@ describe("the NewEntrantOperationForm component", () => {
 
     expect(actionHandler).toHaveBeenCalledWith(
       "registration/operations/002d5a9e-32a6-4191-938c-2c02bfec592d/registration/new-entrant-application",
-      "PUT",
+      "POST",
       "/register-an-operation/002d5a9e-32a6-4191-938c-2c02bfec592d",
       {
-        body: '{"new_entrant_application":"data:application/pdf;name=test.pdf;base64,dGVzdA==","date_of_first_shipment":"On or before March 31, 2024"}',
+        body: expect.any(FormData),
       },
     );
+    const formData = actionHandler.mock.calls[0][3].body;
+    expect(formData.get("new_entrant_application")).toBe(mockFile);
+    expect(formData.get("date_of_first_shipment")).toBe(
+      "On or before March 31, 2024",
+    );
     await waitFor(() => {
       expect(mockPush).toHaveBeenCalledWith(
         "/register-an-operation/002d5a9e-32a6-4191-938c-2c02bfec592d/5",