[ftr] Use default distribution for all tests

Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
elastic · Apr 26, 2021 · f400b6e · f400b6e
1 parent e44e4c7
commit f400b6e
Show file tree

Hide file tree

Showing 34 changed files with 2,802 additions and 624 deletions.
diff --git a/.ci/Jenkinsfile_flaky b/.ci/Jenkinsfile_flaky
@@ -74,9 +74,7 @@ def agentProcess(Map params = [:]) {
         task {
           if (config.needBuild) {
             if (!config.isXpack) {
-              kibanaPipeline.buildOss()
-            } else {
-              kibanaPipeline.buildXpack()
+              kibanaPipeline.buildKibana()
             }
           }
 

diff --git a/.ci/Jenkinsfile_security_cypress b/.ci/Jenkinsfile_security_cypress
@@ -16,7 +16,7 @@ kibanaPipeline(timeoutMinutes: 180) {
         def job = 'xpack-securityCypress'
 
         workers.ci(name: job, size: 'l', ramDisk: true) {
-          kibanaPipeline.bash('test/scripts/jenkins_xpack_build_kibana.sh', 'Build Default Distributable')
+          kibanaPipeline.bash('test/scripts/jenkins_build_kibana.sh', 'Build Distributable')
           kibanaPipeline.functionalTestProcess(job, 'test/scripts/jenkins_security_solution_cypress_chrome.sh')()
           // Temporarily disabled to figure out test flake
           // kibanaPipeline.functionalTestProcess(job, 'test/scripts/jenkins_security_solution_cypress_firefox.sh')()

diff --git a/.ci/es-snapshots/Jenkinsfile_verify_es b/.ci/es-snapshots/Jenkinsfile_verify_es
@@ -37,13 +37,11 @@ kibanaPipeline(timeoutMinutes: 210) {
           ])
 
           task {
-            kibanaPipeline.buildOss(6)
-            tasks.ossCiGroups()
-          }
-
-          task {
-            kibanaPipeline.buildXpack(10)
-            tasks.xpackCiGroups()
+            kibanaPipeline.buildKibana(6)
+            tasks([
+              tasks.ossCiGroups()
+              tasks.xpackCiGroups()
+            ])
           }
         }
       }

diff --git a/packages/kbn-es-archiver/src/lib/indices/kibana_index.ts b/packages/kbn-es-archiver/src/lib/indices/kibana_index.ts
@@ -165,6 +165,7 @@ export async function createDefaultSpace({
     {
       index,
       id: 'space:default',
+      refresh: 'wait_for',
       body: {
         type: 'space',
         updated_at: new Date().toISOString(),

diff --git a/src/dev/ci_setup/setup_env.sh b/src/dev/ci_setup/setup_env.sh
@@ -14,7 +14,7 @@ cacheDir="$HOME/.kibana"
 RED='\033[0;31m'
 C_RESET='\033[0m' # Reset color
 
-export NODE_OPTIONS="$NODE_OPTIONS --max-old-space-size=4096"
+export NODE_OPTIONS="$NODE_OPTIONS --max-old-space-size=5120"
 
 ###
 ### Since the Jenkins logging output collector doesn't look like a TTY

diff --git a/test/accessibility/apps/dashboard_panel.ts b/test/accessibility/apps/dashboard_panel.ts
@@ -63,6 +63,8 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
     it('dashboard panel full screen', async () => {
       const header = await dashboardPanelActions.getPanelHeading('[Flights] Airline Carrier');
       await dashboardPanelActions.toggleContextMenu(header);
+      await dashboardPanelActions.clickContextMenuMoreItem();
+
       await testSubjects.click('embeddablePanelAction-togglePanel');
       await a11y.testAppSnapshot();
     });

diff --git a/test/api_integration/apis/home/sample_data.ts b/test/api_integration/apis/home/sample_data.ts
@@ -43,7 +43,7 @@ export default function ({ getService }: FtrProviderContext) {
 
         expect(resp.body).to.eql({
           elasticsearchIndicesCreated: { kibana_sample_data_flights: 13059 },
-          kibanaSavedObjectsLoaded: 20,
+          kibanaSavedObjectsLoaded: 23,
         });
       });
 

diff --git a/test/api_integration/apis/search/bsearch.ts b/test/api_integration/apis/search/bsearch.ts
@@ -169,7 +169,7 @@ export default function ({ getService }: FtrProviderContext) {
           expect(resp.status).to.be(200);
           parseBfetchResponse(resp).forEach((responseJson, i) => {
             expect(responseJson.id).to.be(i);
-            verifyErrorResponse(responseJson.error, 400, 'search_phase_execution_exception', true);
+            verifyErrorResponse(responseJson.error, 400, 'search_phase_execution_exception', false);
           });
         });
       });

diff --git a/test/api_integration/apis/search/search.ts b/test/api_integration/apis/search/search.ts
@@ -99,26 +99,6 @@ export default function ({ getService }: FtrProviderContext) {
         expect(resp.body.message).to.contain('banana not found');
       });
 
-      it('should return 400 when index type is provided in OSS', async () => {
-        const resp = await supertest
-          .post(`/internal/search/es`)
-          .send({
-            indexType: 'baad',
-            params: {
-              body: {
-                query: {
-                  match_all: {},
-                },
-              },
-            },
-          })
-          .expect(400);
-
-        verifyErrorResponse(resp.body, 400);
-
-        expect(resp.body.message).to.contain('Unsupported index pattern');
-      });
-
       it('should return 400 with illegal ES argument', async () => {
         const resp = await supertest
           .post(`/internal/search/es`)

diff --git a/test/api_integration/apis/search/verify_error.ts b/test/api_integration/apis/search/verify_error.ts
@@ -21,7 +21,5 @@ export const verifyErrorResponse = (
   if (shouldHaveAttrs) {
     expect(r).to.have.property('attributes');
     expect(r.attributes).to.have.property('root_cause');
-  } else {
-    expect(r).not.to.have.property('attributes');
   }
 };
diff --git a/test/api_integration/apis/telemetry/index.js b/test/api_integration/apis/telemetry/index.js
@@ -8,7 +8,6 @@
 
 export default function ({ loadTestFile }) {
   describe('Telemetry', () => {
-    loadTestFile(require.resolve('./telemetry_local'));
     loadTestFile(require.resolve('./opt_in'));
     loadTestFile(require.resolve('./telemetry_optin_notice_seen'));
   });