Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Terminal smoke test failure #156554

Closed
Tracked by #159510
alexr00 opened this issue Jul 28, 2022 · 5 comments
Closed
Tracked by #159510

Terminal smoke test failure #156554

alexr00 opened this issue Jul 28, 2022 · 5 comments

Comments

@alexr00
Copy link
Member

alexr00 commented Jul 28, 2022

  1) VSCode Smoke Tests (Web)
       Terminal
         Terminal Persistence
           detach/attach
             should persist buffer content:
     Error: Timeout: get terminal buffer '#terminal .terminal-wrapper' after 20 seconds.
      at Code.poll (/__w/1/s/test/automation/src/code.ts:256:11)
      at Code.waitForTerminalBuffer (/__w/1/s/test/automation/src/code.ts:225:3)
      at Terminal.waitForTerminalText (/__w/1/s/test/automation/src/terminal.ts:270:4)
      at Context.<anonymous> (src/areas/terminal/terminal-persistence.test.ts:78:5)

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=179605&view=logs&j=e5bbec05-0f06-558d-ebe9-5973d52063d2&t=a671546b-2469-5fb1-b2c9-08e628dcad6d

@meganrogge
Copy link
Contributor

Also skipped the terminal profile tests bc they were failing and holding up PRs #156906

@Tyriar Tyriar added this to the August 2022 milestone Aug 3, 2022
@Tyriar Tyriar self-assigned this Aug 3, 2022
@Tyriar Tyriar modified the milestones: August 2022, September 2022 Aug 22, 2022
@meganrogge
Copy link
Contributor

meganrogge commented Sep 8, 2022

in this particular run, the quickpick is opened and nothing is typed

@meganrogge
Copy link
Contributor

so it appears that this step didn't work according to the trace viewer, since no input happens

// Type value if any
if (value) {
await this.quickInput.type(value);
}

not sure why it didn't report a failure for

async waitForSetValue(selector: string, value: string): Promise<void> {
await this.poll(() => this.driver.setValue(selector, value), () => true, `set value '${selector}'`);
}

@meganrogge
Copy link
Contributor

meganrogge commented Sep 9, 2022

never mind - ignore the above - the snapshots aren't available for most of the important part but going through the actions shows a terminal does get created

@meganrogge
Copy link
Contributor

I investigated this and based on the actions log, this should be passing

@github-actions github-actions bot locked and limited conversation to collaborators Oct 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants