Skip to content

Commit

Permalink
Prefer skip_if_not() to skip_if(!)
Browse files Browse the repository at this point in the history
  • Loading branch information
MichaelChirico authored Nov 25, 2024
1 parent 38eb82d commit 46c6f52
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/testthat/test-c-api.R
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ test_that("client library passes tests", {
return("Disabled")

# Avoid installing into system library by default
skip_if(!nzchar(Sys.getenv("RLANG_FULL_TESTS")))
skip_if_not(nzchar(Sys.getenv("RLANG_FULL_TESTS")))

skip_on_cran()
skip_on_ci()
Expand Down Expand Up @@ -1109,7 +1109,7 @@ test_that("can push to dynamic list-of", {
})

test_that("internal error is thrown with OOB dyn-lof access", {
skip_if(!compiled_by_gcc())
skip_if_not(compiled_by_gcc())
lof <- new_dyn_list_of("integer", 3, 2)
expect_snapshot({
err(lof_arr_push_back(lof, 0, 42L), "Location 0 does not exist")
Expand Down

0 comments on commit 46c6f52

Please sign in to comment.