-
Notifications
You must be signed in to change notification settings - Fork 220
Add test feature #2350
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 test feature #2350
Conversation
|
Thanks for working on this @davidanthoff! This is a much needed feature. It would be a huge improvement to my package-development workflow. |
|
@pfitzseb this is ready for review. I think what we should do is merge this relatively quickly but not announce it widely and then have the feature ship for a while "quietly" with just maybe the folks on the Slack channel knowing about it and trying it. We could also just say the whole thing is still in preview, but in the shipping extension. And then once things have settled, we can announce that it is out of preview. But in general, I think this primarily needs to be tried out by folks now. |
Now somewhat functional.This works quite well now, you can watch a demo at https://youtu.be/Okn_HKihWn8?t=1267 to get a sense how to use this.The general idea is that one can add
anywhere in the Julia code, and then the test runner UI provides a way to execute each of these test items individually inside a new module. https://github.com/julia-vscode/TestItemRunner.jl has some documentation on how to write tests that can be picked up by this feature in the VS Code extension and how to run such new tests via the Base test infrastructure.
https://github.com/julia-vscode/TestItems.jl can be used to write inline tests in say a package.
Please do give this branch a try and report back whether it works!
Lots of things still to do:
@testitemmacro