skip current test

elastic · Nov 6, 2023 · 36ff150 · 36ff150
1 parent 05b18fd
commit 36ff150
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 18 deletions.
diff --git a/...rity_solution/public/common/components/toolbar/bulk_actions/use_bulk_alert_tags_items.tsx b/...rity_solution/public/common/components/toolbar/bulk_actions/use_bulk_alert_tags_items.tsx
@@ -36,18 +36,22 @@ export const useBulkAlertTagsItems = ({ refetch }: UseBulkAlertTagsItemsProps) =
     [setAlertTags]
   );
 
-  const alertTagsItems = hasIndexWrite
-    ? [
-        {
-          key: 'manage-alert-tags',
-          'data-test-subj': 'alert-tags-context-menu-item',
-          name: i18n.ALERT_TAGS_CONTEXT_MENU_ITEM_TITLE,
-          panel: 1,
-          label: i18n.ALERT_TAGS_CONTEXT_MENU_ITEM_TITLE,
-          disableOnQuery: true,
-        },
-      ]
-    : [];
+  const alertTagsItems = useMemo(
+    () =>
+      hasIndexWrite
+        ? [
+            {
+              key: 'manage-alert-tags',
+              'data-test-subj': 'alert-tags-context-menu-item',
+              name: i18n.ALERT_TAGS_CONTEXT_MENU_ITEM_TITLE,
+              panel: 1,
+              label: i18n.ALERT_TAGS_CONTEXT_MENU_ITEM_TITLE,
+              disableOnQuery: true,
+            },
+          ]
+        : [],
+    [hasIndexWrite]
+  );
 
   const TitleContent = useMemo(
     () => (

diff --git a/...urity_solution_cypress/cypress/e2e/detection_response/detection_alerts/alert_status.cy.ts b/...urity_solution_cypress/cypress/e2e/detection_response/detection_alerts/alert_status.cy.ts
@@ -38,12 +38,12 @@ import { visit } from '../../../tasks/navigation';
 import { ALERTS_URL } from '../../../urls/navigation';
 
 // FLAKY: https://github.com/elastic/kibana/issues/169091
-describe('Changing alert status', { tags: ['@ess', '@serverless'] }, () => {
+describe('Changing alert status', () => {
   before(() => {
     cy.task('esArchiverLoad', { archiveName: 'auditbeat_big' });
   });
 
-  context('Opening alerts', () => {
+  context('Opening alerts', { tags: ['@ess', '@serverless'] }, () => {
     beforeEach(() => {
       login();
       deleteAlertsAndRules();
@@ -124,7 +124,7 @@ describe('Changing alert status', { tags: ['@ess', '@serverless'] }, () => {
     });
   });
 
-  context('Marking alerts as acknowledged', () => {
+  context('Marking alerts as acknowledged', { tags: ['@ess', '@serverless'] }, () => {
     beforeEach(() => {
       login();
       deleteAlertsAndRules();
@@ -175,7 +175,7 @@ describe('Changing alert status', { tags: ['@ess', '@serverless'] }, () => {
     });
   });
 
-  context('Closing alerts', () => {
+  context('Closing alerts', { tags: ['@ess', '@serverless'] }, () => {
     beforeEach(() => {
       login();
       deleteAlertsAndRules();
@@ -237,7 +237,10 @@ describe('Changing alert status', { tags: ['@ess', '@serverless'] }, () => {
     });
   });
 
-  context('User is readonly', () => {
+  // This test is unable to be run in serverless as `reader` is not available and viewer is currently reserved
+  // https://github.com/elastic/kibana/pull/169723#issuecomment-1793191007
+  // https://github.com/elastic/kibana/issues/170583
+  context('User is readonly', { tags: ['@ess', '@brokenInServerless'] }, () => {
     beforeEach(() => {
       login();
       visit(ALERTS_URL);
@@ -247,7 +250,6 @@ describe('Changing alert status', { tags: ['@ess', '@serverless'] }, () => {
       visit(ALERTS_URL, { role: ROLES.reader });
       waitForAlertsToPopulate();
     });
-
     it('should not allow users to change a single alert status', () => {
       // This is due to the reader role which makes everything in security 'read only'
       cy.get(TIMELINE_CONTEXT_MENU_BTN).should('not.exist');