#9222: improve doctest coverage of databases/conway.py
---------------------------+------------------------------------------------
   Reporter:  AlexGhitza   |       Owner:  tbd                       
       Type:  enhancement  |      Status:  needs_review              
   Priority:  minor        |   Milestone:  sage-4.4.4                
  Component:  doctest      |    Keywords:  conway polynomial database
     Author:  Alex Ghitza  |    Upstream:  N/A                       
   Reviewer:               |      Merged:                            
Work_issues:               |  
---------------------------+------------------------------------------------
Changes (by newvalueoldvalue):

  * status:  new => needs_review
  * author:  => Alex Ghitza


Comment:

 After the patch:

 {{{
 ERROR: Please add a `TestSuite(s).run()` doctest.
 SCORE conway.py: 85% (6 of 7)

 Missing documentation:
          * _init(self):
 }}}

 I'm not convinced a {{{TestSuite}}} test makes sense here (and I have
 tried to put one in and got a pickling-related error).  Also, I'm not sure
 what {{{_init(self)}}} is meant to be doing, so I'm leaving it alone for
 now.

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