vscode icon indicating copy to clipboard operation
vscode copied to clipboard
verified publisher icon vitest-dev

Add tests

Open sheremet-va opened this issue 1 year ago • 5 comments

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

sheremet-va avatar Feb 19 '24 09:02 sheremet-va

I can take this one.

MilanKovacic avatar Feb 19 '24 23:02 MilanKovacic

Sorry, didn't see this issue. I already have a PR that works for this. See the referenced PR.

ffMathy avatar Feb 24 '24 08:02 ffMathy

No problem, I will add a smoke end-to-end test instead.

MilanKovacic avatar Feb 24 '24 11:02 MilanKovacic

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.

sheremet-va avatar Feb 26 '24 18:02 sheremet-va

I added a "needs test" label. Anyone who wants to help, can look through all issues with that label and write a test 😄

sheremet-va avatar Mar 15 '24 17:03 sheremet-va