[asterisk-dev] [Code Review] 3060: pjsip tests: interactions with chan_sip cause test failures

Mark Michelson reviewboard at asterisk.org
Mon Dec 9 14:27:32 CST 2013


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


I think this will work well for the testsuite. My only suggestion would be a way to override default conflict behavior if desired. It may be that we wish to write some sort of test where we transfer a call between a chan_sip endpoint and a chan_pjsip endpoint or some such. In such a rare circumstance, it would be a good idea to be able to state that the conflict does not apply for this test.

The other question I have is if you plan to go through the tests that use chan_sip and chan_pjsip and add the appropriate deps to the properties in their test-config.yaml files. Were you holding off on that in case this approach was vetoed?

- Mark Michelson


On Dec. 9, 2013, 7:54 p.m., Kevin Harwell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3060/
> -----------------------------------------------------------
> 
> (Updated Dec. 9, 2013, 7:54 p.m.)
> 
> 
> Review request for Asterisk Developers and Matt Jordan.
> 
> 
> Bugs: ASTERISK-22887
>     https://issues.asterisk.org/jira/browse/ASTERISK-22887
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> While chan_sip and chan_pjsip can be loaded and used at the same time it was causing problems in the testsuite because their defaults try to bind to the same host:port.  So for the testsuite made is so any test that depends on res_pjsip will "noload" chan_sip.  Instead of manipulating every current res_pjsip test (through their own modules.conf or config setting), this fix checks to see if the test requires/depends on res_pjsip and if it does it appends a noload => chan_sip.so to the copied modules.conf for that test.
> 
> This functionality is configured via the newly added conflicts.txt file.  Any new conflicts that may arise hopefully can resuse the mechanism.
> 
> 
> Diffs
> -----
> 
>   asterisk/trunk/lib/python/asterisk/asterisk.py 4415 
>   asterisk/trunk/lib/python/asterisk/TestConfig.py 4415 
>   asterisk/trunk/lib/python/asterisk/TestCase.py 4415 
>   asterisk/trunk/configs/modules.conf PRE-CREATION 
>   asterisk/trunk/configs/conflicts.txt PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/3060/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Kevin Harwell
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131209/b6ed7d18/attachment.html>


More information about the asterisk-dev mailing list