#4706: fix race condition between doctest creation and running
----------------------------+----------------------------------------------
       Reporter:  mabshoff  |        Owner:  mabshoff
           Type:  defect    |       Status:  closed
       Priority:  major     |    Milestone:  sage-duplicate/invalid/wontfix
      Component:  doctest   |   Resolution:  invalid
  coverage                  |    Merged in:
       Keywords:            |    Reviewers:
        Authors:            |  Work issues:
Report Upstream:  N/A       |       Commit:
         Branch:            |     Stopgaps:
   Dependencies:            |
----------------------------+----------------------------------------------
Description changed by chapoton:

Old description:

> With high -tp numbers (i.e. 16) on sage.math one will see similar issues
> to the one below:
> {{{
> sage -t -long devel/sage/sage/libs/symmetrica/symmetrica.pyx
>   File "/scratch/mabshoff/release-
> cycle/sage-3.2.alpha0/tmp/.doctest_symmetrica.py", line 29
>     >>> test_integer(Integer(1))###line 539:_sage_    >>> test_integer(1)
>     ^
> IndentationError: unexpected indent
> }}}
>
> This is likely a race condition between creating the file and the actual
> running of the doctest. The issue is not specific to -tp.
>
> A potential solution might be to create all .doctest_$FOO files and then
> start running them. This might also fix the problem with
> {{{
> sage -t -long devel/sage/sage/symbolic/constants.pyx
>          [0.1 s]
> }}}
> in Sage 3.2.1+ which is caused by no doctests being executed since (a)
> either there are no doctests in that file or (b) we are running optional
> doctests only.
>
> Cheers,
>
> Michael
>

> don't apply anything!

New description:

 With high -tp numbers (i.e. 16) on sage.math one will see similar issues
 to the one below:
 {{{
 sage -t -long devel/sage/sage/libs/symmetrica/symmetrica.pyx
   File "/scratch/mabshoff/release-
 cycle/sage-3.2.alpha0/tmp/.doctest_symmetrica.py", line 29
     >>> test_integer(Integer(1))###line 539:_sage_    >>> test_integer(1)
     ^
 IndentationError: unexpected indent
 }}}

 This is likely a race condition between creating the file and the actual
 running of the doctest. The issue is not specific to -tp.

 A potential solution might be to create all .doctest_$FOO files and then
 start running them. This might also fix the problem with
 {{{
 sage -t -long devel/sage/sage/symbolic/constants.pyx
  [0.1 s]
 }}}
 in Sage 3.2.1+ which is caused by no doctests being executed since (a)
 either there are no doctests in that file or (b) we are running optional
 doctests only.

 Cheers,

 Michael


 don't apply anything!

--

--
Ticket URL: <http://trac.sagemath.org/ticket/4706#comment:4>
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.
For more options, visit https://groups.google.com/d/optout.

Reply via email to