tests: Preserve serial test log.d folder when the stdout has 'not ok'

Change-Id: Iee886ffb5fc45f6c6e79e9204f72713d3cb31d65
Signed-off-by: Kienan Stewart <kstewart@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/tests/regression/tests.serial b/tests/regression/tests.serial
index c936309f6ed46af3858ca48b4878ad6c53c0920d..8075a6ab702df7712879755a04ed2eff9367935c 100755 (executable)
--- a/tests/regression/tests.serial
+++ b/tests/regression/tests.serial
@@ -168,7 +168,9 @@ def run_test(test_script):
     with open(str(test_log), "w") as f:
         logging.debug("Wrote test output to '{}'".format(str(test_log)))
         f.write(stdout)
-    if return_code == 0:
+
+    # Some test scripts can fail but still return a zero exit code
+    if return_code == 0 and len(TAP_NOT_OK_LINE_RE.findall(stdout)) == 0:
         if test_log_dir:
             shutil.rmtree(str(test_log_dir))
     return (stdout, return_code)