[Piglit] [PATCH] all.py: Actually add_texwrap_target_tests's target parameter to the test.

Roland Scheidegger sroland at vmware.com
Thu Jun 18 07:54:22 PDT 2015


Am 18.06.2015 um 15:10 schrieb Jose Fonseca:
> The target was only being used for the test name -- all test instances
> were duplicate of one another.
> 
> Noticed this because llvmpipe started failing several tests when offset
> was added, and all test command lines were exactly the same.
> ---
>  tests/all.py | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/all.py b/tests/all.py
> index 2839059..f462972 100755
> --- a/tests/all.py
> +++ b/tests/all.py
> @@ -151,13 +151,15 @@ def add_vpfpgeneric(adder, name):
>  
>  
>  def add_texwrap_target_tests(adder, target):
> -    adder(['texwrap', 'GL_RGBA8'], 'texwrap {}'.format(target))
> -    adder(['texwrap', 'GL_RGBA8', 'offset'],
> +    adder(['texwrap', target, 'GL_RGBA8'],
> +          'texwrap {}'.format(target))
> +    adder(['texwrap', target, 'GL_RGBA8', 'offset'],
>            'texwrap {} offset'.format(target))
> -    adder(['texwrap', 'GL_RGBA8', 'bordercolor'],
> +    adder(['texwrap', target, 'GL_RGBA8', 'bordercolor'],
>            'texwrap {} bordercolor'.format(target))
> -    adder(['texwrap', 'GL_RGBA8', 'proj'], 'texwrap {} proj'.format(target))
> -    adder(['texwrap', 'GL_RGBA8', 'proj', 'bordercolor'],
> +    adder(['texwrap', target, 'GL_RGBA8', 'proj'],
> +          'texwrap {} proj'.format(target))
> +    adder(['texwrap', target, 'GL_RGBA8', 'proj', 'bordercolor'],
>            'texwrap {} proj bordercolor'.format(target))
>  
>  
> 

Reviewed-by: Roland Scheidegger <sroland at vmware.com>


More information about the Piglit mailing list