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.

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

Get shinytest automated app tests working #177

Open
sebdalgarno opened this issue Aug 29, 2024 · 3 comments
Open

Get shinytest automated app tests working #177

sebdalgarno opened this issue Aug 29, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@sebdalgarno
Copy link

openjournals/joss-reviews#7150

I'm getting some test errors in test-run_ccvi_app.R

Can you check on your end?

@see24
Copy link
Member

see24 commented Aug 30, 2024

Yes I get an error on my end as well. I have had a lot of trouble with shinytest and have since given up on it. I would like to leave the file in case I am able to get it working again. For now I just have some manual tests that I run for the app (See #151) . All the non-shiny package functions have testthat tests though.

@sebdalgarno
Copy link
Author

would it be possible to remove (or comment out) the stop call in this test so that all tests pass?

test_that("save works especially for spatial and range change and all related inputs", {
  skip_on_cran()
  skip_on_ci()
  stop("add test")
})

@see24
Copy link
Member

see24 commented Sep 3, 2024

Ok I have skipped the tests locally as well. I will leave this issue open so that I remember to keep working on those automated tests when I have time.

@see24 see24 changed the title R-CMD check test error Get shinytest automated app tests working Sep 3, 2024
@see24 see24 added the bug Something isn't working label Jan 7, 2025
steffilazerte added a commit to steffilazerte/ccviR that referenced this issue Jan 31, 2025
- Add tests for Home, species, spatial
- Simplify modules
steffilazerte added a commit to steffilazerte/ccviR that referenced this issue Feb 4, 2025
steffilazerte added a commit to steffilazerte/ccviR that referenced this issue Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants