Fix tests

x-pack/plugins/cases/public/components/connectors/case/alert_fields.tsx

@@ -12,7 +12,7 @@ import styled from 'styled-components';
 import { EuiCallOut, EuiSpacer } from '@elastic/eui';
 
 import { ActionParamsProps } from '../../../../../triggers_actions_ui/public/types';
-import { CommentType } from '../../../../common';
+import { CommentType, SECURITY_SOLUTION_OWNER } from '../../../../common';
 
 import { CaseActionParams } from './types';
 import { ExistingCase } from './existing_case';
@@ -98,7 +98,7 @@ const CaseParamsFields: React.FunctionComponent<ActionParamsProps<CaseActionPara
       <ExistingCase
         onCaseChanged={onCaseChanged}
         selectedCase={selectedCase}
-        owner="securitySolution"
+        owner={SECURITY_SOLUTION_OWNER}
       />
       <EuiSpacer size="m" />
       <EuiCallOut size="s" title={i18n.CASE_CONNECTOR_CALL_OUT_TITLE} iconType="iInCircle">

x-pack/plugins/cases/public/components/create/form.test.tsx

@@ -15,6 +15,8 @@ import { useConnectors } from '../../containers/configure/use_connectors';
 import { connectorsMock } from '../../containers/mock';
 import { schema, FormProps } from './schema';
 import { CreateCaseForm } from './form';
+import { OwnerProvider } from '../owner_context';
+import { SECURITY_SOLUTION_OWNER } from '../../../common';
 
 jest.mock('../../containers/use_get_tags');
 jest.mock('../../containers/configure/use_connectors');
@@ -41,7 +43,11 @@ describe('CreateCaseForm', () => {
 
     globalForm = form;
 
-    return <Form form={form}>{children}</Form>;
+    return (
+      <OwnerProvider owner={[SECURITY_SOLUTION_OWNER]}>
+        <Form form={form}>{children}</Form>
+      </OwnerProvider>
+    );
   };
 
   beforeEach(() => {

x-pack/plugins/cases/public/components/create/tags.test.tsx

@@ -14,6 +14,8 @@ import { useForm, Form, FormHook } from '../../common/shared_imports';
 import { useGetTags } from '../../containers/use_get_tags';
 import { Tags } from './tags';
 import { schema, FormProps } from './schema';
+import { OwnerProvider } from '../owner_context';
+import { SECURITY_SOLUTION_OWNER } from '../../../common';
 
 jest.mock('../../containers/use_get_tags');
 const useGetTagsMock = useGetTags as jest.Mock;
@@ -31,7 +33,11 @@ describe('Tags', () => {
 
     globalForm = form;
 
-    return <Form form={form}>{children}</Form>;
+    return (
+      <OwnerProvider owner={[SECURITY_SOLUTION_OWNER]}>
+        <Form form={form}>{children}</Form>
+      </OwnerProvider>
+    );
   };
 
   beforeEach(() => {

x-pack/plugins/cases/public/components/owner_context/index.tsx

@@ -7,7 +7,6 @@
 
 import React, { useState } from 'react';
 
-// This context is available to all children of the stateful_event component where the provider is currently set
 export const OwnerContext = React.createContext<string[]>([]);
 
 export const OwnerProvider: React.FC<{

x-pack/plugins/cases/public/components/recent_cases/index.test.tsx

@@ -43,6 +43,7 @@ describe('RecentCases', () => {
     jest.clearAllMocks();
     useGetCasesMock.mockImplementation(() => mockData);
   });
+
   it('is good at loading', () => {
     useGetCasesMock.mockImplementation(() => ({
       ...mockData,
@@ -55,6 +56,7 @@ describe('RecentCases', () => {
     );
     expect(getAllByTestId('loadingPlaceholders')).toHaveLength(3);
   });
+
   it('is good at rendering cases', () => {
     const { getAllByTestId } = render(
       <TestProviders>
@@ -63,14 +65,19 @@ describe('RecentCases', () => {
     );
     expect(getAllByTestId('case-details-link')).toHaveLength(5);
   });
+
   it('is good at rendering max cases', () => {
     render(
       <TestProviders>
         <RecentCases {...{ ...defaultProps, maxCasesToShow: 2 }} />
       </TestProviders>
     );
-    expect(useGetCasesMock).toBeCalledWith({ perPage: 2 });
+    expect(useGetCasesMock).toBeCalledWith({
+      initialQueryParams: { perPage: 2 },
+      initialFilterOptions: { owner: [SECURITY_SOLUTION_OWNER] },
+    });
   });
+
   it('updates filters', () => {
     const { getByTestId } = render(
       <TestProviders>

x-pack/plugins/cases/public/containers/api.test.tsx

@@ -255,7 +255,7 @@ describe('Case Configuration API', () => {
         method: 'GET',
         signal: abortCtrl.signal,
         query: {
-          owner: SECURITY_SOLUTION_OWNER,
+          owner: [SECURITY_SOLUTION_OWNER],
         },
       });
     });
@@ -278,7 +278,7 @@ describe('Case Configuration API', () => {
         method: 'GET',
         signal: abortCtrl.signal,
         query: {
-          owner: SECURITY_SOLUTION_OWNER,
+          owner: [SECURITY_SOLUTION_OWNER],
         },
       });
     });

x-pack/plugins/cases/public/containers/configure/api.test.ts

@@ -62,7 +62,7 @@ describe('Case Configuration API', () => {
         method: 'GET',
         signal: abortCtrl.signal,
         query: {
-          owner: SECURITY_SOLUTION_OWNER,
+          owner: [SECURITY_SOLUTION_OWNER],
         },
       });
     });

x-pack/plugins/cases/public/containers/configure/use_configure.tsx

@@ -155,7 +155,7 @@ export const initialState: State = {
 };
 
 /**
- * Configurations across multiple plugins in
+ * Configurations across multiple plugins is
  * not supported at the moment. For that reason owner: string;
  * instead of owner: string[]
  */

x-pack/plugins/security_solution/public/cases/components/create/index.test.tsx

@@ -48,7 +48,7 @@ describe('Create case', () => {
     );
 
     expect(mockCreateCase).toHaveBeenCalled();
-    expect(mockCreateCase.mock.calls[0][0].owner).toBe([APP_ID]);
+    expect(mockCreateCase.mock.calls[0][0].owner).toEqual([APP_ID]);
   });
 
   it('should redirect to all cases on cancel click', async () => {