CIRADA-Tools/RM-Tools

Unit testing and test coverage

Luke-Pratley opened this issue · 4 comments

Are there any plans for unit tests in RM tools to check sections of code working as expected?

This would help find bugs when someone changes sections of code or introduces anything new.

At the moment, I don't have any plans for any more tests beyond what it already present. I can appreciate the value of having more tests, but at the moment the weight of cost vs value isn't really in favour of further test development, at least for me.

I guess this now partially addressed by #48 ?

I'm not sure. My impression (and Luke can correct me if I'm wrong), is that we were thinking about smaller/more focused tests. The current tests just check the high-level programs (rmsynth1d, rmclean1d, etc), but not the functions inside.

Also, I just realized that the new test framework from #48 isn't running the tests I already wrote (tests/QA_tests.py), so I'll need to figure out how to integrate those together at some point...

I've just put in #53, which now installs all the dependancies (including multinest) runs all the testing in tests.