[asterisk-dev] [Code Review] 4283: Testsuite: Dual channel redirect tests

[Kevin Harwell]

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

(Updated Dec. 24, 2014, 1:01 p.m.)


Review request for Asterisk Developers.


Changes
-------

Added in some dependencies.


Bugs: ASTERISK-24536
    https://issues.asterisk.org/jira/browse/ASTERISK-24536


Repository: testsuite


Description
-------

Wrote three tests:

1) Redirect two channels that are in the Echo application, verify that they are both redirected to the appropriate dialplan location, make sure no surrogate channels are hanging around.

2) Redirect two channels that are bridged, verify that both channels are redirected to the same dialplan location, the bridge they were in gets destroyed, and neither channel is hung up by the process.

3) Redirect two channels that are in two different bridges, verify that both channels are redirected to the same dialplan location, the bridges they were in get destroyed, and neither channel is hung up by the process.

Also, added an option to the event action pluggable module instance that when set will not trigger the action until the count has been met on the event.


Diffs (updated)
-----

  /asterisk/trunk/tests/manager/tests.yaml 6126 
  /asterisk/trunk/tests/manager/redirect/tests.yaml PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/tests.yaml PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/bridge/tests.yaml PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/bridge/same/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/bridge/same/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/bridge/different/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/bridge/different/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/app/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/manager/redirect/dual/app/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/lib/python/asterisk/ami.py 6126 

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


Testing
-------

Ran tests and they passed.


Thanks,

Kevin Harwell

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