fix CRAN issues

Author: lorenzwalthert

NEWS.md

@@ -8,6 +8,7 @@ editor_options:
 
 -   `parsable-R` hook now gives more context on where and how parsing
     fails (#423).
+-   fixed R CMD check issues in devel
 
 # precommit v0.3.1
 

R/roxygen2.R

@@ -105,7 +105,13 @@ roxygen_assert_additional_dependencies <- function() {
       e
     }
   )
-  if (inherits(out, "packageNotFoundError") || ("message" %in% names(out) && grepl("Dependency package(\\(s\\))? .* not available", out$message))) {
+  if (
+    inherits(out, "packageNotFoundError") ||
+      ("message" %in% names(out) && (
+        grepl("Dependency package(\\(s\\))? .* not available", out$message) ||
+          grepl("Failed to load", out$message)
+      ))
+  ) {
     # case used in package but not installed
     rlang::abort(paste0(
       "The roxygenize hook requires all dependencies of your package to be listed in ",

tests/testthat/test-hook-roxygenize.R

@@ -83,7 +83,7 @@ test_that("asserting installed dependencies", {
   writeLines(generate_uninstalled_pkg_call(), "R/core.R")
   testthat::expect_error(
     roxygen_assert_additional_dependencies(),
-    "requires all dependencies of your package"
+    "there is no package called"
   )
 })
 
@@ -125,7 +125,7 @@ test_that("fails gratefully when not installed package is called (packageNotFoun
   # when there is a missing package
   expect_error(
     roxygenize_with_cache(list(getwd()), dirs = dirs_R.cache("roxygenize")),
-    "Please add the package as a dependency"
+    "there is no package called"
   )
 })
 
@@ -139,7 +139,7 @@ test_that("fails gratefully when not installed package is required according to
     suppressWarnings(
       roxygenize_with_cache(list(getwd()), dirs = dirs_R.cache("roxygenize"))
     ),
-    "Please add the package"
+    "The package .* is required"
   )
 })