From c61787ff5b708dea84e9c12bb4e5ba3f88865dee Mon Sep 17 00:00:00 2001
From: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Date: Tue, 12 Apr 2022 16:57:38 -0500
Subject: [PATCH] [SecuritySolution] Remove the cell hovers actions for agent
 status (#130042) (#130056)

* fix: remove the cell actions for agent status

As discussed in https://github.com/elastic/kibana/issues/127010, the agent.status should not have hover actions

* chore: use object-lookup instead of array.includes

(cherry picked from commit 9f37de516b37445d2ec1db7728e8f2d946ef4c9c)

Co-authored-by: Jan Monschke <jan.monschke@elastic.co>
---
 .../table/summary_value_cell.test.tsx         | 38 ++++++++++++++++++-
 .../table/summary_value_cell.tsx              |  6 ++-
 2 files changed, 41 insertions(+), 3 deletions(-)

diff --git a/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.test.tsx b/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.test.tsx
index b51aeb3253eee..10c4c839ee2e6 100644
--- a/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.test.tsx
+++ b/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.test.tsx
@@ -14,6 +14,7 @@ import { TestProviders } from '../../../mock';
 import { EventFieldsData } from '../types';
 import { AlertSummaryRow } from '../helpers';
 import { TimelineId } from '../../../../../common/types';
+import { AGENT_STATUS_FIELD_NAME } from '../../../../timelines/components/timeline/body/renderers/constants';
 
 jest.mock('../../../lib/kibana');
 
@@ -52,6 +53,27 @@ const enrichedHostIpData: AlertSummaryRow['description'] = {
   values: [...hostIpValues],
 };
 
+const enrichedAgentStatusData: AlertSummaryRow['description'] = {
+  data: {
+    field: AGENT_STATUS_FIELD_NAME,
+    format: '',
+    type: '',
+    aggregatable: false,
+    description: '',
+    example: '',
+    category: '',
+    fields: {},
+    indexes: [],
+    name: AGENT_STATUS_FIELD_NAME,
+    searchable: false,
+    readFromDocValues: false,
+    isObjectArray: false,
+  },
+  eventId,
+  values: [],
+  timelineId: TimelineId.test,
+};
+
 describe('SummaryValueCell', () => {
   test('it should render', async () => {
     render(
@@ -64,8 +86,8 @@ describe('SummaryValueCell', () => {
     expect(screen.getAllByTestId('test-filter-out')).toHaveLength(1);
   });
 
-  describe('When in the timeline flyout with timelineId active', () => {
-    test('it should not render the default hover actions', async () => {
+  describe('Without hover actions', () => {
+    test('When in the timeline flyout with timelineId active', async () => {
       render(
         <TestProviders>
           <SummaryValueCell {...enrichedHostIpData} timelineId={TimelineId.active} />
@@ -75,5 +97,17 @@ describe('SummaryValueCell', () => {
       expect(screen.queryByTestId('test-filter-for')).toBeNull();
       expect(screen.queryByTestId('test-filter-out')).toBeNull();
     });
+
+    test('When rendering the host status field', async () => {
+      render(
+        <TestProviders>
+          <SummaryValueCell {...enrichedAgentStatusData} />
+        </TestProviders>
+      );
+
+      expect(screen.getByTestId('event-field-agent.status')).toBeInTheDocument();
+      expect(screen.queryByTestId('test-filter-for')).toBeNull();
+      expect(screen.queryByTestId('test-filter-out')).toBeNull();
+    });
   });
 });
diff --git a/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx b/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx
index ad9ca84429f00..1c9c0292ed912 100644
--- a/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx
+++ b/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx
@@ -12,6 +12,10 @@ import { FieldValueCell } from './field_value_cell';
 import { AlertSummaryRow } from '../helpers';
 import { TimelineId } from '../../../../../common/types';
 
+import { AGENT_STATUS_FIELD_NAME } from '../../../../timelines/components/timeline/body/renderers/constants';
+
+const FIELDS_WITHOUT_ACTIONS: { [field: string]: boolean } = { [AGENT_STATUS_FIELD_NAME]: true };
+
 export const SummaryValueCell: React.FC<AlertSummaryRow['description']> = ({
   data,
   eventId,
@@ -33,7 +37,7 @@ export const SummaryValueCell: React.FC<AlertSummaryRow['description']> = ({
       style={{ flexGrow: 0 }}
       values={values}
     />
-    {timelineId !== TimelineId.active && !isReadOnly && (
+    {timelineId !== TimelineId.active && !isReadOnly && !FIELDS_WITHOUT_ACTIONS[data.field] && (
       <ActionCell
         contextId={timelineId}
         data={data}