Re: tests and coverage

2018-02-22 Thread Sebastian Rühl
Hi Justin, First of all thanks for the Tests (and finding the Bug :). You are absolutely right about your statements but I don’t think its a issue of focus rather than time and resources (at least in my case). My intention was to bring the ADS branch to master a soon as possible to encourage

tests and coverage

2018-02-19 Thread Justin Mclean
Hi, It looks to me that some of the tests are just here to increase coverage and we seem to be missing unit test for some of the more simple classes. Perhaps there’s a bit too much focus on the happy path and we’re not always checking boundary conditions and the like. Obviously this is a good