Fixing flagging test

Author: aespinoza-quoin

app/javascript/components/flagging/components/dialog-tabs/component.spec.js

@@ -15,6 +15,8 @@ describe("<DialogTabs /> - Component", () => {
   });
 
   it("should render the DialogTabs", () => {
+    screen.debug();
+    expect(screen.getByText("flags.flags_tab")).toBeInTheDocument();
     expect(screen.getByText("flags.add_flag_tab")).toBeInTheDocument();
   });
 });

app/javascript/components/flagging/components/flag-dialog/component.spec.js.js app/javascript/components/flagging/components/flag-dialog/component.spec.js

@@ -18,8 +18,13 @@ describe("<FlagDialog />", () => {
     expect(screen.getByText("flags.add_flag_tab")).toBeInTheDocument();
   });
 
+  it("should render the ActionDialog", () => {
+    mountedComponent(<FlagDialog {...props} />);
+    expect(screen.getByText("flags.title")).toBeInTheDocument();
+  });
+
   it("should render the DialogTabs", () => {
     mountedComponent(<DialogTabs {...props} />);
-    expect(screen.getAllByRole("tab")).toBeTruthy();
+    expect(screen.getAllByRole("tab")).toHaveLength(2);
   });
 });

app/javascript/components/flagging/components/flag-form/component.spec.js

@@ -13,7 +13,7 @@ describe("<FlagForm />", () => {
 
   it("renders Form", () => {
     mountedComponent(<FlagForm {...props} />);
-    expect(screen.getAllByText("flags.flag_date")).toBeTruthy();
+    expect(screen.getAllByText("flags.flag_date")).toHaveLength(2);
   });
 
   it("should render the FlagForm", () => {

app/javascript/components/flagging/components/list-flags-item-actions/component.spec.js

@@ -1,4 +1,4 @@
-import { mountedComponent, screen } from "test-utils";
+import { mountedComponent, screen, cleanup } from "test-utils";
 import { fromJS } from "immutable";
 
 import ListFlagsItemActions from "./component";
@@ -30,7 +30,30 @@ describe("<ListFlagsItemActions />", () => {
     expect(screen.getByText("flags.date")).toBeInTheDocument();
   });
 
+  it("should render the FormAction", () => {
+    expect(screen.getByText("flags.resolve_button")).toBeInTheDocument();
+  });
+
   it("should render the DateFlag", () => {
-    expect(screen.getByTestId("dateflag")).toBeInTheDocument();
+    expect(screen.getByTestId("date")).toBeInTheDocument();
+  });
+
+  describe("when user has NOT resolve_any_flag permission", () => {
+    const stateDiffentUser = fromJS({
+      user: {
+        username: "primero_cp"
+      }
+    });
+
+    beforeEach(() => {
+      cleanup();
+      mountedComponent(<ListFlagsItemActions {...props} />, stateDiffentUser);
+    });
+
+    it("should NOT render the ActionButton", () => {
+      const buttonElement = screen.queryByRole("button", { name: /resolve_button/i });
+
+      expect(buttonElement).not.toBeInTheDocument();
+    });
   });
 });

app/javascript/components/flagging/components/list-flags-item/component.spec.js

@@ -1,9 +1,9 @@
 import { mountedComponent, screen } from "test-utils";
 import { fromJS } from "immutable";
 
-import ListFlagsItemActions from "./component";
+import ListFlagsItem from "./component";
 
-describe("<ListFlagsItemActions />", () => {
+describe("<ListFlagsItem />", () => {
   const props = {
     flag: {
       id: 7,
@@ -23,14 +23,22 @@ describe("<ListFlagsItemActions />", () => {
   });
 
   beforeEach(() => {
-    mountedComponent(<ListFlagsItemActions {...props} />, initialState);
+    mountedComponent(<ListFlagsItem {...props} />, initialState);
   });
 
   it("should render the ListFlagsItem", () => {
     expect(screen.getByText("flags.date")).toBeInTheDocument();
   });
 
   it("should render the DateFlag", () => {
-    expect(screen.getByTestId("dateflag")).toBeInTheDocument();
+    expect(screen.getByTestId("date")).toBeInTheDocument();
+  });
+
+  it("should render the ListItemText", () => {
+    expect(screen.getByText("This is a flag 1")).toBeInTheDocument();
+  });
+
+  it("should render the ListFlagsItemActions", () => {
+    expect(screen.getByText("flags.resolve_button")).toBeInTheDocument();
   });
 });

app/javascript/components/flagging/components/list-flags/component.spec.js

@@ -22,7 +22,7 @@ describe("<ListFlags />", () => {
             date: "2019-08-01",
             message: "This is a flag 1",
             flagged_by: "primero",
-            removed: false
+            removed: true
           })
         ]
       }
@@ -38,4 +38,9 @@ describe("<ListFlags />", () => {
     mountedComponent(<ListFlags {...props} />, initialState);
     expect(screen.getByRole("listitem")).toBeInTheDocument();
   });
+
+  it("should render the FlagForm", () => {
+    mountedComponent(<ListFlags {...props} />, initialState);
+    expect(screen.getByText("flags.resolved", { selector: "h3" })).toBeInTheDocument();
+  });
 });

app/javascript/components/flagging/components/unflag/component.spec.js

@@ -26,7 +26,7 @@ describe("<Unflag />", () => {
   });
 
   it("should render the Unflag", () => {
-    expect(screen.getAllByText("flags.resolve_reason")).toBeTruthy();
+    expect(screen.getAllByText("flags.resolve_reason")).toHaveLength(2);
   });
 
   it("renders ActionButton", () => {