[asterisk-dev] [Code Review] 4090: testsuite: add basic valgrind support

Scott Griepentrog reviewboard at asterisk.org
Thu Oct 16 16:23:06 CDT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4090/
-----------------------------------------------------------

Review request for Asterisk Developers.


Repository: testsuite


Description
-------

This adds very basic valgrind support, which is convenient for manual test runs but does not (yet) include support for more automated valgrind usage. 

1) CLI flag '-V' enables valgrind (./runtests -V -t tests/test)

2) Minimal changes to testsuite, other improvements can be added later if desired

3) Valgrind output is picked up by error logging and shown after test run.

4) Unlike previous valgrind attempt, this one works fine on tests with multiple asterisk instances


Diffs
-----

  /asterisk/trunk/runtests.py 5733 
  /asterisk/trunk/lib/python/asterisk/test_case.py 5733 
  /asterisk/trunk/lib/python/asterisk/asterisk.py 5733 

Diff: https://reviewboard.asterisk.org/r/4090/diff/


Testing
-------


Thanks,

Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141016/adfeda56/attachment.html>


More information about the asterisk-dev mailing list