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

Remote Smoke: Goto page finds welcome page on remove #74875

Closed
aeschli opened this issue Jun 5, 2019 · 3 comments
Closed

Remote Smoke: Goto page finds welcome page on remove #74875

aeschli opened this issue Jun 5, 2019 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues verified Verification succeeded
Milestone

Comments

@aeschli
Copy link
Contributor

aeschli commented Jun 5, 2019

Running the remote smoketest against a build shows that search returns more results in a remote window than in a local window.

In a remote window, the results contains the welcome page, where as in a regular window they don't.

  • checkout distro
  • Run yarn smoketest --remote --build ~/vscode/1.35.0-insiders/VSCode-linux-x64/

I added a workaround so the smoke tests passes:

const numResultsExpected = this.app.remote ? 7 : 6;

@vscodebot vscodebot bot added the search Search widget and operation issues label Jun 5, 2019
aeschli added a commit that referenced this issue Jun 5, 2019
@roblourens
Copy link
Member

We handle this for text search but not file search. I can't repro in the product with *.* but I can by just searching 'welcome' with the welcome page open.

@roblourens roblourens added the bug Issue identified by VS Code Team member as probable bug label Jun 5, 2019
@roblourens roblourens added this to the June 2019 milestone Jun 5, 2019
@RMacfarlane RMacfarlane added the verified Verification succeeded label Jun 26, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Jul 20, 2019
@aeschli
Copy link
Contributor Author

aeschli commented Sep 4, 2019

The workaround is still there:

const numResultsExpected = this.app.remote ? 7 : 6;

Shouldn't it be gone a result of the fix?

@roblourens
Copy link
Member

Fixed, thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants