Consider removing library(cmprsk) from test files

[michaelquinn32]

Hi David!

I hope all is well.

There are two test files that have library calls to suggested packages:

• tests/testthat/test-checkInputs.R
• tests/testthat/test-hazrd_tmle.R

I was able to remove these calls from the file without tests failing. Are they spurious?

Otherwise, could you consider a non-failing condition for these? Something like require() or skip_if_not_installed().
https://testthat.r-lib.org/reference/skip.html

Thanks!

[michaelquinn32]

Wait, there is a case in tests/testthat/test-hazrd_tmle.R that relies on the cuminc function. It would be nice to wrap that in a skip_if_not_installed().