[Piglit] [PATCH 3/3] Test reporting: remove extra whitespace

Dylan Baker baker.dylan.c at gmail.com
Wed Jun 11 16:32:33 PDT 2014


This patch removes the whitespace after 'PIGLIT:', this isn't a big
change, but printing and reading one less character per entry might have
some savings.

Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
---
 framework/tests/exectest_test.py | 6 +++---
 tests/util/piglit-util.c         | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/framework/tests/exectest_test.py b/framework/tests/exectest_test.py
index c950499..714d751 100644
--- a/framework/tests/exectest_test.py
+++ b/framework/tests/exectest_test.py
@@ -36,7 +36,7 @@ def test_initialize_piglittest():
 def test_piglittest_interpret_result():
     """ PiglitTest.interpret_result() works no subtests """
     test = PiglitTest('foo')
-    test.result['out'] = 'PIGLIT: {"result": "pass"}\n'
+    test.result['out'] = 'PIGLIT:{"result": "pass"}\n'
     test.interpret_result()
     assert test.result['result'] == 'pass'
 
@@ -44,7 +44,7 @@ def test_piglittest_interpret_result():
 def test_piglittest_interpret_result_subtest():
     """ PiglitTest.interpret_result() works with subtests """
     test = PiglitTest('foo')
-    test.result['out'] = ('PIGLIT: {"result": "pass"}\n'
-                          'PIGLIT: {"subtest": {"subtest": "pass"}}\n')
+    test.result['out'] = ('PIGLIT:{"result": "pass"}\n'
+                          'PIGLIT:{"subtest": {"subtest": "pass"}}\n')
     test.interpret_result()
     assert test.result['subtest']['subtest'] == 'pass'
diff --git a/tests/util/piglit-util.c b/tests/util/piglit-util.c
index dd6134e..ebbcd18 100644
--- a/tests/util/piglit-util.c
+++ b/tests/util/piglit-util.c
@@ -220,7 +220,7 @@ piglit_report_result(enum piglit_result result)
 
 	fflush(stderr);
 
-	printf("PIGLIT: {\"result\": \"%s\" }\n", result_str);
+	printf("PIGLIT:{\"result\": \"%s\" }\n", result_str);
 	fflush(stdout);
 
 	switch(result) {
@@ -241,7 +241,7 @@ piglit_report_subtest_result(enum piglit_result result, const char *format, ...)
 
 	va_start(ap, format);
 
-	printf("PIGLIT: {\"subtest\": {\"");
+	printf("PIGLIT:{\"subtest\": {\"");
 	vprintf(format, ap);
 	printf("\" : \"%s\"}}\n", result_str);
 	fflush(stdout);
-- 
2.0.0



More information about the Piglit mailing list