Merge pull request #115 from OpenLXP/ironbank-dev

Fixing Lint for Ironbank

src/__mocks__/authContextMock.js

@@ -1,4 +1,4 @@
-import { useContext, createContext } from "react";
+import { createContext, useContext } from "react";
 export const AuthContext = createContext()
 export const AuthContextWrapper = ({children}) => {
     const login = jest.fn()

src/__mocks__/data/aggregations.data.js

@@ -1,3 +1,4 @@
+/* eslint-disable import/no-anonymous-default-export */
 export default {
   'Course Type': {
     doc_count_error_upper_bound: 0,

src/__mocks__/data/course.data.js

@@ -1,3 +1,4 @@
+/* eslint-disable import/no-anonymous-default-export */
 export default {
   Course: {
     CourseTitle: 'Test Course Title',

src/__mocks__/data/search.data.js

@@ -1,5 +1,8 @@
+/* eslint-disable import/no-anonymous-default-export */
+
 import aggregationsData from './aggregations.data';
 import courseData from './course.data';
+
 export default {
   hits: [courseData],
   total: 1,

src/__mocks__/data/uiConfig.data.js

@@ -1,3 +1,4 @@
+/* eslint-disable import/no-anonymous-default-export */
 export default {
   course_highlights: [
     {

src/__mocks__/data/userLists.data.js

@@ -1,3 +1,4 @@
+/* eslint-disable import/no-anonymous-default-export */
 export default [
   {
     id: 1,

src/__mocks__/queryClientMock.js

@@ -1,4 +1,4 @@
-import { QueryClientProvider, QueryClient } from 'react-query';
+import { QueryClient, QueryClientProvider } from 'react-query';
 
 export const QueryClientWrapper = ({ children }) => {
   const queryClient = new QueryClient();

src/__tests__/components/buttons/ActionButton.test.js

@@ -1,4 +1,4 @@
-import { render, screen, act, fireEvent } from '@testing-library/react';
+import { act, fireEvent, render, screen } from '@testing-library/react';
 
 import ActionButton from '../../../components/buttons/ActionButton';
 

src/__tests__/components/buttons/Pagination.test.js

@@ -1,4 +1,4 @@
-import { render, act, fireEvent, screen } from '@testing-library/react';
+import { act, fireEvent, render, screen } from '@testing-library/react';
 
 import { Pagination } from '@/components/buttons/Pagination';
 

src/__tests__/components/cards/CourseSpotlight.test.js

@@ -1,12 +1,12 @@
-import { render, act, fireEvent } from '@testing-library/react';
-import { useConfig } from '@/hooks/useConfig';
 import { MemoryRouterProvider } from 'next-router-mock/MemoryRouterProvider';
-import mockRouter from 'next-router-mock';
+import { act, fireEvent, render } from '@testing-library/react';
+import { useAuth } from '@/contexts/AuthContext';
+import { useConfig } from '@/hooks/useConfig';
+import CourseSpotlight from '@/components/cards/CourseSpotlight';
 import courseData from '@/__mocks__/data/course.data';
+import mockRouter from 'next-router-mock';
 import uiConfigData from '@/__mocks__/data/uiConfig.data';
-import CourseSpotlight from '@/components/cards/CourseSpotlight';
 import xAPIMapper from "@/utils/xapi/xAPIMapper";
-import { useAuth } from '@/contexts/AuthContext';
 
 // jest mocks
 jest.mock('next/dist/client/router', () => require('next-router-mock'));
@@ -59,6 +59,7 @@ describe('Course Spotlight', () => {
         };
         const { getByAltText, queryByRole } = renderer(modified);
         expect(getByAltText ('')).toBeInTheDocument();
+
         // expect(queryByRole('img')).toBeInTheDocument();
       });
     });

src/__tests__/components/cards/SearchResult.test.js

@@ -1,15 +1,15 @@
-import { fireEvent, render, act } from '@testing-library/react';
-import SearchResult from '@/components/cards/SearchResult';
-import courseData from '@/__mocks__/data/course.data';
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
-import singletonRouter from 'next/router';
+import { act, fireEvent, render } from '@testing-library/react';
 import {
   useAuthenticatedUser,
   useMockCreateUserList,
   useMockUpdateUserList,
   useMockUserOwnedListsWithoutData,
   useUnauthenticatedUser,
 } from '@/__mocks__/predefinedMocks';
+import SearchResult from '@/components/cards/SearchResult';
+import courseData from '@/__mocks__/data/course.data';
+import singletonRouter from 'next/router';
 
 const renderer = (props) => {
   return render(

src/__tests__/components/inputs/SelectLists.test.js

@@ -1,4 +1,4 @@
-import { act, render, screen, fireEvent } from '@testing-library/react';
+import { act, fireEvent, render, screen } from '@testing-library/react';
 
 import SelectList from '@/components/inputs/SelectList';
 import aggregationsData from '@/__mocks__/data/aggregations.data';

src/__tests__/components/modals/CreateSavedSearch.test.js

@@ -1,14 +1,14 @@
-import { render, fireEvent } from '@testing-library/react';
-import { act } from 'react-dom/test-utils';
-import CreateSavedSearchModal from '@/components/modals/CreateSavedSearch';
-import { useCreateSaveSearch } from '@/hooks/useCreateSaveSearch';
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
+import { act } from 'react-dom/test-utils';
 import {
   createSaveSearchMockFn,
   useAuthenticatedUser,
   useMockCreateSaveSearch,
   useUnauthenticatedUser,
 } from '@/__mocks__/predefinedMocks';
+import { fireEvent, render } from '@testing-library/react';
+import { useCreateSaveSearch } from '@/hooks/useCreateSaveSearch';
+import CreateSavedSearchModal from '@/components/modals/CreateSavedSearch';
 
 const renderer = () => {
   return render(

src/__tests__/components/modals/SaveModal.test.js

@@ -1,14 +1,14 @@
-import { act, screen, render, fireEvent } from '@testing-library/react';
+import { act, fireEvent, render, screen } from '@testing-library/react';
 import SaveModal from '@/components/modals/SaveModal';
 
-import userListData from '@/__mocks__/data/userLists.data';
-import { useUserOwnedLists } from '@/hooks/useUserOwnedLists.js';
-import { useUpdateUserList } from '@/hooks/useUpdateUserList';
-import { useCreateUserList } from '@/hooks/useCreateUserList';
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
-import xAPIMapper from '@/utils/xapi/xAPIMapper';
 import { useAuth } from '@/contexts/AuthContext';
+import { useCreateUserList } from '@/hooks/useCreateUserList';
+import { useUpdateUserList } from '@/hooks/useUpdateUserList';
+import { useUserOwnedLists } from '@/hooks/useUserOwnedLists.js';
 import { xAPISendStatement } from '@/utils/xapi/xAPISendStatement';
+import userListData from '@/__mocks__/data/userLists.data';
+import xAPIMapper from '@/utils/xapi/xAPIMapper';
 
 jest.mock('@/hooks/useUpdateUserList', () => ({
   useUpdateUserList: jest.fn(),

src/__tests__/hooks/useConfig.test.js

@@ -1,5 +1,5 @@
-import { renderHook, act } from '@testing-library/react-hooks';
 import { QueryClient, QueryClientProvider } from 'react-query';
+import { act, renderHook } from '@testing-library/react-hooks';
 import mockAxios from 'jest-mock-axios';
 
 import { useConfig } from '@/hooks/useConfig';

src/__tests__/hooks/useCreateSaveSearch.test.js

@@ -1,7 +1,7 @@
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
-import mockAxios from 'jest-mock-axios';
 import { renderHook } from '@testing-library/react-hooks';
 import { useCreateSaveSearch } from '@/hooks/useCreateSaveSearch';
+import mockAxios from 'jest-mock-axios';
 
 jest.unmock('@/hooks/useCreateSaveSearch');
 
@@ -17,6 +17,7 @@ it('should make an api call', async () => {
     () => useCreateSaveSearch(),
     { wrapper }
   );
+
   // wait for the api call to finish
   await waitForNextUpdate(
     result.current.mutate({ path: 'test', name: 'name' })

src/__tests__/hooks/useCreateUserList.test.js

@@ -1,7 +1,7 @@
-import mockAxios from 'jest-mock-axios';
+import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import { renderHook } from '@testing-library/react-hooks';
 import { useCreateUserList } from '@/hooks/useCreateUserList';
-import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
+import mockAxios from 'jest-mock-axios';
 
 jest.unmock('@/hooks/useCreateUserList');
 

src/__tests__/hooks/useDeleteSavedSearch.test.js

@@ -1,9 +1,9 @@
 // tests for hooks/useDeleteSavedSearch
 
-import { useDeleteSavedSearch } from '@/hooks/useDeleteSavedSearch';
+import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import { renderHook } from '@testing-library/react-hooks';
+import { useDeleteSavedSearch } from '@/hooks/useDeleteSavedSearch';
 import mockAxios from 'jest-mock-axios';
-import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 
 jest.unmock('@/hooks/useDeleteSavedSearch');
 

src/__tests__/hooks/useField.test.js

@@ -1,4 +1,4 @@
-import { renderHook, act } from '@testing-library/react-hooks';
+import { act, renderHook } from '@testing-library/react-hooks';
 import useField from '../../hooks/useField';
 
 describe('useField', () => {

src/__tests__/hooks/useInterestLists.test.js

@@ -1,9 +1,9 @@
 // tests for useInterestLists hook
 
-import { useInterestLists } from '@/hooks/useInterestLists';
-import mockAxios from 'jest-mock-axios';
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import { renderHook } from '@testing-library/react-hooks';
+import { useInterestLists } from '@/hooks/useInterestLists';
+import mockAxios from 'jest-mock-axios';
 import userListsData from '@/__mocks__/data/userLists.data';
 
 jest.unmock('@/hooks/useInterestLists');

src/__tests__/hooks/useMoreCoursesLikeThis.test.js

@@ -1,5 +1,5 @@
-import { renderHook, act } from '@testing-library/react-hooks';
 import { QueryClient, QueryClientProvider } from 'react-query';
+import { act, renderHook } from '@testing-library/react-hooks';
 
 import mockAxios from 'jest-mock-axios';
 

src/__tests__/hooks/useSaveSearch.test.js

@@ -1,6 +1,6 @@
 // test cases for useSaveSearch hook
-import { renderHook, act } from '@testing-library/react-hooks';
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
+import { act, renderHook } from '@testing-library/react-hooks';
 import { useSaveSearchList } from '@/hooks/useSaveSearch';
 
 import mockAxios from 'jest-mock-axios';

src/__tests__/hooks/useSubscribeToList.test.js

@@ -1,5 +1,5 @@
-import mockAxios from 'jest-mock-axios';
 import { renderHook } from '@testing-library/react-hooks';
+import mockAxios from 'jest-mock-axios';
 
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import { useSubscribeToList } from '@/hooks/useSubscribeToList';
@@ -17,8 +17,10 @@ it('should make an api call', async () => {
   const { result, waitForNextUpdate } = renderHook(() => useSubscribeToList(), {
     wrapper,
   });
+
   // wait for the api call to finish
   await waitForNextUpdate(result.current.mutate({ id: '1' }));
+
   // check if the api call was made
   expect(mockAxios.patch).toHaveBeenCalledTimes(1);
 });

src/__tests__/hooks/useSubscribedLists.test.js

@@ -1,9 +1,9 @@
 // tests for hooks/useSubscribedLists.js
-import mockAxios from 'jest-mock-axios';
 import { renderHook } from '@testing-library/react-hooks';
+import mockAxios from 'jest-mock-axios';
 
-import { useSubscribedLists } from '@/hooks/useSubscribedLists';
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
+import { useSubscribedLists } from '@/hooks/useSubscribedLists';
 
 jest.unmock('@/hooks/useSubscribedLists');
 

src/__tests__/hooks/useTimeout.test.js

@@ -1,4 +1,4 @@
-import { renderHook, act } from '@testing-library/react-hooks';
+import { act, renderHook } from '@testing-library/react-hooks';
 import useTimeout from '../../hooks/useTimeout';
 
 jest.useFakeTimers();

src/__tests__/hooks/useUnsubscribeFromList.test.js

@@ -1,8 +1,8 @@
 // tests for useUnsubscribeFromList hook
 
+import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import { renderHook } from '@testing-library/react-hooks';
 import { useUnsubscribeFromList } from '@/hooks/useUnsubscribeFromList';
-import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import mockAxios from 'jest-mock-axios';
 
 jest.unmock('@/hooks/useUnsubscribeFromList');

src/__tests__/hooks/useUpdateUserList.test.js

@@ -1,5 +1,5 @@
-import { renderHook, act } from '@testing-library/react-hooks';
 import { QueryClient, QueryClientProvider } from 'react-query';
+import { act, renderHook } from '@testing-library/react-hooks';
 import mockAxios from 'jest-mock-axios';
 
 import { useUpdateUserList } from '@/hooks/useUpdateUserList';

src/__tests__/hooks/useUserList.test.js

@@ -32,6 +32,7 @@ it('should return null if there is no id', async () => {
   await waitForNextUpdate(result.current.isSuccess);
   expect(result.current.data).toBeUndefined();
   expect(mockAxios.get).toHaveBeenCalledTimes(0);
+
   // expect(result.current.isSuccess).toBe(true);
   // expect(result.current.data).toEqual('success');
   // expect(mockAxios.get).toHaveBeenCalledTimes(1);

src/__tests__/hooks/useUserOwnedLists.test.js

@@ -1,5 +1,5 @@
-import { renderHook, act } from '@testing-library/react-hooks';
 import { QueryClient, QueryClientProvider } from 'react-query';
+import { act, renderHook } from '@testing-library/react-hooks';
 import mockAxios from 'jest-mock-axios';
 
 import {useUserOwnedLists} from '@/hooks/useUserOwnedLists';

src/__tests__/pages/401.test.js

@@ -1,7 +1,7 @@
-import { fireEvent, render, act } from '@testing-library/react';
+import { act, fireEvent, render } from '@testing-library/react';
 
-import Unathorized from '../../pages/401';
 import { MemoryRouterProvider } from 'next-router-mock/MemoryRouterProvider';
+import Unathorized from '../../pages/401';
 import singletonRouter from 'next/router';
 
 // mocks

src/__tests__/pages/403.test.js

@@ -1,7 +1,7 @@
-import { fireEvent, render, act } from '@testing-library/react';
+import { act, fireEvent, render } from '@testing-library/react';
 
-import Forbidden from '../../pages/403';
 import { MemoryRouterProvider } from 'next-router-mock/MemoryRouterProvider';
+import Forbidden from '../../pages/403';
 import singletonRouter from 'next/router';
 
 // mocks

src/__tests__/pages/course/course.test.js

@@ -1,5 +1,4 @@
 import { MemoryRouterProvider } from 'next-router-mock/MemoryRouterProvider';
-import MockRouter from 'next-router-mock';
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import { act, fireEvent, render } from '@testing-library/react';
 import {
@@ -17,6 +16,7 @@ import {
   useUnauthenticatedUser,
 } from '@/__mocks__/predefinedMocks';
 import Course from '@/pages/course/[courseId]';
+import MockRouter from 'next-router-mock';
 import singletonRouter from 'next/router';
 
 const renderer = (isAuth = false) => {

src/__tests__/pages/help.test.js

@@ -1,7 +1,7 @@
 import { QueryClient, QueryClientProvider } from 'react-query';
 import { render } from '@testing-library/react';
-import Help from '../../pages/help';
 import { useAuthenticatedUser, useMockConfig } from '@/__mocks__/predefinedMocks';
+import Help from '../../pages/help';
 
 const queryClient = new QueryClient();
 const renderer = () => {

src/__tests__/pages/index.test.js

@@ -1,9 +1,9 @@
 import { QueryClientWrapper } from '@/__mocks__/queryClientMock.js';
 import { act, fireEvent, render, screen } from '@testing-library/react';
+import { useAuth } from '@/contexts/AuthContext';
 import Home from '@/pages/index';
 import mockRouter from 'next-router-mock';
 import singletonRouter from 'next/router';
-import { useAuth } from '@/contexts/AuthContext';
 import xAPIMapper from "@/utils/xapi/xAPIMapper";
 
 jest.mock('next/dist/client/router', () => require('next-router-mock'));

src/__tests__/pages/lists/list.test.js

@@ -6,12 +6,12 @@ import { act, fireEvent, render } from '@testing-library/react';
 import { useAuth } from '@/contexts/AuthContext';
 import {
   useAuthenticatedUser,
-  useUnauthenticatedUser,
   useListMock,
   useListMockWith401,
   useListMockWith403,
   useListMockWithNoExperiences,
   useMockConfig,
+  useUnauthenticatedUser,
 } from '@/__mocks__/predefinedMocks';
 import { useList } from '@/hooks/useList';
 import List, { getServerSideProps } from '@/pages/lists/[listId]';

src/__tests__/pages/lists/owned.test.js

@@ -1,18 +1,18 @@
-import { render, fireEvent, act } from '@testing-library/react';
-import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
 import { MemoryRouterProvider } from 'next-router-mock/MemoryRouterProvider';
-import Owned from '@/pages/lists/owned';
-import singletonRouter from 'next/router';
-import MockRouter from 'next-router-mock';
+import { QueryClientWrapper } from '@/__mocks__/queryClientMock';
+import { act, fireEvent, render } from '@testing-library/react';
 import {
-  useUnauthenticatedUser,
   useAuthenticatedUser,
+  useMockConfig,
   useMockUserOwnedLists,
   useMockUserOwnedListsWith401,
   useMockUserOwnedListsWith403,
   useMockUserOwnedListsWithoutData,
-  useMockConfig,
+  useUnauthenticatedUser,
 } from '@/__mocks__/predefinedMocks';
+import MockRouter from 'next-router-mock';
+import Owned from '@/pages/lists/owned';
+import singletonRouter from 'next/router';
 
 const renderer = () => {
   MockRouter.setCurrentUrl('/lists/owned');