Fix ESLint unnecessary disable comments

Author: colinrotherham

src/components/content-presentation/table/TableContext.ts

@@ -8,7 +8,6 @@ export interface ITableContext {
 }
 
 export const TableContext = createContext<ITableContext>({
-  /* eslint-disable @typescript-eslint/no-empty-function */
   firstCellIsHeader: false,
   headings: [],
   responsive: false,

src/components/content-presentation/table/components/__tests__/TableCell.test.tsx

@@ -29,9 +29,7 @@ describe('Table.Cell', () => {
       </table>,
     );
 
-    // eslint-disable-next-line no-console
     expect(console.warn).toHaveBeenCalledTimes(1);
-    // eslint-disable-next-line no-console
     expect(console.warn).toHaveBeenLastCalledWith(
       'Table.Cell used outside of a Table.Head or Table.Body component. Unable to determine section type from context.',
     );

src/components/form-elements/checkboxes/CheckboxesContext.ts

@@ -8,7 +8,6 @@ export interface ICheckboxesContext {
 }
 
 export const CheckboxesContext = createContext<ICheckboxesContext>({
-  /* eslint-disable @typescript-eslint/no-empty-function */
   name: '',
   getBoxId: () => undefined,
   leaseReference: () => '',

src/components/form-elements/date-input/DateInput.tsx

@@ -74,7 +74,6 @@ const DateInputComponent = forwardRef<HTMLDivElement, DateInputProps>(
         inputType="dateinput"
         {...rest}
       >
-        {/* eslint-disable-next-line @typescript-eslint/no-unused-vars */}
         {({ className, name, id, error, ...restRenderProps }) => {
           const contextValue: IDateInputContext = {
             id,

src/components/form-elements/date-input/DateInputContext.ts

@@ -10,7 +10,6 @@ export type IDateInputContext = {
 };
 
 export const DateInputContext = createContext<IDateInputContext>({
-  /* eslint-disable @typescript-eslint/no-empty-function */
   id: '',
   name: '',
   handleChange: () => {},

src/components/form-elements/radios/RadiosContext.ts

@@ -10,7 +10,6 @@ export type IRadiosContext = {
 };
 
 export const RadiosContext = createContext<IRadiosContext>({
-  /* eslint-disable @typescript-eslint/no-empty-function */
   name: '',
   selectedRadio: '',
   getRadioId: () => '',

src/components/navigation/header/HeaderContext.ts

@@ -24,7 +24,6 @@ export interface IHeaderContext {
 }
 
 export const HeaderContext = createContext<IHeaderContext>({
-  /* eslint-disable @typescript-eslint/no-empty-function */
   logoProps: undefined,
   serviceProps: undefined,
   organisationProps: undefined,

src/components/utils/__tests__/HeadingLevel.test.tsx

@@ -18,7 +18,6 @@ describe('HeadingLevel', () => {
   });
 
   it('console.warn when headingLevel is invalid', () => {
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     const consoleSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
     // @ts-expect-error - testing invalid prop
     render(<HeadingLevel headingLevel="h7" />);

src/util/hooks/UseDevWarning.tsx

@@ -6,7 +6,6 @@ type ConditionFn = () => boolean;
 export const useDevWarning = (warning: string, condition: ConditionFn = () => true): void => {
   useEffect(() => {
     if (isDev() && condition()) {
-      // eslint-disable-next-line no-console
       console.warn(warning);
     }
   }, [warning, condition]);