PR changes applied

web/package.json

@@ -39,7 +39,6 @@
   "scripts": {
     "start": "craco start",
     "build": "craco build",
-    "test:work": "craco test --watchAll=false --coverage ./src/hooks --collectCoverageFrom='src/hooks/**/*.(tsx|ts)'",
     "test:unit": "craco test --watchAll=false --coverage",
     "test:integration": "npm run cy:run",
     "test:lint": "eslint ./src/**/*.{ts,tsx}",
@@ -79,8 +78,7 @@
   "jest": {
     "transformIgnorePatterns": [
       "/node_modules/(?!antd|@ant-design|rc-.+?|@babel/runtime).+(js|jsx)$"
-    ],
-    "testMatch": [ "<rootDir>/src/hooks/**/*.test.tsx" ]
+    ]
   },
   "resolutions": {
     "styled-components": "^5"

web/src/hooks/useDAGChart.test.tsx → web/src/hooks/__tests__/useDAGChart.test.tsx

@@ -1,6 +1,6 @@
 import {renderHook} from '@testing-library/react-hooks';
-import {SemanticGroupNames} from '../constants/SemanticGroupNames.constants';
-import {useDAGChart} from './useDAGChart';
+import {SemanticGroupNames} from '../../constants/SemanticGroupNames.constants';
+import {useDAGChart} from '../useDAGChart';
 
 test('useDAGChart with empty span', () => {
   const {result} = renderHook(() => useDAGChart({}));

web/src/hooks/useDoubleClick.test.tsx → web/src/hooks/__tests__/useDoubleClick.test.tsx

@@ -1,5 +1,5 @@
 import {act, renderHook} from '@testing-library/react-hooks';
-import {useDoubleClick} from './useDoubleClick';
+import {useDoubleClick} from '../useDoubleClick';
 
 test('useDoubleClick', () => {
   const doubleClick = jest.fn();

web/src/hooks/useElementSize.test.tsx → web/src/hooks/__tests__/useElementSize.test.tsx

@@ -1,5 +1,5 @@
 import {renderHook} from '@testing-library/react-hooks';
-import {useElementSize} from './useElementSize';
+import {useElementSize} from '../useElementSize';
 
 test('useElementSize', () => {
   const {result} = renderHook(() => useElementSize());

web/src/hooks/useGuidedTour.test.tsx → web/src/hooks/__tests__/useGuidedTour.test.tsx

@@ -1,6 +1,6 @@
 import {renderHook} from '@testing-library/react-hooks';
-import {GuidedTours} from '../services/GuidedTour.service';
-import useGuidedTour from './useGuidedTour';
+import {GuidedTours} from '../../services/GuidedTour.service';
+import useGuidedTour from '../useGuidedTour';
 
 test('useGuidedTour', () => {
   const {result} = renderHook(() => useGuidedTour(GuidedTours.Home));

web/src/hooks/usePolling.test.tsx → web/src/hooks/__tests__/usePolling.test.tsx

@@ -1,5 +1,5 @@
 import {renderHook} from '@testing-library/react-hooks';
-import usePolling from './usePolling';
+import usePolling from '../usePolling';
 
 test('usePolling', async () => {
   const callback = jest.fn();

web/src/hooks/useDAGChart.ts

@@ -3,12 +3,12 @@ import {Dag} from 'd3-dag';
 import _ from 'lodash';
 import {TSpanMap} from '../components/Diagram/components/DAG';
 
-export const useDAGChart = (
-  spanMap: TSpanMap = {}
-): void | {
+interface DAGResponse {
   dag: Dag<{id: string; parentIds: string[]}, undefined>;
   layout: {width: number; height: number};
-} => {
+}
+
+export const useDAGChart = (spanMap: TSpanMap = {}): void | DAGResponse => {
   if (_.isEmpty(spanMap)) {
     return;
   }

web/src/hooks/usePolling.ts

@@ -2,28 +2,23 @@ import {useEffect} from 'react';
 
 interface usePollingProps {
   callback(): void;
+
   delay: number;
   isPolling: boolean;
 }
 
 const usePolling = ({callback, delay = 1000, isPolling}: usePollingProps): void => {
   useEffect(() => {
     let interval: any = null;
-
-    console.log(1);
     interval = setInterval(() => {
-      console.log(2);
       if (isPolling) {
-        console.log(3);
         callback();
       } else {
-        console.log(4);
         interval && clearInterval(interval);
       }
     }, delay);
 
     return () => {
-      console.log(5);
       return interval && clearInterval(interval);
     };
   }, [delay, isPolling, callback]);