-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Code] more history test cases #35216
[Code] more history test cases #35216
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. please wait until the CI passes.
await retry.try(async () => { | ||
expect(await testSubjects.exists('codeFileTreeNode-Directory-src')).to.be(true); | ||
}); | ||
await testGoToDefinition(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the refactoring.
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
jenkins test this |
💔 Build Failed |
@WangQianliang any updates on this? please change the base branch to |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💚 Build Succeeded |
#35819) * test(code/frontend): more history functional test cases (#35216) * fix(code/frontend): should not collapse folder after click a breadcrumb (#35458) * [Code] fix typeahead highlight file (#35585) * fix(code/frontend): should not collapse folder after click a breadcrumb * fix(code/frontend): should highlight file when jump from typeahead * separate to a independent test case * fix(code/frontend): show loading in directory view (#35720) * fix(code/frontend): fix highlight symbol (#35531) * feature(code/frontend): retry loading symbols if language server is initializing (#35715)
Summary
in source view page, switch tab can go back and forward
in search page, apply search filter can go back and forward
go to definition can go back and go to definition again, cover this issue