LGTM.

Not sure I like this long term, but will let us play with stuff for now.

On Mon, Mar 29, 2010 at 7:59 AM, John Admanski <[email protected]> wrote:
> Change the hierarchial tests to use : instead of . to indicate the test
> hierarchy. Since using . makes it impossible to determine what the actual
> test and tag are, this new feature breaks existing behaviour. Pending a
> better solution we should use a separate character that doesn't already
> have a better meaning.
>
> Signed-off-by: John Admanski <[email protected]>
>
> Index: trunk/client/common_lib/test.py
> ===================================================================
> --- client/common_lib/test.py   (revision 4351)
> +++ client/common_lib/test.py   (working copy)
> @@ -627,7 +627,7 @@
>         # Tests in site_testdir override tests defined in testdir
>         testname = path = url
>         testgroup = ''
> -        path = re.sub('\.', '/', testname)
> +        path = re.sub(':', '/', testname)
>         modulename = os.path.basename(path)
>         classname = '%s.%s' % (modulename, modulename)
>
> _______________________________________________
> Autotest mailing list
> [email protected]
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
>
_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to