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

[Paul Belanger]

> On Aug. 16, 2011, 7:10 p.m., mjordan wrote:
> > asterisk/trunk/runtests.py, line 127
> > <https://reviewboard.asterisk.org/r/1366/diff/2/?file=18455#file18455line127>
> >
> >     As a potential future improvement, it may be worth caching this Asterisk() instance so that the cost of creating it is only incurred once.

/me nods.  Next patch.


- Paul


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


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/dab7fa40/attachment.htm>