#2227: [with patch, positive review] sage-2.10.2.alpha1 -- doctest broken in
sageinspect.py because I added a new option
------------------------------------+---------------------------
       Reporter:  was               |        Owner:  mabshoff
           Type:  defect            |       Status:  closed
       Priority:  blocker           |    Milestone:  sage-2.10.2
      Component:  doctest coverage  |   Resolution:  fixed
       Keywords:                    |    Merged in:
        Authors:                    |    Reviewers:
Report Upstream:  N/A               |  Work issues:
         Branch:                    |       Commit:
   Dependencies:                    |     Stopgaps:
------------------------------------+---------------------------
Changes (by chapoton):

 * upstream:   => N/A


Old description:

> {{{
>          [8.8 s]
> sage -t  devel/sage-main/sage/misc/sageinspect.py
> **********************************************************************
> File "sageinspect.py", line 412:
>     sage: sage_getdef(sage.rings.integer.Integer.factor,
> obj_name='factor')
> Expected:
>     "factor(algorithm='pari', proof='True')"
> Got:
>     "factor(algorithm='pari', proof='True', limit='None')"
> **********************************************************************
> 1 items had failures:
>    1 of  24 in __main__.example_10
> ***Test Failed*** 1 failures.
>
> }}}
>
> FIX -- just change the doctest -- this makes perfect sense.
> (I added a new limit option to factor).

New description:

 {{{
  [8.8 s]
 sage -t  devel/sage-main/sage/misc/sageinspect.py
 **********************************************************************
 File "sageinspect.py", line 412:
     sage: sage_getdef(sage.rings.integer.Integer.factor,
 obj_name='factor')
 Expected:
     "factor(algorithm='pari', proof='True')"
 Got:
     "factor(algorithm='pari', proof='True', limit='None')"
 **********************************************************************
 1 items had failures:
    1 of  24 in __main__.example_10
 ***Test Failed*** 1 failures.

 }}}

 FIX -- just change the doctest -- this makes perfect sense.
 (I added a new limit option to factor).

--

--
Ticket URL: <http://trac.sagemath.org/ticket/2227#comment:5>
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