From 778d574b1aa4d45742cab04c0cc4d71141f0420a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Drago=C8=99=20Moldovan-Gr=C3=BCnfeld?= Date: Fri, 29 Jul 2022 19:44:12 +0100 Subject: [PATCH] ARROW-17166: [R] [CI] force_tests() cannot return TRUE (#13680) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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](https://github.com/apache/arrow/pull/13680#issuecomment-1191873953).~ Authored-by: Dragoș Moldovan-Grünfeld Signed-off-by: Jonathan Keane --- r/tests/testthat/helper-skip.R | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/r/tests/testthat/helper-skip.R b/r/tests/testthat/helper-skip.R index fd1ce1a76c325..7a6c2687ed895 100644 --- a/r/tests/testthat/helper-skip.R +++ b/r/tests/testthat/helper-skip.R @@ -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) {