laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified
pySim-shell_test/utils: display pySim-shell logfile content

When we configure the tests to display file content, we only display files that
we compare, let's also display log file contents from pySim-shell. This will
be useful in situations where we only have log output from the tests, but no
access to the file system of the test host.

Related: OS#6601
Change-Id: Ibf6f78d7e71c213c7ca1caaf21c4c890e892261e
---
M tests/pySim-shell_test/utils.py
1 file changed, 5 insertions(+), 0 deletions(-)

diff --git a/tests/pySim-shell_test/utils.py b/tests/pySim-shell_test/utils.py
index 6acbd70..348c9b1 100644
--- a/tests/pySim-shell_test/utils.py
+++ b/tests/pySim-shell_test/utils.py
@@ -272,6 +272,11 @@
# Check for exceptions
logfile = open(logfile_name)
logfile_content = logfile.read()
+ if self.print_content:
+ print("pySim-shell logfile content: (%s)" % os.path.basename(logfile_name))
+ print("-----------------------8<-----------------------")
+ print(logfile_content)
+ print("-----------------------8<-----------------------")
logfile.close()
exception_regex_compiled = re.compile('.*EXCEPTION.*')
exceptions_strings = re.findall(exception_regex_compiled, logfile_content)

To view, visit change 38545. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ibf6f78d7e71c213c7ca1caaf21c4c890e892261e
Gerrit-Change-Number: 38545
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>