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

Vinson Lee vlee at freedesktop.org
Mon Mar 16 23:30:49 PDT 2015


Signed-off-by: Vinson Lee <vlee at freedesktop.org>
---
 framework/backends/junit.py | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/framework/backends/junit.py b/framework/backends/junit.py
index 53b6086..98a85b6 100644
--- a/framework/backends/junit.py
+++ b/framework/backends/junit.py
@@ -120,7 +120,7 @@ class JUnitBackend(FileBackend):
             if lname in self._expected_failures:
                 expected_result = "failure"
                 # a test can either fail or crash, but not both
-                assert( lname not in self._expected_crashes )
+                assert lname not in self._expected_crashes
 
             if lname in self._expected_crashes:
                 expected_result = "error"
@@ -131,17 +131,20 @@ class JUnitBackend(FileBackend):
             if data['result'] == 'skip':
                 res = etree.SubElement(element, 'skipped')
 
-            elif data['result'] in ['warn', 'fail', 'dmesg-warn', 'dmesg-fail']:
+            elif data['result'] in ['warn', 'fail', 'dmesg-warn',
+                                    'dmesg-fail']:
                 if expected_result == "failure":
                     err.text += "\n\nWARN: passing test as an expected failure"
-                    res = etree.SubElement(element, 'skipped', message='expected failure')
+                    res = etree.SubElement(element, 'skipped',
+                                           message='expected failure')
                 else:
                     res = etree.SubElement(element, 'failure')
 
             elif data['result'] == 'crash':
                 if expected_result == "error":
                     err.text += "\n\nWARN: passing test as an expected crash"
-                    res = etree.SubElement(element, 'skipped', message='expected crash')
+                    res = etree.SubElement(element, 'skipped',
+                                           message='expected crash')
                 else:
                     res = etree.SubElement(element, 'error')
 
-- 
2.3.2



More information about the Piglit mailing list