[Piglit] [Patch v3 4/5] results_v0_tests.py: Use TestrunResult.load instead of load_results
Dylan Baker
baker.dylan.c at gmail.com
Thu Oct 30 16:32:41 PDT 2014
framework.results.load_results calls update_results() internally. This
means that as soon as a second method is added that updates results to
version 2 then the tests will start failing.
v2: - add this patch
Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
---
framework/tests/results_v0_tests.py | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/framework/tests/results_v0_tests.py b/framework/tests/results_v0_tests.py
index fd8be26..c9ed91e 100644
--- a/framework/tests/results_v0_tests.py
+++ b/framework/tests/results_v0_tests.py
@@ -100,8 +100,9 @@ DATA['tests'].update({
},
})
-with utils.with_tempfile(json.dumps(DATA)) as f:
- RESULT = results._update_zero_to_one(results.load_results(f))
+with utils.with_tempfile(json.dumps(DATA)) as t:
+ with open(t, 'r') as f:
+ RESULT = results._update_zero_to_one(results.TestrunResult.load(f))
def test_dmesg():
@@ -188,8 +189,9 @@ def test_info_split():
data['tests']['sometest']['info'] = \
'Returncode: 1\n\nErrors:stderr\n\nOutput: stdout\n\nmore\n\nstuff'
- with utils.with_tempfile(json.dumps(data)) as f:
- results._update_zero_to_one(results.load_results(f))
+ with utils.with_tempfile(json.dumps(data)) as t:
+ with open(t, 'r') as f:
+ results._update_zero_to_one(results.TestrunResult.load(f))
def test_subtests_with_slash():
--
2.1.2
More information about the Piglit
mailing list