Skip to content

Commit

Permalink
Remove assertion about upsell tooltip when suppression is disabled
Browse files Browse the repository at this point in the history
See TODO for details.
  • Loading branch information
rylnd committed Apr 9, 2024
1 parent 4a35e59 commit f974273
Showing 1 changed file with 2 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down Expand Up @@ -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', () => {
Expand Down

0 comments on commit f974273

Please sign in to comment.