style: fix lint errors across migrated files

arbrandes · claude · arbrandes · commit 43737befb384 · 2026-03-05T08:23:44.000-03:00
Auto-fixed stylistic issues (indentation, semicolons, brace style, type→interface, Array<T>→T[]) and manually fixed display-name errors (named wrapper functions), @ts-ignore→@ts-expect-error, and Function→explicit signature. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
diff --git a/src/containers/CourseCard/components/CourseCardBanners/CreditBanner/hooks.js b/src/containers/CourseCard/components/CourseCardBanners/CreditBanner/hooks.js
@@ -39,7 +39,9 @@ export const useCreditBannerData = (cardId) => {
       requestStatus: creditData.requestStatus,
     };
   }, [courseData]);
-  if (!credit.isEligible || !courseData?.credit?.isEligible) { return null; }
+  if (!credit.isEligible || !courseData?.credit?.isEligible) {
+    return null;
+  }
 
   const { error, purchased, requestStatus } = credit;
   let ContentComponent = EligibleContent;
diff --git a/src/containers/CourseCard/components/CourseCardDetails/hooks.js b/src/containers/CourseCard/components/CourseCardDetails/hooks.js
@@ -11,7 +11,9 @@ export const useAccessMessage = ({ cardId }) => {
   const { courseRun, enrollment } = courseData || {};
   const formatDate = utilHooks.useFormatDate();
   if (!courseRun.isStarted) {
-    if (!courseRun.startDate && !courseRun.advertisedStart) { return null; }
+    if (!courseRun.startDate && !courseRun.advertisedStart) {
+      return null;
+    }
     const startDate = courseRun.advertisedStart ? courseRun.advertisedStart : formatDate(courseRun.startDate);
     return formatMessage(messages.courseStarts, { startDate });
   }
@@ -28,7 +30,9 @@ export const useAccessMessage = ({ cardId }) => {
         { accessExpirationDate: formatDate(accessExpirationDate) },
       );
     }
-    if (!endDate) { return null; }
+    if (!endDate) {
+      return null;
+    }
     return formatMessage(
       isArchived ? messages.courseEnded : messages.courseEnds,
       { endDate: formatDate(endDate) },
diff --git a/src/containers/CourseCard/components/CourseCardMenu/hooks.js b/src/containers/CourseCard/components/CourseCardMenu/hooks.js
@@ -33,7 +33,9 @@ export const useHandleToggleDropdown = (cardId) => {
     cardId,
   );
   return (isOpen) => {
-    if (isOpen) { trackCourseEvent(); }
+    if (isOpen) {
+      trackCourseEvent();
+    }
   };
 };
 
