[asterisk-dev] [Code Review] Testsuite: Disallow MoH upon hold option

Kevin Harwell reviewboard at asterisk.org
Mon Feb 18 14:30:39 CST 2013


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

Review request for Asterisk Developers.


Summary
-------

Tests to make sure the music on hold event is not triggered if the "discard_remote_hold_retrieval" option is set to "yes".  In the test multiple scenarios are ran where one SIP phone puts another SIP phone on hold by sending a re-INVITE with a modified SDP containing either a restricted audio direction, an IP address of 0.0.0.0, or a combination thereof.  This is tested both for a local RTP bridge, and a non-bridged scenario.


This addresses bug ABE-2899.
    https://issues.asterisk.org/jira/browse/ABE-2899


Diffs
-----

  /asterisk/trunk/tests/channels/SIP/sip_hold_no_moh/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/sip_hold_no_moh/configs/ast1/sip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/sip_hold_no_moh/sipp/phone_A.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/sip_hold_no_moh/sipp/phone_B_IP_media_restrict.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/sip_hold_no_moh/sipp/phone_B_IP_restrict.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/sip_hold_no_moh/sipp/phone_B_media_restrict.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/sip_hold_no_moh/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/tests.yaml 3642 

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


Testing
-------

Ran the test and made sure all scenarios passed.  Also set the discard_remote_hold_retrieval to "no" and made sure the test failed. 


Thanks,

Kevin

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


More information about the asterisk-dev mailing list