Use test seams instead of DI in useTimeoutToggle

.eslintrc

@@ -6,8 +6,5 @@
   "parserOptions": {
     "project": "./tsconfig.json"
   },
-  "ignorePatterns": ["src/service*.ts"],
-  "rules": {
-    "react-hooks/rules-of-hooks": "off"
-  }
+  "ignorePatterns": ["src/service*.ts"]
 }

src/utils/helpers/hooks.ts

@@ -1,18 +1,22 @@
 import { parseQuery } from '@shlinkio/shlink-frontend-kit';
-import { useRef, useState } from 'react';
+import { useCallback, useRef, useState } from 'react';
 import { useLocation, useNavigate } from 'react-router-dom';
 
 const DEFAULT_DELAY = 2000;
 
-export type TimeoutToggle = (initialValue?: boolean, delay?: number) => [boolean, () => void];
+export type TimeoutToggle = typeof useTimeoutToggle;
 
 export const useTimeoutToggle = (
-  setTimeout: (callback: Function, timeout: number) => number,
-  clearTimeout: (timer: number) => void,
-): TimeoutToggle => (initialValue = false, delay = DEFAULT_DELAY) => {
+  initialValue = false,
+  delay = DEFAULT_DELAY,
+
+  // Test seams
+  setTimeout = window.setTimeout,
+  clearTimeout = window.clearTimeout,
+): [boolean, () => void] => {
   const [flag, setFlag] = useState<boolean>(initialValue);
   const timeout = useRef<number | undefined>(undefined);
-  const callback = () => {
+  const callback = useCallback(() => {
     setFlag(!initialValue);
 
     if (timeout.current) {
@@ -20,7 +24,7 @@ export const useTimeoutToggle = (
     }
 
     timeout.current = setTimeout(() => setFlag(initialValue), delay);
-  };
+  }, [clearTimeout, delay, initialValue, setTimeout]);
 
   return [flag, callback];
 };

src/utils/services/provideServices.ts

@@ -12,7 +12,5 @@ export const provideServices = (bottle: Bottle) => {
   bottle.constant('csvToJson', csvToJson);
   bottle.constant('jsonToCsv', jsonToCsv);
 
-  bottle.constant('setTimeout', window.setTimeout);
-  bottle.constant('clearTimeout', window.clearTimeout);
-  bottle.serviceFactory('useTimeoutToggle', useTimeoutToggle, 'setTimeout', 'clearTimeout');
+  bottle.serviceFactory('useTimeoutToggle', () => useTimeoutToggle);
 };