Skip to content

Commit

Permalink
ARROW-17166: [R] [CI] force_tests() cannot return TRUE (#13680)
Browse files Browse the repository at this point in the history
OOMing no longer occurs with large memory tests, so this PR no longer removes those. During the investigation, I uncovered a bug with `force_tests()` - which became the aim of the PR.  
~It looks like setting the docker `TZ` to `"MART"` triggers a fail in the Ubuntu CI jobs.~
~It turns out I misinterpreted the CI output and this was an issue with OOM in the large memory tests. See [comment](#13680 (comment)

Authored-by: Dragoș Moldovan-Grünfeld <[email protected]>
Signed-off-by: Jonathan Keane <[email protected]>
  • Loading branch information
dragosmg authored Jul 29, 2022
1 parent 036fdf2 commit 778d574
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion r/tests/testthat/helper-skip.R
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ build_features <- c(
)

force_tests <- function() {
identical(tolower(Sys.getenv("ARROW_R_force_tests()")), "true")
identical(tolower(Sys.getenv("ARROW_R_FORCE_TESTS")), "true")
}

skip_if_not_available <- function(feature) {
Expand Down

0 comments on commit 778d574

Please sign in to comment.