-
Notifications
You must be signed in to change notification settings - Fork 92
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
Integrate 'testthat' into the Tests feature for R packages #1365
Comments
The first groundwork for this has been laid by @juliasilge in #915. That creates the scaffold for hooking into VS Code's test UI. At the complete other extreme, I have standalone code that extracts the relevant test info (e.g. coordinates for The next phase is to create the plumbing that connects those two pieces. Re: 'Have a "Configure R Tests" button that helps new users set up testthat for the first time' ☝️ if one can use R code, that is |
I think it's fine to lean on R code; it'd be nicer to have our own UI but not essential. We'll need an RStudio-like mechanism for helping users get bootstrapped by installing the packages needed for this kind of functionality ("configuring R tests requires usethis and testthat, would you like to install those now?"). |
With #1445 we can now run tests in a little self-contained R session or alternatively in the console. |
With #915 merged in, we can now move on to a next step, such as parsing the test files. |
In build 1214 I see:
When I click to run a test, it looks like it is going but it never finishes (will wait for minutes) or returns success/failure info to the UI. |
Having experienced this myself now in build 1214, my leading hypothesis is that it's this, quoting @jmcphers:
It's clear (from logging) that the hang is at the test file parsing stage. |
Oh lordy, we're back in business and I think this is fixed now in release builds. A simple fix but rather hard to divine. |
Currently, if you have an R package and open the Testing view, it says that 'No tests have been found in the workspace', even if the workspace contains
testthat
tests.We should:
testthat
for the first timeThe text was updated successfully, but these errors were encountered: