[Piglit] [PATCH v2] Ignore piglit warning status in JUnit

Mark Janes mark.a.janes at intel.com
Fri Jun 26 15:35:12 PDT 2015


JUnit has no concept of "warning".  It supports the following
statuses:

 - skip
 - success
 - fail
 - error

A test which emits a warning is more accurately represented as
"success" in JUnit.

v2: Continue to report failure for "dmesg-warn", which is more serious
    than "warn".  (from Ilia Mirkin)
---
 framework/backends/junit.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/framework/backends/junit.py b/framework/backends/junit.py
index 632e516..8c8639e 100644
--- a/framework/backends/junit.py
+++ b/framework/backends/junit.py
@@ -142,8 +142,7 @@ 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 ['fail', 'dmesg-warn', 'dmesg-fail']:
                 if expected_result == "failure":
                     err.text += "\n\nWARN: passing test as an expected failure"
                     res = etree.SubElement(element, 'skipped',
-- 
2.1.4



More information about the Piglit mailing list