Fix tests

elastic · Jul 3, 2020 · 888234c · 888234c
1 parent 1810185
commit 888234c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/x-pack/plugins/security_solution/public/overview/pages/overview.test.tsx b/x-pack/plugins/security_solution/public/overview/pages/overview.test.tsx
@@ -19,6 +19,11 @@ import { Overview } from './index';
 
 jest.mock('../../common/lib/kibana');
 jest.mock('../../common/containers/source');
+jest.mock('../../common/containers/use_global_time', () => ({
+ useGlobalTime: jest
+ .fn()
+ .mockReturnValue({ from: 0, isInitializing: false, to: 0, setQuery: jest.fn() }),
+}));
 
 // Test will fail because we will to need to mock some core services to make the test work
 // For now let's forget about SiemSearchBar and QueryBar

diff --git a/x-pack/plugins/security_solution/public/overview/pages/overview.tsx b/x-pack/plugins/security_solution/public/overview/pages/overview.tsx
@@ -5,7 +5,7 @@
  */
 
 import { EuiFlexGroup, EuiFlexItem, EuiSpacer } from '@elastic/eui';
-import React, { useState, useMemo } from 'react';
+import React, { useCallback, useState, useMemo } from 'react';
 import { connect, ConnectedProps } from 'react-redux';
 import { StickyContainer } from 'react-sticky';
 import { Query, Filter } from 'src/plugins/data/public';
@@ -60,10 +60,10 @@ const OverviewComponent: React.FC<PropsFromRedux> = ({
  );
 
  const [dismissMessage, setDismissMessage] = useState<boolean>(hasDismissEndpointNoticeMessage);
- const dismissEndpointNotice = () => {
+ const dismissEndpointNotice = useCallback(() => {
  setDismissMessage(true);
  addMessage('management', 'dismissEndpointNotice');
- };
+ }, [addMessage]);
 
  return (
  <>