Merge pull request microsoft#209702 from microsoft/tyriar/terminal_test_2

Clean up terminal tests follow up

Author: Tyriar

src/vs/workbench/contrib/terminal/test/browser/terminalProfileService.integrationTest.ts

@@ -168,13 +168,10 @@ suite('TerminalProfileService', () => {
 		terminalInstanceService = new TestTerminalInstanceService();
 		extensionService = new TestTerminalExtensionService();
 		environmentService = { remoteAuthority: undefined } as IWorkbenchEnvironmentService;
-		// instantiationService = new TestInstantiationService();
 
 		const themeService = new TestThemeService();
 		const terminalContributionService = new TestTerminalContributionService();
-		// const contextKeyService = new MockContextKeyService();
 
-		// instantiationService.stub(IContextKeyService, contextKeyService);
 		instantiationService.stub(IExtensionService, extensionService);
 		instantiationService.stub(IConfigurationService, configurationService);
 		instantiationService.stub(IRemoteAgentService, remoteAgentService);

src/vs/workbench/contrib/terminal/test/browser/terminalService.test.ts

@@ -115,7 +115,6 @@ suite('Workbench - TerminalService', () => {
 			await terminalService.safeDisposeTerminal({
 				target: TerminalLocation.Panel,
 				hasChildProcesses: true,
-				// onExit: onExitEmitter.event,
 				dispose: () => fail()
 			} as Partial<ITerminalInstance> as any);
 			dialogService.setConfirmResult({ confirmed: true });
@@ -148,7 +147,6 @@ suite('Workbench - TerminalService', () => {
 			await terminalService.safeDisposeTerminal({
 				target: TerminalLocation.Panel,
 				hasChildProcesses: true,
-				// onExit: onExitEmitter.event,
 				dispose: () => fail()
 			} as Partial<ITerminalInstance> as any);
 			dialogService.setConfirmResult({ confirmed: true });