[asterisk-dev] [Code Review] 2770: TestSuite: ARI Bridge Tests

David Lee reviewboard at asterisk.org
Tue Aug 27 14:32:33 CDT 2013


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

(Updated Aug. 27, 2013, 2:32 p.m.)


Review request for Asterisk Developers.


Changes
-------

* Merged some changes from Matt's latest patch
* Removed vestigial code
* Replaced nasty caller line printing with simply throwing an exception


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


Repository: testsuite


Description
-------

TestSuite tests that test the expected behavior for adding/removing
channel to/from bridges using ARI.


Diffs (updated)
-----

  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/tests.yaml 4089 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/unhappy/test-config.yaml PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/unhappy/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/unhappy/bridge_unhappy.py PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/tests.yaml PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/move/test-config.yaml PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/move/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/move/bridge_move.py PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/lib/python/asterisk/TestCase.py 4089 
  /asterisk/team/dlee/matts-ari-patch/lib/python/asterisk/TestConfig.py 4089 
  /asterisk/team/dlee/matts-ari-patch/lib/python/asterisk/ari.py 4089 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/hangup/bridge_hangup.py PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/hangup/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/hangup/test-config.yaml PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/happy/bridge_happy.py PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/happy/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/team/dlee/matts-ari-patch/tests/rest_api/bridges/happy/test-config.yaml PRE-CREATION 

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


Testing
-------

Tests pass.

https://reviewboard.asterisk.org/r/2769/


Thanks,

David Lee

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


More information about the asterisk-dev mailing list