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

Matt Jordan reviewboard at asterisk.org
Tue Dec 10 13:22:28 CST 2013


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



asterisk/trunk/configs/modules.conf
<https://reviewboard.asterisk.org/r/3060/#comment19768>

    Before you go....
    
    Supplying a default modules.conf means you should also disable the console channel drivers. Otherwise this will result in a lot of tests failing on the build agents.
    
    I would go ahead and noload chan_alsa, chan_oss, and chan_console. I'm pretty sure all of the old console tests no longer use chan_console.
    


- Matt Jordan


On Dec. 9, 2013, 10:52 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, 10:52 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/20131210/1f7601b6/attachment.html>


More information about the asterisk-dev mailing list