[Piglit] [piglit][PATCH 1/2] core.py: fix subtest json reporting

Dylan Baker baker.dylan.c at gmail.com
Wed Aug 28 12:04:06 PDT 2013

Previously every subtest wrote an entry in the file as a full test, this
is a bug since the fake-fulltest (actually subtest) reported with the
full test value, not it's own value.

This patch changes the behvaior so that a test with subtests only gets
one entry, with the full test result value as it's value.

Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
 framework/core.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/core.py b/framework/core.py
index b1a5726..661b21d 100644
--- a/framework/core.py
+++ b/framework/core.py
@@ -476,7 +476,7 @@ class Test:
                 for test in result['subtest'].keys():
                     if serious_level(result['subtest'][test]) >= serious_level(result['result']):
                         result['result'] = result['subtest'][test]
-                    json_writer.write_dict_item(path + '/' + test, result)
+                json_writer.write_dict_item(path + '/' + test, result)
                 json_writer.write_dict_item(path, result)

More information about the Piglit mailing list