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 <[email protected]>
---
 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

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to