Merge branch 'main' into pipelines

mlr-org · Nov 22, 2024 · d566c85 · d566c85
2 parents 055cafc + b21a0ad
commit d566c85
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/tests/testthat/test_AutoTuner.R b/tests/testthat/test_AutoTuner.R
@@ -132,8 +132,7 @@ test_that("AutoTuner works with graphlearner", {
 })
 
 test_that("Nested resampling works with graphlearner", {
-  skip_if_not_installed("mlr3pipelines")
-  skip_if(packageVersion("mlr3pipelines") < "0.5.3")
+  skip_if_not_installed("mlr3pipelines", "0.5.3")
 
   gl = MAKE_GL()
   task = tsk("iris")
@@ -273,8 +272,7 @@ test_that("search space from TuneToken works", {
 })
 
 test_that("AutoTuner get_base_learner method works", {
-  skip_if_not_installed("mlr3pipelines")
-  skip_if(packageVersion("mlr3pipelines") < "0.5.3")
+  skip_if_not_installed("mlr3pipelines", "0.5.3")
   `%>>%` = getFromNamespace("%>>%", asNamespace("mlr3pipelines"))
 
   # simple learner

diff --git a/tests/testthat/test_Tuner.R b/tests/testthat/test_Tuner.R
@@ -266,9 +266,7 @@ test_that("proper error when primary search space is empty", {
 })
 
 test_that("internal tuning: branching", {
-  skip_if_not_installed("mlr3pipelines")
-  skip_if(packageVersion("mlr3pipelines") < "0.5.3")
-
+  skip_if_not_installed("mlr3pipelines", "0.5.3")
   # this case is special, because not all internally tuned parameters are present in every iteration, only those that
   # are in the active branch are
   glrn = mlr3pipelines::ppl("branch", graphs = list(