test: Upadte test cases

theopensystemslab · Dec 9, 2024 · 5a3e427 · 5a3e427
1 parent a6f1452
commit 5a3e427
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 21 deletions.
diff --git a/editor.planx.uk/src/@planx/components/Pay/Editor/Editor.test.tsx b/editor.planx.uk/src/@planx/components/Pay/Editor/Editor.test.tsx
@@ -117,7 +117,7 @@ describe("Pay component - Editor Modal", () => {
             <PayComponent
               id="test"
               handleSubmit={handleSubmit}
-              node={{ data: { fn: "fee" } }}
+              node={{ data: { fn: "application.fee.payable" } }}
             />
           </DndProvider>,
         );
@@ -146,7 +146,7 @@ describe("Pay component - Editor Modal", () => {
       act(() => setState({ user: mockUser, flowName: "test flow" }));
       const mockNode = {
         data: {
-          fn: "fee",
+          fn: "application.fee.payable",
           govPayMetadata: [
             { key: "flow", value: "flowName" },
             { key: "source", value: "PlanX" },
@@ -260,7 +260,7 @@ describe("Pay component - Editor Modal", () => {
           <PayComponent
             id="test"
             handleSubmit={handleSubmit}
-            node={{ data: { fn: "fee" } }}
+            node={{ data: { fn: "application.fee.payable" } }}
           />
         </DndProvider>,
       );

diff --git a/editor.planx.uk/src/@planx/components/Pay/Public/Pay.fixture.tsx b/editor.planx.uk/src/@planx/components/Pay/Public/Pay.fixture.tsx
@@ -8,7 +8,7 @@ export default function Fixture() {
       handleSubmit={console.log}
       title="Pay"
       description=""
-      fn="fee"
+      fn="application.fee.payable"
       color="#efefef"
       govPayMetadata={[]}
     />

diff --git a/editor.planx.uk/src/@planx/components/Pay/Public/Pay.test.tsx b/editor.planx.uk/src/@planx/components/Pay/Public/Pay.test.tsx
@@ -32,18 +32,10 @@ const flowWithUndefinedFee: Store.Flow = {
   _root: {
     edges: ["setValue", "pay"],
   },
-  setValue: {
-    type: TYPES.SetValue,
-    edges: ["pay"],
-    data: {
-      fn: "application.fee.payable",
-      val: "0",
-    },
-  },
   pay: {
     type: TYPES.Pay,
     data: {
-      fn: "application.fee.typo",
+      fn: "application.fee.payable",
     },
   },
 };
@@ -92,22 +84,21 @@ const defaultProps = {
 };
 
 describe("Pay component when fee is undefined or £0", () => {
-  beforeEach(() => {
-    getState().resetPreview();
-  });
+  beforeAll(() => (initialState = getState()));
+  afterEach(() => act(() => setState(initialState)));
 
   it("Shows an error if fee is undefined", () => {
     const handleSubmit = vi.fn();
 
-    setState({ flow: flowWithUndefinedFee, breadcrumbs: breadcrumbs });
+    setState({ flow: flowWithUndefinedFee, breadcrumbs: {} });
     expect(getState().computePassport()).toEqual({
-      data: { "application.fee.payable": ["0"] },
+      data: { "application.fee.payable": undefined },
     });
 
     setup(
       <Pay
         title="Pay for your application"
-        fn="application.fee.typo"
+        fn="application.fee.payable"
         handleSubmit={handleSubmit}
         govPayMetadata={[]}
       />,
@@ -456,6 +447,8 @@ describe("Confirm component in information-only mode", () => {
 });
 
 describe("the demo user view", () => {
+  beforeAll(() => (initialState = getState()));
+
   beforeEach(() => {
     act(() =>
       setState({
@@ -464,14 +457,16 @@ describe("the demo user view", () => {
     );
   });
 
+  afterEach(() => act(() => setState(initialState)));
+
   it("should render an error when teamSlug is demo", async () => {
     const handleSubmit = vi.fn();
     const { queryByText } = setup(
       <Pay
-        title="Pay for your application"
-        fn="application.fee.typo"
+        fn="application.fee.payable"
         handleSubmit={handleSubmit}
         govPayMetadata={[]}
+        {...defaultProps}
       />,
     );
     const errorHeader = queryByText("GOV.UK Pay is not enabled for demo users");