[asterisk-dev] [Code Review] 4226: Testsuite: Add tests for external interactions with ARI/Stasis

opticron reviewboard at asterisk.org
Wed Dec 3 15:16:27 CST 2014


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

(Updated Dec. 3, 2014, 3:16 p.m.)


Review request for Asterisk Developers and Mark Michelson.


Changes
-------

Add comment on modifications in the python ARI library.


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


Repository: testsuite


Description (updated)
-------

This reorganizes existing tests that affect channels in Stasis() via external mechanisms and adds new tests to cover other interactions not yet represented in the testsuite. An additional, more comprehensive test for attended transfers has been added as well as tests for bridged and unbridged channels in Stasis() being affected by AMI Bridge and Redirect actions. The bug reported in ASTERISK-24591 was found while writing these tests.

This also modifies the ARI python libraries to allow parameters to have substituted variables.


Diffs
-----

  asterisk/trunk/tests/rest_api/tests.yaml 6018 
  asterisk/trunk/tests/rest_api/external_interaction/tests.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/blind_transfer/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/blind_transfer/configs/ast1/pjsip.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/blind_transfer/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/blind_transfer/blind_transfer.py PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/tests.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/swap_in/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/swap_in/sipp/referer.xml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/swap_in/sipp/referee.xml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/swap_in/configs/ast1/pjsip.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/swap_in/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/swap_in/attended_transfer.py PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/masq_in/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/masq_in/sipp/referer.xml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/masq_in/sipp/referee.xml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/masq_in/configs/ast1/pjsip.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/masq_in/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/attended_transfer/masq_in/attended_transfer.py PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_redirect/unbridged/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_redirect/unbridged/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_redirect/tests.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_redirect/bridged/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_redirect/bridged/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_bridge/unbridged/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_bridge/unbridged/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_bridge/tests.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_bridge/bridged/test-config.yaml PRE-CREATION 
  asterisk/trunk/tests/rest_api/external_interaction/ami_bridge/bridged/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/trunk/tests/rest_api/bridges/tests.yaml 6018 
  asterisk/trunk/tests/rest_api/bridges/blind_transfer/test-config.yaml 6018 
  asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/pjsip.conf 6018 
  asterisk/trunk/tests/rest_api/bridges/blind_transfer/configs/ast1/extensions.conf 6018 
  asterisk/trunk/tests/rest_api/bridges/blind_transfer/blind_transfer.py 6018 
  asterisk/trunk/tests/rest_api/bridges/attended_transfer/test-config.yaml 6018 
  asterisk/trunk/tests/rest_api/bridges/attended_transfer/sipp/referer.xml 6018 
  asterisk/trunk/tests/rest_api/bridges/attended_transfer/sipp/referee.xml 6018 
  asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/pjsip.conf 6018 
  asterisk/trunk/tests/rest_api/bridges/attended_transfer/configs/ast1/extensions.conf 6018 
  asterisk/trunk/tests/rest_api/bridges/attended_transfer/attended_transfer.py 6018 
  asterisk/trunk/lib/python/asterisk/ari.py 6018 

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


Testing
-------

Ran the tests and verified they passed with the expected events when the fix in 4213 is applied.


Thanks,

opticron

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


More information about the asterisk-dev mailing list