[asterisk-dev] [Code Review] TestSuite: Add tests for codec negotiation

[mjordan]

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

Ship it!


The only test I don't see is image combined with some other stream type.  Not sure if that's possible or easy to set up, but that would probably cover all eventualities.


asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/multistream.xml
<https://reviewboard.asterisk.org/r/1520/#comment8729>

    Why is this optional?  I would have thought that sending two stream types we should expect a 488 at some point.
    
    If this is required, apply this comment to the other tests as well.


I'm assuming you've tested this with the existing udptl / t38 tests as well

- mjordan


On Oct. 12, 2011, 1:09 p.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1520/
> -----------------------------------------------------------
> 
> (Updated Oct. 12, 2011, 1:09 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Add codec and stream negotiation tests for varying single stream, incompatible stream, and multistream situations. This requires the changes in https://reviewboard.asterisk.org/r/1516/ to pass since it corrects several issues with how Asterisk deals with SDP.
> 
> 
> Diffs
> -----
> 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/configs/ast1/extensions.conf PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/configs/ast1/sip.conf PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/run-test PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/avt_streams.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/decline_incompat_audio.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/decline_incompat_text.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/decline_incompat_video.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/multistream.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/orderstream.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/single_audio.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/single_image.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/single_image_inverse.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/single_text.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/single_text_inverse.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/single_video.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/sipp/single_video_inverse.xml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/codec_negotiation/test-config.yaml PRE-CREATION 
>   asterisk/trunk/tests/channels/SIP/tests.yaml 2515 
> 
> Diff: https://reviewboard.asterisk.org/r/1520/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> opticron
> 
>

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