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

Add basic lsp integration test to check language support #1236

Closed
wants to merge 2 commits into from

Conversation

michael-weng
Copy link
Contributor

  • Roll relevant test under language folder for better organization.
  • Given how build all task for defaultPacakge will need to be executed as a set up for this test, remove @slow for other tests that's running this task as the cost will be atmortized in CI.
  • The test check lsp client is running and exercise Goto Definition Request and Find References Request.

Issue: #1235

- Roll relevant test under language folder for better organization.
- Given how build all task for defaultPacakge will need to be executed
as a set up for this test, remove @slow for other tests that's running
this task as the cost will be atmortized in CI.
- The test check lsp client is running and exercise Goto Definition
Request and Find References Request.

Issue: swiftlang#1235
award999 added a commit to award999/vscode-swift that referenced this pull request Dec 10, 2024
* Moving macros into common file with tests from swiftlang#1236
* Fix some flakyness and failures
@award999
Copy link
Contributor

Superseded by #1259

@award999 award999 closed this Dec 10, 2024
award999 added a commit to award999/vscode-swift that referenced this pull request Dec 12, 2024
* Moving macros into common file with tests from swiftlang#1236
* Fix some flakyness and failures
award999 added a commit that referenced this pull request Dec 12, 2024
* Some e2e/smoke tests for our integration with sourcekit-lsp

* Moving macros into common file with tests from #1236
* Fix some flakyness and failures

* Mark as slow

* Fix for new snippet file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants