On Tue, Jul 28, 2009 at 5:17 AM, Vinzent
Steinberg<vinzent.steinb...@googlemail.com> wrote:
> 2009/7/28 Vinzent Steinberg <vinzent.steinb...@googlemail.com>:
>> 2009/7/27 Ondrej Certik <ond...@certik.cz>:
>>>
>>> +    else:
>>> +        paths = [p for p in glob.glob('sympy/*/tests/test_*.py')
>>> +                 if any(a in p for a in args)]
>>> +        t.add_paths(paths)
>>>
>>> ^^ Is this going to work for more nested tests, like
>>> sympy/core/assumptions/tests?
>>>
>>> If so, then +1.
>>
>> Yeah, it should, because '*' also matches '/'.
>
> No, sadly '*' does not match '/'. I changed it to include
> sympy/*/tests/test_*.py and sympy/*/*/tests/test_*.py, this covers all
> tests currently in sympy. Is this sufficient? (This limitation does
> not apply for running the tests without arguments.)

So in this case, just use the get_paths() method. That way all the
hacks are concentrated on one place.


Ondrej

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sympy-patches" group.
To post to this group, send email to sympy-patches@googlegroups.com
To unsubscribe from this group, send email to 
sympy-patches+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/sympy-patches?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to