On Thu, Oct 29, 2015 at 10:42 AM, Nathann Cohen <nathann.co...@gmail.com> wrote:
>> That's what
>>
>>     # long time
>>
>> is supposed to be for.
>
> I don't think so. You wouldn't want to see a doctest take one hour to
> run, even if it is flagged with 'long time'. I don't know about you,
> but I run 'long' doctests several time per day on some files.
>
>>> The same happens for some doctests that rely on optional LP solvers:
>>> they can take <1s with CPLEX/Gurobi, but 'standard sage' (with GLPK)
>>> cannot be expected to ever solve them.
>>
>> So if a user tries to use them they will be effectively broken?
>
> Broken? Why? They do return the expected output, they only do it very
> very slowly (and perhaps not at all) when you don't have "one of
> several solvers" installed. When you do, it can be instantaneous.
> Thus, it is probably better to not test them every time somebody types
> 'make ptestlong'.

There is no place for tests that cannot be automatically run somehow
in our test suite.   Absolutely any instance of this whatsoever is a
shortcoming in our test suite that we should *plan* to find a way to
address.  (I'm of course not saying we should delete/change everything
right now.)

If you have tests that take too long, then we need another tag "# really long".
If there are tests that create git branches or have other "undesirable
side effects", then get better at writing tests that properly run in a
sandbox that addresses such side effects.

William


-- 
William (http://wstein.org)

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to