[Piglit] [PATCH] oclconform: Fix PEP 8 issues.

Vinson Lee vlee at freedesktop.org
Sat Nov 14 23:20:33 PST 2015


Signed-off-by: Vinson Lee <vlee at freedesktop.org>
---
 framework/test/oclconform.py | 39 +++++++++++++++++++++++++--------------
 1 file changed, 25 insertions(+), 14 deletions(-)

diff --git a/framework/test/oclconform.py b/framework/test/oclconform.py
index 14ce902..eb3adb4 100644
--- a/framework/test/oclconform.py
+++ b/framework/test/oclconform.py
@@ -41,20 +41,24 @@ __all__ = [
 def get_test_section_name(test):
     return 'oclconform-{}'.format(test)
 
-class OCLConform(Test):
 
+class OCLConform(Test):
     def interpret_result(self):
         if self.result.returncode != 0 or 'FAIL' in self.result.out:
             self.result.result = 'fail'
         else:
             self.result.result = 'pass'
 
+
 def add_sub_test(profile, test_name, subtest_name, subtest):
-    profile.test_list[grouptools.join('oclconform', test_name, subtest_name)] = subtest
+    profile.test_list[grouptools.join('oclconform', test_name,
+                                      subtest_name)] = subtest
+
 
 def add_test(profile, test_name, test):
     profile.test_list[grouptools.join('oclconform', test_name)] = test
 
+
 def add_oclconform_tests(profile):
     section_name = 'oclconform'
     if not PIGLIT_CONFIG.has_section(section_name):
@@ -68,30 +72,37 @@ def add_oclconform_tests(profile):
     for test in tests:
         test_section_name = get_test_section_name(test)
         if not PIGLIT_CONFIG.has_section(test_section_name):
-            print("Warning: no section defined for {}".format(test), file=stderr)
+            print("Warning: no section defined for {}".format(test),
+                  file=stderr)
             continue
 
         test_name = PIGLIT_CONFIG.get(test_section_name, 'test_name')
-        should_run_concurrent = PIGLIT_CONFIG.has_option(test_section_name, 'concurrent')
+        should_run_concurrent = PIGLIT_CONFIG.has_option(test_section_name,
+                                                         'concurrent')
         if PIGLIT_CONFIG.has_option(test_section_name, 'list_subtests'):
-            list_tests = PIGLIT_CONFIG.get(test_section_name, 'list_subtests')
-            subtest_regex = PIGLIT_CONFIG.get(test_section_name, 'subtest_regex')
+            list_tests = PIGLIT_CONFIG.get(test_section_name,
+                                           'list_subtests')
+            subtest_regex = PIGLIT_CONFIG.get(test_section_name,
+                                              'subtest_regex')
             subtest_regex.encode('string_escape')
             run_subtests = PIGLIT_CONFIG.get(test_section_name, 'run_subtest')
-            list_tests =list_tests.split()
+            list_tests = list_tests.split()
 
-            subtests = subprocess.check_output(args=list_tests, cwd=bindir).split('\n')
+            subtests = subprocess.check_output(args=list_tests,
+                                               cwd=bindir).split('\n')
             for subtest in subtests:
                 m = re.match(subtest_regex, subtest)
                 if not m:
                     continue
                 subtest = m.group(1)
-                subtest_command = join(bindir, run_subtests.replace('<subtest>', subtest))
+                subtest_command = join(bindir,
+                                       run_subtests.replace('<subtest>',
+                                                            subtest))
                 add_sub_test(profile, test_name, subtest,
-		             OCLConform(command=subtest_command.split(),
-			                run_concurrent=should_run_concurrent))
+                             OCLConform(command=subtest_command.split(),
+                                        run_concurrent=should_run_concurrent))
         else:
             run_test = PIGLIT_CONFIG.get(test_section_name, 'run_test')
-            add_test(profile, test_name, OCLConform(command=run_test.split(),
-	                                            run_concurrent=should_run_concurrent))
-
+            add_test(profile, test_name,
+                     OCLConform(command=run_test.split(),
+                                run_concurrent=should_run_concurrent))
-- 
2.4.9 (Apple Git-60)



More information about the Piglit mailing list