Hi, The unit test framework is shaping up nicely! I now understand what you meant about the renaming, with the 'module' to 'package' changes for the unit test directories. I have a few minor points about this commit. In the docstring where you have the text "@return: a file system path for the current operating system", what exactly is 'a file system path'? I'm also confused by the docstring with the text "@param pattern_list: a lsit of regular expression patterns which identifie a file as on containing a unit test TestCase" near line 294. If I would like to use this function how should I construct the list (I'm making the assumption I have only read the API docs)? I.e. is there a set of pre-defined elements? Or am I free to supply the following:
pattern_list=['generic_fns/test*.py', 'data/test*.py', 'data/__init__.py'] For the text "@rtype: a hierachy of pyunit testSuites and testCases", is the hierachy a string or some other object? A few portability issues include that the module variables 'system_path_pattern', 'unit_test_path_pattern', and 'test_case_patterns' need to be made OS independent (the last could use CURRENT_DIRECTORY rather than a dot). And is CURRENT_DIRECTORY always '.' on all OSs Python supports? If you like Gary, I can make the necessary changes tomorrow. Cheers, Edward On 3/13/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
Author: varioustoxins Date: Tue Mar 13 10:35:15 2007 New Revision: 3173 URL: http://svn.gna.org/viewcvs/relax?rev=3173&view=rev Log: directory independant running of unit tests, improved docs, code cleanup Modified: 1.3/test_suite/unit_tests/unit_test_runner.py [This mail would be too long, it was shortened to contain the URLs only.] Modified: 1.3/test_suite/unit_tests/unit_test_runner.py URL: http://svn.gna.org/viewcvs/relax/1.3/test_suite/unit_tests/unit_test_runner.py?rev=3173&r1=3172&r2=3173&view=diff _______________________________________________ relax (http://nmr-relax.com) This is the relax-commits mailing list [email protected] To unsubscribe from this list, get a password reminder, or change your subscription options, visit the list information page at https://mail.gna.org/listinfo/relax-commits
_______________________________________________ relax (http://nmr-relax.com) This is the relax-devel mailing list [email protected] To unsubscribe from this list, get a password reminder, or change your subscription options, visit the list information page at https://mail.gna.org/listinfo/relax-devel
