Fixing attachments tests

Author: aespinoza-quoin

app/javascript/components/record-form/form/field-types/attachments/attachment-field.spec.js

@@ -1,4 +1,4 @@
-import { mountedComponent, screen } from "../../../../../test-utils";
+import { mountedComponent, screen, fireEvent } from "../../../../../test-utils";
 
 import { ATTACHMENT_TYPES } from "./constants";
 import AttachmentField from "./attachment-field";
@@ -21,8 +21,9 @@ describe("<AttachmentField />", () => {
     initialValues: {}
   };
 
-  it("should render ActionDialog", () => {
+  it("should render component", () => {
     mountedComponent(<AttachmentField {...props} />, {}, [], {}, formProps);
+    expect(document.querySelector(".uploadBox")).toBeInTheDocument();
     expect(screen.getByText("fields.file_upload_box.select_file_button_text")).toBeInTheDocument();
   });
 
@@ -32,14 +33,27 @@ describe("<AttachmentField />", () => {
   });
 
   describe("when value contains attachmentUrl", () => {
+    const newProps = {
+      ...props,
+      value: {
+        attachment_url: "random-string"
+      }
+    };
+
+    it("should render ActionDialog", () => {
+      mountedComponent(<AttachmentField {...newProps} />, {}, [], {}, formProps);
+      fireEvent.click(screen.getByRole("button"));
+      expect(screen.getByText("fields.remove attachment_field_test")).toBeInTheDocument();
+    });
+
     it("should not render the AttachmentInput", () => {
-      mountedComponent(<AttachmentField {...props} />, {}, [], {}, formProps);
+      mountedComponent(<AttachmentField {...newProps} />, {}, [], {}, formProps);
       expect(screen.queryByRole("textbox")).toBeNull();
     });
 
     it("should render the AttachmentPreview", () => {
-      mountedComponent(<AttachmentField {...props} />, {}, [], {}, formProps);
-      expect(screen.getByText("fields.file_upload_box.select_file_button_text")).toBeInTheDocument();
+      mountedComponent(<AttachmentField {...newProps} />, {}, [], {}, formProps);
+      expect(screen.getByRole("img")).toBeInTheDocument();
     });
   });
 

app/javascript/components/record-form/form/field-types/attachments/attachment-preview.spec.js

@@ -23,6 +23,7 @@ describe("<AttachmentPreview />", () => {
 
   it("should render an img", () => {
     mountedComponent(<AttachmentPreview {...props} />, {}, [], {}, formProps);
+    expect(screen.getByRole("img")).toBeInTheDocument();
     expect(screen.getByTestId("attachment")).toBeInTheDocument();
   });
 });

app/javascript/components/record-form/form/field-types/attachments/render-preview.spec.js

@@ -18,7 +18,6 @@ describe("renderPreview", () => {
     };
 
     simpleMountedComponent(renderPreview(ATTACHMENT_TYPES.document, file, css, deleteButton));
-    renderPreview(ATTACHMENT_TYPES.document, file, css, deleteButton);
     expect(screen.getByText(/test.txt/i)).toBeInTheDocument();
   });