test: skips vdiffr tests on CRAN. This is mainly so that we can run `…

…devtools::check(env_vars = c(NOT_CRAN = "false"))` to be able to check everything else, but _not_ run the vdiffr tests, because they fail in `devtools::check()`. We tried debugging some of these failures but, per this comment from Hadley r-lib/devtools#2258 (comment) , it has is very difficult and is a clear drawback of vdiffr. That said, the tests pass with devtools::test() and serve their purpose of ensuring future development doesn't break existing functionality. For this reason, we are moving forward with this approach for now.
metrumresearchgroup · Feb 9, 2022 · 223e8f6 · 223e8f6
1 parent 957aff1
commit 223e8f6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/tests/testthat/test-base-plot.R b/tests/testthat/test-base-plot.R
@@ -1,3 +1,4 @@
+skip_on_cran() # vdiffr flakiness
 
 dataDir <- system.file("test-data", package = "pmforest")
 specDir <- file.path(dataDir, "spec")

diff --git a/tests/testthat/test-nsim-plot.R b/tests/testthat/test-nsim-plot.R
@@ -1,3 +1,4 @@
+skip_on_cran() # vdiffr flakiness
 
 dataDir <- system.file("test-data", package = "pmforest")
 specDir <- file.path(dataDir, "spec")