[asterisk-dev] [Code Review] 2505: Add tests for new Bridging-related AMI actions and events

opticron reviewboard at asterisk.org
Tue May 7 08:14:00 CDT 2013


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

(Updated May 7, 2013, 8:14 a.m.)


Review request for Asterisk Developers.


Changes
-------

Fix red blobs.


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


Repository: testsuite


Description
-------

Modify BridgeTestCase to handle the events generated by the new bridging architecture and get the simple_bridge test into a working condition with additional tests for AMI Bridge* events.  The CEL tests have been commented out for the time being since the CEL log is missing events that will be restored when the Stasis-Core CEL work is completed.

This also adds a new test (tests/manager/bridge_actions) based on a simplified simple_bridge test that tests bridge-related AMI actions BridgeInfo and BridgeList to ensure proper functionality.

This code will remain in testsuite/asterisk/team/group/bridge_construction until bridge_construction is merged into Asterisk trunk.


Diffs (updated)
-----

  asterisk/trunk/lib/python/asterisk/BridgeTestCase.py 3764 
  asterisk/trunk/tests/bridge/simple_bridge/test-config.yaml 3764 
  asterisk/trunk/tests/manager/bridge_actions/check_bridge.py PRE-CREATION 
  asterisk/trunk/tests/manager/bridge_actions/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/manager/bridge_actions/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/manager/tests.yaml 3764 

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


Testing
-------

Verified that test event logs were as expected for these tests.


Thanks,

opticron

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


More information about the asterisk-dev mailing list