Remove unused variable

src/vs/platform/telemetry/common/telemetryService.ts

@@ -21,7 +21,6 @@ export interface ITelemetryServiceConfig {
 	appender: ITelemetryAppender;
 	commonProperties?: TPromise<{ [name: string]: any }>;
 	piiPaths?: string[];
-	userOptIn?: boolean;
 }
 
 export class TelemetryService implements ITelemetryService {
@@ -46,7 +45,7 @@ export class TelemetryService implements ITelemetryService {
 		this._appender = config.appender;
 		this._commonProperties = config.commonProperties || TPromise.as({});
 		this._piiPaths = config.piiPaths || [];
-		this._userOptIn = typeof config.userOptIn === 'undefined' ? true : config.userOptIn;
+		this._userOptIn = true;
 
 		// static cleanup pattern for: `file:///DANGEROUS/PATH/resources/app/Useful/Information`
 		this._cleanupPatterns = [/file:\/\/\/.*?\/resources\/app\//gi];

src/vs/platform/telemetry/test/electron-browser/telemetryService.test.ts

@@ -14,8 +14,6 @@ import * as Errors from 'vs/base/common/errors';
 import * as sinon from 'sinon';
 import { getConfigurationValue } from 'vs/platform/configuration/common/configuration';
 
-const optInStatusEventName: string = 'optInStatus';
-
 class TestTelemetryAppender implements ITelemetryAppender {
 
 	public events: any[];
@@ -719,29 +717,9 @@ suite('TelemetryService', () => {
 		}
 	}));
 
-	test('Telemetry Service respects user opt-in settings', sinon.test(function () {
-		let testAppender = new TestTelemetryAppender();
-		let service = new TelemetryService({ userOptIn: false, appender: testAppender }, undefined);
-
-		return service.publicLog('testEvent').then(() => {
-			assert.equal(testAppender.getEventsCount(), 0);
-			service.dispose();
-		});
-	}));
-
-	test('Telemetry Service does not sent optInStatus when user opted out', sinon.test(function () {
+	test('Telemetry Service sends events when enableTelemetry is on', sinon.test(function () {
 		let testAppender = new TestTelemetryAppender();
-		let service = new TelemetryService({ userOptIn: false, appender: testAppender }, undefined);
-
-		return service.publicLog(optInStatusEventName, { optIn: false }).then(() => {
-			assert.equal(testAppender.getEventsCount(), 0);
-			service.dispose();
-		});
-	}));
-
-	test('Telemetry Service sends events when enableTelemetry is on even user optin is on', sinon.test(function () {
-		let testAppender = new TestTelemetryAppender();
-		let service = new TelemetryService({ userOptIn: true, appender: testAppender }, undefined);
+		let service = new TelemetryService({ appender: testAppender }, undefined);
 
 		return service.publicLog('testEvent').then(() => {
 			assert.equal(testAppender.getEventsCount(), 1);