As discussed in #502, this PR adds a few additional tests to score() in tests/testthat/test-score.R.
Addressing #502 raised two new issues (#800, #801). I suggest addressing #801 (done) and then adding more tests as suggested in #502.
This PR closes #587.
As discussed in #587, I went through all exported functions and added additional tests to make sure the output class is exactly what we expect it to be. In some places, I cleaned tests by
moving headings around,
removing duplicated tests
adding small additional tests
Checklist
[x] My PR is based on a package issue and I have explicitly linked it.
[x] I have included the target issue or issues in the PR title as follows: issue-number: PR title
[x] I have tested my changes locally.
[x] I have added or updated unit tests where necessary.
[x] I have updated the documentation if required.
[x] I have built the package locally and run rebuilt docs using roxygen2.
[x] My code follows the established coding standards and I have run lintr::lint_package() to check for style issues introduced by my changes.
[ ] I have added a news item linked to this PR.
[ ] I have reviewed CI checks for this PR and addressed them as far as I am able.
Description
This PR partly addresses #502.
As discussed in #502, this PR adds a few additional tests to
score()
intests/testthat/test-score.R
. Addressing #502 raised two new issues (#800, #801). I suggest addressing #801 (done) and then adding more tests as suggested in #502.This PR closes #587. As discussed in #587, I went through all exported functions and added additional tests to make sure the output class is exactly what we expect it to be. In some places, I cleaned tests by
Checklist
lintr::lint_package()
to check for style issues introduced by my changes.