[data views] Reenable MKI test - change data import tool to be server…

…less compatible (#181349) ## Summary Change data import / export method so test works in MKI environment. Closes #171479 and #180568 Verified by running functional test runner run locally against MKI environment.
elastic · May 9, 2024 · 1bcec40 · 1bcec40
1 parent 02a22fd
commit 1bcec40
Show file tree

Hide file tree

Showing 9 changed files with 11 additions and 26 deletions.
diff --git a/...lugins/data_view_management/public/components/index_pattern_table/index_pattern_table.tsx b/...lugins/data_view_management/public/components/index_pattern_table/index_pattern_table.tsx
@@ -293,7 +293,7 @@ export const IndexPatternTable = ({
       fill={true}
       iconType="plusInCircle"
       onClick={() => setShowCreateDialog(true)}
-      data-test-subj="createIndexPatternButton"
+      data-test-subj="createDataViewButton"
     >
       <FormattedMessage
         id="indexPatternManagement.dataViewTable.createBtn"

diff --git a/test/functional/page_objects/settings_page.ts b/test/functional/page_objects/settings_page.ts
@@ -621,18 +621,7 @@ export class SettingsPageObject extends FtrService {
 
   async clickAddNewIndexPatternButton() {
     await this.common.scrollKibanaBodyTop();
-
-    // if showing no data view prompt
-    const noDataView = await this.testSubjects.exists('createDataViewButton');
-    if (noDataView) {
-      await this.testSubjects.click('createDataViewButton');
-      return;
-    }
-
-    const tableView = await this.testSubjects.exists('createIndexPatternButton');
-    if (tableView) {
-      await this.testSubjects.click('createIndexPatternButton');
-    }
+    await this.testSubjects.click('createDataViewButton');
   }
 
   async selectRollupIndexPatternType() {

diff --git a/...less/api_integration/test_suites/common/data_views/fields_for_wildcard_route/conflicts.ts b/...less/api_integration/test_suites/common/data_views/fields_for_wildcard_route/conflicts.ts
@@ -17,7 +17,6 @@ export default function ({ getService }: FtrProviderContext) {
   const svlCommonApi = getService('svlCommonApi');
 
   describe('conflicts', function () {
-    this.tags('failsOnMKI');
     before(() =>
       esArchiver.load('test/api_integration/fixtures/es_archiver/index_patterns/conflicts')
     );
@@ -31,7 +30,7 @@ export default function ({ getService }: FtrProviderContext) {
         .set(ELASTIC_HTTP_VERSION_HEADER, INITIAL_REST_VERSION_INTERNAL)
         // TODO: API requests in Serverless require internal request headers
         .set(svlCommonApi.getInternalRequestHeader())
-        .query({ pattern: 'logs-*' })
+        .query({ pattern: 'logs-2017.01.*' })
         .expect(200)
         .then((resp) => {
           expect(resp.body).to.eql({

diff --git a/...t_serverless/functional/es_archives/kibana_sample_data_flights_index_pattern/data.json.gz b/...t_serverless/functional/es_archives/kibana_sample_data_flights_index_pattern/data.json.gz
diff --git a/...erverless/functional/test_suites/common/management/data_views/_data_view_create_delete.ts b/...erverless/functional/test_suites/common/management/data_views/_data_view_create_delete.ts
@@ -19,8 +19,6 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
   const PageObjects = getPageObjects(['settings', 'common', 'header']);
 
   describe('creating and deleting default data view', function describeIndexTests() {
-    // failsOnMKI, see https://github.com/elastic/kibana/issues/171479
-    this.tags(['failsOnMKI']);
     before(async function () {
       // TODO: emptyKibanaIndex fails in Serverless with
       // "index_not_found_exception: no such index [.kibana_ingest]",
@@ -31,7 +29,11 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
       await esArchiver.loadIfNeeded(
         'x-pack/test_serverless/functional/es_archives/kibana_sample_data_flights_index_pattern'
       );
+      await kibanaServer.importExport.load(
+        'test/functional/fixtures/kbn_archiver/kibana_sample_data_flights_index_pattern'
+      );
       await esArchiver.loadIfNeeded('test/functional/fixtures/es_archiver/logstash_functional');
+
       await kibanaServer.uiSettings.replace({});
       // TODO: Navigation to Data View Management is different in Serverless
       await PageObjects.common.navigateToApp('management');
@@ -41,10 +43,13 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
     after(async function () {
       // TODO: Loading this from `es_archives` in `test_serverless`
       // instead since minor modifications were required
+
       await esArchiver.unload(
         'x-pack/test_serverless/functional/es_archives/kibana_sample_data_flights_index_pattern'
       );
-
+      await kibanaServer.importExport.unload(
+        'test/functional/fixtures/kbn_archiver/kibana_sample_data_flights_index_pattern'
+      );
       await esArchiver.unload('test/functional/fixtures/es_archiver/logstash_functional');
     });
 

diff --git a/x-pack/test_serverless/functional/test_suites/common/management/data_views/_edit_field.ts b/x-pack/test_serverless/functional/test_suites/common/management/data_views/_edit_field.ts
@@ -15,8 +15,6 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
   const testSubjects = getService('testSubjects');
 
   describe('edit field', function () {
-    // failsOnMKI, see https://github.com/elastic/kibana/issues/180568
-    this.tags(['failsOnMKI']);
     before(async function () {
       await kibanaServer.importExport.load('test/functional/fixtures/kbn_archiver/discover');
     });

diff --git a/...t_serverless/functional/test_suites/common/management/data_views/_index_pattern_filter.ts b/...t_serverless/functional/test_suites/common/management/data_views/_index_pattern_filter.ts
@@ -16,8 +16,6 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
   const es = getService('es');
 
   describe('index pattern filter', function describeIndexTests() {
-    // https://github.com/elastic/kibana/issues/178733
-    this.tags('failsOnMKI');
     before(async function () {
       await kibanaServer.savedObjects.cleanStandardList();
       await kibanaServer.uiSettings.replace({});

diff --git a/...ck/test_serverless/functional/test_suites/common/management/data_views/_runtime_fields.ts b/...ck/test_serverless/functional/test_suites/common/management/data_views/_runtime_fields.ts
@@ -17,8 +17,6 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
   const testSubjects = getService('testSubjects');
 
   describe('runtime fields', function () {
-    // Bug: https://github.com/elastic/kibana/issues/178939
-    this.tags('failsOnMKI');
     before(async function () {
       await browser.setWindowSize(1200, 800);
       await kibanaServer.importExport.load('test/functional/fixtures/kbn_archiver/discover');

diff --git a/x-pack/test_serverless/functional/test_suites/search/default_dataview.ts b/x-pack/test_serverless/functional/test_suites/search/default_dataview.ts
@@ -14,8 +14,6 @@ export default function ({ getPageObject, getService }: FtrProviderContext) {
   const svlCommonPage = getPageObject('svlCommonPage');
 
   describe('default dataView', function () {
-    // Error: expected testSubject(kbnOverviewElasticsearchGettingStarted) to exist
-    this.tags(['failsOnMKI']);
     before(async () => {
       await svlCommonPage.login();
       await svlSearchNavigation.navigateToLandingPage();