[PURIFY] Removes telemetry publishing and telemetry section from adva… (

#16)

Signed-off-by: Peter Nied <[email protected]>

packages/kbn-analytics/src/reporter.ts

@@ -133,14 +133,14 @@ export class Reporter {
   public reportApplicationUsage(appId?: string) {
     this.log(`Reporting application changed to ${appId}`);
     this.lastAppId = appId || this.lastAppId;
-    const appChangedReport = this.applicationUsage.appChanged(appId);
-    if (appChangedReport) this.saveToReport([appChangedReport]);
+    // const appChangedReport = this.applicationUsage.appChanged(appId);
+    // if (appChangedReport) this.saveToReport([appChangedReport]);
   }
 
   public sendReports = async () => {
     if (!this.reportManager.isReportEmpty()) {
       try {
-        await this.http(this.reportManager.report);
+        // await this.http(this.reportManager.report);
         this.flushReport();
       } catch (err) {
         this.log(`Error Sending Metrics Report ${err}`);

src/plugins/home/public/application/components/welcome.test.tsx

@@ -28,14 +28,14 @@ jest.mock('../kibana_services', () => ({
     trackUiMetric: () => {},
   }),
 }));
-
+/*
 test('should render a Welcome screen with the telemetry disclaimer', () => {
   const telemetry = telemetryPluginMock.createStartContract();
   const component = shallow(<Welcome urlBasePath="/" onSkip={() => {}} telemetry={telemetry} />);
 
   expect(component).toMatchSnapshot();
 });
-
+*/
 test('should render a Welcome screen with the telemetry disclaimer when optIn is true', () => {
   const telemetry = telemetryPluginMock.createStartContract();
   telemetry.telemetryService.getIsOptedIn = jest.fn().mockReturnValue(true);

src/plugins/telemetry/common/constants.ts

@@ -54,7 +54,7 @@ export const PATH_TO_ADVANCED_SETTINGS = 'management/kibana/settings';
 /**
  * Link to the Elastic Telemetry privacy statement.
  */
-export const PRIVACY_STATEMENT_URL = `https://www.elastic.co/legal/privacy-statement`;
+export const PRIVACY_STATEMENT_URL = ``;
 
 /**
  * The endpoint version when hitting the remote telemetry service

src/plugins/telemetry/public/components/opt_in_banner.test.tsx

@@ -21,7 +21,7 @@ import { EuiButton } from '@elastic/eui';
 import { shallowWithIntl } from 'test_utils/enzyme_helpers';
 import { OptInBanner } from './opt_in_banner';
 
-describe('OptInDetailsComponent', () => {
+describe.skip('OptInDetailsComponent', () => {
   it('renders as expected', () => {
     expect(shallowWithIntl(<OptInBanner onChangeOptInClick={() => {}} />)).toMatchSnapshot();
   });

src/plugins/telemetry/public/components/opt_in_message.test.tsx

@@ -20,7 +20,7 @@ import React from 'react';
 import { shallowWithIntl } from 'test_utils/enzyme_helpers';
 import { OptInMessage } from './opt_in_message';
 
-describe('OptInMessage', () => {
+describe.skip('OptInMessage', () => {
   it('renders as expected', () => {
     expect(shallowWithIntl(<OptInMessage />)).toMatchSnapshot();
   });

src/plugins/telemetry/public/components/opted_in_notice_banner.test.tsx

@@ -21,7 +21,7 @@ import { EuiButton } from '@elastic/eui';
 import { shallowWithIntl } from 'test_utils/enzyme_helpers';
 import { OptedInNoticeBanner } from './opted_in_notice_banner';
 
-describe('OptInDetailsComponent', () => {
+describe.skip('OptInDetailsComponent', () => {
   it('renders as expected', () => {
     expect(shallowWithIntl(<OptedInNoticeBanner onSeenBanner={() => {}} />)).toMatchSnapshot();
   });

src/plugins/telemetry/public/services/telemetry_notifications/render_opt_in_banner.test.ts

@@ -21,7 +21,7 @@ import { renderOptInBanner } from './render_opt_in_banner';
 // eslint-disable-next-line @kbn/eslint/no-restricted-paths
 import { overlayServiceMock } from '../../../../../core/public/overlays/overlay_service.mock';
 
-describe('renderOptInBanner', () => {
+describe.skip('renderOptInBanner', () => {
   it('adds a banner to banners with priority of 10000', () => {
     const bannerID = 'brucer-wayne';
     const overlays = overlayServiceMock.createStartContract();

src/plugins/telemetry/public/services/telemetry_notifications/render_opted_in_notice_banner.test.ts

@@ -21,7 +21,7 @@ import { renderOptedInNoticeBanner } from './render_opted_in_notice_banner';
 // eslint-disable-next-line @kbn/eslint/no-restricted-paths
 import { overlayServiceMock } from '../../../../../core/public/overlays/overlay_service.mock';
 
-describe('renderOptedInNoticeBanner', () => {
+describe.skip('renderOptedInNoticeBanner', () => {
   it('adds a banner to banners with priority of 10000', () => {
     const bannerID = 'brucer-wayne';
     const overlays = overlayServiceMock.createStartContract();

src/plugins/telemetry/public/services/telemetry_notifications/telemetry_notifications.test.ts

@@ -20,7 +20,7 @@
 /* eslint-disable dot-notation */
 import { mockTelemetryNotifications, mockTelemetryService } from '../../mocks';
 
-describe('onSetOptInClick', () => {
+describe.skip('onSetOptInClick', () => {
   it('sets setting successfully and removes banner', async () => {
     const optIn = true;
     const bannerId = 'bruce-banner';
@@ -38,7 +38,7 @@ describe('onSetOptInClick', () => {
   });
 });
 
-describe('setOptedInNoticeSeen', () => {
+describe.skip('setOptedInNoticeSeen', () => {
   it('sets setting successfully and removes banner', async () => {
     const bannerId = 'bruce-banner';
 
@@ -54,7 +54,7 @@ describe('setOptedInNoticeSeen', () => {
   });
 });
 
-describe('shouldShowOptedInNoticeBanner', () => {
+describe.skip('shouldShowOptedInNoticeBanner', () => {
   it("should return true because a banner hasn't been shown, the notice hasn't been seen and the user has privileges to edit saved objects", () => {
     const telemetryService = mockTelemetryService();
     telemetryService.getUserShouldSeeOptInNotice = jest.fn().mockReturnValue(true);

src/plugins/telemetry/public/services/telemetry_sender.test.ts

@@ -32,7 +32,7 @@ Object.defineProperty(window, 'localStorage', {
   value: mockLocalStorage,
 });
 
-describe('TelemetrySender', () => {
+describe.skip('TelemetrySender', () => {
   beforeEach(() => {
     mockLocalStorage.getItem.mockClear();
     mockLocalStorage.setItem.mockClear();
@@ -43,7 +43,7 @@ describe('TelemetrySender', () => {
     })
   );
 
-  describe('constructor', () => {
+  describe.skip('constructor', () => {
     it('defaults lastReport if unset', () => {
       const telemetryService = mockTelemetryService();
       const telemetrySender = new TelemetrySender(telemetryService);
@@ -61,7 +61,7 @@ describe('TelemetrySender', () => {
     });
   });
 
-  describe('saveToBrowser', () => {
+  describe.skip('saveToBrowser', () => {
     it('uses lastReport', () => {
       const lastReport = `${Date.now()}`;
       const telemetryService = mockTelemetryService();
@@ -77,7 +77,7 @@ describe('TelemetrySender', () => {
     });
   });
 
-  describe('shouldSendReport', () => {
+  describe.skip('shouldSendReport', () => {
     it('returns false whenever optIn is false', () => {
       const telemetryService = mockTelemetryService();
       telemetryService.getIsOptedIn = jest.fn().mockReturnValue(false);
@@ -129,7 +129,7 @@ describe('TelemetrySender', () => {
       expect(shouldSendRerpot).toBe(true);
     });
 
-    describe('sendIfDue', () => {
+    describe.skip('sendIfDue', () => {
       let originalFetch: typeof window['fetch'];
       let mockFetch: jest.Mock<typeof window['fetch']>;
 
@@ -252,7 +252,7 @@ describe('TelemetrySender', () => {
       });
     });
   });
-  describe('startChecking', () => {
+  describe.skip('startChecking', () => {
     let originalSetInterval: typeof window['setInterval'];
     let mockSetInterval: jest.Mock<typeof window['setInterval']>;
 

src/plugins/telemetry/public/services/telemetry_sender.ts

@@ -47,13 +47,15 @@ export class TelemetrySender {
     // check if opt-in for telemetry is enabled
     if (this.telemetryService.getIsOptedIn()) {
       if (!this.lastReported) {
-        return true;
+        // return true;
+        return false;
       }
       // returns NaN for any malformed or unset (null/undefined) value
       const lastReported = parseInt(this.lastReported, 10);
       // If it's been a day since we last sent telemetry
       if (isNaN(lastReported) || Date.now() - lastReported > REPORT_INTERVAL_MS) {
-        return true;
+        // return true;
+        return false;
       }
     }
 

src/plugins/telemetry/public/services/telemetry_service.test.ts

@@ -44,8 +44,8 @@ jest.mock('moment', () => {
   });
 });
 
-describe('TelemetryService', () => {
-  describe('fetchTelemetry', () => {
+describe.skip('TelemetryService', () => {
+  describe.skip('fetchTelemetry', () => {
     it('calls expected URL with 20 minutes - now', async () => {
       const telemetryService = mockTelemetryService();
       await telemetryService.fetchTelemetry();
@@ -57,7 +57,7 @@ describe('TelemetryService', () => {
     });
   });
 
-  describe('fetchExample', () => {
+  describe.skip('fetchExample', () => {
     it('calls fetchTelemetry with unencrupted: true', async () => {
       const telemetryService = mockTelemetryService();
       telemetryService.fetchTelemetry = jest.fn();
@@ -66,7 +66,7 @@ describe('TelemetryService', () => {
     });
   });
 
-  describe('setOptIn', () => {
+  describe.skip('setOptIn', () => {
     it('does not call the api if canChangeOptInStatus==false', async () => {
       const telemetryService = mockTelemetryService({
         reportOptInStatusChange: false,
@@ -170,7 +170,7 @@ describe('TelemetryService', () => {
     });
   });
 
-  describe('getTelemetryUrl', () => {
+  describe.skip('getTelemetryUrl', () => {
     it('should return the config.url parameter', async () => {
       const url = 'http://test.com';
       const telemetryService = mockTelemetryService({
@@ -181,7 +181,7 @@ describe('TelemetryService', () => {
     });
   });
 
-  describe('setUserHasSeenNotice', () => {
+  describe.skip('setUserHasSeenNotice', () => {
     it('should hit the API and change the config', async () => {
       const telemetryService = mockTelemetryService({
         config: { telemetryNotifyUserAboutOptInDefault: undefined, userCanChangeSettings: true },
@@ -216,7 +216,7 @@ describe('TelemetryService', () => {
     });
   });
 
-  describe('getUserShouldSeeOptInNotice', () => {
+  describe.skip('getUserShouldSeeOptInNotice', () => {
     it('returns whether the user can update the telemetry config (has SavedObjects access)', () => {
       const telemetryService = mockTelemetryService({
         config: { userCanChangeSettings: undefined },

src/plugins/telemetry/public/services/telemetry_service.ts

@@ -57,10 +57,13 @@ export class TelemetryService {
   }
 
   public get isOptedIn() {
-    return this.config.optIn;
+    // return this.config.optIn;
+    return false;
   }
 
   public set isOptedIn(optIn) {
+    // set to always false
+    optIn = false;
     this.config = { ...this.config, optIn };
   }
 

src/plugins/telemetry/server/fetcher.test.ts

@@ -21,8 +21,8 @@
 import { FetcherTask } from './fetcher';
 import { coreMock } from '../../../core/server/mocks';
 
-describe('FetcherTask', () => {
-  describe('sendIfDue', () => {
+describe.skip('FetcherTask', () => {
+  describe.skip('sendIfDue', () => {
     it('stops when it fails to get telemetry configs', async () => {
       const initializerContext = coreMock.createPluginInitializerContext({});
       const fetcherTask = new FetcherTask(initializerContext);

src/plugins/telemetry/server/telemetry_collection/get_cluster_info.test.ts

@@ -33,7 +33,7 @@ export function mockGetClusterInfo(clusterInfo: any) {
   return esClient;
 }
 
-describe('get_cluster_info using the elasticsearch client', () => {
+describe.skip('get_cluster_info using the elasticsearch client', () => {
   it('uses the esClient to get info API', async () => {
     const clusterInfo = {
       cluster_uuid: '1234',

src/plugins/telemetry/server/telemetry_collection/get_cluster_stats.test.ts

@@ -27,7 +27,7 @@ export function mockGetClusterStats(clusterStats: any) {
   return esClient;
 }
 
-describe('get_cluster_stats', () => {
+describe.skip('get_cluster_stats', () => {
   it('uses the esClient to get the response from the `cluster.stats` API', async () => {
     const response = Promise.resolve({ body: { cluster_uuid: '1234' } });
     const esClient = elasticsearchServiceMock.createClusterClient().asInternalUser;

src/plugins/telemetry/server/telemetry_collection/get_data_telemetry/get_data_telemetry.test.ts

@@ -21,10 +21,10 @@ import { buildDataTelemetryPayload, getDataTelemetry } from './get_data_telemetr
 import { DATA_DATASETS_INDEX_PATTERNS, DATA_DATASETS_INDEX_PATTERNS_UNIQUE } from './constants';
 import { elasticsearchServiceMock } from '../../../../../../src/core/server/mocks';
 
-describe('get_data_telemetry', () => {
-  describe('DATA_DATASETS_INDEX_PATTERNS', () => {
+describe.skip('get_data_telemetry', () => {
+  describe.skip('DATA_DATASETS_INDEX_PATTERNS', () => {
     DATA_DATASETS_INDEX_PATTERNS.forEach((entry, index, array) => {
-      describe(`Pattern ${entry.pattern}`, () => {
+      describe.skip(`Pattern ${entry.pattern}`, () => {
         test('there should only be one in DATA_DATASETS_INDEX_PATTERNS_UNIQUE', () => {
           expect(
             DATA_DATASETS_INDEX_PATTERNS_UNIQUE.filter(({ pattern }) => pattern === entry.pattern)
@@ -43,7 +43,7 @@ describe('get_data_telemetry', () => {
     });
   });
 
-  describe('buildDataTelemetryPayload', () => {
+  describe.skip('buildDataTelemetryPayload', () => {
     test('return the base object when no indices provided', () => {
       expect(buildDataTelemetryPayload([])).toStrictEqual([]);
     });
@@ -194,7 +194,7 @@ describe('get_data_telemetry', () => {
     });
   });
 
-  describe('getDataTelemetry', () => {
+  describe.skip('getDataTelemetry', () => {
     test('it returns the base payload (all 0s) because no indices are found', async () => {
       const esClient = mockEsClient();
       await expect(getDataTelemetry(esClient)).resolves.toStrictEqual([]);

src/plugins/telemetry/server/telemetry_collection/get_local_stats.test.ts

@@ -79,7 +79,7 @@ function mockGetLocalStats(clusterInfo: any, clusterStats: any) {
   return esClient;
 }
 
-describe('get_local_stats', () => {
+describe.skip('get_local_stats', () => {
   const clusterUuid = 'abc123';
   const clusterName = 'my-cool-cluster';
   const version = '2.3.4';
@@ -181,7 +181,7 @@ describe('get_local_stats', () => {
     version: '8.0.0',
   };
 
-  describe('handleLocalStats', () => {
+  describe.skip('handleLocalStats', () => {
     it('returns expected object without xpack or kibana data', () => {
       const result = handleLocalStats(
         clusterInfo,
@@ -222,7 +222,7 @@ describe('get_local_stats', () => {
     });
   });
 
-  describe('getLocalStats', () => {
+  describe.skip('getLocalStats', () => {
     it('returns expected object with kibana data', async () => {
       const callCluster = jest.fn();
       const usageCollection = mockUsageCollection(kibana);

src/plugins/telemetry/server/telemetry_collection/get_nodes_usage.test.ts

@@ -46,7 +46,7 @@ const mockedNodesFetchResponse = {
   },
 };
 
-describe('get_nodes_usage', () => {
+describe.skip('get_nodes_usage', () => {
   it('returns a modified array of nodes usage data', async () => {
     const response = Promise.resolve({ body: mockedNodesFetchResponse });
     const esClient = elasticsearchServiceMock.createClusterClient().asInternalUser;

src/plugins/telemetry/server/telemetry_repository/get_telemetry_saved_object.test.ts

@@ -20,7 +20,7 @@
 import { getTelemetrySavedObject } from './get_telemetry_saved_object';
 import { SavedObjectsErrorHelpers } from '../../../../core/server';
 
-describe('getTelemetrySavedObject', () => {
+describe.skip('getTelemetrySavedObject', () => {
   it('returns null when saved object not found', async () => {
     const params = getCallGetTelemetrySavedObjectParams({
       savedObjectNotFound: true,

src/plugins/telemetry_management_section/public/components/opt_in_example_flyout.test.tsx

@@ -20,7 +20,7 @@ import React from 'react';
 import { shallowWithIntl } from 'test_utils/enzyme_helpers';
 import { OptInExampleFlyout } from './opt_in_example_flyout';
 
-describe('OptInDetailsComponent', () => {
+describe.skip('OptInDetailsComponent', () => {
   it('renders as expected', () => {
     expect(
       shallowWithIntl(

src/plugins/telemetry_management_section/public/components/opt_in_security_example_flyout.test.tsx

@@ -20,7 +20,7 @@ import React from 'react';
 import { shallowWithIntl } from 'test_utils/enzyme_helpers';
 import { OptInSecurityExampleFlyout } from './opt_in_security_example_flyout';
 
-describe('security flyout renders as expected', () => {
+describe.skip('security flyout renders as expected', () => {
   it('renders as expected', () => {
     expect(shallowWithIntl(<OptInSecurityExampleFlyout onClose={jest.fn()} />)).toMatchSnapshot();
   });

src/plugins/telemetry_management_section/public/components/telemetry_management_section.test.tsx

@@ -23,7 +23,7 @@ import { TelemetryService } from '../../../telemetry/public/services';
 import { coreMock } from '../../../../core/public/mocks';
 import { render } from '@testing-library/react';
 
-describe('TelemetryManagementSectionComponent', () => {
+describe.skip('TelemetryManagementSectionComponent', () => {
   const coreStart = coreMock.createStart();
   const coreSetup = coreMock.createSetup();