[Piglit] [PATCH] gtest: Fix bug introduced in 90a256411

Dylan Baker baker.dylan.c at gmail.com
Thu Feb 13 10:59:13 PST 2014


In the mentioned commit the signature of Test.interpretResult was
changed from (self, out, returncode, results, dmesg) to (self, out,
returncode, results), however, this was not applied to GTest, which
retained the dmesg argument.

This patch fixes that bug.

Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>

---
 framework/gtest.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/gtest.py b/framework/gtest.py
index 7b474f3..00ed6db 100644
--- a/framework/gtest.py
+++ b/framework/gtest.py
@@ -29,7 +29,7 @@ import re
 from framework.exectest import ExecTest
 
 class GTest(ExecTest):
-    def interpretResult(self, out, returncode, results, dmesg):
+    def interpretResult(self, out, returncode, results):
         # Since gtests can have several subtets, if any of the subtests fail
         # then we need to report fail.
         if len(re.findall('FAILED', out, re.MULTILINE)) > 0:
-- 
1.8.5.4



More information about the Piglit mailing list