[asterisk-dev] [Code Review] 2876: testsuite: Fix 200 OKs lacking SDPs on numerous tests

[jrose]

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

Review request for Asterisk Developers, Matt Jordan and Mark Michelson.


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


Repository: testsuite


Description
-------

https://reviewboard.asterisk.org/r/2827/
This patch was pulled following cascading test failures due to bad things happening when SIPp tests failed earlier. Since then the issues with lingering SIPp processes have been addressed and all that was left to be dealt with were the failures that this patch actually introduced.

It turned out all of them were due to the lack of SDPs in SIPp scenarios, which is what the patch was targeting anyway. As soon as these tests are altered to include SDPs, this patch will be able to go back in.


Diffs
-----

  /asterisk/trunk/tests/cdr/originate-cdr-disposition/sipp/wait-for-call.xml 4215 
  /asterisk/trunk/tests/channels/SIP/session_timers/uac_multiple_422_accept/sipp/uas-no-hangup.xml 4215 
  /asterisk/trunk/tests/channels/SIP/session_timers/uac_multiple_422_originate/sipp/uas-no-hangup.xml 4215 

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


Testing
-------

Ran each of these tests before and after the patch with r/2827 applied.  Failure confirmed without the patch, averted with the patch.

Every test that I was able to run on my test box between fax, CDR, and SIP successfully before the patch now runs successfully with the patch as long as these changes to the testsuite are in place. I haven't ventured too far outside of those areas though. Luckily it'll be a lot more obvious where problems are now that we no longer have to worry about lingering SIPp processes.


Thanks,

jrose

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