JOSS: Add tests
leeper opened this issue · 1 comments
leeper commented
While the package has thorough examples, please add a test suite in /tests
that provides coverage of most of the functionality in the package. At a minimum it should provides tests demonstrating that functions run with the documented arguments; even better would be thorough tests that check whether function outputs are correct and match documented return values.
For JOSS: openjournals/joss-reviews#754
strengejacke commented
I added a lot of tests, all of them checking the expected output (and not just if functions are working). Does the current test suite cover the reviewer's needs?