[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