Don't log empty test result messages

The messages obtained from test results may contain stuff like detailed
failure/error information, tracebacks etc. In many cases, the message
will be empty, in which case it doesn't need to be logged.

For an example, run test with many test cases, e.g.
verify/test_verify.py, and inspect the tail of the pytest.log.txt before
and after this commit.

(cherry picked from commit 0a6b0cf68c6553f7eeef06fb507c1b28b9c61f38)

diff --git a/bin/tests/system/conftest.py b/bin/tests/system/conftest.py
index 7a548ea202587837746ef5589b6e79c0318ad0b8..20ede74822da9359419269b0e9ecb8775a049b70 100644 (file)
--- a/bin/tests/system/conftest.py
+++ b/bin/tests/system/conftest.py
@@ -415,7 +415,8 @@ def system_test_dir(request, env, system_test_name, expected_artifacts):
             isctest.log.debug("%s %s", result.outcome.upper(), node)
             messages.extend(result.messages.values())
         for message in messages:
-            isctest.log.debug("\n" + message)
+            if message:
+                isctest.log.debug("\n" + message)
         failed = any(res.outcome == "failed" for res in test_results.values())
         skipped = any(res.outcome == "skipped" for res in test_results.values())
         if failed: