From: Karol L. <kar...@kn...> - 2007-09-03 23:23:45
|
I think it is a good idea to also clean up the unittests a bit before releasing 0.8. In the interest of this, I added a "SKIPPED" column in the test summary and the appropriate code for monitoring this (currently based on PASS being in the docstring). This way it easier to see where we are skipping unittests. Also, in my last commit I merged testmodule into testall as Noel suggested (although the name 'testall' does not reflect its function anymore!). Other issues that you might want to comment on: 1) there should be more coverage of unittests for methods(?) 2) should unittests for parsers and methods be either better integrated (possibly run through the same function) or better separated (in separate directories)? Presently, testall.py runs only parser unittests, so shouldn't it be renamed to testparsers.py? 3) I noticed that there are a number of tests in GeoOpt that can be done for SP, so should they be only in SP or in both? Cheers, Karol -- written by Karol Langner Tue Sep 4 01:02:43 EDT 2007 |