[Security Solution] [Cases] Deleting files from security solution and removing feature flag

packages/kbn-optimizer/limits.yml

@@ -69,7 +69,7 @@ pageLoadAssetSize:
   searchprofiler: 67080
   security: 189428
   securityOss: 30806
-  securitySolution: 283440
+  securitySolution: 187863
   share: 99061
   snapshotRestore: 79032
   spaces: 387915

x-pack/plugins/cases/README.md

@@ -52,14 +52,39 @@ Arguments:
 |disabledStatuses?|`CaseStatuses[];` array of disabled statuses
 |getCaseDetailsHref|`(caseDetails: CaseDetailsHrefSchema) => string;` callback to generate the case details url from the case id
 |isModal?|`boolean;` is All Cases table a modal
-|onCaseDetailsNavClick|`(caseDetails: CaseDetailsHrefSchema) => void;` callback for nav click
-|onConfigureCasesNavClick?|`(ev: React.MouseEvent) => void;` callback for nav click
-|onCreateCaseNavClick?|`(ev: React.MouseEvent) => void;` callback for nav click
+|onCaseDetailsNavClick|`(caseDetails: CaseDetailsHrefSchema) => void;` callback for case details nav click
+|onConfigureCasesNavClick?|`(ev: React.MouseEvent) => void;` callback for configure case nav click
+|onCreateCaseNavClick?|`(ev: React.MouseEvent) => void;` callback for create case nav click
 |onRowClick?|`(theCase?: Case ! SubCase) => void;` callback for row click, passing case in row
-|userCanCrud|boolean; user permissions to crud
+|userCanCrud|`boolean;` user permissions to crud
 
 UI component:
  ![All Cases Component][all-cases-img] 
+
+### `getCaseView`
+Arguments: 
+
+|Property|Description|
+|---|---|
+|allCasesHref|`string;` route for all cases page
+|backToAllCasesOnClick|`(ev: MouseEvent) => void;` handler for back to all cases click
+|caseDetailsHref|`string;` route for configure cases page
+|caseId|`string;` ID of the case
+|configureCasesHref|`string;` route for configure cases page
+|getCaseDetailHrefWithCommentId|`(commentId: string) => string;` callback to generate the case details url with a comment id reference from the case id and comment id
+|getRuleDetailsHref|`(ruleId: string, null, undefined) => string;` callback to generate the rule details url from the rule id
+|onComponentInitialized?|`() => void;` callback when component has initialized
+|onConfigureCasesNavClick|`(ev: React.MouseEvent) => void;` callback for configure case nav click
+|onRuleDetailsClick|`(ruleId: string, null, undefined) => void;` callback for rule details nav click
+|renderInvestigateInTimelineActionComponent?|: `(alertIds: string[]) => JSX.Element;` space to render `InvestigateInTimelineActionComponent`
+|renderTimelineDetailsPanel?|: `() => JSX.Element;` space to render `TimelineDetailsPanel`
+|showAlertDetails|: `(alertId: string, index: string) => void;` callback to show alert details
+|subCaseId?|: `string;` subcase id
+|useFetchAlertData|: `(alertIds: string[]) => [boolean, Record<string, Ecs>];` fetch alerts
+|userCanCrud|: `boolean;` user permissions to crud
+
+UI component:
+ ![Case View Component][case-view-img] 
 
 ### `getCreateCase`
 Arguments: 
@@ -83,6 +108,21 @@ UI component:
  UI component:
   ![Configure Component][configure-img] 
 
+### `getRecentCases`
+Arguments: 
+
+|Property|Description|
+|---|---|
+|allCasesHref|`string;` href of all cases page
+|createCaseHref|`string;` 
+|getCaseDetailsHref|`(caseDetails: CaseDetailsHrefSchema) => string;` 
+|goToAllCases|`(ev: React.MouseEvent) => void;` callback for all cases link click
+|onCaseDetailsNavClick|`(caseDetails: CaseDetailsHrefSchema) => void;` callback for case details nav click
+|perPage|`number;` number of cases to show in widget
+
+UI component:
+ ![Recent Cases Component][recent-cases-img] 
+
 ## Case Action Type
 
 See [Kibana Actions](https://github.com/elastic/kibana/tree/master/x-pack/plugins/actions) for more information.
@@ -192,4 +232,6 @@ For IBM Resilient connectors:
 [configure-img]: images/configure.png
 [create-img]: images/create.png
 [all-cases-img]: images/all_cases.png
+[recent-cases-img]: images/recent_cases.png
+[case-view-img]: images/case_view.png
 

x-pack/plugins/cases/common/index.ts

@@ -7,3 +7,4 @@
 
 export * from './constants';
 export * from './api';
+export * from './ui/types';

x-pack/plugins/security_solution/public/cases/components/add_comment/translations.ts → x-pack/plugins/cases/common/ui/index.ts

@@ -5,4 +5,4 @@
  * 2.0.
  */
 
-export * from '../../translations';
+export * from './types';

x-pack/plugins/security_solution/public/cases/containers/types.ts → x-pack/plugins/cases/common/ui/types.ts

@@ -16,10 +16,12 @@ import {
   User,
   UserAction,
   UserActionField,
-} from '../../../../cases/common';
-import { CaseStatusWithAllStatus } from '../components/status';
+} from '../api';
 
