From 37838803dbdd042ac43c8cb860b67b5e4422978f Mon Sep 17 00:00:00 2001 From: Angus Gratton Date: Wed, 18 Nov 2020 15:48:27 +1100 Subject: [PATCH 1/2] ci: Log failure to close any DUT --- tools/ci/python_packages/tiny_test_fw/Env.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/ci/python_packages/tiny_test_fw/Env.py b/tools/ci/python_packages/tiny_test_fw/Env.py index d54535351e..90d84e3405 100644 --- a/tools/ci/python_packages/tiny_test_fw/Env.py +++ b/tools/ci/python_packages/tiny_test_fw/Env.py @@ -18,6 +18,7 @@ import threading import functools import netifaces +import traceback from . import EnvConfig @@ -190,6 +191,7 @@ class Env(object): dut.print_debug_info() dut.close() except Exception as e: + traceback.print_exc() dut_close_errors.append(e) self.allocated_duts = dict() return dut_close_errors From 26fb74bc972ecf435c59f23ffbaa6ce3779aa221 Mon Sep 17 00:00:00 2001 From: Angus Gratton Date: Tue, 24 Nov 2020 18:32:09 +1100 Subject: [PATCH 2/2] ci: ttfw: Encode serial port data to whatever the console encoding is This is a bit of a hack, but gives us a way to always log it --- tools/ci/python_packages/tiny_test_fw/DUT.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tools/ci/python_packages/tiny_test_fw/DUT.py b/tools/ci/python_packages/tiny_test_fw/DUT.py index e01f8bb67b..ca54269a72 100644 --- a/tools/ci/python_packages/tiny_test_fw/DUT.py +++ b/tools/ci/python_packages/tiny_test_fw/DUT.py @@ -40,6 +40,7 @@ If they using different port then need to implement their DUTPort class as well. from __future__ import print_function import time import re +import sys import threading import copy import functools @@ -78,11 +79,12 @@ def _expect_lock(func): def _decode_data(data): """ for python3, if the data is bytes, then decode it to string """ if isinstance(data, bytes): - # convert bytes to string + # convert bytes to string. This is a bit of a hack, we know that we want to log this + # later so encode to the stdout encoding with backslash escapes for anything non-encodable try: - data = data.decode("utf-8", "ignore") - except UnicodeDecodeError: - data = data.decode("iso8859-1", ) + return data.decode(sys.stdout.encoding, "backslashreplace") + except UnicodeDecodeError: # Python <3.5 doesn't support backslashreplace + return data.decode(sys.stdout.encoding, "replace") return data