bdr cran fixes

DESCRIPTION

@@ -47,7 +47,6 @@ Depends:
 Imports:
   ggplot2,
   magrittr,
-  maps,
   mvtnorm,
   purrr,
   rlang (>= 0.3.0),
@@ -62,10 +61,11 @@ BugReports: https://github.com/padrinoDB/Rpadrino/issues
 URL: https://github.com/padrinoDB/Rpadrino, https://padrinoDB.github.io/Rpadrino/
 RoxygenNote: 7.1.2
 Suggests: 
-    testthat,
-    knitr,
     covr,
-    roxygen2
+    knitr,
+    maps,
+    roxygen2,
+    testthat
 VignetteBuilder: knitr
 Config/testthat/parallel: true
 Config/testthat/edition: 3

cran-comments.md

@@ -1,4 +1,4 @@
-This is a resubmission of the first submission of Rpadrino. We have added "'"s around software names in the DESCRIPTION, updated the documentation to include \value tags for all exported functions, and ensured our tests now reset user graphical options in the testing suite. 
+This is a resubmission of the first submission of Rpadrino. We have updated our tests to run on Solaris in the absence of Pandoc, and a note about imported packages on Fedora.
 
 Additional information on this submission:
 

tests/testthat/test-ipmr_generics.R

@@ -152,7 +152,7 @@ test_that("convergence diagnostics", {
 
   y <- pdb_make_ipm(x)
 
-  old_par <- par()
+  old_par <- par(no.readonly = TRUE)
 
   par(mfrow = c(2,2))
 

tests/testthat/test-pdb_report.R

@@ -2,6 +2,8 @@ data(pdb)
 
 test_that("pdb_report generates correct output", {
 
+  skip_if_not(rmarkdown::pandoc_available(version = "1.12.3"))
+
   out_temp <- tempdir()
   dir.create(out_temp, FALSE)
 
@@ -32,6 +34,8 @@ test_that("pdb_report generates correct output", {
 
 test_that("full filenames work too", {
 
+  skip_if_not(rmarkdown::pandoc_available(version = "1.12.3"))
+
   out_temp <- tempdir()
   dir.create(out_temp, FALSE)