[Piglit] [PATCH 3/3] framework/exectest.py: Only record one instance of subtests

Dylan Baker baker.dylan.c at gmail.com
Thu Apr 24 17:20:42 PDT 2014


Since subtests are better handled in the summary module there is no need
to record these multiple times. This significantly reduces the size of
the results file, (from 45M to 10M)
---
 framework/exectest.py | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/framework/exectest.py b/framework/exectest.py
index 3056a78..abd10f1 100644
--- a/framework/exectest.py
+++ b/framework/exectest.py
@@ -96,13 +96,7 @@ class Test(object):
             log.log(path, self.result['result'])
             log.post_log(log_current, self.result['result'])
 
-            if 'subtest' in self.result and len(self.result['subtest']) > 1:
-                for test in self.result['subtest']:
-                    self.result['result'] = self.result['subtest'][test]
-                    json_writer.write_dict_item(os.path.join(path, test),
-                                                self.result)
-            else:
-                json_writer.write_dict_item(path, self.result)
+            json_writer.write_dict_item(path, self.result)
         else:
             log.log(path, 'dry-run')
             log.post_log(log_current, 'dry-run')
-- 
2.0.0.rc0



More information about the Piglit mailing list