Clean up

Author: Tyriar

src/vs/platform/terminal/common/capabilities/commandDetection/promptInputModel.ts

@@ -52,20 +52,18 @@ export class PromptInputModel extends Disposable implements IPromptInputModel {
 		private readonly _xterm: Terminal,
 		onCommandStart: Event<ITerminalCommand>,
 		onCommandExecuted: Event<ITerminalCommand>,
-		private readonly _logService: ILogService
+		@ILogService private readonly _logService: ILogService
 	) {
 		super();
 
 		this._register(this._xterm.onData(e => this._handleInput(e)));
 		this._register(this._xterm.onCursorMove(() => this._sync()));
-		// TODO: Listen to the xterm textarea focus event?
 
 		this._register(onCommandStart(e => this._handleCommandStart(e as { marker: IMarker })));
 		this._register(onCommandExecuted(() => this._handleCommandExecuted()));
 	}
 
 	setContinuationPrompt(value: string): void {
-		console.log('setContinuationPrompt', value);
 		this._continuationPrompt = value;
 	}
 
@@ -77,7 +75,6 @@ export class PromptInputModel extends Disposable implements IPromptInputModel {
 		this._state = PromptInputState.Input;
 		this._commandStartMarker = command.marker;
 		this._commandStartX = this._xterm.buffer.active.cursorX;
-		console.log('commandStart', command.marker.line, this._commandStartX);
 		this._onDidStartInput.fire();
 	}
 
@@ -91,7 +88,6 @@ export class PromptInputModel extends Disposable implements IPromptInputModel {
 	}
 
 	private _handleInput(data: string) {
-		this._logService.trace(`PromptInputModel#_handleInput data=${data}`);
 		this._sync();
 	}
 

src/vs/platform/terminal/test/common/capabilities/commandDetection/promptInputModel.test.ts

@@ -4,13 +4,25 @@
  *--------------------------------------------------------------------------------------------*/
 
 import { ensureNoDisposablesAreLeakedInTestSuite } from 'vs/base/test/common/utils';
-import type { PromptInputModel } from 'vs/platform/terminal/common/capabilities/commandDetection/promptInputModel';
+import { NullLogService } from 'vs/platform/log/common/log';
+import { PromptInputModel } from 'vs/platform/terminal/common/capabilities/commandDetection/promptInputModel';
+import { Emitter } from 'vs/base/common/event';
+import type { ITerminalCommand } from 'vs/platform/terminal/common/capabilities/capabilities';
+
+// eslint-disable-next-line local/code-import-patterns, local/code-amd-node-module
+import { Terminal } from '@xterm/headless';
 
 suite('RequestStore', () => {
 	const store = ensureNoDisposablesAreLeakedInTestSuite();
 	let promptInputModel: PromptInputModel;
+	let xterm: Terminal;
+	let onCommandStart: Emitter<ITerminalCommand>;
+	let onCommandExecuted: Emitter<ITerminalCommand>;
 
 	setup(() => {
-
+		xterm = new Terminal();
+		onCommandStart = new Emitter();
+		onCommandExecuted = new Emitter();
+		promptInputModel = store.add(new PromptInputModel(xterm, onCommandStart.event, onCommandExecuted.event, new NullLogService));
 	});
 });