From: Dan Streetman <ddstreet@canonical.com>
Date: Fri, 30 Apr 2021 00:21:10 +0000 (-0400)
Subject: test: combine stdout/stderr from failed test
X-Git-Tag: v249-rc1~223
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=d57e871c60805e3f1f57729ea8022477a611697d;p=thirdparty%2Fsystemd.git

test: combine stdout/stderr from failed test

Printing stdout and stderr from a failed test makes it harder to
interpret what the specific problem was; instead let's print out
the lines in order as we got them when the test was run

Also save failed test output to file if ARTIFACT_DIRECTORY is defined
---

diff --git a/test/run-unit-tests.py b/test/run-unit-tests.py
index 9a75cd421ec..f0dea5c07fe 100755
--- a/test/run-unit-tests.py
+++ b/test/run-unit-tests.py
@@ -4,6 +4,7 @@ import argparse
 import dataclasses
 import glob
 import os
+import pathlib
 import subprocess
 import sys
 try:
@@ -27,6 +28,8 @@ def argument_parser():
     p = argparse.ArgumentParser()
     p.add_argument('-u', '--unsafe', action='store_true',
                    help='run "unsafe" tests too')
+    p.add_argument('-A', '--artifact_directory',
+                   help='store output from failed tests in this dir')
     return p
 
 opts = argument_parser().parse_args()
@@ -35,11 +38,14 @@ tests = glob.glob('/usr/lib/systemd/tests/test-*')
 if opts.unsafe:
     tests += glob.glob('/usr/lib/systemd/tests/unsafe/test-*')
 
+if not opts.artifact_directory and os.getenv('ARTIFACT_DIRECTORY'):
+    opts.artifact_directory = os.getenv('ARTIFACT_DIRECTORY')
+
 total = Total(total=len(tests))
 for test in tests:
     name = os.path.basename(test)
 
-    ex = subprocess.run(test, stdin=subprocess.DEVNULL, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+    ex = subprocess.run(test, stdin=subprocess.DEVNULL, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
     if ex.returncode == 0:
         print(f'{GREEN}PASS: {name}{RESET_ALL}')
         total.good += 1
@@ -50,12 +56,23 @@ for test in tests:
         print(f'{RED}FAIL: {name}{RESET_ALL}')
         total.fail += 1
 
-        # stdout/stderr might not be valid unicode, let's just dump it to the terminal.
-        # Also let's reset the style afterwards, in case our output sets something.
-        sys.stdout.buffer.write(ex.stdout)
-        print(f'{RESET_ALL}{BRIGHT}')
-        sys.stdout.buffer.write(ex.stderr)
-        print(f'{RESET_ALL}')
+        output_file = None
+        if opts.artifact_directory:
+            output_dir = pathlib.Path(opts.artifact_directory) / 'unit-tests'
+            output_dir.mkdir(parents=True, exist_ok=True)
+            output_file = output_dir / name
+            output_file.write_bytes(ex.stdout)
+
+        try:
+            print(ex.stdout.decode('utf-8'))
+        except UnicodeDecodeError:
+            print(f'{BRIGHT}Note, some test output shown here is not UTF-8')
+            if output_file:
+                print(f'For actual test output see artifact file {output_file}')
+            print(f'{RESET_ALL}')
+            print(ex.stdout.decode('utf-8', errors='replace'))
+    sys.stdout.flush()
+
 
 print(f'{BRIGHT}OK: {total.good} SKIP: {total.skip} FAIL: {total.fail}{RESET_ALL}')
 sys.exit(total.fail > 0)