Skip to content

Commit

Permalink
More conditional skips
Browse files Browse the repository at this point in the history
  • Loading branch information
wlandau committed Nov 6, 2023
1 parent 1edb2e9 commit 3e42728
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
5 changes: 5 additions & 0 deletions tests/testthat/test-6-plans.R
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ test_with_dir("warn about <- and -> in drake_plan()", {

test_with_dir("File functions handle input", {
skip_on_cran() # CRAN gets essential tests only (check time limits).
skip_if_not_installed("styler")
expect_equal(
file_in(1, "x", "y"), c("1", "x", "y")
)
Expand Down Expand Up @@ -470,6 +471,7 @@ test_with_dir("plan_to_code()", {

test_with_dir("plan_to_notebook()", {
skip_on_cran()
skip_if_not_installed("styler")
skip_if_not_installed("knitr")
skip_if_not_installed("tibble")
expect_false(file.exists("report.md"))
Expand Down Expand Up @@ -556,6 +558,7 @@ test_with_dir("drake_plan does tidy eval", {
# From Alex Axthelm: https://github.com/ropensci/drake/issues/200
test_with_dir("drake_plan tidy eval can be customized and disabled", {
skip_on_cran() # CRAN gets essential tests only (check time limits).
skip_if_not_installed("styler")
my_variable <- 5
plan1 <- drake_plan(
a = !!my_variable,
Expand Down Expand Up @@ -608,6 +611,7 @@ test_with_dir("stringsAsFactors can be TRUE", {

test_with_dir("case sensitivity", {
skip_on_cran()
skip_if_not_installed("styler")
plan <- drake_plan(
a = 1,
b = 2,
Expand Down Expand Up @@ -635,6 +639,7 @@ test_with_dir("Strings stay strings, not symbols", {
})

test_with_dir("missing symbols get replaced (#1299)", {
skip_if_not_installed("styler")
plan <- drake_plan(x = NULL)
expect_silent(make(plan, verbose = 0L, session_info = FALSE))
})
Expand Down
1 change: 1 addition & 0 deletions tests/testthat/test-7-deprecate.R
Original file line number Diff line number Diff line change
Expand Up @@ -1598,6 +1598,7 @@ test_with_dir("move to caching = \"main\" at the top level", {
})

test_with_dir("move to caching = \"main\" at the target level", {
skip_if_not_installed("future")
plan <- drake_plan(x = target(1, caching = "master"))
expect_warning(
make(plan, caching = "worker", parallelism = "future"),
Expand Down

0 comments on commit 3e42728

Please sign in to comment.