Upgrade testthat to 3rd edition

[nikolas-burkoff]

From https://github.com/insightsengineering/coredev-tasks/issues/154

To-Do:

• Add >= 3.0.0 to testthat version in DESC file
• Add to DESC file:

Config/testthat/edition: 3
Config/testthat/parallel: true

• Fix all errors and warnings in tests -> for example using the above tern has 37 failures and 1118 passes (with 150 warnings)
• Update testthat/testthat.R at the moment locally it runs ParallelReporter (using default_reporter() which is also parallel seems nicer to me) and on CI it uses JunitReporter which is not parallelizable - do we want it parallel there?

Repos:

• tern Upgrade testthat to use 3rd edition tern#656 (parallel remains)
• teal
• ...

[donyunardi]

Update: teal, teal.slice, and teal.transform already have testthat (>= 3.1.5)

[nikolas-burkoff]

@donyunardi note there is a difference between using a testthat version and a testthat edition https://testthat.r-lib.org/articles/third-edition.html

[donyunardi]

Thanks Nik.
I think we should try from the least risky teal package first and use that as learning experience before we moves to the bigger one.

[averissimo]

Example of steps needed for teal.data

• Add Config/testthat/edition: 3 to DESCRIPTION
• Migrate 15 instances of expect_is as the function is deprecated (in favor of checkmate::expect_r6 or testthat::expect_type)
  • Should be 0 instances once TealData is removed
• Deprecation warnings are detected and should be recognized in tests
  • Should be 0 instances once TealData is removed

Some of the immediate benefits of this transition (taken from duplicated issue)