Execute the addins in interactive mode (#2350)

Co-authored-by: Jim Hester <james.f.hester@gmail.com>

Author: jennybc

NEWS.md

@@ -1,5 +1,7 @@
 # devtools (development version)
 
+* RStudio addins now run in interactive mode, rather than background mode (@jennybc, #2350)
+
 * `install(upgrade)` now defaults to 'default' rather than 'ask'. This allows you to control the default asking behavior with the `R_REMOTES_UPGRADE` environment variable (#2345)
 
 * `build_readme()` now uses the `path` argument, as designed (#2344)

R/active.R

@@ -8,7 +8,10 @@ find_active_file <- function(arg = "file") {
 find_test_file <- function(path) {
   type <- test_file_type(path)
   if (any(is.na(type))) {
-    rlang::abort(c("Don't know how to find tests for: ", path[is.na(type)]))
+    rlang::abort(c(
+      "Don't know how to find tests associated with the active file:",
+      path[is.na(type)]
+    ))
   }
 
   is_test <- type == "test"

inst/rstudio/addins.dcf

@@ -1,19 +1,19 @@
 Name: Run a test file
 Description: Run the current test file, using `devtools::test_active_file()`.
 Binding: test_active_file
-Interactive: false
+Interactive: true
 
 Name: Report test coverage for a file
 Description: Calculate and report test coverage for the current test file, using `devtools::test_coverage_active_file()`.
 Binding: test_coverage_active_file
-Interactive: false
+Interactive: true
 
 Name: Report test coverage for a package
 Description: Calculate and report the test coverage for the current package, using `devtools::test_coverage()`.
 Binding: test_coverage
-Interactive: false
+Interactive: true
 
 Name: Document a package
 Description: A wrapper for `roxygen`'s `roxygen2::roxygenize()`
 Binding: document
-Interactive: false
+Interactive: true