library(mlr3pipelines) makes it required for tests, as opposed to the skip_if_not_installed() usage elsewhere

[MichaelChirico]

These two lines are contradictory:

mlr3tuning/tests/testthat/helper.R

Line 4 in 4b6c958

library(mlr3pipelines)

mlr3tuning/tests/testthat/test_Tuner.R

Line 72 in 4b6c958

skip_if_not_installed("mlr3pipelines")

Because helper.R is always sourced before running a test_ file.

IMO if {mlr3pipelines} is required for the tests, it is better to put the library() call in tests/testthat.R -- this is the first place a reader will look for test requirements.

If it's indeed optional, I would remove the library() call in helper.R.

[be-marc]

Thanks for raising the issue. Fixed in #471