[Piglit] [PATCH] framework/summary: Fix regressions and fixes list to not include skip

Dylan Baker baker.dylan.c at gmail.com
Thu Sep 24 14:31:53 PDT 2015


This fixes the lists of regressions and fixes to not include skip or
notrun statuses.

This fixes a regression introduced in the recent summary rework of
piglit.

Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
---
 framework/summary/common.py | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/framework/summary/common.py b/framework/summary/common.py
index 246de09..96562e5 100644
--- a/framework/summary/common.py
+++ b/framework/summary/common.py
@@ -126,11 +126,15 @@ class Names(object):
 
     @lazy_property
     def regressions(self):
-        return self.__diff(operator.lt)
+        # By ensureing tha min(x, y) is >= so.PASS we eleminate NOTRUN and SKIP
+        # from these pages
+        return self.__diff(lambda x, y: x < y and min(x, y) >= so.PASS)
 
     @lazy_property
     def fixes(self):
-        return self.__diff(operator.gt)
+        # By ensureing tha min(x, y) is >= so.PASS we eleminate NOTRUN and SKIP
+        # from these pages
+        return self.__diff(lambda x, y: x > y and min(x, y) >= so.PASS)
 
     @lazy_property
     def enabled(self):
-- 
2.5.3



More information about the Piglit mailing list