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

Test: Smoke Tests #91337

Closed
6 tasks done
Tyriar opened this issue Feb 25, 2020 · 7 comments
Closed
6 tasks done

Test: Smoke Tests #91337

Tyriar opened this issue Feb 25, 2020 · 7 comments

Comments

@Tyriar
Copy link
Member

Tyriar commented Feb 25, 2020

#91278

Complexity: 2

Run the smoke tests and report issues for any test failures https://github.com/Microsoft/vscode/blob/master/test/smoke/README.md#run. ⚠️ Desktop tests MUST run with --stable-build argument ⚠️

If the automated tests fail, create and issue for that and run the tests manually: https://github.com/microsoft/vscode/wiki/Smoke-Test

@joaomoreno
Copy link
Member

@chrmarti I switched my slot with yours since I do not have macOS.

@bpasero
Copy link
Member

bpasero commented Feb 25, 2020

@sbatten @mjbvz fyi those tests that target --stable will not function for this iteration because the smoke test was improved to detect opened editors more reliably. Unfortunately that required changes in the workbench itself which are not part of the 1.42 release.

So I suggest to not run with --stable for this time and do the following manual test:

  • open stable on a user data dir and build up some UI state
  • verify this UI state is restored opening that in insiders

Same for dirty files (untitled and normal files).

@bpasero
Copy link
Member

bpasero commented Feb 26, 2020

@Tyriar I am not sure these instructions make it clear that we must run with --build to test against insiders (both for Electron and Web!) and not out of sources. Not sure if you copied this from some template but then we should update that.

@joaomoreno
Copy link
Member

Yeah, Search tests fail on Linux Web Firefox as well... and that triggers everything else to fail, so I'll wait for Search to be fixed and run this again tomorrow. cc @JacksonKearl @roblourens

image

@Tyriar
Copy link
Member Author

Tyriar commented Feb 26, 2020

@joaomoreno you could comment out or use .skip on the search test to unblock yourself?

@joaomoreno
Copy link
Member

Commenting out search just makes others fail. Anything related to quick open seems not to work at all. More specifically: anything related to input boxes.

I know people have tested web on Firefox (@bpasero ?), so this is maybe Linux specific?

@joaomoreno
Copy link
Member

Created #91609

@joaomoreno joaomoreno removed their assignment Feb 26, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2020
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

8 participants