#12415: Update doctesting framework
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
Reporter: robertwb
|
Owner: jdemeyer
Type: enhancement
|
Status: needs_review
Priority: major
|
Milestone: sage-5.8
Component: doctest
|
Resolution:
Keywords:
|
Work issues:
Report Upstream: N/A
|
Reviewers: Jeroen Demeyer, David Roe
Authors: David Roe, Robert Bradshaw, Jeroen Demeyer
|
Merged in:
Dependencies: #13147,#13146, #13145, #12723, #12392, #12393, #12395,
#12396, #12397, #12381, #12382, #12383, #12384, #11871, #13195, #13121, #13748,
#13899, #12719, #5155, #14070, #14079, #14150, #14158, #14182, #14184, #14054
| Stopgaps:
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
Comment (by jdemeyer):
Sorry, too fast. This is dangerous:
{{{
sage: W.start()
sage: W.join()
sage: while W.rmessages is not None:
....: W.read_messages()
sage: len(W.messages) > 0
True
}}}
You should read the messages ''while'' the worker is running, because
otherwise the pipe buffer might get full, leading to a block. So put
`W.join()` at the end instead of at the beginning. Apart from that, it's a
useful and good test.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12415#comment:249>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
