diff options
author | Stephen Warren <swarren@nvidia.com> | 2018-09-20 16:55:03 -0600 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2018-10-06 09:04:18 -0400 |
commit | dddea0d0d72e45f60fadddd45e22e30065bd010f (patch) | |
tree | b08d971e78e810bbfb14d12ce0d9ec1dde398f2e | |
parent | 14573fb78fef99266f7691ac9af2ddb388f9491f (diff) | |
download | u-boot-dddea0d0d72e45f60fadddd45e22e30065bd010f.zip u-boot-dddea0d0d72e45f60fadddd45e22e30065bd010f.tar.gz u-boot-dddea0d0d72e45f60fadddd45e22e30065bd010f.tar.bz2 |
test/py: ignore console read exceptions after test failure
After a test has failed, test/py drains the U-Boot console log to ensure
that any relevant output is captured. At this point, we don't care about
detecting any additional errors, since the test is already known to have
failed, and U-Boot will be restarted. To ensure that the test cleanup code
is not interrupted, and can correctly terminate the log sections for the
failed test, ignore any exception that occurs while reading the U-Boot
console output during this limited period of time.
Signed-off-by: Stephen Warren <swarren@nvidia.com>
-rw-r--r-- | test/py/u_boot_console_base.py | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/test/py/u_boot_console_base.py b/test/py/u_boot_console_base.py index a14bad6..326b2ac 100644 --- a/test/py/u_boot_console_base.py +++ b/test/py/u_boot_console_base.py @@ -304,7 +304,17 @@ class ConsoleBase(object): # Wait for something U-Boot will likely never send. This will # cause the console output to be read and logged. self.p.expect(['This should never match U-Boot output']) - except u_boot_spawn.Timeout: + except: + # We expect a timeout, since U-Boot won't print what we waited + # for. Squash it when it happens. + # + # Squash any other exception too. This function is only used to + # drain (and log) the U-Boot console output after a failed test. + # The U-Boot process will be restarted, or target board reset, once + # this function returns. So, we don't care about detecting any + # additional errors, so they're squashed so that the rest of the + # post-test-failure cleanup code can continue operation, and + # correctly terminate any log sections, etc. pass finally: self.p.timeout = orig_timeout |