Remove component from reduxt testing

Author: imnasnainaec

src/components/Login/Captcha.tsx

@@ -4,7 +4,6 @@ import { useTranslation } from "react-i18next";
 import { toast } from "react-toastify";
 
 import { verifyCaptchaToken } from "backend";
-import i18n from "i18n";
 import { RuntimeConfig } from "types/runtimeConfig";
 import theme from "types/theme";
 
@@ -17,7 +16,7 @@ export interface CaptchaProps {
 export default function Captcha(props: CaptchaProps): ReactElement {
   const setSuccess = props.setSuccess;
   const isRequired = useRef(RuntimeConfig.getInstance().captchaRequired());
-  const { t } = useTranslation();
+  const { i18n, t } = useTranslation();
 
   useEffect(() => {
     setSuccess(!isRequired.current);

src/components/GoalTimeline/tests/GoalRedux.test.tsx renamed to src/goals/Redux/tests/GoalRedux.test.ts

@@ -1,9 +1,8 @@
 import "@testing-library/jest-dom";
 import { act, cleanup } from "@testing-library/react";
 
-import { Edit, MergeUndoIds, Permission, User, UserEdit } from "api/models";
+import { Edit, MergeUndoIds, User, UserEdit } from "api/models";
 import * as LocalStorage from "backend/localStorage";
-import GoalTimeline from "components/GoalTimeline";
 import {
   CharInvChanges,
   CharInvData,
@@ -32,48 +31,31 @@ import { GoalStatus, GoalType } from "types/goals";
 import { Path } from "types/path";
 import { newUser } from "types/user";
 import * as goalUtilities from "utilities/goalUtilities";
-import { renderWithProviders } from "utilities/testingLibraryUtilities";
 
 jest.mock("backend", () => ({
   addGoalToUserEdit: (...args: any[]) => mockAddGoalToUserEdit(...args),
-  addStepToGoal: (...args: any[]) => mockAddStepToGoal(...args),
+  addStepToGoal: () => jest.fn(),
   createUserEdit: () => mockCreateUserEdit(),
-  findDuplicates: () => Promise.resolve(),
-  getCurrentPermissions: () => mockGetCurrentPermissions(),
+  findDuplicates: () => jest.fn(),
   getGraylistEntries: () => Promise.resolve([]),
-  getUser: (id: string) => mockGetUser(id),
   getUserEditById: (...args: any[]) => mockGetUserEditById(...args),
-  hasGraylistEntries: () => Promise.resolve(false),
+  hasGraylistEntries: () => jest.fn(),
   retrieveDuplicates: () => mockRetrieveDuplicates(),
-  updateUser: (user: User) => mockUpdateUser(user),
 }));
-jest.mock("components/Project/ProjectActions", () => ({}));
-jest.mock("components/Pronunciations/Recorder");
 jest.mock("router/browserRouter", () => ({
   navigate: (path: Path) => mockNavigate(path),
 }));
 
 const mockAddGoalToUserEdit = jest.fn();
-const mockAddStepToGoal = jest.fn();
 const mockCreateUserEdit = jest.fn();
-const mockGetCurrentPermissions = jest.fn();
-const mockGetUser = jest.fn();
 const mockGetUserEditById = jest.fn();
 const mockNavigate = jest.fn();
 const mockRetrieveDuplicates = jest.fn();
-const mockUpdateUser = jest.fn();
 function setMockFunctions(): void {
   mockAddGoalToUserEdit.mockResolvedValue(0);
-  mockAddStepToGoal.mockResolvedValue(0);
   mockCreateUserEdit.mockResolvedValue(mockUser());
-  mockGetCurrentPermissions.mockResolvedValue([
-    Permission.CharacterInventory,
-    Permission.MergeAndReviewEntries,
-  ]);
-  mockGetUser.mockResolvedValue(mockUser());
   mockGetUserEditById.mockResolvedValue(mockUserEdit(true));
   mockRetrieveDuplicates.mockResolvedValue(goalDataMock.plannedWords);
-  mockUpdateUser.mockResolvedValue(mockUser());
 }
 
 const mockProjectId = "123";
@@ -124,7 +106,6 @@ describe("setCurrentGoal", () => {
   it("calls setCurrentGoal() with no arguments", async () => {
     const store = setupStore();
     await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
       store.dispatch(setCurrentGoal());
     });
     expect(store.getState().goalsState.currentGoal.goalType).toEqual(
@@ -136,9 +117,6 @@ describe("setCurrentGoal", () => {
 describe("asyncGetUserEdits", () => {
   it("backend returns user edits", async () => {
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
     const convertEditToGoalSpy = jest.spyOn(goalUtilities, "convertEditToGoal");
     await act(async () => {
       await store.dispatch(asyncGetUserEdits());
@@ -148,14 +126,9 @@ describe("asyncGetUserEdits", () => {
   });
 
   it("backend returns no user edits", async () => {
-    // render the GoalTimeline
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
 
     // setup mocks for testing the action/reducers
-    jest.clearAllMocks();
     const convertEditToGoalSpy = jest.spyOn(goalUtilities, "convertEditToGoal");
     mockGetUserEditById.mockResolvedValueOnce(mockUserEdit(false));
 
@@ -169,9 +142,6 @@ describe("asyncGetUserEdits", () => {
 
   it("creates new user edits", async () => {
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
     LocalStorage.setCurrentUser(newUser());
     await act(async () => {
       await store.dispatch(asyncGetUserEdits());
@@ -184,10 +154,6 @@ describe("asyncGetUserEdits", () => {
 describe("asyncAddGoal", () => {
   it("adds new MergeDups goal", async () => {
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
-
     await act(async () => {
       await store.dispatch(asyncAddGoal(new MergeDups()));
     });
@@ -201,10 +167,6 @@ describe("asyncAddGoal", () => {
 
   it("adds new CreateCharInv goal", async () => {
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
-
     await act(async () => {
       await store.dispatch(asyncAddGoal(new CreateCharInv()));
     });
@@ -225,10 +187,6 @@ describe("asyncAddGoal", () => {
 describe("asyncLoadNewGoalData", () => {
   it("loads data for MergeDups goal", async () => {
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
-
     await act(async () => {
       await store.dispatch(asyncAddGoal(new MergeDups()));
       await store.dispatch(asyncLoadNewGoalData());
@@ -247,9 +205,6 @@ describe("asyncAdvanceStep", () => {
   it("advance MergeDups goal", async () => {
     // setup the test scenario
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
     // create mergeDups goal
     await act(async () => {
       await store.dispatch(asyncAddGoal(new MergeDups()));
@@ -282,9 +237,6 @@ describe("asyncAdvanceStep", () => {
   it("advance CreateCharInv goal", async () => {
     // setup the test scenario
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
     // create character inventory goal
     const goal = new CreateCharInv();
     await act(async () => {
@@ -304,9 +256,6 @@ describe("asyncUpdateGoal", () => {
   it("update CreateCharInv goal", async () => {
     // setup the test scenario
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
     // create CreateCharInv goal
     const goal = new CreateCharInv();
     await act(async () => {
@@ -327,9 +276,6 @@ describe("asyncUpdateGoal", () => {
   it("update MergeDups goal", async () => {
     // setup the test scenario
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
     // create MergeDups goal
     const goal = new MergeDups();
     await act(async () => {
@@ -352,9 +298,6 @@ describe("asyncUpdateGoal", () => {
   it("update ReviewDeferredDups goal", async () => {
     // setup the test scenario
     const store = setupStore();
-    await act(async () => {
-      renderWithProviders(<GoalTimeline />, { store: store });
-    });
     // create ReviewDeferredDups goal
     const goal = new ReviewDeferredDups();
     await act(async () => {