Merge branch 'de_on_week/ai_assistant_query_auto_update' of https://g…

…ithub.com/vitaliidm/kibana into de_on_week/ai_assistant_query_auto_update

config/serverless.yml

@@ -100,6 +100,8 @@ xpack.index_management.enableIndexActions: false
 xpack.index_management.enableLegacyTemplates: false
 # Disable index stats information from Index Management UI
 xpack.index_management.enableIndexStats: false
+# Enable size and doc count information via metering API from Index Management UI
+xpack.index_management.enableSizeAndDocCount: true
 # Disable data stream stats information from Index Management UI
 xpack.index_management.enableDataStreamStats: false
 # Only limited index settings can be edited

x-pack/plugins/cases/public/common/mock/test_providers.tsx

@@ -9,7 +9,7 @@
 
 import React, { useMemo } from 'react';
 import { MemoryRouter } from 'react-router-dom';
-import { render as reactRender } from '@testing-library/react';
+import { render as reactRender, waitFor } from '@testing-library/react';
 import type { RenderOptions, RenderResult } from '@testing-library/react';
 import type { ILicense } from '@kbn/licensing-plugin/public';
 import type { ScopedFilesClient } from '@kbn/files-plugin/public';
@@ -124,6 +124,7 @@ export interface AppMockRenderer {
   queryClient: QueryClient;
   AppWrapper: React.FC<{ children: React.ReactNode }>;
   getFilesClient: () => ScopedFilesClient;
+  clearQueryCache: () => Promise<void>;
 }
 
 export const testQueryClient = new QueryClient({
@@ -202,6 +203,12 @@ export const createAppMockRenderer = ({
     });
   };
 
+  const clearQueryCache = async () => {
+    queryClient.getQueryCache().clear();
+
+    await waitFor(() => expect(queryClient.isFetching()).toBe(0));
+  };
+
   return {
     coreStart: services,
     queryClient,
@@ -210,5 +217,6 @@ export const createAppMockRenderer = ({
     externalReferenceAttachmentTypeRegistry,
     persistableStateAttachmentTypeRegistry,
     getFilesClient,
+    clearQueryCache,
   };
 };

x-pack/plugins/cases/public/components/all_cases/columns_popover.test.tsx

@@ -14,15 +14,18 @@ import type { AppMockRenderer } from '../../common/mock';
 import { createAppMockRenderer } from '../../common/mock';
 import { ColumnsPopover } from './columns_popover';
 
-// FLAKY: https://github.com/elastic/kibana/issues/174682
-describe.skip('ColumnsPopover', () => {
+describe('ColumnsPopover', () => {
   let appMockRenderer: AppMockRenderer;
 
   beforeEach(() => {
     appMockRenderer = createAppMockRenderer();
     jest.clearAllMocks();
   });
 
+  afterEach(async () => {
+    await appMockRenderer.clearQueryCache();
+  });
+
   const selectedColumns = [
     { field: 'title', name: 'Title', isChecked: true },
     { field: 'category', name: 'Category', isChecked: false },

x-pack/plugins/cases/public/components/all_cases/use_actions.test.tsx

@@ -28,15 +28,18 @@ import {
 jest.mock('../../containers/api');
 jest.mock('../../containers/user_profiles/api');
 
-// FLAKY: https://github.com/elastic/kibana/issues/188361
-describe.skip('useActions', () => {
+describe('useActions', () => {
   let appMockRender: AppMockRenderer;
 
   beforeEach(() => {
     appMockRender = createAppMockRenderer();
     jest.clearAllMocks();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('renders column actions', async () => {
     const { result } = renderHook(() => useActions({ disableActions: false }), {
       wrapper: appMockRender.AppWrapper,

x-pack/plugins/cases/public/components/case_form_fields/severity.test.tsx

@@ -6,7 +6,9 @@
  */
 
 import React from 'react';
-import { screen, waitFor, render } from '@testing-library/react';
+import { screen, waitFor } from '@testing-library/react';
+import type { AppMockRenderer } from '../../common/mock';
+import { createAppMockRenderer } from '../../common/mock';
 import { Severity } from './severity';
 import userEvent from '@testing-library/user-event';
 import { waitForEuiPopoverOpen } from '@elastic/eui/lib/test/rtl';
@@ -15,8 +17,18 @@ import { FormTestComponent } from '../../common/test_utils';
 const onSubmit = jest.fn();
 
 describe('Severity form field', () => {
+  let appMockRender: AppMockRenderer;
+
+  beforeEach(() => {
+    appMockRender = createAppMockRenderer();
+  });
+
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('renders', async () => {
-    render(
+    appMockRender.render(
       <FormTestComponent onSubmit={onSubmit}>
         <Severity isLoading={false} />
       </FormTestComponent>
@@ -28,7 +40,7 @@ describe('Severity form field', () => {
 
   // default to LOW in this test configuration
   it('defaults to the correct value', async () => {
-    render(
+    appMockRender.render(
       <FormTestComponent onSubmit={onSubmit}>
         <Severity isLoading={false} />
       </FormTestComponent>
@@ -39,7 +51,7 @@ describe('Severity form field', () => {
   });
 
   it('selects the correct value when changed', async () => {
-    render(
+    appMockRender.render(
       <FormTestComponent onSubmit={onSubmit}>
         <Severity isLoading={false} />
       </FormTestComponent>
@@ -61,7 +73,7 @@ describe('Severity form field', () => {
   });
 
   it('disables when loading data', async () => {
-    render(
+    appMockRender.render(
       <FormTestComponent onSubmit={onSubmit}>
         <Severity isLoading={true} />
       </FormTestComponent>

x-pack/plugins/cases/public/components/case_form_fields/sync_alerts_toggle.test.tsx

@@ -14,8 +14,7 @@ import { FormTestComponent } from '../../common/test_utils';
 import type { AppMockRenderer } from '../../common/mock';
 import { createAppMockRenderer } from '../../common/mock';
 
-// Failing: See https://github.com/elastic/kibana/issues/190270
-describe.skip('SyncAlertsToggle', () => {
+describe('SyncAlertsToggle', () => {
   let appMockRender: AppMockRenderer;
   const onSubmit = jest.fn();
   const defaultFormProps = {
@@ -31,6 +30,10 @@ describe.skip('SyncAlertsToggle', () => {
     appMockRender = createAppMockRenderer();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('it renders', async () => {
     appMockRender.render(
       <FormTestComponent>

x-pack/plugins/cases/public/components/case_view/components/case_view_files.test.tsx

@@ -26,8 +26,7 @@ const caseData: CaseUI = {
   comments: [...basicCase.comments, alertCommentWithIndices],
 };
 
-// FLAKY: https://github.com/elastic/kibana/issues/185046
-describe.skip('Case View Page files tab', () => {
+describe('Case View Page files tab', () => {
   let appMockRender: AppMockRenderer;
 
   useGetCaseFilesMock.mockReturnValue({
@@ -37,10 +36,11 @@ describe.skip('Case View Page files tab', () => {
 
   beforeEach(() => {
     appMockRender = createAppMockRenderer();
+    jest.clearAllMocks();
   });
 
-  afterEach(() => {
-    jest.clearAllMocks();
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
   });
 
   it('should render the utility bar for the files table', async () => {

x-pack/plugins/cases/public/components/case_view/components/custom_fields.test.tsx

@@ -16,16 +16,17 @@ import { customFieldsMock, customFieldsConfigurationMock } from '../../../contai
 import userEvent from '@testing-library/user-event';
 import { CustomFieldTypes } from '../../../../common/types/domain';
 
-describe.skip('Case View Page files tab', () => {
+describe('Case View Page files tab', () => {
   const onSubmit = jest.fn();
   let appMockRender: AppMockRenderer;
 
   beforeEach(() => {
     appMockRender = createAppMockRenderer();
+    jest.clearAllMocks();
   });
 
-  afterEach(() => {
-    jest.clearAllMocks();
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
   });
 
   it('should render the custom fields correctly', async () => {

x-pack/plugins/cases/public/components/connector_selector/form.test.tsx

@@ -19,8 +19,7 @@ jest.mock('../../common/lib/kibana');
 
 const useKibanaMock = useKibana as jest.Mocked<typeof useKibana>;
 
-// FLAKY: https://github.com/elastic/kibana/issues/189530
-describe.skip('ConnectorSelector', () => {
+describe('ConnectorSelector', () => {
   const handleChange = jest.fn();
   const defaultProps = {
     connectors: [],
@@ -45,6 +44,10 @@ describe.skip('ConnectorSelector', () => {
     });
   });
 
+  afterEach(async () => {
+    await appMock.clearQueryCache();
+  });
+
   it('should render', async () => {
     appMock.render(
       <FormTestComponent>

x-pack/plugins/cases/public/components/create/template.test.tsx

@@ -13,8 +13,7 @@ import { createAppMockRenderer } from '../../common/mock';
 import { templatesConfigurationMock } from '../../containers/mock';
 import { TemplateSelector } from './templates';
 
-// FLAKY: https://github.com/elastic/kibana/issues/178457
-describe.skip('CustomFields', () => {
+describe('CustomFields', () => {
   let appMockRender: AppMockRenderer;
   const onTemplateChange = jest.fn();
 
@@ -23,6 +22,10 @@ describe.skip('CustomFields', () => {
     appMockRender = createAppMockRenderer();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('renders correctly', async () => {
     appMockRender.render(
       <TemplateSelector

x-pack/plugins/cases/public/components/custom_fields/form_fields.test.tsx

@@ -15,8 +15,7 @@ import { FormTestComponent } from '../../common/test_utils';
 import { CustomFieldTypes } from '../../../common/types/domain';
 import { FormFields } from './form_fields';
 
-// FLAKY: https://github.com/elastic/kibana/issues/188450
-describe.skip('FormFields ', () => {
+describe('FormFields ', () => {
   let appMockRender: AppMockRenderer;
   const onSubmit = jest.fn();
 
@@ -25,6 +24,10 @@ describe.skip('FormFields ', () => {
     appMockRender = createAppMockRenderer();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('renders correctly', async () => {
     appMockRender.render(
       <FormTestComponent onSubmit={onSubmit}>

x-pack/plugins/cases/public/components/description/index.test.tsx

@@ -27,8 +27,7 @@ const defaultProps = {
   isLoadingDescription: false,
 };
 
-// Failing: See https://github.com/elastic/kibana/issues/185879
-describe.skip('Description', () => {
+describe('Description', () => {
   const onUpdateField = jest.fn();
   let appMockRender: AppMockRenderer;
 
@@ -37,6 +36,10 @@ describe.skip('Description', () => {
     appMockRender = createAppMockRenderer();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('renders description correctly', async () => {
     appMockRender.render(<Description {...defaultProps} onUpdateField={onUpdateField} />);
 

x-pack/plugins/cases/public/components/files/file_attachment_event.test.tsx

@@ -16,15 +16,18 @@ import { createAppMockRenderer } from '../../common/mock';
 import { basicFileMock } from '../../containers/mock';
 import { FileAttachmentEvent } from './file_attachment_event';
 
-// FLAKY: https://github.com/elastic/kibana/issues/174661
-describe.skip('FileAttachmentEvent', () => {
+describe('FileAttachmentEvent', () => {
   let appMockRender: AppMockRenderer;
 
   beforeEach(() => {
     jest.clearAllMocks();
     appMockRender = createAppMockRenderer();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('renders clickable name', async () => {
     appMockRender.render(
       <FileAttachmentEvent file={basicFileMock as unknown as DownloadableFile} />

x-pack/plugins/cases/public/components/files/file_preview.test.tsx

@@ -16,15 +16,18 @@ import { createAppMockRenderer, mockedTestProvidersOwner } from '../../common/mo
 import { basicFileMock } from '../../containers/mock';
 import { FilePreview } from './file_preview';
 
-// FLAKY: https://github.com/elastic/kibana/issues/182364
-describe.skip('FilePreview', () => {
+describe('FilePreview', () => {
   let appMockRender: AppMockRenderer;
 
   beforeEach(() => {
     jest.clearAllMocks();
     appMockRender = createAppMockRenderer();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('FilePreview rendered correctly', async () => {
     appMockRender.render(<FilePreview closePreview={jest.fn()} selectedFile={basicFileMock} />);
 

x-pack/plugins/cases/public/components/files/file_type.test.tsx

@@ -17,8 +17,7 @@ import { basicCase, basicFileMock } from '../../containers/mock';
 import { getFileType } from './file_type';
 import { FILE_ATTACHMENT_TYPE } from '../../../common/constants';
 
-// FLAKY: https://github.com/elastic/kibana/issues/175841
-describe.skip('getFileType', () => {
+describe('getFileType', () => {
   const fileType = getFileType();
 
   it('invalid props return blank FileAttachmentViewObject', () => {
@@ -45,6 +44,10 @@ describe.skip('getFileType', () => {
       jest.clearAllMocks();
     });
 
+    afterEach(async () => {
+      await appMockRender.clearQueryCache();
+    });
+
     it('event renders a clickable name if the file is an image', async () => {
       appMockRender = createAppMockRenderer();
 
@@ -142,7 +145,10 @@ describe.skip('getFileType', () => {
 
     it('empty externalReferenceMetadata returns blank FileAttachmentViewObject', () => {
       expect(
-        fileType.getAttachmentViewObject({ ...attachmentViewProps, externalReferenceMetadata: {} })
+        fileType.getAttachmentViewObject({
+          ...attachmentViewProps,
+          externalReferenceMetadata: {},
+        })
       ).toEqual({
         event: 'added an unknown file',
         hideDefaultActions: true,

x-pack/plugins/cases/public/components/filter_popover/index.test.tsx

@@ -15,8 +15,7 @@ import { createAppMockRenderer } from '../../common/mock';
 
 import { FilterPopover } from '.';
 
-// FLAKY: https://github.com/elastic/kibana/issues/176679
-describe.skip('FilterPopover ', () => {
+describe('FilterPopover ', () => {
   let appMockRender: AppMockRenderer;
   const onSelectedOptionsChanged = jest.fn();
   const tags: string[] = ['coke', 'pepsi'];
@@ -26,6 +25,10 @@ describe.skip('FilterPopover ', () => {
     jest.clearAllMocks();
   });
 
+  afterEach(async () => {
+    await appMockRender.clearQueryCache();
+  });
+
   it('renders button label correctly', async () => {
     appMockRender.render(
       <FilterPopover