[asterisk-dev] [Code Review] 2852: Convert the IAX2 basic call test to use pluggable modules; update for Asterisk 12

[opticron]

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

Ship it!


- opticron


On Sept. 13, 2013, 9:45 a.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2852/
> -----------------------------------------------------------
> 
> (Updated Sept. 13, 2013, 9:45 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22219
>     https://issues.asterisk.org/jira/browse/ASTERISK-22219
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> This patch does the following:
> 
> * It updates CDR parsing/checking to handle checking CDR entries across multiple instances of Asterisk. This is backwards compatible with existing CDR checking.
> * It adds a new pluggable module that will automagically end a test once the last channel has hung up. This is useful when using the 'bare bones' test object TestCase.TestCaseModule, which doesn't by itself provide a mechanism for ending a test.
> * It updates the IAX2 basic call test to use all this, and checks for CDRs/CEL entries in Asterisk 12. (It's kind of hard to reliably check for CEL entries prior to this version, so I've punted the Asterisk 1.8/11 CEL checking)
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/lib/python/asterisk/PluggableModules.py 4182 
>   /asterisk/trunk/lib/python/asterisk/cdr.py 4182 
>   /asterisk/trunk/tests/channels/iax2/basic-call/configs/ast1/extensions.conf 4182 
>   /asterisk/trunk/tests/channels/iax2/basic-call/run-test 4182 
>   /asterisk/trunk/tests/channels/iax2/basic-call/test-config.yaml 4182 
> 
> Diff: https://reviewboard.asterisk.org/r/2852/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matt Jordan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130926/6b2e5da9/attachment-0001.html>