On Mittwoch, 24. August 2016 20:08:17 CEST Dale R. Worley wrote:
> In the file testenv/README is:
>
> Next, is the const variable, TEST_NAME that defines the name of the
> Test.
>
> Both, the HTTPTest and FTPTest modules have the same prototype:
> {
> name,
> pre_hook,
> test_options,
> post_hook,
> protocols
> }
> name should be a string, and is usually passed to the TEST_NAME
> variable,
>
> Remember to always name the Test correctly using the TEST_NAME variable.
> This is essential since a directory with the Test Name is created and this
> can cause synchronization problems when the Parallel Test Harness is used.
> One can use the following command on Unix systems to check for TEST_NAME
> clashes:
> $ grep -r -h "TEST_NAME =" | cut -c13- | uniq -c -d
>
> However, *none* of the tests in testenv has a TEST_NAME parameter.
>
> In addition, the HTTPTest class does not have a "name" argument for its
> constructor. (See testenv/test/http_test.py, line 15.)
>
> What should be done about this?Very good finding. name/Test_NAME are remnants. We (well, I :-) removed that some while ago (using the file name of the test as unique identifier) and just forgot to amend the README. See commit 926e42d4678689195a0bbed210c6d027db7cc390 A patch to correct this would be welcome ! Regards, Tim
signature.asc
Description: This is a digitally signed message part.
