[7.x] [SIEM] Fix eslint errors (#49713) (#51875)

* [SIEM] Fix eslint errors (#49713)

* Update use_kibana_ui_setting.ts

Author: patrykkopycinski

.eslintrc.js

@@ -202,13 +202,6 @@ module.exports = {
         'jsx-a11y/click-events-have-key-events': 'off',
       },
     },
-    {
-      files: ['x-pack/legacy/plugins/siem/**/*.{js,ts,tsx}'],
-      rules: {
-        'react-hooks/exhaustive-deps': 'off',
-        'react-hooks/rules-of-hooks': 'off',
-      },
-    },
     {
       files: ['x-pack/legacy/plugins/snapshot_restore/**/*.{js,ts,tsx}'],
       rules: {
@@ -827,6 +820,8 @@ module.exports = {
         // might be introduced after the other warns are fixed
         // 'react/jsx-sort-props': 'error',
         'react/jsx-tag-spacing': 'error',
+        // might be introduced after the other warns are fixed
+        'react-hooks/exhaustive-deps': 'off',
         'require-atomic-updates': 'error',
         'rest-spread-spacing': ['error', 'never'],
         'symbol-description': 'error',

x-pack/legacy/plugins/siem/public/components/embeddables/embedded_map.test.tsx

@@ -10,13 +10,18 @@ import * as React from 'react';
 import { EmbeddedMapComponent } from './embedded_map';
 import { SetQuery } from './types';
 import { useKibanaCore } from '../../lib/compose/kibana_core';
+import { useIndexPatterns } from '../../hooks/use_index_patterns';
 
 jest.mock('../search_bar', () => ({
   siemFilterManager: {
     addFilters: jest.fn(),
   },
 }));
 
+const mockUseIndexPatterns = useIndexPatterns as jest.Mock;
+jest.mock('../../hooks/use_index_patterns');
+mockUseIndexPatterns.mockImplementation(() => [true, []]);
+
 const mockUseKibanaCore = useKibanaCore as jest.Mock;
 jest.mock('../../lib/compose/kibana_core');
 mockUseKibanaCore.mockImplementation(() => ({

x-pack/legacy/plugins/siem/public/components/flyout/pane/index.test.tsx

@@ -24,6 +24,10 @@ mockUseKibanaCore.mockImplementation(() => ({
   uiSettings: mockUiSettings,
 }));
 
+jest.mock('ui/vis/lib/timezone', () => ({
+  timezoneProvider: () => () => 'America/New_York',
+}));
+
 describe('Pane', () => {
   test('renders correctly against snapshot', () => {
     const EmptyComponent = shallow(

x-pack/legacy/plugins/siem/public/components/link_to/redirect_wrapper.tsx

@@ -6,13 +6,13 @@
 
 import React from 'react';
 import { Redirect } from 'react-router-dom';
-import { scrollToTop } from '../scroll_to_top';
+import { useScrollToTop } from '../scroll_to_top';
 
 export interface RedirectWrapperProps {
   to: string;
 }
 
 export const RedirectWrapper = ({ to }: RedirectWrapperProps) => {
-  scrollToTop();
+  useScrollToTop();
   return <Redirect to={to} />;
 };

x-pack/legacy/plugins/siem/public/components/resize_handle/is_resizing.tsx

@@ -6,7 +6,7 @@
 
 import { useState } from 'react';
 
-export const isContainerResizing = () => {
+export const useIsContainerResizing = () => {
   const [isResizing, setIsResizing] = useState(false);
 
   return {

x-pack/legacy/plugins/siem/public/components/scroll_to_top/index.test.tsx

@@ -8,7 +8,7 @@ import { mount } from 'enzyme';
 import * as React from 'react';
 
 import { globalNode, HookWrapper } from '../../mock';
-import { scrollToTop } from '.';
+import { useScrollToTop } from '.';
 
 const spyScroll = jest.fn();
 const spyScrollTo = jest.fn();
@@ -21,15 +21,15 @@ describe('Scroll to top', () => {
 
   test('scroll have been called', () => {
     Object.defineProperty(globalNode.window, 'scroll', { value: spyScroll });
-    mount(<HookWrapper hook={() => scrollToTop()} />);
+    mount(<HookWrapper hook={() => useScrollToTop()} />);
 
     expect(spyScroll).toHaveBeenCalledWith(0, 0);
   });
 
   test('scrollTo have been called', () => {
     Object.defineProperty(globalNode.window, 'scroll', { value: null });
     Object.defineProperty(globalNode.window, 'scrollTo', { value: spyScrollTo });
-    mount(<HookWrapper hook={() => scrollToTop()} />);
+    mount(<HookWrapper hook={() => useScrollToTop()} />);
     expect(spyScrollTo).toHaveBeenCalled();
   });
 });

x-pack/legacy/plugins/siem/public/components/scroll_to_top/index.tsx

@@ -6,7 +6,7 @@
 
 import { useEffect } from 'react';
 
-export const scrollToTop = () => {
+export const useScrollToTop = () => {
   useEffect(() => {
     // trying to use new API - https://developer.mozilla.org/en-US/docs/Web/API/Window/scrollTo
     if (window.scroll) {

x-pack/legacy/plugins/siem/public/components/timeline/body/column_headers/index.test.tsx

@@ -19,7 +19,7 @@ import { ColumnHeadersComponent } from '.';
 
 jest.mock('../../../resize_handle/is_resizing', () => ({
   ...jest.requireActual('../../../resize_handle/is_resizing'),
-  isContainerResizing: () => ({
+  useIsContainerResizing: () => ({
     isResizing: true,
     setIsResizing: jest.fn(),
   }),

x-pack/legacy/plugins/siem/public/components/timeline/body/column_headers/index.tsx

@@ -18,7 +18,7 @@ import {
 import { DraggableFieldBadge } from '../../../draggables/field_badge';
 import { StatefulFieldsBrowser } from '../../../fields_browser';
 import { FIELD_BROWSER_HEIGHT, FIELD_BROWSER_WIDTH } from '../../../fields_browser/helpers';
-import { isContainerResizing } from '../../../resize_handle/is_resizing';
+import { useIsContainerResizing } from '../../../resize_handle/is_resizing';
 import {
   OnColumnRemoved,
   OnColumnResized,
@@ -71,7 +71,7 @@ export const ColumnHeadersComponent = ({
   timelineId,
   toggleColumn,
 }: Props) => {
-  const { isResizing, setIsResizing } = isContainerResizing();
+  const { isResizing, setIsResizing } = useIsContainerResizing();
 
   return (
     <EventsThead data-test-subj="column-headers">

x-pack/legacy/plugins/siem/public/components/timeline/body/renderers/endgame/endgame_security_event_details_line.tsx

@@ -18,7 +18,7 @@ import {
   getHumanReadableLogonType,
   getUserDomainField,
   getUserNameField,
-  useTargetUserAndTargetDomain,
+  getTargetUserAndTargetDomain,
 } from './helpers';
 
 import * as i18n from './translations';
@@ -65,10 +65,10 @@ export const EndgameSecurityEventDetailsLine = React.memo<Props>(
     userName,
     winlogEventId,
   }) => {
-    const domain = useTargetUserAndTargetDomain(eventAction) ? endgameTargetDomainName : userDomain;
+    const domain = getTargetUserAndTargetDomain(eventAction) ? endgameTargetDomainName : userDomain;
     const eventDetails = getEventDetails(eventAction);
     const hostNameSeparator = getHostNameSeparator(eventAction);
-    const user = useTargetUserAndTargetDomain(eventAction) ? endgameTargetUserName : userName;
+    const user = getTargetUserAndTargetDomain(eventAction) ? endgameTargetUserName : userName;
     const userDomainField = getUserDomainField(eventAction);
     const userNameField = getUserNameField(eventAction);