Fix tests

elastic · May 14, 2021 · 11b01b0 · 11b01b0
1 parent 3b4634e
commit 11b01b0
Show file tree

Hide file tree

Showing 6 changed files with 166 additions and 56 deletions.
diff --git a/x-pack/plugins/cases/public/components/recent_cases/index.test.tsx b/x-pack/plugins/cases/public/components/recent_cases/index.test.tsx
@@ -74,7 +74,6 @@ describe('RecentCases', () => {
     );
     expect(useGetCasesMock).toBeCalledWith({
       initialQueryParams: { perPage: 2 },
-      initialFilterOptions: { owner: [SECURITY_SOLUTION_OWNER] },
     });
   });
 

diff --git a/x-pack/plugins/cases/public/containers/api.test.tsx b/x-pack/plugins/cases/public/containers/api.test.tsx
@@ -137,6 +137,7 @@ describe('Case Configuration API', () => {
           ...DEFAULT_QUERY_PARAMS,
           reporters: [],
           tags: [],
+          owner: [SECURITY_SOLUTION_OWNER],
         },
         signal: abortCtrl.signal,
       });
@@ -163,6 +164,7 @@ describe('Case Configuration API', () => {
           tags: ['"coke"', '"pepsi"'],
           search: 'hello',
           status: CaseStatuses.open,
+          owner: [SECURITY_SOLUTION_OWNER],
         },
         signal: abortCtrl.signal,
       });
@@ -191,6 +193,7 @@ describe('Case Configuration API', () => {
           tags: ['"("', '"\\"double\\""'],
           search: 'hello',
           status: CaseStatuses.open,
+          owner: [SECURITY_SOLUTION_OWNER],
         },
         signal: abortCtrl.signal,
       });

diff --git a/x-pack/plugins/cases/public/containers/configure/use_configure.test.tsx b/x-pack/plugins/cases/public/containers/configure/use_configure.test.tsx
@@ -5,6 +5,7 @@
  * 2.0.
  */
 
+import React from 'react';
 import { renderHook, act } from '@testing-library/react-hooks';
 import {
   initialState,
@@ -15,6 +16,7 @@ import {
 import { mappings, caseConfigurationCamelCaseResponseMock } from './mock';
 import * as api from './api';
 import { ConnectorTypes } from '../../../common';
+import { TestProviders } from '../../common/mock';
 
 const mockErrorToast = jest.fn();
 const mockSuccessToast = jest.fn();
@@ -49,8 +51,11 @@ describe('useConfigure', () => {
 
   test('init', async () => {
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       expect(result.current).toEqual({
@@ -67,8 +72,11 @@ describe('useConfigure', () => {
 
   test('fetch case configuration', async () => {
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -99,8 +107,11 @@ describe('useConfigure', () => {
     const spyOnGetCaseConfigure = jest.spyOn(api, 'getCaseConfigure');
 
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -111,8 +122,11 @@ describe('useConfigure', () => {
 
   test('correctly sets mappings', async () => {
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -124,8 +138,11 @@ describe('useConfigure', () => {
 
   test('set isLoading to true when fetching case configuration', async () => {
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -137,8 +154,11 @@ describe('useConfigure', () => {
 
   test('persist case configuration', async () => {
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -166,8 +186,11 @@ describe('useConfigure', () => {
     );
 
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -192,8 +215,11 @@ describe('useConfigure', () => {
     );
 
     await act(async () => {
-      const { waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -221,8 +247,11 @@ describe('useConfigure', () => {
     );
 
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -245,8 +274,11 @@ describe('useConfigure', () => {
     );
 
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
       await waitForNextUpdate();
       await waitForNextUpdate();
@@ -266,8 +298,11 @@ describe('useConfigure', () => {
     });
 
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
 
       await waitForNextUpdate();
@@ -302,8 +337,11 @@ describe('useConfigure', () => {
     });
 
     await act(async () => {
-      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(() =>
-        useCaseConfigure()
+      const { result, waitForNextUpdate } = renderHook<string, ReturnUseCaseConfigure>(
+        () => useCaseConfigure(),
+        {
+          wrapper: ({ children }) => <TestProviders>{children}</TestProviders>,
+        }
       );
 
       await waitForNextUpdate();
-Original file line number
+Diff line change
@@ Expand Up / @@ -74,7 +74,6 @@ describe('RecentCases', () => { @@
         );
         expect(useGetCasesMock).toBeCalledWith({
           initialQueryParams: { perPage: 2 },
-          initialFilterOptions: { owner: [SECURITY_SOLUTION_OWNER] },
         });
       });
@@ Expand Down @@