[asterisk-commits] Expand checks for core dumps. (testsuite[master])

[SVN commits to the Asterisk project]

Matt Jordan has submitted this change and it was merged.

Change subject: Expand checks for core dumps.
......................................................................


Expand checks for core dumps.

Tests using lua (asttest) change to the test directory.  This causes
any core dumps from Asterisk to be placed there instead of the
testsuite root directory as expected.  Running with the -V option
produces a file named vgcore.${PID}.  These variations of name and
location are now recognized and processed.

Change-Id: Idf83bff672cb4c3559214864498a9baf404c1fc5
---
M runtests.py
1 file changed, 9 insertions(+), 2 deletions(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/runtests.py b/runtests.py
index 17d7ff7..bf773e9 100755
--- a/runtests.py
+++ b/runtests.py
@@ -150,11 +150,18 @@
         self.time = time.time() - start_time
 
     def _check_for_core(self):
-        contents = os.listdir('.')
         core_files = []
+
+        contents = os.listdir('.')
         for item in contents:
-            if item.startswith('core'):
+            if item.startswith('core') or item.startswith('vgcore'):
                 core_files.append(item)
+
+        contents = os.listdir(self.test_name)
+        for item in contents:
+            if item.startswith('core') or item.startswith('vgcore'):
+                core_files.append(os.path.join(self.test_name, item))
+
         return core_files
 
     def _archive_core_dumps(self, core_dumps):

-- 
To view, visit https://gerrit.asterisk.org/553
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idf83bff672cb4c3559214864498a9baf404c1fc5
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>