[asterisk-dev] [Code Review]: fix that ./runtests.py -t something doesn't match something_else

[opticron]

> On Feb. 24, 2012, 8:37 a.m., Matt Jordan wrote:
> > Ironically, that's one of the things I *like* about that command-line switch.  Its handy to be able to run a subset of tests that fit within a particular genre.  Particularly things like run all SIP tests:
> > 
> > ./runtests.py --test=tests/channels/SIP
> > 
> > I'm fine with changing this to run just one test, but I'd prefer there to still be an option that does matching.

If this patch is committed, we can always use the tag system to run subsets of tests.  The downside is that not all tests are tagged properly and some non-trivial amount of time will be required to ensure that they are.


- opticron


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


On Feb. 24, 2012, 3:02 a.m., wdoekes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1771/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2012, 3:02 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> ./runtests.py -t tests/channels/SIP/sip_register
> 
> matched 
> 
> tests/channels/SIP/sip_register
> tests/channels/SIP/sip_register_domain_acl
> 
> with patch it only matches
> 
> tests/channels/SIP/sip_register
> 
> 
> (The whole -t bit could use a bit of refactoring, this is just a quick-fix.)
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/runtests.py 3064 
> 
> Diff: https://reviewboard.asterisk.org/r/1771/diff
> 
> 
> Testing
> -------
> 
> tests/channels/SIP
> 
> still matches all SIP tests.
> 
> 
> Thanks,
> 
> wdoekes
> 
>

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