Plans on adding unit tests?

[welch16]

Hi @sjbeckett,

I was wondering if there are any plans to add unit tests with testthat or an alternative package. I know that other organizations like bioconductor or ropensci recommend adding unit tests.

I know the amount of work required to add tests to a package of the scope as this one, but I feel it would be enriched in the end.

Thanks

[sjbeckett]

Hi @welch16, many thanks for this suggestion!

I think we can implement unit testing on some, but not all of our core functionality - and wanted to scope out what this may look like. Given the dynamic nature of the datasets that we are working with within the package, the output of individual load function calls is non-deterministic, which we think would make unit testing challenging to implement across the entire package. Given this challenge, we have made some adjustments in our revisions to help user experience.

However, I think we could implement unit testing on some of our utility functions including tidy_Data and calcRisk.

[sjbeckett]

Unit tests for some core functions added with #67