diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/remote_clusters_add.helpers.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/add/remote_clusters_add.helpers.js similarity index 100% rename from x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/remote_clusters_add.helpers.js rename to x-pack/plugins/remote_clusters/__jest__/client_integration/add/remote_clusters_add.helpers.js diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_add.test.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/add/remote_clusters_add.test.js similarity index 97% rename from x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_add.test.js rename to x-pack/plugins/remote_clusters/__jest__/client_integration/add/remote_clusters_add.test.js index 6e1b2f6266584..05a4a2e330325 100644 --- a/x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_add.test.js +++ b/x-pack/plugins/remote_clusters/__jest__/client_integration/add/remote_clusters_add.test.js @@ -4,10 +4,9 @@ * you may not use this file except in compliance with the Elastic License. */ -import { pageHelpers, nextTick, setupEnvironment } from './helpers'; -import { NON_ALPHA_NUMERIC_CHARS, ACCENTED_CHARS } from './helpers/constants'; - -const { setup } = pageHelpers.remoteClustersAdd; +import { nextTick, setupEnvironment } from '../helpers'; +import { NON_ALPHA_NUMERIC_CHARS, ACCENTED_CHARS } from './special_characters'; +import { setup } from './remote_clusters_add.helpers'; describe('Create Remote cluster', () => { describe('on component mount', () => { diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/constants.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/add/special_characters.js similarity index 83% rename from x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/constants.js rename to x-pack/plugins/remote_clusters/__jest__/client_integration/add/special_characters.js index bdc965b4037dd..d2ef598127572 100644 --- a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/constants.js +++ b/x-pack/plugins/remote_clusters/__jest__/client_integration/add/special_characters.js @@ -4,14 +4,6 @@ * you may not use this file except in compliance with the Elastic License. */ -export const REMOTE_CLUSTER_EDIT_NAME = 'new-york'; - -export const REMOTE_CLUSTER_EDIT = { - name: REMOTE_CLUSTER_EDIT_NAME, - seeds: ['localhost:9400'], - skipUnavailable: true, -}; - export const NON_ALPHA_NUMERIC_CHARS = [ '#', '@', diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/remote_clusters_edit.helpers.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/edit/remote_clusters_edit.helpers.js similarity index 85% rename from x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/remote_clusters_edit.helpers.js rename to x-pack/plugins/remote_clusters/__jest__/client_integration/edit/remote_clusters_edit.helpers.js index 294861405b9d5..b5402f3b017f0 100644 --- a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/remote_clusters_edit.helpers.js +++ b/x-pack/plugins/remote_clusters/__jest__/client_integration/edit/remote_clusters_edit.helpers.js @@ -11,7 +11,13 @@ import { RemoteClusterEdit } from '../../../public/application/sections/remote_c import { createRemoteClustersStore } from '../../../public/application/store'; import { registerRouter } from '../../../public/application/services/routing'; -import { REMOTE_CLUSTER_EDIT_NAME } from './constants'; +export const REMOTE_CLUSTER_EDIT_NAME = 'new-york'; + +export const REMOTE_CLUSTER_EDIT = { + name: REMOTE_CLUSTER_EDIT_NAME, + seeds: ['localhost:9400'], + skipUnavailable: true, +}; const testBedConfig = { store: createRemoteClustersStore, diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_edit.test.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/edit/remote_clusters_edit.test.js similarity index 87% rename from x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_edit.test.js rename to x-pack/plugins/remote_clusters/__jest__/client_integration/edit/remote_clusters_edit.test.js index a5905227f49b8..b0e0832cb0831 100644 --- a/x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_edit.test.js +++ b/x-pack/plugins/remote_clusters/__jest__/client_integration/edit/remote_clusters_edit.test.js @@ -6,12 +6,14 @@ import { act } from 'react-dom/test-utils'; -import { RemoteClusterForm } from '../../public/application/sections/components/remote_cluster_form'; -import { pageHelpers, setupEnvironment } from './helpers'; -import { REMOTE_CLUSTER_EDIT, REMOTE_CLUSTER_EDIT_NAME } from './helpers/constants'; - -const { setup } = pageHelpers.remoteClustersEdit; -const { setup: setupRemoteClustersAdd } = pageHelpers.remoteClustersAdd; +import { RemoteClusterForm } from '../../../public/application/sections/components/remote_cluster_form'; +import { setupEnvironment } from '../helpers'; +import { setup as setupRemoteClustersAdd } from '../add/remote_clusters_add.helpers'; +import { + setup, + REMOTE_CLUSTER_EDIT, + REMOTE_CLUSTER_EDIT_NAME, +} from './remote_clusters_edit.helpers'; describe('Edit Remote cluster', () => { let server; diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/http_requests.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/http_requests.js index 182700b0c8fbd..e174a8faa3c57 100644 --- a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/http_requests.js +++ b/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/http_requests.js @@ -8,29 +8,22 @@ import sinon from 'sinon'; // Register helpers to mock HTTP Requests const registerHttpRequestMockHelpers = (server) => { - const mockResponse = (defaultResponse, response) => [ + const mockResponse = (response) => [ 200, { 'Content-Type': 'application/json' }, - JSON.stringify({ ...defaultResponse, ...response }), + JSON.stringify(response), ]; const setLoadRemoteClustersResponse = (response) => { - const defaultResponse = []; - server.respondWith('GET', '/api/remote_clusters', [ 200, { 'Content-Type': 'application/json' }, - JSON.stringify(response ? response : defaultResponse), + JSON.stringify(response), ]); }; const setDeleteRemoteClusterResponse = (response) => { - const defaultResponse = { - itemsDeleted: [], - errors: [], - }; - - server.respondWith('DELETE', /api\/remote_clusters/, mockResponse(defaultResponse, response)); + server.respondWith('DELETE', /api\/remote_clusters/, mockResponse(response)); }; return { diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/index.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/index.js index d70ba2a21e176..6ef0e10b6ae15 100644 --- a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/index.js +++ b/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/index.js @@ -4,16 +4,5 @@ * you may not use this file except in compliance with the Elastic License. */ -import { setup as remoteClustersAddSetup } from './remote_clusters_add.helpers'; -import { setup as remoteClustersEditSetup } from './remote_clusters_edit.helpers'; -import { setup as remoteClustersListSetup } from './remote_clusters_list.helpers'; - export { nextTick, getRandomString, findTestSubject } from '../../../../../test_utils'; - export { setupEnvironment } from './setup_environment'; - -export const pageHelpers = { - remoteClustersAdd: { setup: remoteClustersAddSetup }, - remoteClustersEdit: { setup: remoteClustersEditSetup }, - remoteClustersList: { setup: remoteClustersListSetup }, -}; diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/remote_clusters_list.helpers.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/list/remote_clusters_list.helpers.js similarity index 100% rename from x-pack/plugins/remote_clusters/__jest__/client_integration/helpers/remote_clusters_list.helpers.js rename to x-pack/plugins/remote_clusters/__jest__/client_integration/list/remote_clusters_list.helpers.js diff --git a/x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_list.test.js b/x-pack/plugins/remote_clusters/__jest__/client_integration/list/remote_clusters_list.test.js similarity index 96% rename from x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_list.test.js rename to x-pack/plugins/remote_clusters/__jest__/client_integration/list/remote_clusters_list.test.js index f19d92fc63f3a..3c393cd70009a 100644 --- a/x-pack/plugins/remote_clusters/__jest__/client_integration/remote_clusters_list.test.js +++ b/x-pack/plugins/remote_clusters/__jest__/client_integration/list/remote_clusters_list.test.js @@ -5,20 +5,14 @@ */ import { act } from 'react-dom/test-utils'; -import { - pageHelpers, - setupEnvironment, - nextTick, - getRandomString, - findTestSubject, -} from './helpers'; +import { getRouter } from '../../../public/application/services'; +import { getRemoteClusterMock } from '../../../fixtures/remote_cluster'; -import { getRouter } from '../../public/application/services'; -import { getRemoteClusterMock } from '../../fixtures/remote_cluster'; +import { PROXY_MODE } from '../../../common/constants'; -import { PROXY_MODE } from '../../common/constants'; +import { setupEnvironment, nextTick, getRandomString, findTestSubject } from '../helpers'; -const { setup } = pageHelpers.remoteClustersList; +import { setup } from './remote_clusters_list.helpers'; // FLAKY: https://github.com/elastic/kibana/issues/58681 describe.skip('', () => { @@ -34,8 +28,7 @@ describe.skip('', () => { }); beforeEach(() => { - // Set "default" mock responses by not providing any arguments - httpRequestsMockHelpers.setLoadRemoteClustersResponse(); + httpRequestsMockHelpers.setLoadRemoteClustersResponse([]); }); describe('on component mount', () => {