Add option to skip_if_not_installed() to fail unless _R_CHECK_FORCE_SUGGESTS_=false?

[MichaelChirico]

As implemented by @gaborcsardi in a few places:

r-lib/gitcreds#48
r-lib/processx#372

That way developers can opt out (or in, depending on the default) of soft-passing tests with "skip" when the specified package is unavailable.

I don't have a good suggestion for how to name the argument, maybe strict?

Happy to file a PR.

[hadley]

I'd rather we figure out how to do this automatically. library() and friends throw a packageNotFoundError error with a $package component so what if we used that to auto-skip?

[MichaelChirico]

Let me check your thinking, the idea is to get test_check() (and friends?) to get a setup hook like:

suggested_packages <- drop(read.dcf("DESCRIPTION", "Suggests"))
loadNamespace <- function(package, ...) {
  tryCatch(
    base::loadNamespace(package, ...),
    packageNotFoundError = \(c) if (package %in% suggested_packages) skip(...) else stop(c)
  )
}

(noting that loadNamespace(), via find.package(), also throws that class error)

[hadley]

Yeah, something like that. There's already a tryCatch() in test_code() that we could maybe use?

[hadley]

I think this is a duplicate of #1598