[ska][x-pack] relocate api_integration tests [1] #218470
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,49 @@ | ||
| /* | ||
| * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
| * or more contributor license agreements. Licensed under the Elastic License | ||
| * 2.0; you may not use this file except in compliance with the Elastic License | ||
| * 2.0. | ||
| */ | ||
|
|
||
| import { ScoutTestRunConfigCategory } from '@kbn/scout-info'; | ||
| import { FtrConfigProviderContext } from '@kbn/test'; | ||
| import { services } from './services'; | ||
|
|
||
| export async function getApiIntegrationConfig({ readConfigFile }: FtrConfigProviderContext) { | ||
| const xPackFunctionalTestsConfig = await readConfigFile( | ||
| require.resolve('@kbn/test-suites-xpack/functional/config.base') | ||
| ); | ||
|
|
||
| return { | ||
| services, | ||
| testConfigCategory: ScoutTestRunConfigCategory.API_TEST, | ||
| servers: xPackFunctionalTestsConfig.get('servers'), | ||
| security: xPackFunctionalTestsConfig.get('security'), | ||
| junit: { | ||
| reportName: 'X-Pack API Integration Tests', | ||
| }, | ||
| kbnTestServer: { | ||
| ...xPackFunctionalTestsConfig.get('kbnTestServer'), | ||
| serverArgs: [ | ||
| ...xPackFunctionalTestsConfig.get('kbnTestServer.serverArgs'), | ||
| '--xpack.security.session.idleTimeout=3600000', // 1 hour | ||
| '--telemetry.optIn=true', | ||
| '--xpack.fleet.agents.pollingRequestTimeout=5000', // 5 seconds | ||
| '--xpack.ruleRegistry.write.enabled=true', | ||
| '--xpack.ruleRegistry.write.enabled=true', | ||
| '--xpack.ruleRegistry.write.cache.enabled=false', | ||
| '--monitoring_collection.opentelemetry.metrics.prometheus.enabled=true', | ||
| ], | ||
| }, | ||
| esTestCluster: { | ||
| ...xPackFunctionalTestsConfig.get('esTestCluster'), | ||
| serverArgs: [ | ||
| ...xPackFunctionalTestsConfig.get('esTestCluster.serverArgs'), | ||
| 'node.attr.name=apiIntegrationTestNode', | ||
| `path.repo=/tmp/repo,/tmp/repo_1,/tmp/repo_2,/tmp/cloud-snapshots/`, | ||
| ], | ||
| }, | ||
| }; | ||
| } | ||
|
|
||
| export default getApiIntegrationConfig; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| /* | ||
| * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
| * or more contributor license agreements. Licensed under the Elastic License | ||
| * 2.0; you may not use this file except in compliance with the Elastic License | ||
| * 2.0. | ||
| */ | ||
|
|
||
| import { GenericFtrProviderContext } from '@kbn/test'; | ||
|
|
||
| import { services } from './services'; | ||
|
|
||
| export type FtrProviderContext = GenericFtrProviderContext<typeof services, {}>; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,52 @@ | ||
| /* | ||
| * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
| * or more contributor license agreements. Licensed under the Elastic License | ||
| * 2.0; you may not use this file except in compliance with the Elastic License | ||
| * 2.0. | ||
| */ | ||
|
|
||
| import { FtrProviderContext } from '../ftr_provider_context'; | ||
|
|
||
| export function DataViewApiProvider({ getService }: FtrProviderContext) { | ||
|
Member
There was a problem hiding this comment. I see this is also defined under
Contributor
Author
There was a problem hiding this comment. "DA" services are unique by design, more restricted and generally may not work in non-DA tests:
Your point is valid, but I'm afraid we have to keep "DA" services separately even if it means file/name duplication. We discussed it with @pheyos and conclusion was that SKA test relocation focuses around x-pack tests split and we don't refactor/unify services to not slow process. When tests / services are split, we hopefully can refactor and remove duplicates. But with Scout development being the priority, I'm not sure how much of this work we plan in the near term |
||
| const supertest = getService('supertest'); | ||
|
|
||
| return { | ||
| async create({ id, name, title }: { id: string; name: string; title: string }) { | ||
| const { body } = await supertest | ||
| .post(`/api/content_management/rpc/create`) | ||
| .set('kbn-xsrf', 'foo') | ||
| .set('x-elastic-internal-origin', 'foo') | ||
| .send({ | ||
| contentTypeId: 'index-pattern', | ||
| data: { | ||
| fieldAttrs: '{}', | ||
| title, | ||
| timeFieldName: '@timestamp', | ||
| sourceFilters: '[]', | ||
| fields: '[]', | ||
| fieldFormatMap: '{}', | ||
| typeMeta: '{}', | ||
| runtimeFieldMap: '{}', | ||
| name, | ||
| }, | ||
| options: { id }, | ||
| version: 1, | ||
| }); | ||
| return body; | ||
| }, | ||
|
|
||
| async delete({ id }: { id: string }) { | ||
| const { body } = await supertest | ||
| .post(`/api/content_management/rpc/delete`) | ||
| .set('kbn-xsrf', 'foo') | ||
| .set('x-elastic-internal-origin', 'foo') | ||
| .send({ | ||
| contentTypeId: 'index-pattern', | ||
| id, | ||
| options: { force: true }, | ||
| version: 1, | ||
| }); | ||
| return body; | ||
| }, | ||
| }; | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| /* | ||
| * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
| * or more contributor license agreements. Licensed under the Elastic License | ||
| * 2.0; you may not use this file except in compliance with the Elastic License | ||
| * 2.0. | ||
| */ | ||
|
|
||
| import { format as formatUrl } from 'url'; | ||
| import supertest from 'supertest'; | ||
|
|
||
| import { FtrProviderContext } from '../ftr_provider_context'; | ||
|
|
||
| /** | ||
| * Supertest provider that doesn't include user credentials into base URL that is passed | ||
| * to the supertest. | ||
| */ | ||
| export function EsSupertestWithoutAuthProvider({ getService }: FtrProviderContext) { | ||
| const config = getService('config'); | ||
| const elasticsearchServerConfig = config.get('servers.elasticsearch'); | ||
|
|
||
| return supertest( | ||
| formatUrl({ | ||
| ...elasticsearchServerConfig, | ||
| auth: false, | ||
| }) | ||
| ); | ||
| } |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should every entry has an associated team? I guess this line can be deleted
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is automatically added by CI bot, I will explicitly list the owners for what's inside of it and has no owner yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found entries for:
so I guess it is expected to be added?