Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance test coverage for derive_var_transposed() #2575

Closed
1 task
manciniedoardo opened this issue Nov 27, 2024 · 2 comments · Fixed by #2597
Closed
1 task

Enhance test coverage for derive_var_transposed() #2575

manciniedoardo opened this issue Nov 27, 2024 · 2 comments · Fixed by #2597
Assignees
Labels

Comments

@manciniedoardo
Copy link
Collaborator

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 derive_vars_transposed()) 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 derive_vars_transposed():
    image
@bms63
Copy link
Collaborator

bms63 commented Nov 27, 2024

This is a duplicated issue - #2520

But yours sounds nicer so will close the other one

@manciniedoardo
Copy link
Collaborator Author

ah sorry - cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 participants