Enhance test coverage for use_ad_template()

[manciniedoardo]

Background Information

It would be great to get {admiral}'s test coverage to 100%. However, this should not come at the expense of development time/effort from {admiral} core developers. As such, I have made a set of issues tagged "Unit Tests" (such as this one for use_ad_template()) where users who wish to dip their feet into the {admiral} ecosystem can:

• Learn a bit about unit testing for R packages
• Make a meaningful contribution to {admiral}

by adding or modifying unit tests to account for lines not previously covered.

Important note: Given the above, this issue is for newcomers to {admiral} or users who wish to contribute to the package, and should not be taken on by core {admiral} developers.

Definition of Done

• Unit test(s) added/modified to test the lines in red in use_ad_template():
  

[bms63]

Hey @manciniedoardo - I think we decided to move roxygen_param_by_vars() to admiraldev.

For use_ad_template() - I don't think we can write a test for it since it opens an interactive sessions, but I did find this https://github.com/r-lib/covr?tab=readme-ov-file#exclusion-comments - covr is the package used in the test report. @bundfussr

Can we close this issue please and open one for admiradev and a other issue for looking at using # nocov . Happy to create these

[manciniedoardo]

@bms63 sorry, you're right, I shouldn't have included roxygen_param_by_vars(). Maybe for this issue we could just add #nocov to the use_ad_template() line?

[bms63]

Sounds good to me!

[bms63]

Created issue for roxygen_param_by_vars() : pharmaverse/admiraldev#477