Skip to content

Commit

Permalink
[ObsUX][LogsUX] Split infra directory tests ownership (elastic#171166)
Browse files Browse the repository at this point in the history
## πŸ““ Summary

Closes elastic#170938 

This work split the `test/functional/apps/infra` test folder accordingly
with the new ownership.

Co-authored-by: Marco Antonio Ghiani <[email protected]>
  • Loading branch information
tonyghiani and Marco Antonio Ghiani authored Nov 14, 2023
1 parent 4b1d6e4 commit 0592518
Show file tree
Hide file tree
Showing 8 changed files with 16 additions and 15 deletions.
3 changes: 2 additions & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -989,7 +989,8 @@ packages/kbn-monaco/src/esql @elastic/kibana-visualizations
/x-pack/plugins/infra/public/common @elastic/obs-ux-infra_services-team
/x-pack/plugins/infra/public/observability_logs @elastic/obs-ux-logs-team
/x-pack/plugins/infra/public/services @elastic/obs-ux-infra_services-team
/x-pack/test/functional/apps/infra @elastic/infra-monitoring-ui @elastic/obs-ux-infra_services-team @elastic/obs-ux-logs-team
/x-pack/test/functional/apps/infra @elastic/infra-monitoring-ui @elastic/obs-ux-infra_services-team
/x-pack/test/functional/apps/infra/logs @elastic/infra-monitoring-ui @elastic/obs-ux-logs-team
/x-pack/test/api_integration/apis/infra @elastic/infra-monitoring-ui @elastic/obs-ux-infra_services-team

# Observability UX management team
Expand Down
12 changes: 6 additions & 6 deletions x-pack/test/functional/apps/infra/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,12 @@ export default ({ loadTestFile }: FtrProviderContext) => {
});

describe('Logs UI', function () {
loadTestFile(require.resolve('./log_entry_categories_tab'));
loadTestFile(require.resolve('./log_entry_rate_tab'));
loadTestFile(require.resolve('./logs_source_configuration'));
loadTestFile(require.resolve('./log_stream_date_nano'));
loadTestFile(require.resolve('./link_to'));
loadTestFile(require.resolve('./log_stream'));
loadTestFile(require.resolve('./logs/log_entry_categories_tab'));
loadTestFile(require.resolve('./logs/log_entry_rate_tab'));
loadTestFile(require.resolve('./logs/logs_source_configuration'));
loadTestFile(require.resolve('./logs/log_stream_date_nano'));
loadTestFile(require.resolve('./logs/link_to'));
loadTestFile(require.resolve('./logs/log_stream'));
});
});
};
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

import expect from '@kbn/expect';
import { URL } from 'url';
import { FtrProviderContext } from '../../ftr_provider_context';
import { FtrProviderContext } from '../../../ftr_provider_context';

const ONE_HOUR = 60 * 60 * 1000;
const LOG_VIEW_ID = 'testView';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

import expect from '@kbn/expect';

import { FtrProviderContext } from '../../ftr_provider_context';
import { FtrProviderContext } from '../../../ftr_provider_context';

export default ({ getService }: FtrProviderContext) => {
const esArchiver = getService('esArchiver');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

import expect from '@kbn/expect';

import { FtrProviderContext } from '../../ftr_provider_context';
import { FtrProviderContext } from '../../../ftr_provider_context';

export default ({ getService }: FtrProviderContext) => {
const logsUi = getService('logsUi');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

import expect from '@kbn/expect';
import { URL } from 'url';
import { FtrProviderContext } from '../../ftr_provider_context';
import { FtrProviderContext } from '../../../ftr_provider_context';

const SERVICE_ID = '49a18510598271e924253ed2581d7ada';

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@

import expect from '@kbn/expect';
import { URL } from 'url';
import { FtrProviderContext } from '../../ftr_provider_context';
import { DATES } from './constants';
import { FtrProviderContext } from '../../../ftr_provider_context';
import { DATES } from '../constants';

export default ({ getPageObjects, getService }: FtrProviderContext) => {
const retry = getService('retry');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ import {
ELASTIC_HTTP_VERSION_HEADER,
X_ELASTIC_INTERNAL_ORIGIN_REQUEST,
} from '@kbn/core-http-common';
import { DATES } from './constants';
import { DATES } from '../constants';

import { FtrProviderContext } from '../../ftr_provider_context';
import { FtrProviderContext } from '../../../ftr_provider_context';

const COMMON_REQUEST_HEADERS = {
'kbn-xsrf': 'some-xsrf-token',
Expand Down

0 comments on commit 0592518

Please sign in to comment.