Issues #502 and #587- Add additional tests #802
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.