#11937: Doctest fails in gap_reps.py when Kash is installed
----------------------------+-----------------------------------------------
   Reporter:  dkrenn        |          Owner:  mvngu                  
       Type:  defect        |         Status:  positive_review        
   Priority:  trivial       |      Milestone:  sage-4.8               
  Component:  doctest       |       Keywords:  doctest, gap_reps, kash
Work_issues:                |       Upstream:  N/A                    
   Reviewer:  Johan Bosman  |         Author:  Daniel Krenn           
     Merged:                |   Dependencies:                         
----------------------------+-----------------------------------------------
Changes (by johanbosman):

  * keywords:  doctest, gap_reps => doctest, gap_reps, kash
  * status:  needs_review => positive_review


Old description:

> The doctest in gap_reps.py
>
> {{{
> EllipticCurve([Integer(1),-Integer(1),Integer(1),-Integer(2680),-Integer(50053)]).galois_representation().image_type(Integer(7))   
> # long time
> }}}
> fails by doctest -long because
>
> {{{
> Expected:
>     'The image is a group of order 18.'
> Got:
>     'The image is a non-abelian group of order 18.'
> }}}
> That error is hided by the doctest command mentioned in #11936, which
> produces a timeout.
>
> One could think about an extra error message saying that there was an
> error '''and''' a timeout

New description:

 The doctest in gap_reps.py

 {{{
 
EllipticCurve([Integer(1),-Integer(1),Integer(1),-Integer(2680),-Integer(50053)]).galois_representation().image_type(Integer(7))   
 # long time
 }}}
 fails by doctest -long when Kash is installed:

 {{{
 Expected:
     'The image is a group of order 18.'
 Got:
     'The image is a non-abelian group of order 18.'
 }}}
 That error is hidden by the doctest command mentioned in #11936, which
 produces a timeout.

 Apply [attachment:trac_11937_gal_reps_doctest_fail_wrong_output.4.patch]
 to the Sage library.

--

Comment:

 I've tested it with and without Kash installed (in the former case with a
 fix of #11936).  Everything works perfectly.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11937#comment:14>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to