Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Test file doesn't open up in RStudio despite what documentation says #194

Closed
daattali opened this issue May 2, 2022 · 0 comments · Fixed by #199
Closed

Test file doesn't open up in RStudio despite what documentation says #194

daattali opened this issue May 2, 2022 · 0 comments · Fixed by #199
Labels
bug Something isn't working
Milestone
v0.2.0

Comments

@daattali
Copy link
Contributor

daattali commented May 2, 2022

I installed the latest {shinytest2} and RStudio IDE today. {shinytest2} v 0.1.0 and RStudio build 485.

Documentation says that when the recorder quites, "If you are running in the RStudio IDE, it will automatically open this file in the editor"

I just tested it out, and after clicking the "Save test and exit" button, RStudio did not open up the file.
@schloerke schloerke added this to the v0.2.0 milestone May 3, 2022
@schloerke schloerke added the bug Something isn't working label May 3, 2022
schloerke added a commit that referenced this issue May 3, 2022
@schloerke
Make sure test file opens in RStudio when done
315b76c 
Fixes #194
@schloerke schloerke mentioned this issue May 3, 2022
Merged
1 task
schloerke added a commit that referenced this issue May 3, 2022
@schloerke
Post release bugs found by Dean (#199)
c29b78e 
* Adjust logic for `write_union` to not require usethis

Fixes #193

* Make sure test file opens in RStudio when done

Fixes #194

* Disable dev website. Changes need to be made now

* Make sure images are always included

Fixes #195

* `$get_screenshot()` is displayed in the graphics device or plots tab

Fixes #196

* Safe guard vdiffr

* Mention installing `{diffviewer}`

Fixes #198

* Rename `app_screenshot()` -> `app_get_screenshot()`

* Update NEWS.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
v0.2.0
Development

Successfully merging a pull request may close this issue.

Post release bugs found by Dean rstudio/shinytest2
2 participants
@schloerke @daattali