From f974273fa1e49d9c4a64a024afb6ac93ee31b131 Mon Sep 17 00:00:00 2001 From: Ryland Herrick Date: Mon, 8 Apr 2024 19:05:05 -0500 Subject: [PATCH] Remove assertion about upsell tooltip when suppression is disabled See TODO for details. --- .../event_correlation_rule_suppression_ess_basic.cy.ts | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/x-pack/test/security_solution_cypress/cypress/e2e/detection_response/detection_engine/rule_creation/event_correlation_rule_suppression_ess_basic.cy.ts b/x-pack/test/security_solution_cypress/cypress/e2e/detection_response/detection_engine/rule_creation/event_correlation_rule_suppression_ess_basic.cy.ts index 1b3a53c6f8252..138bb4ea777bc 100644 --- a/x-pack/test/security_solution_cypress/cypress/e2e/detection_response/detection_engine/rule_creation/event_correlation_rule_suppression_ess_basic.cy.ts +++ b/x-pack/test/security_solution_cypress/cypress/e2e/detection_response/detection_engine/rule_creation/event_correlation_rule_suppression_ess_basic.cy.ts @@ -12,10 +12,7 @@ import { getDetails } from '../../../../tasks/rule_details'; import { CREATE_RULE_URL } from '../../../../urls/navigation'; import { deleteAlertsAndRules } from '../../../../tasks/api_calls/common'; import { selectEqlRuleType } from '../../../../tasks/create_new_rule'; -import { - ALERT_SUPPRESSION_FIELDS_INPUT, - ALERT_SUPPRESSION_FIELDS, -} from '../../../../screens/create_new_rule'; +import { ALERT_SUPPRESSION_FIELDS_INPUT } from '../../../../screens/create_new_rule'; import { DEFINITION_DETAILS, SUPPRESS_FOR_DETAILS, @@ -61,10 +58,7 @@ describe( selectEqlRuleType(); cy.get(ALERT_SUPPRESSION_FIELDS_INPUT).should('be.disabled'); - cy.get(ALERT_SUPPRESSION_FIELDS).trigger('mouseover'); - - // Platinum license is required, tooltip on disabled alert suppression checkbox should tell this - cy.get(TOOLTIP).contains('Platinum license'); + // TODO the upsell tooltip is displayed here, but triggering it via a disabled element is very flaky (both in cypress and in the actual UI). Until we can figure out a better way to do this, we're just going to assert the disablement itself. }); it('shows upselling message on rule details with suppression on basic license', () => {