You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Yes, we definitely need some good coverage for our lower-level APIs here, at least. (I'm not sure what VS Code offers in the area of extension-based testing.)
We'll need to tweak some of our components where we invoke Bazel, Buildifier, etc. so that we can inject fakes that mock the results. We should probably wait on that until I make the changes that replace the XML-based processing with the proto output.
We currently have no tests for the extension. VS Code extension testing framework described here https://code.visualstudio.com/docs/extensions/testing-extensions
We will need to add a starter test and include it in the PR verification CI job
The text was updated successfully, but these errors were encountered: