From e208c8604ea532b3c031a1dbbff4e198f3cc2f70 Mon Sep 17 00:00:00 2001 From: Daniel Imms <2193314+Tyriar@users.noreply.github.com> Date: Mon, 29 Nov 2021 11:46:57 -0800 Subject: [PATCH] Remove timeout from automation terminal util I don't believe this was ever required, awaiting writeInTerminal should make sure the data event is triggered in xterm, the following enter should just trigger another data event. This doesn't guarantee it was accepted by the process yet, just that xterm.js emitted the event. Part of #137847 --- test/automation/src/terminal.ts | 2 -- 1 file changed, 2 deletions(-) diff --git a/test/automation/src/terminal.ts b/test/automation/src/terminal.ts index b87457f4ea1..947ff2fe924 100644 --- a/test/automation/src/terminal.ts +++ b/test/automation/src/terminal.ts @@ -80,8 +80,6 @@ export class Terminal { async runCommandInTerminal(commandText: string): Promise { await this.code.writeInTerminal(Selector.Xterm, commandText); - // hold your horses - await new Promise(c => setTimeout(c, 500)); await this.code.dispatchKeybinding('enter'); }