Merge pull request #9168 from JuhPuur/unittest_logging_fix

Fix to unit test losing process output due to timing issue

UNITTESTS/unit_test/utils.py

@@ -51,8 +51,13 @@ def execute_program(args, error_msg="An error occurred!", success_msg=None):
  stdout=subprocess.PIPE,
  stderr=subprocess.STDOUT)
 
+ # Output is stripped to remove newline character. logging adds its own
+ # so we avoid double newlines.
+ # Because the process can terminate before the loop has read all lines,
+ # we read the output remnant just in case. Otherwise we lose it.
  while process.poll() is None:
- logging.info(process.stdout.readline().decode("utf8"))
+ logging.info(process.stdout.readline().decode('utf8').rstrip('\n'))
+ logging.info(process.stdout.read().decode('utf8').rstrip('\n'))
 
  retcode = process.wait()