-export { CaseConnector, ActionConnector, CaseStatuses } from '../../../../cases/common';
+export const StatusAll = 'all' as const;
+export type StatusAllType = typeof StatusAll;
+
+export type CaseStatusWithAllStatus = CaseStatuses | StatusAllType;
 
 export type Comment = CommentRequest & {
   associationType: AssociationType;

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

@@ -14,7 +14,7 @@ import { TestProviders } from '../../common/mock';
 import { casesStatus, useGetCasesMockState, collectionCase } from '../../containers/mock';
 import * as i18n from './translations';
 
-import { CaseStatuses, CaseType } from '../../../common';
+import { CaseStatuses, CaseType, StatusAll } from '../../../common';
 import { getEmptyTagValue } from '../empty_value';
 import { useDeleteCases } from '../../containers/use_delete_cases';
 import { useGetCases } from '../../containers/use_get_cases';
@@ -23,7 +23,6 @@ import { useUpdateCases } from '../../containers/use_bulk_update_case';
 import { useGetActionLicense } from '../../containers/use_get_action_license';
 import { getCasesColumns } from './columns';
 import { AllCases } from '.';
-import { StatusAll } from '../status';
 
 jest.mock('../../containers/use_bulk_update_case');
 jest.mock('../../containers/use_delete_cases');

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

@@ -22,7 +22,7 @@ import styled, { css } from 'styled-components';
 import classnames from 'classnames';
 
 import * as i18n from './translations';
-import { CaseStatuses, CaseType } from '../../../common';
+import { CaseStatuses, CaseType, CommentRequestAlertType, CommentType } from '../../../common';
 import { getCasesColumns } from './columns';
 import { Case, DeleteCase, FilterOptions, SortFieldCase, SubCase } from '../../containers/types';
 import { useGetCases, UpdateCase } from '../../containers/use_get_cases';
@@ -44,6 +44,7 @@ import { getActions } from './actions';
 import { CasesTableFilters } from './table_filters';
 import { useUpdateCases } from '../../containers/use_bulk_update_case';
 import { useGetActionLicense } from '../../containers/use_get_action_license';
+import { usePostComment } from '../../containers/use_post_comment';
 import { getActionLicenseError } from '../use_push_to_service/helpers';
 import { CaseCallOut } from '../callout';
 import { ConfigureCaseButton } from '../configure_cases/button';
@@ -112,9 +113,9 @@ const BasicTable = styled(EuiBasicTable)`
 BasicTable.displayName = 'BasicTable';
 
 export interface AllCasesProps {
+  alertData?: Omit<CommentRequestAlertType, 'type'>;
   configureCasesHref: string;
   createCaseHref: string;
-  disabledCases?: CaseType[];
   disabledStatuses?: CaseStatuses[];
   getCaseDetailsHref: (caseDetails: CaseDetailsHrefSchema) => string;
   isModal?: boolean;
@@ -123,13 +124,14 @@ export interface AllCasesProps {
   onCreateCaseNavClick?: (ev: React.MouseEvent) => void;
   onRowClick?: (theCase?: Case | SubCase) => void;
   userCanCrud: boolean;
+  updateCase?: (newCase: Case) => void;
 }
 
 export const AllCases = React.memo<AllCasesProps>(
   ({
+    alertData,
     configureCasesHref,
     createCaseHref,
-    disabledCases = [],
     disabledStatuses,
     getCaseDetailsHref,
     isModal = false,
@@ -138,6 +140,7 @@ export const AllCases = React.memo<AllCasesProps>(
     onCreateCaseNavClick,
     onRowClick,
     userCanCrud,
+    updateCase,
   }) => {
     const { actionLicense } = useGetActionLicense();
     const {
@@ -177,6 +180,10 @@ export const AllCases = React.memo<AllCasesProps>(
       isUpdated,
       updateBulkStatus,
     } = useUpdateCases();
+
+    // Post Comment to Case
+    const { postComment, isLoading: isCommentsUpdating } = usePostComment();
+
     const [deleteThisCase, setDeleteThisCase] = useState<DeleteCase>({
       title: '',
       id: '',
@@ -371,12 +378,12 @@ export const AllCases = React.memo<AllCasesProps>(
           onCaseDetailsNavClick,
         }),
       [
-        userCanCrud,
         actions,
         filterOptions.status,
-        isModal,
         getCaseDetailsHref,
+        isModal,
         onCaseDetailsNavClick,
+        userCanCrud,
       ]
     );
 
@@ -422,7 +429,17 @@ export const AllCases = React.memo<AllCasesProps>(
 
     const tableRowProps = useCallback(
       (theCase: Case) => {
-        const onTableRowClick = memoize(() => {
+        const onTableRowClick = memoize(async () => {
+          if (alertData != null) {
+            await postComment({
+              caseId: theCase.id,
+              data: {
+                type: CommentType.alert,
+                ...alertData,
+              },
+              updateCase,
+            });
+          }
           if (onRowClick) {
             onRowClick(theCase);
           }
@@ -434,7 +451,7 @@ export const AllCases = React.memo<AllCasesProps>(
           ...(isModal && theCase.type !== CaseType.collection ? { onClick: onTableRowClick } : {}),
         };
       },
-      [isModal, onRowClick]
+      [isModal, alertData, onRowClick, postComment, updateCase]
     );
 
     const enableBuckActions = userCanCrud && !isModal;
@@ -503,7 +520,7 @@ export const AllCases = React.memo<AllCasesProps>(
             </EuiFlexGroup>
           </CaseHeaderPage>
         )}
-        {(isCasesLoading || isDeleting || isUpdating) && !isDataEmpty && (
+        {(isCasesLoading || isDeleting || isUpdating || isCommentsUpdating) && !isDataEmpty && (
           <ProgressLoader size="xs" color="accent" className="essentialAnimation" />
         )}
         <TableWrap data-test-subj="table-wrap" loading={!isModal ? isCasesLoading : undefined}>
@@ -565,6 +582,7 @@ export const AllCases = React.memo<AllCasesProps>(
                 itemId="id"
                 items={data.cases}
                 itemIdToExpandedRowMap={itemIdToExpandedRowMap}
+                loading={isCommentsUpdating}
                 noItemsMessage={
                   <EuiEmptyPrompt
                     title={<h3>{i18n.NO_CASES}</h3>}

x-pack/plugins/cases/public/components/all_cases/status_filter.test.tsx

@@ -9,9 +9,8 @@ import React from 'react';
 import { mount } from 'enzyme';
 import { waitFor } from '@testing-library/react';
 
-import { CaseStatuses } from '../../../common';
+import { CaseStatuses, StatusAll } from '../../../common';
 import { StatusFilter } from './status_filter';
-import { StatusAll } from '../status';
 
 const stats = {
   [StatusAll]: 0,

x-pack/plugins/cases/public/components/all_cases/status_filter.tsx

@@ -7,7 +7,8 @@
 
 import React, { memo } from 'react';
 import { EuiSuperSelect, EuiSuperSelectOption, EuiFlexGroup, EuiFlexItem } from '@elastic/eui';
-import { Status, statuses, StatusAll, CaseStatusWithAllStatus } from '../status';
+import { Status, statuses } from '../status';
+import { CaseStatusWithAllStatus, StatusAll } from '../../../common';
 
 interface Props {
   stats: Record<CaseStatusWithAllStatus, number | null>;

x-pack/plugins/cases/public/components/all_cases/table_filters.tsx

@@ -10,12 +10,11 @@ import { isEqual } from 'lodash/fp';
 import styled from 'styled-components';
 import { EuiFlexGroup, EuiFlexItem, EuiFieldSearch, EuiFilterGroup } from '@elastic/eui';
 
-import { CaseStatuses } from '../../../common';
+import { CaseStatuses, CaseStatusWithAllStatus, StatusAll } from '../../../common';
 import { FilterOptions } from '../../containers/types';
 import { useGetTags } from '../../containers/use_get_tags';
 import { useGetReporters } from '../../containers/use_get_reporters';
 import { FilterPopover } from '../filter_popover';
-import { CaseStatusWithAllStatus, StatusAll } from '../status';
 import { StatusFilter } from './status_filter';
 
 import * as i18n from './translations';

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

@@ -8,8 +8,8 @@
 import React from 'react';
 import { EuiContextMenuItem } from '@elastic/eui';
 
-import { CaseStatuses } from '../../../common';
-import { statuses, CaseStatusWithAllStatus } from '../status';
+import { CaseStatuses, CaseStatusWithAllStatus } from '../../../common';
+import { statuses } from '../status';
 import * as i18n from './translations';
 import { Case } from '../../containers/types';
 

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

@@ -17,8 +17,7 @@ import {
   EuiHorizontalRule,
 } from '@elastic/eui';
 
-import { CaseStatuses, CaseAttributes, CaseType } from '../../../common';
-import { Case, CaseConnector } from '../../containers/types';
+import { CaseStatuses, CaseAttributes, CaseType, Case, CaseConnector } from '../../../common';
 import { HeaderPage } from '../header_page';
 import { EditableTitle } from '../header_page/editable_title';
 import { TagList } from '../tag_list';

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

@@ -36,8 +36,6 @@ const CaseParamsFields: React.FunctionComponent<ActionParamsProps<CaseActionPara
   actionParams,
   editAction,
   index,
-  errors,
-  messageVariables,
   actionConnector,
 }) => {
   const { caseId = null, comment = defaultAlertComment } = actionParams.subActionParams ?? {};

x-pack/plugins/cases/public/components/connectors/index.ts

@@ -17,7 +17,7 @@ import {
   ResilientFieldsType,
 } from '../../../common';
 
-export { getActionType as getCaseConnectorUI } from './case';
+export { getActionType as getCaseConnectorUi } from './case';
 
 export * from './config';
 export * from './types';

x-pack/plugins/cases/public/components/connectors/jira/search_issues.tsx

@@ -9,7 +9,7 @@ import React, { useMemo, useEffect, useCallback, useState, memo } from 'react';
 import { EuiComboBox, EuiComboBoxOptionOption } from '@elastic/eui';
 
 import { useKibana } from '../../../common/lib/kibana';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { useGetIssues } from './use_get_issues';
 import { useGetSingleIssue } from './use_get_single_issue';
 import * as i18n from './translations';

x-pack/plugins/cases/public/components/connectors/jira/use_get_fields_by_issue_type.tsx

@@ -7,7 +7,7 @@
 
 import { useState, useEffect, useRef } from 'react';
 import { HttpSetup, ToastsApi } from 'kibana/public';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { getFieldsByIssueType } from './api';
 import { Fields } from './types';
 import * as i18n from './translations';

x-pack/plugins/cases/public/components/connectors/jira/use_get_issue_types.tsx

@@ -7,7 +7,7 @@
 
 import { useState, useEffect, useRef } from 'react';
 import { HttpSetup, ToastsApi } from 'kibana/public';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { getIssueTypes } from './api';
 import { IssueTypes } from './types';
 import * as i18n from './translations';

x-pack/plugins/cases/public/components/connectors/jira/use_get_issues.tsx

@@ -8,7 +8,7 @@
 import { isEmpty, debounce } from 'lodash/fp';
 import { useState, useEffect, useRef } from 'react';
 import { HttpSetup, ToastsApi } from 'kibana/public';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { getIssues } from './api';
 import { Issues } from './types';
 import * as i18n from './translations';

x-pack/plugins/cases/public/components/connectors/jira/use_get_single_issue.tsx

@@ -7,7 +7,7 @@
 
 import { useState, useEffect, useRef } from 'react';
 import { HttpSetup, ToastsApi } from 'kibana/public';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { getIssue } from './api';
 import { Issue } from './types';
 import * as i18n from './translations';

x-pack/plugins/cases/public/components/connectors/resilient/use_get_incident_types.tsx

@@ -7,7 +7,7 @@
 
 import { useState, useEffect, useRef } from 'react';
 import { HttpSetup, ToastsApi } from 'kibana/public';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { getIncidentTypes } from './api';
 import * as i18n from './translations';
 

x-pack/plugins/cases/public/components/connectors/resilient/use_get_severity.tsx

@@ -7,7 +7,7 @@
 
 import { useState, useEffect, useRef } from 'react';
 import { HttpSetup, ToastsApi } from 'kibana/public';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { getSeverity } from './api';
 import * as i18n from './translations';
 

x-pack/plugins/cases/public/components/connectors/servicenow/use_get_choices.test.tsx

@@ -8,7 +8,7 @@
 import { renderHook } from '@testing-library/react-hooks';
 
 import { useKibana } from '../../../common/lib/kibana';
-import { ActionConnector } from '../../../containers/types';
+import { ActionConnector } from '../../../../common';
 import { choices } from '../mock';
 import { useGetChoices, UseGetChoices, UseGetChoicesProps } from './use_get_choices';
 import * as api from './api';