better unit test

Enterprise-CMCS · Feb 6, 2025 · e41a089 · e41a089
1 parent 104f6bc
commit e41a089
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 5 deletions.
diff --git a/services/ui-src/src/components/report/MeasureFooter.test.tsx b/services/ui-src/src/components/report/MeasureFooter.test.tsx
@@ -38,23 +38,39 @@ describe("Measure Footer", () => {
       />
     );
 
+    //click previous
     const previousLink = screen.getByRole("button", {
       name: "Previous",
     });
     await userEvent.click(previousLink);
     const prevRoute = "/report/QMS/CO/undefined/mock-prev-link";
     expect(mockUseNavigate).toHaveBeenCalledWith(prevRoute);
 
+    //click next
     const nextLink = screen.getByRole("button", {
       name: "Next",
     });
     await userEvent.click(nextLink);
     const nextRoute = "/report/QMS/CO/undefined/mock-next-link";
     expect(mockUseNavigate).toHaveBeenCalledWith(nextRoute);
 
-    expect(screen.getByText("Clear measure data")).toBeInTheDocument();
-    expect(screen.getByText("Complete section")).toBeInTheDocument();
-    expect(screen.getByText("Complete measure")).toBeInTheDocument();
+    //click clear
+    const clearBtn = screen.getByRole("button", {
+      name: "Clear measure data",
+    });
+    await userEvent.click(clearBtn);
+
+    //click complete measure
+    const completeMeasureBtn = screen.getByRole("button", {
+      name: "Complete measure",
+    });
+    await userEvent.click(completeMeasureBtn);
+
+    //click complete section
+    const completeSectionBtn = screen.getByRole("button", {
+      name: "Complete section",
+    });
+    await userEvent.click(completeSectionBtn);
   });
 
   it("Test Measure Footer component without optional fields", () => {

diff --git a/services/ui-src/src/components/report/MeasureFooter.tsx b/services/ui-src/src/components/report/MeasureFooter.tsx
@@ -47,12 +47,12 @@ export const MeasureFooterElement = (props: PageElementProps) => {
             </Button>
           )}
           {footer.completeMeasure && (
-            <Button disabled={true} onClick={() => onCompleteMeasure()}>
+            <Button onClick={() => onCompleteMeasure()}>
               Complete measure
             </Button>
           )}
           {footer.completeSection && (
-            <Button disabled={true} onClick={() => onCompleteSection()}>
+            <Button onClick={() => onCompleteSection()}>
               Complete section
             </Button>
           )}