Merge pull request #457 from primeroIMS/trigyn-testcases-record-actions-transitions-components-referrals

Trigyn testcases record actions transitions components referrals

Author: aespinoza-quoin

app/javascript/components/record-actions/transitions/components/referrals/component.spec.js

@@ -2,7 +2,7 @@
 
 import { fromJS } from "immutable";
 
-import { mountedComponent, screen, cleanup } from "../../../../../test-utils";
+import { mountedComponent, screen } from "../../../../../test-utils";
 import users from "../../mocked-users";
 
 import ReferralForm from "./component";

app/javascript/components/record-actions/transitions/components/referrals/form-internal.spec.js

@@ -5,7 +5,7 @@ import { mountedComponent, screen } from "../../../../../test-utils";
 
 import FormInternal from "./form-internal";
 
-const InternalForm = props => {
+function InternalForm(props) {
   const formProps = {
     initialValues: {
       agency: "",
@@ -21,7 +21,8 @@ const InternalForm = props => {
       </Form>
     </Formik>
   );
-};
+}
+InternalForm.displayName = "InternalForm";
 
 describe("<FormInternal />", () => {
   const props = {
@@ -56,11 +57,6 @@ describe("<FormInternal />", () => {
 
   it("renders Field", () => {
     mountedComponent(<InternalForm {...props} />);
-    expect(screen.queryAllByRole("textbox")).toHaveLength(3);
-  });
-
-  it("renders TextFieldProps from SearchableSelect with valid props", () => {
-    mountedComponent(<InternalForm {...props} />);
-    expect(screen.queryAllByRole("textbox")).toHaveLength(3);
+    expect(document.querySelectorAll("input")).toHaveLength(3);
   });
 });

app/javascript/components/record-actions/transitions/components/referrals/provided-consent.spec.js

@@ -1,6 +1,5 @@
 /* eslint-disable no-unused-expressions */
 import { mountedComponent, screen } from "../../../../../test-utils";
-import { RECORD_TYPES } from "../../../../../config";
 
 import ProvidedConsent from "./provided-consent";
 
@@ -53,15 +52,4 @@ describe("<ProvidedConsent /> - referrals", () => {
     expect(screen.getByText(/referral.provided_consent_label/i)).toBeInTheDocument();
   });
 
-  it("should render <ProvidedForm> with valid props", () => {
-    const props = {
-      canConsentOverride: true,
-      providedConsent: false,
-      setDisabled: () => {},
-      recordType: RECORD_TYPES.cases
-    };
-
-    mountedComponent(<ProvidedConsent {...props} />, {}, [], [], formProps);
-    expect(screen.queryAllByTestId("form-control")).toHaveLength(1);
-  });
 });

app/javascript/components/record-actions/transitions/components/referrals/provided-form.spec.js

@@ -24,14 +24,15 @@ describe("<ProvidedForm /> - referrals", () => {
   });
 
   it("should render some components when user can not override consent", () => {
-    const props = {
+    const canNotOverrideProps = {
       canConsentOverride: false
     };
 
-    mountedComponent(<ProvidedForm {...props} />, {}, [], [], formProps);
-
+    mountedComponent(<ProvidedForm {...canNotOverrideProps} />, {}, [], [], formProps);
     expect(screen.queryAllByTestId("grid")).toHaveLength(3);
     expect(screen.queryByTestId("form-control")).toBeNull();
+    expect(screen.queryAllByRole("checkbox")).toHaveLength(0);
     expect(screen.getByText(/referral.provided_consent_label/i)).toBeInTheDocument();
+
   });
 });