Merge pull request #243 from ajit3190/trigyn-testcase-summary-inciden…

…t-mrm Trigyn testcase summary incident mrm
primeroIMS · Aug 21, 2023 · ab5ed27 · ab5ed27
2 parents 838a5a0 + 22513cd
commit ab5ed27
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 41 deletions.
diff --git a/app/javascript/components/pages/admin/forms-list/components/drag-indicator/component.jsx b/app/javascript/components/pages/admin/forms-list/components/drag-indicator/component.jsx
@@ -8,7 +8,7 @@ const Component = ({ isDragDisabled, ...props }) => {
   const classes = isDragDisabled ? { classes: { root: css.dragIndicator } } : {};
 
   return (
-    <Icon {...props} {...classes}>
+    <Icon {...props} {...classes} data-testid="error-icon">
       <DragIndicatorIcon />
     </Icon>
   );

diff --git a/...script/components/pages/admin/forms-list/components/filters-expansion-panel/component.jsx b/...script/components/pages/admin/forms-list/components/filters-expansion-panel/component.jsx
@@ -6,7 +6,7 @@ import FilterInput from "../filter-input";
 
 const Component = ({ name, handleSetFilterValue, options, id, filterValues }) => {
   return (
-    <Accordion elevation={3} defaultExpanded>
+    <Accordion elevation={3} defaultExpanded data-testid="test">
       <AccordionSummary expandIcon={<ExpandMoreIcon />}>{name}</AccordionSummary>
       <AccordionDetails>
         <FilterInput

diff --git a/...ultiple-violations/component.unit.test.js → ...ren-multiple-violations/component.spec.js b/...ultiple-violations/component.unit.test.js → ...ren-multiple-violations/component.spec.js
@@ -1,13 +1,11 @@
 import { fromJS } from "immutable";
+import { mountedComponent, screen } from "test-utils";
 
-import { setupMountedComponent } from "../../../../test";
-import SubformField from "../../../record-form/form/subforms";
 import { FieldRecord, FormSectionRecord } from "../../../record-form/records";
 
 import ChildrenMultipleViolations from "./component";
 
 describe("<ChildrenMultipleViolations />", () => {
-  let component;
   const individualVictims = [
     {
       unique_id: "1234abcd",
@@ -48,20 +46,10 @@ describe("<ChildrenMultipleViolations />", () => {
   };
 
   beforeEach(() => {
-    ({ component } = setupMountedComponent(ChildrenMultipleViolations, props, {}, [], { values: individualVictims }));
-  });
-
-  it("should render <ChildrenMultipleViolations /> component", () => {
-    expect(component.find(ChildrenMultipleViolations)).to.have.lengthOf(1);
-  });
-
-  it("should render 1 <SubformField /> component", () => {
-    expect(component.find(SubformField)).to.have.lengthOf(1);
+    mountedComponent(<ChildrenMultipleViolations {...props} />, {}, {}, [], { values: individualVictims });
   });
 
   it("should render custom title", () => {
-    expect(component.find(SubformField).find("h3").text().trim()).to.equal(
-      "incidents.summary_mrm.fields.children_multiple_violation.label"
-    );
+    expect(screen.getByText("incidents.summary_mrm.fields.children_multiple_violation.label")).toBeInTheDocument();
   });
 });
diff --git a/...nts/summary-fields/component.unit.test.js → ...mponents/summary-fields/component.spec.js b/...nts/summary-fields/component.unit.test.js → ...mponents/summary-fields/component.spec.js
@@ -1,10 +1,8 @@
-import { setupMountedComponent } from "../../../../test";
-import { FormSectionField } from "../../../record-form";
+import { mountedComponent, screen } from "test-utils";
 
 import SummaryFields from "./component";
 
 describe("<SummaryFields />", () => {
-  let component;
   const values = {
     age: 10,
     case_id_display: "1234abcd",
@@ -29,15 +27,10 @@ describe("<SummaryFields />", () => {
   };
 
   beforeEach(() => {
-    ({ component } = setupMountedComponent(SummaryFields, props, {}, [], { values }));
+    mountedComponent(<SummaryFields {...props} />, {}, {}, [], { values });
   });
 
-  // TODO: Skipping for now. Causing timeout issue in cancelIdleCallback, rewrite in rtl
-  it.skip("should render <SummaryFields /> component", () => {
-    expect(component.find(SummaryFields)).to.have.lengthOf(1);
-  });
-
-  it.skip("should render 6 <FormSectionField /> component", () => {
-    expect(component.find(FormSectionField)).to.have.lengthOf(6);
+  it("should render custom title", () => {
+    expect(screen.getByText("incidents.summary_mrm.fields.summary_of_incident.label")).toBeInTheDocument();
   });
 });
diff --git a/...iolations-subforms/component.unit.test.js → ...nts/violations-subforms/component.spec.js b/...iolations-subforms/component.unit.test.js → ...nts/violations-subforms/component.spec.js
@@ -1,13 +1,11 @@
 import { fromJS } from "immutable";
+import { mountedComponent, screen } from "test-utils";
 
-import { setupMountedComponent } from "../../../../test";
-import SubformField from "../../../record-form/form/subforms";
 import { FieldRecord, FormSectionRecord } from "../../../record-form/records";
 
 import ViolationsSubforms from "./component";
 
 describe("<ViolationsSubforms />", () => {
-  let component;
   const values = {
     age: 10,
     case_id_display: "1234abcd",
@@ -44,18 +42,10 @@ describe("<ViolationsSubforms />", () => {
   };
 
   beforeEach(() => {
-    ({ component } = setupMountedComponent(ViolationsSubforms, props, {}, [], { values }));
-  });
-
-  it("should render <ViolationsSubforms /> component", () => {
-    expect(component.find(ViolationsSubforms)).to.have.lengthOf(1);
-  });
-
-  it("should render 1 <SubformField /> component", () => {
-    expect(component.find(SubformField)).to.have.lengthOf(1);
+    mountedComponent(<ViolationsSubforms {...props} />, {}, {}, [], { values });
   });
 
   it("should render custom title", () => {
-    expect(component.find(SubformField).find("h3").text().trim()).to.equal("Killing of Children");
+    expect(screen.getByText("Killing of Children")).toBeInTheDocument();
   });
 });