diff --git a/src/vs/test/utils/servicesTestUtils.ts b/src/vs/test/utils/servicesTestUtils.ts index 688ff9a4f96b778eedb5b9b1851086096011acba..8ea86ac52476ecdb505d9762b461f3ba49e5a0b5 100644 --- a/src/vs/test/utils/servicesTestUtils.ts +++ b/src/vs/test/utils/servicesTestUtils.ts @@ -354,6 +354,7 @@ export class TestEditorGroupService implements IEditorGroupService { services.set(IWorkspaceContextService, new TestContextService()); const lifecycle = new TestLifecycleService(); services.set(ILifecycleService, lifecycle); + services.set(ITelemetryService, NullTelemetryService); let inst = new InstantiationService(services); diff --git a/src/vs/workbench/common/editor/editorStacksModel.ts b/src/vs/workbench/common/editor/editorStacksModel.ts index 91f6ff99986d27ad42481d88d188948eca0f9539..85f6722844915099c929f45d97992e527a99cc0d 100644 --- a/src/vs/workbench/common/editor/editorStacksModel.ts +++ b/src/vs/workbench/common/editor/editorStacksModel.ts @@ -1122,9 +1122,7 @@ export class EditorStacksModel implements IEditorStacksModel { } private handleOnEditorOpened(editor: EditorInput): void { - if (this.telemetryService) { - this.telemetryService.publicLog('editorOpened', editor.getTelemetryDescriptor()); - } + this.telemetryService.publicLog('editorOpened', editor.getTelemetryDescriptor()); } private handleOnEditorClosed(event: GroupEvent): void { @@ -1144,9 +1142,7 @@ export class EditorStacksModel implements IEditorStacksModel { } } - if (this.telemetryService) { - this.telemetryService.publicLog('editorClosed', editor.getTelemetryDescriptor()); - } + this.telemetryService.publicLog('editorClosed', editor.getTelemetryDescriptor()); } public isOpen(resource: URI): boolean; diff --git a/src/vs/workbench/test/browser/editorStacksModel.test.ts b/src/vs/workbench/test/browser/editorStacksModel.test.ts index a55d91e090ca009c54e7e2c7e2eb1c92be9e653d..7e5a616daeb0730f721997278fbe5cf9e0ea0fce 100644 --- a/src/vs/workbench/test/browser/editorStacksModel.test.ts +++ b/src/vs/workbench/test/browser/editorStacksModel.test.ts @@ -19,6 +19,7 @@ import { IWorkspaceContextService } from 'vs/platform/workspace/common/workspace import { Registry } from 'vs/platform/platform'; import { Position, Direction } from 'vs/platform/editor/common/editor'; import { IInstantiationService } from 'vs/platform/instantiation/common/instantiation'; +import { ITelemetryService, NullTelemetryService } from 'vs/platform/telemetry/common/telemetry'; import { DiffEditorInput } from 'vs/workbench/common/editor/diffEditorInput'; import 'vs/workbench/browser/parts/editor/baseEditor'; @@ -27,6 +28,7 @@ function create(): EditorStacksModel { inst.stub(IStorageService, new TestStorageService()); inst.stub(ILifecycleService, new TestLifecycleService()); inst.stub(IWorkspaceContextService, new TestContextService()); + inst.stub(ITelemetryService, NullTelemetryService); const config = new TestConfigurationService(); config.setUserConfiguration('workbench', { editor: { openPositioning: 'right' } }); @@ -641,6 +643,7 @@ suite('Editor Stacks Model', () => { inst.stub(IStorageService, new TestStorageService()); inst.stub(ILifecycleService, new TestLifecycleService()); inst.stub(IWorkspaceContextService, new TestContextService()); + inst.stub(ITelemetryService, NullTelemetryService); const config = new TestConfigurationService(); inst.stub(IConfigurationService, config); @@ -1175,6 +1178,8 @@ suite('Editor Stacks Model', () => { inst.stub(IWorkspaceContextService, new TestContextService()); const lifecycle = new TestLifecycleService(); inst.stub(ILifecycleService, lifecycle); + inst.stub(ITelemetryService, NullTelemetryService); + const config = new TestConfigurationService(); config.setUserConfiguration('workbench', { editor: { openPositioning: 'right' } }); inst.stub(IConfigurationService, config); @@ -1217,6 +1222,8 @@ suite('Editor Stacks Model', () => { inst.stub(IWorkspaceContextService, new TestContextService()); const lifecycle = new TestLifecycleService(); inst.stub(ILifecycleService, lifecycle); + inst.stub(ITelemetryService, NullTelemetryService); + const config = new TestConfigurationService(); config.setUserConfiguration('workbench', { editor: { openPositioning: 'right' } }); inst.stub(IConfigurationService, config); @@ -1298,6 +1305,8 @@ suite('Editor Stacks Model', () => { inst.stub(IWorkspaceContextService, new TestContextService()); const lifecycle = new TestLifecycleService(); inst.stub(ILifecycleService, lifecycle); + inst.stub(ITelemetryService, NullTelemetryService); + const config = new TestConfigurationService(); config.setUserConfiguration('workbench', { editor: { openPositioning: 'right' } }); inst.stub(IConfigurationService, config); @@ -1347,6 +1356,8 @@ suite('Editor Stacks Model', () => { inst.stub(IWorkspaceContextService, new TestContextService()); const lifecycle = new TestLifecycleService(); inst.stub(ILifecycleService, lifecycle); + inst.stub(ITelemetryService, NullTelemetryService); + const config = new TestConfigurationService(); config.setUserConfiguration('workbench', { editor: { openPositioning: 'right' } }); inst.stub(IConfigurationService, config); @@ -1387,6 +1398,8 @@ suite('Editor Stacks Model', () => { inst.stub(IStorageService, new TestStorageService()); const lifecycle = new TestLifecycleService(); inst.stub(ILifecycleService, lifecycle); + inst.stub(ITelemetryService, NullTelemetryService); + const config = new TestConfigurationService(); config.setUserConfiguration('workbench', { editor: { openPositioning: 'right' } }); inst.stub(IConfigurationService, config);