[asterisk-dev] [Code Review] Add the ability to define asterisk module dependencies from a test.

[mjordan]

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

Ship it!



asterisk/trunk/runtests.py
<https://reviewboard.asterisk.org/r/1366/#comment8025>

    As a potential future improvement, it may be worth caching this Asterisk() instance so that the cost of creating it is only incurred once.


- mjordan


On Aug. 16, 2011, 4:10 p.m., Paul Belanger wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1366/
> -----------------------------------------------------------
> 
> (Updated Aug. 16, 2011, 4:10 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Our tests blindly assume the host machine has the required modules for a test, this is not always the case.  So, I've added a simple dependency check for a test writer to define which asterisk modules their test requires.
> 
> 
> Diffs
> -----
> 
>   asterisk/trunk/runtests.py 1857 
>   asterisk/trunk/tests/pbx/pbx_lua_background/test-config.yaml 1857 
> 
> Diff: https://reviewboard.asterisk.org/r/1366/diff
> 
> 
> Testing
> -------
> 
> local development box.
> 
> 
> Thanks,
> 
> Paul
> 
>

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