Skip to content

Commit

Permalink
Revert "[FTR][CI] Use default distribution for all tests (elastic#94968
Browse files Browse the repository at this point in the history
…)"

This reverts commit 537be25.
  • Loading branch information
Tyler Smalley committed Jun 3, 2021
1 parent 2956993 commit 0280b31
Show file tree
Hide file tree
Showing 32 changed files with 672 additions and 2,841 deletions.
6 changes: 5 additions & 1 deletion .ci/Jenkinsfile_flaky
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,11 @@ def agentProcess(Map params = [:]) {
]) {
task {
if (config.needBuild) {
kibanaPipeline.buildKibana()
if (!config.isXpack) {
kibanaPipeline.buildOss()
} else {
kibanaPipeline.buildXpack()
}
}

for(def i = 0; i < config.agentExecutions; i++) {
Expand Down
2 changes: 1 addition & 1 deletion .ci/Jenkinsfile_security_cypress
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ kibanaPipeline(timeoutMinutes: 180) {
def job = 'xpack-securityCypress'

workers.ci(name: job, size: 'l', ramDisk: true) {
kibanaPipeline.bash('test/scripts/jenkins_build_kibana.sh', 'Build Distributable')
kibanaPipeline.bash('test/scripts/jenkins_xpack_build_kibana.sh', 'Build Default 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')()
Expand Down
6 changes: 5 additions & 1 deletion .ci/es-snapshots/Jenkinsfile_verify_es
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,12 @@ kibanaPipeline(timeoutMinutes: 210) {
])

task {
kibanaPipeline.buildKibana(16)
kibanaPipeline.buildOss(6)
tasks.ossCiGroups()
}

task {
kibanaPipeline.buildXpack(10, true)
tasks.xpackCiGroups()
tasks.xpackCiGroupDocker()
}
Expand Down
1 change: 0 additions & 1 deletion packages/kbn-es-archiver/src/lib/indices/kibana_index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,6 @@ export async function createDefaultSpace({
{
index,
id: 'space:default',
refresh: 'wait_for',
body: {
type: 'space',
updated_at: new Date().toISOString(),
Expand Down
2 changes: 0 additions & 2 deletions test/accessibility/apps/dashboard_panel.ts
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,6 @@ 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();
});
Expand Down
2 changes: 1 addition & 1 deletion test/api_integration/apis/home/sample_data.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ export default function ({ getService }: FtrProviderContext) {

expect(resp.body).to.eql({
elasticsearchIndicesCreated: { kibana_sample_data_flights: 13059 },
kibanaSavedObjectsLoaded: 23,
kibanaSavedObjectsLoaded: 20,
});
});

Expand Down
20 changes: 20 additions & 0 deletions test/api_integration/apis/search/search.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,26 @@ 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`)
Expand Down
1 change: 1 addition & 0 deletions test/api_integration/apis/telemetry/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@

export default function ({ loadTestFile }) {
describe('Telemetry', () => {
loadTestFile(require.resolve('./telemetry_local'));
loadTestFile(require.resolve('./opt_in'));
loadTestFile(require.resolve('./telemetry_optin_notice_seen'));
});
Expand Down
Loading

0 comments on commit 0280b31

Please sign in to comment.