-
-
Notifications
You must be signed in to change notification settings - Fork 85
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 tests #249
Comments
I can take this one. |
Sorry, didn't see this issue. I already have a PR that works for this. See the referenced PR. |
No problem, I will add a smoke end-to-end test instead. |
Yeah, this issue is about integration/e2e tests for the extension to catch regressions earlier. |
I added a "needs test" label. Anyone who wants to help, can look through all issues with that label and write a test 😄 |
Currently, we only have unit tests for some of the functionality, but it would be nice to also have integration tests using Electron API to catch regressions: https://code.visualstudio.com/api/working-with-extensions/testing-extension
The text was updated successfully, but these errors were encountered: