[asterisk-dev] [Code Review] A pile of Dial tests for Asterisk 12

[Paul Belanger]

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


I see a lot of duplicate extensions.conf and sip.conf files.  Might I suggest either setting up symlink for the ast1 folders, or do what we do for the fastagi tests.  This should help cut down the administrative tasks related to maintaining the testsuite.

- Paul


On March 15, 2013, 4:34 p.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2384/
> -----------------------------------------------------------
> 
> (Updated March 15, 2013, 4:34 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> For the CDR work being done for Asterisk 12, we need channel information as it pertains to Dialing to be pushed over the Stasis-Core message bus. As well, AMI (per the specification here https://wiki.asterisk.org/wiki/display/AST/AMI+1.4+Specification) requires knowledge of dialing to be able to raise the corresponding DialBegin/DialEnd events.
> 
> While the CDR work isn't ready yet for testing, AMI is much easier to port over to Stasis-Core - see Review https://reviewboard.asterisk.org/r/2383. These tests can be extended to verify any functionality that consumes the state of channels during a Dial operation - CDRs, CEL, etc.
> 
> This review provides a number of tests to cover Dial operations and the AMI events they produce. The following scenarios are covered:
> 
> -- Single Party --
> 
> * Dialing an endpoint and having it Answer
> * Dialing an endpoint and having it reply back with a Busy
> * Dialing an endpoint and having it be Unavailable
> * Dialing an endpoint and having it reply back as Congested
> * Dialing an endpoint and having the operation time out
> * Dialing an endpoint and having the caller press a DTMF key that hangs them up, cancelling the dial operation
> 
> -- Multi Party --
> 
> * Dialing two endpoints, having one Answer and the verifying that the other one gets Cancelled
> * Dialing two endpoints, having both be Busy/one Answer and the other be Busy
> * Dialing two endpoints, having both be Congested/one Answer and the other be Congested
> * Dialing two endpoints, having both be Unavailable/one Answer and the other be Unavailable
> * Dialing two endpoints and having the caller press a DTMF key that hangs them up, cancelling both dial operations
> 
> 
> This addresses bug ASTERISK-21196.
>     https://issues.asterisk.org/jira/browse/ASTERISK-21196
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/lib/python/asterisk/PluggableModules.py 3666 
>   /asterisk/trunk/lib/python/asterisk/ami.py 3666 
>   /asterisk/trunk/tests/apps/dial/dial_answer/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_answer/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_answer/sipp/charlie.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_answer/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_busy/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_busy/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_busy/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_busy/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_congestion/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_congestion/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_congestion/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_congestion/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_dtmf_hangup_cancel/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_dtmf_hangup_cancel/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_dtmf_hangup_cancel/send_dtmf.py PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_dtmf_hangup_cancel/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_dtmf_hangup_cancel/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_no_answer/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_no_answer/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_no_answer/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_no_answer/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_busy/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_busy/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_busy/sipp/ua_busy.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_busy/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_cancel/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_cancel/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_cancel/send_dtmf.py PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_cancel/sipp/ua_wait_cancel.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_cancel/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_congestion/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_congestion/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_congestion/sipp/ua_congestion.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_congestion/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_no_answer/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_no_answer/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_no_answer/sipp/ua_no_answer.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_no_answer/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_unavail/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_unavail/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_unavail/sipp/ua_unavail.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_all_unavail/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_answer_cancel/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_answer_cancel/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_answer_cancel/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_answer_cancel/sipp/charlie.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_answer_cancel/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_busy/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_busy/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_busy/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_busy/sipp/charlie.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_busy/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_congestion/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_congestion/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_congestion/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_congestion/sipp/charlie.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_congestion/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_no_answer/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_no_answer/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_no_answer/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_no_answer/sipp/charlie.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_no_answer/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_unavail/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_unavail/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_unavail/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_unavail/sipp/charlie.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/dial_parallel_single_unavail/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_parallel/tests.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_timeout/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_timeout/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_timeout/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_timeout/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_unavail/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_unavail/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_unavail/sipp/bob.xml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/dial_unavail/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/dial/tests.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/tests.yaml 3666 
> 
> Diff: https://reviewboard.asterisk.org/r/2384/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matt
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130316/e4ccce7d/attachment-0001.htm>