[Piglit] [patch v2 1/2] tests/all.py: Fix name of arb_tessellation_shader test

Ilia Mirkin imirkin at alum.mit.edu
Fri Aug 29 06:36:21 PDT 2014


On Fri, Aug 29, 2014 at 2:18 AM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> '/' is not allowed in the name of tests because it is the delimiter for
> group names. This test includes '/' which causes the HTML summary to
> render in odd ways.
>
> v2: - use nested group structure rather than adding a test directly to
>       the test list (Ilia)

Thanks :)

Series is Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

>
> Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
> ---
>  tests/all.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/all.py b/tests/all.py
> index 20ebec2..05a2c8e 100644
> --- a/tests/all.py
> +++ b/tests/all.py
> @@ -1513,7 +1513,7 @@ add_concurrent_test(arb_tessellation_shader, 'arb_tessellation_shader-minmax')
>  add_concurrent_test(arb_tessellation_shader, 'arb_tessellation_shader-invalid-get-program-params')
>  add_concurrent_test(arb_tessellation_shader, 'arb_tessellation_shader-invalid-patch-vertices-range')
>  add_concurrent_test(arb_tessellation_shader, 'arb_tessellation_shader-invalid-primitive')
> -add_concurrent_test(arb_tessellation_shader, 'built-in-constants tests/spec/arb_tessellation_shader/minimum-maximums.txt')
> +arb_tessellation_shader['built-in-constants'] = concurrent_test('built-in-constants tests/spec/arb_tessellation_shader/minimum-maximums.txt')
>  add_concurrent_test(arb_tessellation_shader, 'arb_tessellation_shader-minmax')
>  import_glsl_parser_tests(arb_tessellation_shader,
>                           os.path.join(testsDir, 'spec',
> --
> 2.1.0
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list