test: thunks

openedx · Dec 18, 2024 · 780d05b · 780d05b
1 parent 67fc392
commit 780d05b
Show file tree

Hide file tree

Showing 3 changed files with 105 additions and 5 deletions.
diff --git a/src/optimizer-page/data/thunks.test.js b/src/optimizer-page/data/thunks.test.js
@@ -2,6 +2,7 @@ import { startLinkCheck, fetchLinkCheckStatus } from "./thunks";
 import * as api from "./api";
 import { LINK_CHECK_STATUSES } from "./constants";
 import { RequestStatus } from "../../data/constants";
+import mockApiResponse from "../mocks/mockApiResponse";
 
 describe("startLinkCheck thunk", () => {
   const dispatch = jest.fn();
@@ -69,3 +70,103 @@ describe("startLinkCheck thunk", () => {
     });
   });
 });
+
+describe("fetchLinkCheckStatus thunk", () => {
+  describe("successful request", () => {
+    it("should return scan result", async () => {
+      const dispatch = jest.fn();
+      const getState = jest.fn();
+      const courseId = "course-123";
+      const mockGetLinkCheckStatus = jest
+        .spyOn(api, "getLinkCheckStatus")
+        .mockResolvedValue({
+          linkCheckStatus: mockApiResponse.LinkCheckStatus,
+          linkCheckOutput: mockApiResponse.LinkCheckOutput,
+          linkCheckCreatedAt: mockApiResponse.LinkCheckCreatedAt,
+        });
+
+      await fetchLinkCheckStatus(courseId)(dispatch, getState);
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: false,
+        type: "courseOptimizer/updateLinkCheckInProgress",
+      });
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: 2,
+        type: "courseOptimizer/updateCurrentStage",
+      });
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: mockApiResponse.LinkCheckOutput,
+        type: "courseOptimizer/updateLinkCheckResult",
+      });
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: { status: RequestStatus.SUCCESSFUL },
+        type: "courseOptimizer/updateLoadingStatus",
+      });
+    });
+  });
+
+  describe("failed request", () => {
+    it("should set request status to failed", async () => {
+      const dispatch = jest.fn();
+      const getState = jest.fn();
+      const courseId = "course-123";
+      const mockGetLinkCheckStatus = jest
+        .spyOn(api, "getLinkCheckStatus")
+        .mockRejectedValue(new Error("error"));
+
+      await fetchLinkCheckStatus(courseId)(dispatch, getState);
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: { status: RequestStatus.FAILED },
+        type: "courseOptimizer/updateLoadingStatus",
+      });
+    });
+  });
+
+  describe("failed scan", () => {
+    it("should set error message", async () => {
+      const mockGetLinkCheckStatus = jest
+        .spyOn(api, "getLinkCheckStatus")
+        .mockResolvedValue({
+          linkCheckStatus: LINK_CHECK_STATUSES.FAILED,
+          linkCheckOutput: mockApiResponse.LinkCheckOutput,
+          linkCheckCreatedAt: mockApiResponse.LinkCheckCreatedAt,
+        });
+
+      const dispatch = jest.fn();
+      const getState = jest.fn();
+      const courseId = "course-123";
+
+      await fetchLinkCheckStatus(courseId)(dispatch, getState);
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: true,
+        type: "courseOptimizer/updateIsErrorModalOpen",
+      });
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: { msg: "Link Check Failed" },
+        type: "courseOptimizer/updateError",
+      });
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: { status: RequestStatus.SUCCESSFUL },
+        type: "courseOptimizer/updateLoadingStatus",
+      });
+
+      expect(dispatch).toHaveBeenCalledWith({
+        payload: -1,
+        type: "courseOptimizer/updateCurrentStage",
+      });
+
+      expect(dispatch).not.toHaveBeenCalledWith({
+        payload: expect.anything(),
+        type: "courseOptimizer/updateLinkCheckResult",
+      });
+    });
+  });
+});
diff --git a/src/optimizer-page/data/thunks.ts b/src/optimizer-page/data/thunks.ts
@@ -55,6 +55,7 @@ export function fetchLinkCheckStatus(courseId) {
       } else {
         dispatch(updateLinkCheckInProgress(false));
       }
+      console.log('linkCheckStatus:', linkCheckStatus);
 
       dispatch(updateCurrentStage(SCAN_STAGES[linkCheckStatus]));
 
@@ -65,9 +66,7 @@ export function fetchLinkCheckStatus(courseId) {
       ) {
         dispatch(updateError({ msg: 'Link Check Failed' }));
         dispatch(updateIsErrorModalOpen(true));
-      }
-
-      if (linkCheckOutput) {
+      } else if (linkCheckOutput) {
         dispatch(updateLinkCheckResult(linkCheckOutput));
       }
 
@@ -78,7 +77,7 @@ export function fetchLinkCheckStatus(courseId) {
         dispatch(updateLoadingStatus({ status: RequestStatus.DENIED }));
       } else {
         dispatch(
-          updateLoadingStatus({ courseId, status: RequestStatus.FAILED }),
+          updateLoadingStatus({ status: RequestStatus.FAILED }),
         );
       }
       return false;

diff --git a/src/optimizer-page/mocks/mockApiResponse.js b/src/optimizer-page/mocks/mockApiResponse.js
@@ -1,5 +1,5 @@
 const mockApiResponse = {
-  LinkCheckStatus: 'In-Progress',
+  LinkCheckStatus: 'Succeeded',
   LinkCheckCreatedAt: '2024-12-14T00:26:50.838350Z',
   LinkCheckOutput: {
     sections: [