laforge has submitted this change. (
https://gerrit.osmocom.org/c/pysim/+/38543?usp=email
)
Change subject: pySim-shell_test/utils: delete log files in general
......................................................................
pySim-shell_test/utils: delete log files in general
When we get rid of temporary files, we delete those using a wildcard,
but for the logs from pySim-shell we explicitly memorize the name
of the pySim-shell logfile and delete it later by this explicit
name. This is not necessary, let's just delete all log files present
using a wildcard.
Related: OS#6601
Change-Id: I09dc7e59d1a3dcb68f54e3a8dccb86a1bc6c9ee6
---
M tests/pySim-shell_test/utils.py
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
laforge: Looks good to me, approved
Jenkins Builder: Verified
osmith: Looks good to me, but someone else must approve
diff --git a/tests/pySim-shell_test/utils.py b/tests/pySim-shell_test/utils.py
index fe8386a..943eb48 100644
--- a/tests/pySim-shell_test/utils.py
+++ b/tests/pySim-shell_test/utils.py
@@ -220,7 +220,7 @@
print("testcase execution done -- cleaning up ...")
if not self.keepfiles:
os.system("rm -f ./*.tmp")
- os.system("rm -f ./" + self.__pySim_shell_logfile_name)
+ os.system("rm -f ./*.log")
for template in self.__templates_generated:
os.system("rm -f ./" + template)
@@ -241,7 +241,6 @@
"""
logfile_name = "pySim-shell_" + self._testMethodName +
".log"
- self.__pySim_shell_logfile_name = logfile_name
# Make sure the script file is available
if not os.access(script, os.R_OK):
--
To view, visit
https://gerrit.osmocom.org/c/pysim/+/38543?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I09dc7e59d1a3dcb68f54e3a8dccb86a1bc6c9ee6
Gerrit-Change-Number: 38543
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>