From b6b16c6887fbff20d2e15c8e7ac47d8ec80de11c Mon Sep 17 00:00:00 2001 From: Alison Goryachev Date: Wed, 5 May 2021 13:05:33 -0400 Subject: [PATCH 1/3] unskip test to run via flaky test runner --- x-pack/test/api_integration/apis/index.ts | 50 +++++++++---------- .../api_integration/apis/management/index.js | 14 +++--- .../management/index_management/indices.js | 3 +- 3 files changed, 33 insertions(+), 34 deletions(-) diff --git a/x-pack/test/api_integration/apis/index.ts b/x-pack/test/api_integration/apis/index.ts index e0328444701777..8e2f28ee085850 100644 --- a/x-pack/test/api_integration/apis/index.ts +++ b/x-pack/test/api_integration/apis/index.ts @@ -11,31 +11,31 @@ export default function ({ loadTestFile }: FtrProviderContext) { describe('apis', function () { this.tags('ciGroup6'); - loadTestFile(require.resolve('./search')); - loadTestFile(require.resolve('./es')); - loadTestFile(require.resolve('./security')); - loadTestFile(require.resolve('./spaces')); - loadTestFile(require.resolve('./monitoring')); - loadTestFile(require.resolve('./xpack_legacy')); - loadTestFile(require.resolve('./features')); - loadTestFile(require.resolve('./telemetry')); - loadTestFile(require.resolve('./logstash')); - loadTestFile(require.resolve('./kibana')); - loadTestFile(require.resolve('./metrics_ui')); - loadTestFile(require.resolve('./beats')); - loadTestFile(require.resolve('./console')); + // loadTestFile(require.resolve('./search')); + // loadTestFile(require.resolve('./es')); + // loadTestFile(require.resolve('./security')); + // loadTestFile(require.resolve('./spaces')); + // loadTestFile(require.resolve('./monitoring')); + // loadTestFile(require.resolve('./xpack_legacy')); + // loadTestFile(require.resolve('./features')); + // loadTestFile(require.resolve('./telemetry')); + // loadTestFile(require.resolve('./logstash')); + // loadTestFile(require.resolve('./kibana')); + // loadTestFile(require.resolve('./metrics_ui')); + // loadTestFile(require.resolve('./beats')); + // loadTestFile(require.resolve('./console')); loadTestFile(require.resolve('./management')); - loadTestFile(require.resolve('./uptime')); - loadTestFile(require.resolve('./maps')); - loadTestFile(require.resolve('./security_solution')); - loadTestFile(require.resolve('./short_urls')); - loadTestFile(require.resolve('./lens')); - loadTestFile(require.resolve('./ml')); - loadTestFile(require.resolve('./transform')); - loadTestFile(require.resolve('./lists')); - loadTestFile(require.resolve('./upgrade_assistant')); - loadTestFile(require.resolve('./searchprofiler')); - loadTestFile(require.resolve('./painless_lab')); - loadTestFile(require.resolve('./file_upload')); + // loadTestFile(require.resolve('./uptime')); + // loadTestFile(require.resolve('./maps')); + // loadTestFile(require.resolve('./security_solution')); + // loadTestFile(require.resolve('./short_urls')); + // loadTestFile(require.resolve('./lens')); + // loadTestFile(require.resolve('./ml')); + // loadTestFile(require.resolve('./transform')); + // loadTestFile(require.resolve('./lists')); + // loadTestFile(require.resolve('./upgrade_assistant')); + // loadTestFile(require.resolve('./searchprofiler')); + // loadTestFile(require.resolve('./painless_lab')); + // loadTestFile(require.resolve('./file_upload')); }); } diff --git a/x-pack/test/api_integration/apis/management/index.js b/x-pack/test/api_integration/apis/management/index.js index 596c8086c95ba7..bbe718794a89fc 100644 --- a/x-pack/test/api_integration/apis/management/index.js +++ b/x-pack/test/api_integration/apis/management/index.js @@ -7,13 +7,13 @@ export default function ({ loadTestFile }) { describe('management', () => { - loadTestFile(require.resolve('./advanced_settings')); - loadTestFile(require.resolve('./cross_cluster_replication')); - loadTestFile(require.resolve('./remote_clusters')); - loadTestFile(require.resolve('./rollup')); + // loadTestFile(require.resolve('./advanced_settings')); + // loadTestFile(require.resolve('./cross_cluster_replication')); + // loadTestFile(require.resolve('./remote_clusters')); + // loadTestFile(require.resolve('./rollup')); loadTestFile(require.resolve('./index_management')); - loadTestFile(require.resolve('./index_lifecycle_management')); - loadTestFile(require.resolve('./ingest_pipelines')); - loadTestFile(require.resolve('./snapshot_restore')); + // loadTestFile(require.resolve('./index_lifecycle_management')); + // loadTestFile(require.resolve('./ingest_pipelines')); + // loadTestFile(require.resolve('./snapshot_restore')); }); } diff --git a/x-pack/test/api_integration/apis/management/index_management/indices.js b/x-pack/test/api_integration/apis/management/index_management/indices.js index 3653d9916466d0..cef1bdbba754b8 100644 --- a/x-pack/test/api_integration/apis/management/index_management/indices.js +++ b/x-pack/test/api_integration/apis/management/index_management/indices.js @@ -34,8 +34,7 @@ export default function ({ getService }) { clearCache, } = registerHelpers({ supertest }); - // Failing: See https://github.com/elastic/kibana/issues/64473 - describe.skip('indices', () => { + describe('indices', () => { after(() => Promise.all([cleanUpEsResources()])); describe('clear cache', () => { From 58a606da2e1f6ab8ace8a7d91fc3ad2ef184bc78 Mon Sep 17 00:00:00 2001 From: Alison Goryachev Date: Thu, 6 May 2021 11:37:04 -0400 Subject: [PATCH 2/3] uncomment test files and use describe.only --- x-pack/test/api_integration/apis/index.ts | 50 +++++++++---------- .../api_integration/apis/management/index.js | 14 +++--- .../management/index_management/indices.js | 2 +- 3 files changed, 33 insertions(+), 33 deletions(-) diff --git a/x-pack/test/api_integration/apis/index.ts b/x-pack/test/api_integration/apis/index.ts index 8e2f28ee085850..e0328444701777 100644 --- a/x-pack/test/api_integration/apis/index.ts +++ b/x-pack/test/api_integration/apis/index.ts @@ -11,31 +11,31 @@ export default function ({ loadTestFile }: FtrProviderContext) { describe('apis', function () { this.tags('ciGroup6'); - // loadTestFile(require.resolve('./search')); - // loadTestFile(require.resolve('./es')); - // loadTestFile(require.resolve('./security')); - // loadTestFile(require.resolve('./spaces')); - // loadTestFile(require.resolve('./monitoring')); - // loadTestFile(require.resolve('./xpack_legacy')); - // loadTestFile(require.resolve('./features')); - // loadTestFile(require.resolve('./telemetry')); - // loadTestFile(require.resolve('./logstash')); - // loadTestFile(require.resolve('./kibana')); - // loadTestFile(require.resolve('./metrics_ui')); - // loadTestFile(require.resolve('./beats')); - // loadTestFile(require.resolve('./console')); + loadTestFile(require.resolve('./search')); + loadTestFile(require.resolve('./es')); + loadTestFile(require.resolve('./security')); + loadTestFile(require.resolve('./spaces')); + loadTestFile(require.resolve('./monitoring')); + loadTestFile(require.resolve('./xpack_legacy')); + loadTestFile(require.resolve('./features')); + loadTestFile(require.resolve('./telemetry')); + loadTestFile(require.resolve('./logstash')); + loadTestFile(require.resolve('./kibana')); + loadTestFile(require.resolve('./metrics_ui')); + loadTestFile(require.resolve('./beats')); + loadTestFile(require.resolve('./console')); loadTestFile(require.resolve('./management')); - // loadTestFile(require.resolve('./uptime')); - // loadTestFile(require.resolve('./maps')); - // loadTestFile(require.resolve('./security_solution')); - // loadTestFile(require.resolve('./short_urls')); - // loadTestFile(require.resolve('./lens')); - // loadTestFile(require.resolve('./ml')); - // loadTestFile(require.resolve('./transform')); - // loadTestFile(require.resolve('./lists')); - // loadTestFile(require.resolve('./upgrade_assistant')); - // loadTestFile(require.resolve('./searchprofiler')); - // loadTestFile(require.resolve('./painless_lab')); - // loadTestFile(require.resolve('./file_upload')); + loadTestFile(require.resolve('./uptime')); + loadTestFile(require.resolve('./maps')); + loadTestFile(require.resolve('./security_solution')); + loadTestFile(require.resolve('./short_urls')); + loadTestFile(require.resolve('./lens')); + loadTestFile(require.resolve('./ml')); + loadTestFile(require.resolve('./transform')); + loadTestFile(require.resolve('./lists')); + loadTestFile(require.resolve('./upgrade_assistant')); + loadTestFile(require.resolve('./searchprofiler')); + loadTestFile(require.resolve('./painless_lab')); + loadTestFile(require.resolve('./file_upload')); }); } diff --git a/x-pack/test/api_integration/apis/management/index.js b/x-pack/test/api_integration/apis/management/index.js index bbe718794a89fc..596c8086c95ba7 100644 --- a/x-pack/test/api_integration/apis/management/index.js +++ b/x-pack/test/api_integration/apis/management/index.js @@ -7,13 +7,13 @@ export default function ({ loadTestFile }) { describe('management', () => { - // loadTestFile(require.resolve('./advanced_settings')); - // loadTestFile(require.resolve('./cross_cluster_replication')); - // loadTestFile(require.resolve('./remote_clusters')); - // loadTestFile(require.resolve('./rollup')); + loadTestFile(require.resolve('./advanced_settings')); + loadTestFile(require.resolve('./cross_cluster_replication')); + loadTestFile(require.resolve('./remote_clusters')); + loadTestFile(require.resolve('./rollup')); loadTestFile(require.resolve('./index_management')); - // loadTestFile(require.resolve('./index_lifecycle_management')); - // loadTestFile(require.resolve('./ingest_pipelines')); - // loadTestFile(require.resolve('./snapshot_restore')); + loadTestFile(require.resolve('./index_lifecycle_management')); + loadTestFile(require.resolve('./ingest_pipelines')); + loadTestFile(require.resolve('./snapshot_restore')); }); } diff --git a/x-pack/test/api_integration/apis/management/index_management/indices.js b/x-pack/test/api_integration/apis/management/index_management/indices.js index cef1bdbba754b8..2e08d44834c61a 100644 --- a/x-pack/test/api_integration/apis/management/index_management/indices.js +++ b/x-pack/test/api_integration/apis/management/index_management/indices.js @@ -34,7 +34,7 @@ export default function ({ getService }) { clearCache, } = registerHelpers({ supertest }); - describe('indices', () => { + describe.only('indices', () => { after(() => Promise.all([cleanUpEsResources()])); describe('clear cache', () => { From 81f91059f54f8026d397e9490e16dfcdfac6d6a3 Mon Sep 17 00:00:00 2001 From: Alison Goryachev Date: Thu, 6 May 2021 19:55:48 -0400 Subject: [PATCH 3/3] remove .only from indices tests --- .../api_integration/apis/management/index_management/indices.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/x-pack/test/api_integration/apis/management/index_management/indices.js b/x-pack/test/api_integration/apis/management/index_management/indices.js index 2e08d44834c61a..cef1bdbba754b8 100644 --- a/x-pack/test/api_integration/apis/management/index_management/indices.js +++ b/x-pack/test/api_integration/apis/management/index_management/indices.js @@ -34,7 +34,7 @@ export default function ({ getService }) { clearCache, } = registerHelpers({ supertest }); - describe.only('indices', () => { + describe('indices', () => { after(() => Promise.all([cleanUpEsResources()])); describe('clear cache', () => {