Skip to content

Use @vscode/test-cli to run tests #949

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

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

plemarquand
Copy link
Contributor

Migrates to the new @vscode/test-cli package to download the appropriate VS Code and run the tests. This streamlines the test setup code, and should be a more reliable way to run the tests as VS Code evolves.

Hopefully this also helps with the occasional SIGSEGV errors that happen in CI on startup.

@plemarquand plemarquand force-pushed the vscode-test-cli branch 2 times, most recently from 602bcd9 to 7732978 Compare July 3, 2024 16:51
@award999
Copy link
Contributor

award999 commented Jul 3, 2024

I have this setup with my WIP for unit testing, I'll push that and we can reconcile this patch with that

@plemarquand plemarquand marked this pull request as draft July 3, 2024 16:57
@plemarquand
Copy link
Contributor Author

@award999 ah, sorry I didn't realize you already had some code going for this. I wrote most of this a while ago and revived it after seeing @matthewbastien getting those SIGSEGV errors in CI hoping this would help.

@award999
Copy link
Contributor

award999 commented Jul 3, 2024

@award999 ah, sorry I didn't realize you already had some code going for this. I wrote most of this a while ago and revived it after seeing @matthewbastien getting those SIGSEGV errors in CI hoping this would help.

NP, that patch will take longer to get in so let's use this one for just the test runner portion

Migrates to the new @vscode/test-cli package to download the appropriate
VS Code and run the tests. This streamlines the test setup code, and
should be a more reliable way to run the tests as VS Code evolves.

Hopefully this also helps with the occasional SIGSEGV errors that happen
in CI on startup.
@plemarquand plemarquand marked this pull request as ready for review July 3, 2024 19:07
@plemarquand
Copy link
Contributor Author

plemarquand commented Jul 3, 2024

Thanks @award999 for sharing the code you’d written for this.

Copy link
Contributor

@adam-fowler adam-fowler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@plemarquand plemarquand merged commit a9aa6bc into swiftlang:main Jul 5, 2024
@plemarquand plemarquand deleted the vscode-test-cli branch July 5, 2024 12:21
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.

3 participants