Merge pull request #131 from MichaelChirico/sugg-alluvial

Optionally skip tests when Suggests dep 'alluvial' is not installed
corybrunson · Aug 10, 2024 · ecf582c · ecf582c
2 parents 5b42a51 + 58bbdc4
commit ecf582c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/tests/testthat/test-geom-alluvium.r b/tests/testthat/test-geom-alluvium.r
@@ -1,14 +1,16 @@
 # visual tests
 
 test_that("`geom_alluvium` draws correctly", {
+  skip_on_cran()
+  skip_if_not_installed("vdiffr")
+  skip_if_not_installed("alluvial")
+
   d1 <- as.data.frame(Titanic)
   a1 <- aes(y = Freq, axis1 = Class, axis2 = Sex, axis3 = Age, fill = Survived)
   a2 <- aes(y = Freq, axis1 = Class, axis2 = Sex)
   d2 <- alluvial::Refugees
   a3 <- aes(y = refugees, x = year, alluvium = country)
 
-  skip_on_cran()
-  skip_if_not_installed("vdiffr")
   vdiffr::expect_doppelganger(
     "`geom_alluvium`: basic",
     ggplot(d1, a1) + geom_alluvium()

diff --git a/tests/testthat/test-stat-alluvium.r b/tests/testthat/test-stat-alluvium.r
@@ -81,6 +81,7 @@ test_that("`stat_alluvium` orders alluvia correctly according to `aes.bind`", {
 # exceptional data
 
 test_that("`stat_flow` handles exceptional data with out errors", {
+  skip_if_not_installed("alluvial")
   data(Refugees, package = "alluvial")
   refugees_sub <- subset(Refugees, year %in% c(2003, 2005, 2010, 2013))
   gg <- ggplot(refugees_sub, aes(x = year, y = refugees, alluvium = country)) +