[asterisk-dev] [Code Review] 2482: Add tests for SDP SRTP negotiation

[opticron]

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

(Updated April 29, 2013, 3:58 p.m.)


Review request for Asterisk Developers.


Changes
-------

Added another test to cover an inverted scenario.


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


Repository: testsuite


Description (updated)
-------

Add tests for SRTP negotiation in SDP for cases where:
* Call declined
** Encryption enabled
*** Unknown cryptographic suite in offer
*** Missing crypto attribute in offer
*** Encryption not offered (attribute present)
** Encryption not enabled
* Call accepted
** Encryption not enabled, crypto attribute present and ignored
** Encryption enabled


Diffs (updated)
-----

  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/configs/ast1/extensions.conf PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/configs/ast1/modules.conf PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/configs/ast1/res_sip.conf PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/sipp/accept_disabled_with_attrib.xml PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/sipp/accept_nominal.xml PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/sipp/decline_enabled.xml PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/sipp/decline_no_attrib.xml PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/sipp/decline_not_enabled.xml PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/sipp/decline_unknown_suite.xml PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/srtp_negotiation/test-config.yaml PRE-CREATION 
  asterisk/team/group/pimp_my_sip/tests/channels/gulp/tests.yaml 3735 

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


Testing
-------


Thanks,

opticron

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