follow @jennybc advice

[pachadotdev]

this is a new version of #1921

@jennybc I took some time to type with 1 hand and I used the logic you proposed here #1921 (comment)

[jennybc]

It appears that use_src() does exactly this already? Can you help me understand why use_cpp11() needs to do that again?

usethis/R/rcpp.R

Lines 74 to 75 in 93f4410

	roxygen_ns_append(glue("@useDynLib {project_name()}, .registration = TRUE")) &&
	roxygen_remind()

[jennybc]

In terms of skips and mocks, I would expect this to look very much like the same test for use_rcpp():

usethis/tests/testthat/test-rcpp.R

Lines 6 to 21 in a919437

	test_that("use_rcpp() creates files/dirs, edits DESCRIPTION and .gitignore", {
	create_local_package()
	use_roxygen_md()
	# pretend Rcpp is installed
	local_check_installed()
	use_rcpp("test")
	expect_match(desc::desc_get("LinkingTo"), "Rcpp")
	expect_match(desc::desc_get("Imports"), "Rcpp")
	expect_proj_dir("src")
	expect_proj_file("src", "test.cpp")
	ignores <- read_utf8(proj_path("src", ".gitignore"))
	expect_true(all(c("*.o", "*.so", "*.dll") %in% ignores))
	})

[pachadotdev]

I changed it to reflect this. The test "use_rcpp_armadillo() creates Makevars..." is something I can implement later for cpp11armadillo.

[jennybc]

Whew, we got there!