diff --git a/src/containers/CourseCard/components/CourseCardMenu/hooks.test.js b/src/containers/CourseCard/components/CourseCardMenu/hooks.test.js
@@ -69,7 +69,9 @@ describe('CourseCardMenu hooks', () => {
   });
 
   describe('useHandleToggleDropdown', () => {
-    beforeEach(() => { out = hooks.useHandleToggleDropdown(cardId); });
+    beforeEach(() => {
+      out = hooks.useHandleToggleDropdown(cardId);
+    });
     describe('behavior', () => {
       it('initializes course event tracker with event name and card ID', () => {
         expect(useCourseTrackingEvent).toHaveBeenCalledWith(
diff --git a/src/containers/Dashboard/hooks.test.js b/src/containers/Dashboard/hooks.test.js
@@ -43,7 +43,9 @@ describe('CourseCard hooks', () => {
   });
 
   describe('useDashboardLayoutData', () => {
-    beforeEach(() => { state.mock(); });
+    beforeEach(() => {
+      state.mock();
+    });
     describe('behavior', () => {
       it('initializes sidebarShowing to default true value', () => {
         hooks.useDashboardLayoutData();
diff --git a/src/data/context/BackedData.test.tsx b/src/data/context/BackedData.test.tsx
@@ -3,8 +3,7 @@ import { renderHook, act } from '@testing-library/react';
 import { BackedDataProvider, useBackedData } from './BackedDataProvider';
 
 describe('BackedDataProvider and useBackedData', () => {
-  // eslint-disable-next-line func-names
-  const createWrapper = () => function ({ children }: { children: React.ReactNode }) {
+  const createWrapper = () => function Wrapper({ children }: { children: React.ReactNode }) {
     return <BackedDataProvider>{children}</BackedDataProvider>;
   };
 
diff --git a/src/data/context/BackedDataProvider.tsx b/src/data/context/BackedDataProvider.tsx
@@ -4,21 +4,23 @@ import React, {
 } from 'react';
 
 interface BackedDataContextType {
-  backUpData: any;
-  setBackUpData: (backUpData: any) => void;
+  backUpData: any,
+  setBackUpData: (backUpData: any) => void,
 }
 
 const BackedDataContext = createContext<BackedDataContextType | null>(null);
 
 interface BackedDataState {
-  backUpData: any;
+  backUpData: any,
 }
 
 const initialState: BackedDataState = {
   backUpData: undefined,
 };
 
-type BackedDataAction = { type: 'SET_DATA'; payload: any };
+interface BackedDataAction {
+  type: 'SET_DATA', payload: any,
+}
 
 const backedDataReducer = (state: BackedDataState, action: BackedDataAction): BackedDataState => {
   switch (action.type) {
@@ -34,7 +36,7 @@ const backedDataReducer = (state: BackedDataState, action: BackedDataAction): Ba
 };
 
 interface BackedDataProviderProps {
-  children: ReactNode;
+  children: ReactNode,
 }
 
 export const BackedDataProvider: React.FC<BackedDataProviderProps> = ({ children }) => {
diff --git a/src/data/context/Filters.test.tsx b/src/data/context/Filters.test.tsx
@@ -3,8 +3,7 @@ import { renderHook, act } from '@testing-library/react';
 import { FiltersProvider, useFilters } from './FiltersProvider';
 
 describe('FiltersProvider and useFilters', () => {
-  // eslint-disable-next-line func-names
-  const createWrapper = () => function ({ children }: { children: React.ReactNode }) {
+  const createWrapper = () => function Wrapper({ children }: { children: React.ReactNode }) {
     return <FiltersProvider>{children}</FiltersProvider>;
   };
 
@@ -647,7 +646,7 @@ describe('FiltersProvider and useFilters', () => {
     });
 
     it('should handle provider re-renders without losing state', () => {
-      const TestWrapper = ({ rerenderTrigger, children }: { rerenderTrigger: number; children: React.ReactNode }) => (
+      const TestWrapper = ({ rerenderTrigger, children }: { rerenderTrigger: number, children: React.ReactNode }) => (
         <FiltersProvider>
           <div data-testid={`rerender-${rerenderTrigger}`}>
             {children}
@@ -759,7 +758,7 @@ describe('FiltersProvider and useFilters', () => {
       const { result } = renderHook(() => useFilters(), {
         wrapper: createWrapper(),
       });
-      const validSortOptions: Array<'enrolled' | 'title'> = ['enrolled', 'title'];
+      const validSortOptions: ('enrolled' | 'title')[] = ['enrolled', 'title'];
 
       validSortOptions.forEach(option => {
         act(() => {
diff --git a/src/data/context/FiltersProvider.tsx b/src/data/context/FiltersProvider.tsx
@@ -6,23 +6,23 @@ import React, {
 type SortOption = 'enrolled' | 'title';
 
 interface FiltersContextType {
-  filters: string[];
-  sortBy: SortOption;
-  pageNumber: number;
-  setFilters: (newFilters: string[]) => void;
-  addFilter: (filter: string) => void;
-  removeFilter: (filter: string) => void;
-  clearFilters: () => void;
-  setSortBy: (sortBy: SortOption) => void;
-  setPageNumber: (pageNumber: number) => void;
+  filters: string[],
+  sortBy: SortOption,
+  pageNumber: number,
+  setFilters: (newFilters: string[]) => void,
+  addFilter: (filter: string) => void,
+  removeFilter: (filter: string) => void,
+  clearFilters: () => void,
+  setSortBy: (sortBy: SortOption) => void,
+  setPageNumber: (pageNumber: number) => void,
 }
 
 const FiltersContext = createContext<FiltersContextType | null>(null);
 
 interface FiltersState {
-  filters: string[];
-  sortBy: SortOption;
-  pageNumber: number;
+  filters: string[],
+  sortBy: SortOption,
+  pageNumber: number,
 }
 
 const initialState: FiltersState = {
@@ -32,12 +32,12 @@ const initialState: FiltersState = {
 };
 
 type FiltersAction =
-  | { type: 'SET_FILTERS'; payload: string[] }
-  | { type: 'ADD_FILTER'; payload: string }
-  | { type: 'REMOVE_FILTER'; payload: string }
+  | { type: 'SET_FILTERS', payload: string[] }
+  | { type: 'ADD_FILTER', payload: string }
+  | { type: 'REMOVE_FILTER', payload: string }
   | { type: 'CLEAR_FILTERS' }
-  | { type: 'SET_SORT_BY'; payload: SortOption }
-  | { type: 'SET_PAGE_NUMBER'; payload: number };
+  | { type: 'SET_SORT_BY', payload: SortOption }
+  | { type: 'SET_PAGE_NUMBER', payload: number };
 
 const filtersReducer = (state: FiltersState, action: FiltersAction): FiltersState => {
   switch (action.type) {
diff --git a/src/data/context/Masquerade.test.tsx b/src/data/context/Masquerade.test.tsx
@@ -3,8 +3,7 @@ import { renderHook, act } from '@testing-library/react';
 import { MasqueradeProvider, useMasquerade } from './MasqueradeProvider';
 
 describe('MasqueradeProvider and useMasquerade', () => {
-  // eslint-disable-next-line func-names
-  const createWrapper = () => function ({ children }: { children: React.ReactNode }) {
+  const createWrapper = () => function Wrapper({ children }: { children: React.ReactNode }) {
     return <MasqueradeProvider>{children}</MasqueradeProvider>;
   };
 
@@ -352,7 +351,7 @@ describe('MasqueradeProvider and useMasquerade', () => {
     });
 
     it('should handle provider re-renders without losing state', () => {
-      const TestWrapper = ({ rerenderTrigger, children }: { rerenderTrigger: number; children: React.ReactNode }) => (
+      const TestWrapper = ({ rerenderTrigger, children }: { rerenderTrigger: number, children: React.ReactNode }) => (
         <MasqueradeProvider>
           <div data-testid={`rerender-${rerenderTrigger}`}>
             {children}
@@ -512,8 +511,8 @@ describe('MasqueradeProvider and useMasquerade', () => {
         wrapper: createWrapper(),
       });
 
-      const stringUser: string = 'typed-user@test.com';
-      const undefinedUser: undefined = undefined;
+      const stringUser = 'typed-user@test.com';
+      const undefinedUser = undefined;
 
       act(() => {
         result.current.setMasqueradeUser(stringUser);
diff --git a/src/data/context/MasqueradeProvider.tsx b/src/data/context/MasqueradeProvider.tsx
@@ -4,21 +4,23 @@ import React, {
 } from 'react';
 
 interface MasqueradeContextType {
-  masqueradeUser: string | undefined;
-  setMasqueradeUser: (user: string | undefined) => void;
+  masqueradeUser: string | undefined,
+  setMasqueradeUser: (user: string | undefined) => void,
 }
 
 const MasqueradeContext = createContext<MasqueradeContextType | null>(null);
 
 interface MasqueradeState {
-  masqueradeUser: string | undefined;
+  masqueradeUser: string | undefined,
 }
 
 const initialState: MasqueradeState = {
   masqueradeUser: undefined,
 };
 
-type MasqueradeAction = { type: 'SET_MASQUERADE_USER'; payload: string | undefined };
+interface MasqueradeAction {
+  type: 'SET_MASQUERADE_USER', payload: string | undefined,
+}
 
 const masqueradeReducer = (state: MasqueradeState, action: MasqueradeAction): MasqueradeState => {
   switch (action.type) {
@@ -34,7 +36,7 @@ const masqueradeReducer = (state: MasqueradeState, action: MasqueradeAction): Ma
 };
 
 interface MasqueradeProviderProps {
-  children: ReactNode;
+  children: ReactNode,
 }
 
 export const MasqueradeProvider: React.FC<MasqueradeProviderProps> = ({ children }) => {
diff --git a/src/data/context/SelectSession.test.tsx b/src/data/context/SelectSession.test.tsx
@@ -3,8 +3,7 @@ import { renderHook, act } from '@testing-library/react';
 import { SelectSessionModalProvider, useSelectSessionModal } from './SelectSessionProvider';
 
 describe('SelectSessionModalProvider and useSelectSessionModal', () => {
-  // eslint-disable-next-line func-names
-  const createWrapper = () => function ({ children }: { children: React.ReactNode }) {
+  const createWrapper = () => function Wrapper({ children }: { children: React.ReactNode }) {
     return <SelectSessionModalProvider>{children}</SelectSessionModalProvider>;
   };
 
@@ -380,7 +379,7 @@ describe('SelectSessionModalProvider and useSelectSessionModal', () => {
     });
 
     it('should handle provider re-renders without losing state', () => {
-      const TestWrapper = ({ rerenderTrigger, children }: { rerenderTrigger: number; children: React.ReactNode }) => (
+      const TestWrapper = ({ rerenderTrigger, children }: { rerenderTrigger: number, children: React.ReactNode }) => (
         <SelectSessionModalProvider>
           <div data-testid={`rerender-${rerenderTrigger}`}>
             {children}
@@ -596,8 +595,8 @@ describe('SelectSessionModalProvider and useSelectSessionModal', () => {
         wrapper: createWrapper(),
       });
 
-      const stringCardId: string = 'typed-card-123';
-      const nullCardId: null = null;
+      const stringCardId = 'typed-card-123';
+      const nullCardId = null;
 
       act(() => {
         result.current.updateSelectSessionModal(stringCardId);
diff --git a/src/data/context/SelectSessionProvider.tsx b/src/data/context/SelectSessionProvider.tsx
@@ -4,23 +4,23 @@ import React, {
 } from 'react';
 
 interface SelectSessionModalState {
-  cardId: string | null;
+  cardId: string | null,
 }
 
 interface SelectSessionModalContextType {
-  selectSessionModal: SelectSessionModalState;
-  updateSelectSessionModal: (cardId: string | null) => void;
-  closeSelectSessionModal: () => void;
+  selectSessionModal: SelectSessionModalState,
+  updateSelectSessionModal: (cardId: string | null) => void,
+  closeSelectSessionModal: () => void,
 }
 
 const SelectSessionModalContext = createContext<SelectSessionModalContextType | null>(null);
 
 interface State {
-  selectSessionModal: SelectSessionModalState;
+  selectSessionModal: SelectSessionModalState,
 }
 
 type Action =
-  | { type: 'UPDATE_SELECT_SESSION_MODAL'; payload: string | null }
+  | { type: 'UPDATE_SELECT_SESSION_MODAL', payload: string | null }
   | { type: 'CLOSE_SELECT_SESSION_MODAL' };
 
 const initialState: State = {
@@ -46,7 +46,7 @@ const selectSessionModalReducer = (state: State, action: Action): State => {
 };
 
 interface SelectSessionModalProviderProps {
-  children: ReactNode;
+  children: ReactNode,
 }
 
 export const SelectSessionModalProvider: React.FC<SelectSessionModalProviderProps> = ({ children }) => {
diff --git a/src/data/context/index.tsx b/src/data/context/index.tsx
@@ -3,9 +3,9 @@ import { MasqueradeProvider, useMasquerade } from './MasqueradeProvider';
 import { FiltersProvider, useFilters } from './FiltersProvider';
 import { SelectSessionModalProvider, useSelectSessionModal } from './SelectSessionProvider';
 
-type ContextProvidersProps = {
-  children: React.ReactNode;
-};
+interface ContextProvidersProps {
+  children: React.ReactNode,
+}
 
 const ContextProviders = ({ children }: ContextProvidersProps) => (
   <BackedDataProvider>
diff --git a/src/data/hooks/mutationHooks.test.tsx b/src/data/hooks/mutationHooks.test.tsx
@@ -35,8 +35,7 @@ const createWrapper = () => {
     },
   });
 
-  // eslint-disable-next-line func-names
-  return function ({ children }: { children: React.ReactNode }) {
+  return function Wrapper({ children }: { children: React.ReactNode }) {
     return (
       <QueryClientProvider client={queryClient}>
         {children}
@@ -332,12 +331,10 @@ describe('mutationHooks', () => {
 
       for (const { hook, params, arg } of hooks) {
         invalidateQueriesSpy.mockClear();
-        // ts-ignore to handle varying params
-        // @ts-ignore
+        // @ts-expect-error handle varying params
         const { result } = renderHook(() => (arg ? hook(arg) : hook()), { wrapper });
 
-        // @ts-ignore
-        // eslint-disable-next-line no-await-in-loop
+        // @ts-expect-error handle varying mutateAsync params
         await result.current.mutateAsync(params);
 
         expect(invalidateQueriesSpy).toHaveBeenCalledWith({
diff --git a/src/data/hooks/mutationHooks.ts b/src/data/hooks/mutationHooks.ts
diff --git a/src/data/hooks/queryHooks.test.tsx b/src/data/hooks/queryHooks.test.tsx
diff --git a/src/hooks/useCourseTrackingEvent.ts b/src/hooks/useCourseTrackingEvent.ts
diff --git a/src/hooks/useEntitlementInfo.ts b/src/hooks/useEntitlementInfo.ts
diff --git a/src/utils/dataTransformers.test.ts b/src/utils/dataTransformers.test.ts
diff --git a/src/utils/dataTransformers.ts b/src/utils/dataTransformers.ts
