Remove legacyEs test service (#107567)

* migrate uptime tests to the new client. it didn't use legacy one anyway * remove legacyEs service
elastic · Aug 4, 2021 · 41b6a99
1 parent 2ad7d68
commit 41b6a99
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 61 deletions.
diff --git a/test/common/services/index.ts b/test/common/services/index.ts
@@ -7,7 +7,6 @@
  */
 
 import { DeploymentService } from './deployment';
-import { LegacyEsProvider } from './legacy_es';
 import { ElasticsearchProvider } from './elasticsearch';
 import { EsArchiverProvider } from './es_archiver';
 import { KibanaServerProvider } from './kibana_server';
@@ -19,7 +18,6 @@ import { SavedObjectInfoService } from './saved_object_info';
 
 export const services = {
   deployment: DeploymentService,
-  legacyEs: LegacyEsProvider,
   es: ElasticsearchProvider,
   esArchiver: EsArchiverProvider,
   kibanaServer: KibanaServerProvider,

diff --git a/test/common/services/legacy_es.ts b/test/common/services/legacy_es.ts
diff --git a/x-pack/test/api_integration/apis/uptime/rest/helper/make_ping.ts b/x-pack/test/api_integration/apis/uptime/rest/helper/make_ping.ts
@@ -7,13 +7,14 @@
 
 import uuid from 'uuid';
 import { merge } from 'lodash';
+import { KibanaClient } from '@elastic/elasticsearch/api/kibana';
 import { makeTls, TlsProps } from './make_tls';
 
 const DEFAULT_INDEX_NAME = 'heartbeat-8-generated-test';
 const DATA_STREAM_INDEX_NAME = 'synthetics-http-default';
 
 export const makePing = async (
-  es: any,
+  es: KibanaClient,
   monitorId: string,
   fields: { [key: string]: any },
   mogrify: (doc: any) => any,

diff --git a/x-pack/test/api_integration/apis/uptime/rest/telemetry_collectors_fleet.ts b/x-pack/test/api_integration/apis/uptime/rest/telemetry_collectors_fleet.ts
@@ -12,7 +12,6 @@ import { makeChecksWithStatus } from './helper/make_checks';
 
 export default function ({ getService }: FtrProviderContext) {
   const supertest = getService('supertest');
-  const es = getService('legacyEs');
   const client = getService('es');
 
   describe('telemetry collectors fleet', () => {
@@ -36,7 +35,7 @@ export default function ({ getService }: FtrProviderContext) {
       };
 
       await makeChecksWithStatus(
-        es,
+        client,
         'upMonitorId',
         1,
         1,
@@ -55,7 +54,7 @@ export default function ({ getService }: FtrProviderContext) {
       );
 
       await makeChecksWithStatus(
-        es,
+        client,
         'downMonitorId',
         1,
         1,
@@ -74,7 +73,7 @@ export default function ({ getService }: FtrProviderContext) {
       );
 
       await makeChecksWithStatus(
-        es,
+        client,
         'noGeoNameMonitor',
         1,
         1,
@@ -91,7 +90,7 @@ export default function ({ getService }: FtrProviderContext) {
         true
       );
       await makeChecksWithStatus(
-        es,
+        client,
         'downMonitorId',
         1,
         1,
@@ -110,7 +109,7 @@ export default function ({ getService }: FtrProviderContext) {
       );
 
       await makeChecksWithStatus(
-        es,
+        client,
         'mixMonitorId',
         1,
         1,
@@ -121,7 +120,7 @@ export default function ({ getService }: FtrProviderContext) {
         undefined,
         true
       );
-      await es.indices.refresh();
+      await client.indices.refresh();
     });
 
     after('unload heartbeat index', () => {
@@ -140,7 +139,7 @@ export default function ({ getService }: FtrProviderContext) {
     });
 
     beforeEach(async () => {
-      await es.indices.refresh();
+      await client.indices.refresh();
     });
 
     it('should receive expected results for fleet managed monitors after calling monitor logging', async () => {

diff --git a/x-pack/test/api_integration/services/index.ts b/x-pack/test/api_integration/services/index.ts