[Piglit] [PATCH 01/12] framework/summary.py: Replace Section heading with docstrings

Dylan Baker baker.dylan.c at gmail.com
Thu May 9 23:09:15 PDT 2013


The sections headings are not really that useful, but they might as well
be kept as class level docstrings instead of being thrown away.

Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
---
 framework/summary.py | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/framework/summary.py b/framework/summary.py
index a72bb4b..0b15814 100644
--- a/framework/summary.py
+++ b/framework/summary.py
@@ -23,10 +23,11 @@
 import core
 
 
-#############################################################################
-##### Vector indicating the number of subtests that have passed/failed/etc.
-#############################################################################
 class PassVector:
+    """
+    Vector indicting the number of tests that have a particular status,
+    pass/fail/skip/etc.
+    """
 	def __init__(self, p, w, f, s, c):
 		self.passnr = p
 		self.warnnr = w
@@ -42,11 +43,10 @@ class PassVector:
 		self.crashnr += o.crashnr
 
 
-#############################################################################
-##### TestSummary: Summarize the results for one test across a
-##### number of testruns
-#############################################################################
 class TestSummary:
+    """
+    Summarize the results for one test across a number of testruns
+    """
 	def isRegression(self, statiList):
 		# Regression is:
 		# - if an item is neither 'pass' nor 'skip'
@@ -102,10 +102,11 @@ results is an array of TestResult instances, one per testrun
 	def allTests(self):
 		return [self]
 
-#############################################################################
-##### GroupSummary: Summarize a group of tests
-#############################################################################
+
 class GroupSummary:
+    """
+    Summarize a group of tests
+    """
 	def createDummyGroup(self, result, test_name):
 		new_group = core.GroupResult()
 		new_group[' ' + test_name + '(All Tests)'] = result[test_name]
@@ -225,10 +226,11 @@ Returns an array of all child TestSummary instances.
 """
 		return [t for name in self.children for t in self.children[name].allTests()]
 
-#############################################################################
-##### Summary: Summarize an array of testruns
-#############################################################################
+
 class Summary:
+    """
+    Summarize an array of testruns
+    """
 	def __init__(self, testruns):
 		"""\
 testruns is an array of TestrunResult instances
-- 
1.8.1.4



More information about the Piglit mailing list