[Piglit] [PATCH] es3conform.py: Fix PEP 8 issues.

Dylan Baker baker.dylan.c at gmail.com
Mon Mar 9 10:46:44 PDT 2015


Reviewed-by: Dylan Baker <baker.dylan.c at gmail.com>

On Sun, Mar 08, 2015 at 01:44:17PM -0700, Vinson Lee wrote:
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  tests/es3conform.py | 15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)
> 
> diff --git a/tests/es3conform.py b/tests/es3conform.py
> index 540a739..20b408b 100644
> --- a/tests/es3conform.py
> +++ b/tests/es3conform.py
> @@ -32,11 +32,11 @@ from framework.test import TEST_BIN_DIR, Test
>  __all__ = ['profile']
>  
>  #############################################################################
> -##### GTFTest: Execute a sub-test of the Khronos ES 3.0 Conformance suite.
> -#####
> -##### To use this, create a 'GTF3' symlink in piglit/bin which points to the
> -##### Khronos 'GTF' executable.  Piglit will automatically add all .test
> -##### files into the 'gtf' category.
> +# GTFTest: Execute a sub-test of the Khronos ES 3.0 Conformance suite.
> +#
> +# To use this, create a 'GTF3' symlink in piglit/bin which points to the
> +# Khronos 'GTF' executable.  Piglit will automatically add all .test
> +# files into the 'gtf' category.
>  #############################################################################
>  
>  if not path.exists(path.join(TEST_BIN_DIR, 'GTF3')):
> @@ -47,8 +47,10 @@ profile = TestProfile()
>  # Chase the piglit/bin/GTF symlink to find where the tests really live.
>  gtfroot = path.dirname(path.realpath(path.join(TEST_BIN_DIR, 'GTF3')))
>  
> +
>  class GTFTest(Test):
> -    pass_re = re.compile(r'(Conformance|Regression) PASSED all (?P<passed>\d+) tests')
> +    pass_re = re.compile(
> +        r'(Conformance|Regression) PASSED all (?P<passed>\d+) tests')
>  
>      def __init__(self, testpath):
>          super(GTFTest, self).__init__([path.join(TEST_BIN_DIR, 'GTF3'),
> @@ -62,6 +64,7 @@ class GTFTest(Test):
>          else:
>              self.result['result'] = 'fail'
>  
> +
>  def populateTests(runfile):
>      "Read a .run file, adding any .test files to the profile"
>      with open(runfile, 'r') as f:
> -- 
> 2.3.1
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150309/055c4afe/attachment.sig>


More information about the Piglit mailing list