From 84fe0fb70e9047a73114df163d9e01f8628cc010 Mon Sep 17 00:00:00 2001
From: Jatin Kathuria <jatin.kathuria@elastic.co>
Date: Thu, 28 Mar 2024 10:49:19 +0100
Subject: [PATCH] fix: review feedback cypress

---
 .../timelines/unified_components/query_tab.cy.ts      | 11 +++++++++--
 .../cypress/tasks/discover.ts                         |  2 +-
 .../cypress/tasks/unified_timeline.ts                 |  5 -----
 3 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/x-pack/test/security_solution_cypress/cypress/e2e/investigations/timelines/unified_components/query_tab.cy.ts b/x-pack/test/security_solution_cypress/cypress/e2e/investigations/timelines/unified_components/query_tab.cy.ts
index 8cef9077aaa190..2571f529dcf084 100644
--- a/x-pack/test/security_solution_cypress/cypress/e2e/investigations/timelines/unified_components/query_tab.cy.ts
+++ b/x-pack/test/security_solution_cypress/cypress/e2e/investigations/timelines/unified_components/query_tab.cy.ts
@@ -11,7 +11,12 @@ import {
   openHostDetailsFlyout,
   openUserDetailsFlyout,
 } from '../../../../tasks/unified_timeline';
-import { GET_UNIFIED_DATA_GRID_CELL_HEADER } from '../../../../screens/unified_timeline';
+import {
+  GET_UNIFIED_DATA_GRID_CELL_HEADER,
+  HOST_DETAILS_FLYOUT,
+  TIMELINE_DETAILS_FLYOUT,
+  USER_DETAILS_FLYOUT,
+} from '../../../../screens/unified_timeline';
 import { GET_DISCOVER_DATA_GRID_CELL_HEADER } from '../../../../screens/discover';
 import { addFieldToTable, removeFieldFromTable } from '../../../../tasks/discover';
 import { login } from '../../../../tasks/login';
@@ -54,13 +59,15 @@ describe(
       it('should be able to open/close details details/host/user flyout', () => {
         cy.log('Event Details Flyout');
         openEventDetailsFlyout(0);
+        cy.get(TIMELINE_DETAILS_FLYOUT).should('be.visible');
         closeTimelineFlyout();
         cy.log('Host Details Flyout');
         openHostDetailsFlyout(0);
+        cy.get(HOST_DETAILS_FLYOUT).should('be.visible');
         closeTimelineFlyout();
         cy.log('User Details Flyout');
         openUserDetailsFlyout(0);
-        closeTimelineFlyout();
+        cy.get(USER_DETAILS_FLYOUT).should('be.visible');
       });
     });
   }
diff --git a/x-pack/test/security_solution_cypress/cypress/tasks/discover.ts b/x-pack/test/security_solution_cypress/cypress/tasks/discover.ts
index 8bc39cc7ccd5e2..e888f75149b2de 100644
--- a/x-pack/test/security_solution_cypress/cypress/tasks/discover.ts
+++ b/x-pack/test/security_solution_cypress/cypress/tasks/discover.ts
@@ -113,7 +113,7 @@ export const addFieldToTable = (fieldId: string) => {
 };
 
 export const removeFieldFromTable = (fieldId: string) => {
-  cy.get(GET_DISCOVER_COLUMN_TOGGLE_BTN(fieldId)).first().trigger('click');
+  cy.get(GET_DISCOVER_COLUMN_TOGGLE_BTN(fieldId)).first().click();
 };
 
 export const createAdHocDataView = (name: string, indexPattern: string, save: boolean = false) => {
diff --git a/x-pack/test/security_solution_cypress/cypress/tasks/unified_timeline.ts b/x-pack/test/security_solution_cypress/cypress/tasks/unified_timeline.ts
index c89b8745b1c73d..d53ef784a8d246 100644
--- a/x-pack/test/security_solution_cypress/cypress/tasks/unified_timeline.ts
+++ b/x-pack/test/security_solution_cypress/cypress/tasks/unified_timeline.ts
@@ -8,28 +8,23 @@
 import {
   GET_UNIFIED_DATA_GRID_CELL,
   GET_UNIFIED_DATA_GRID_CELL_HEADER,
-  HOST_DETAILS_FLYOUT,
   HOST_DETAILS_LINK,
   TIMELINE_DETAILS_FLYOUT,
   TIMELINE_DETAILS_FLYOUT_BTN,
   TIMELINE_DETAILS_FLYOUT_CLOSE_BTN,
-  USER_DETAILS_FLYOUT,
   USER_DETAILS_LINK,
 } from '../screens/unified_timeline';
 
 export const openEventDetailsFlyout = (rowIndex: number) => {
   cy.get(TIMELINE_DETAILS_FLYOUT_BTN).eq(rowIndex).click();
-  cy.get(TIMELINE_DETAILS_FLYOUT).should('be.visible');
 };
 
 export const openHostDetailsFlyout = (rowIndex: number) => {
   cy.get(HOST_DETAILS_LINK).eq(rowIndex).click();
-  cy.get(HOST_DETAILS_FLYOUT).should('be.visible');
 };
 
 export const openUserDetailsFlyout = (rowIndex: number) => {
   cy.get(USER_DETAILS_LINK).eq(rowIndex).click();
-  cy.get(USER_DETAILS_FLYOUT).should('be.visible');
 };
 
 export const getUnifiedTableHeaderColumn = (columnName: string) => {