#12415: Update doctesting framework
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
       Reporter:  robertwb                                                      
                                                                                
                                |         Owner:  jdemeyer      
           Type:  enhancement                                                   
                                                                                
                                |        Status:  needs_work    
       Priority:  major                                                         
                                                                                
                                |     Milestone:  sage-5.8      
      Component:  doctest                                                       
                                                                                
                                |    Resolution:                
       Keywords:                                                                
                                                                                
                                |   Work issues:                
Report Upstream:  N/A                                                           
                                                                                
                                |     Reviewers:  Jeroen Demeyer
        Authors:  David Roe, Robert Bradshaw                                    
                                                                                
                                |     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  |      Stopgaps:                
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+

Comment (by roed):

 Replying to [comment:193 jdemeyer]:
 > Could you please review [attachment:12415_doctest_review.patch]? It
 contains all my reviewer changes for doctests in the Sage library (i.e.
 all changes not for the doctest framework).

 I'm running tests on OS X 10.6.8 with sage 5.7.beta1.

 * Your changes to `sage.calculus.test_sympy` break doctests on my machine:
 they were correct before the change.  Presumably these results are system
 dependent and should be changed somehow.
 * I get a bad exit in sage.misc.interpreter at line 132 (in
 `get_test_shell()`): it prints out a bunch of help text from IPython,
 ending with
 {{{
 [SageTerminalApp] Bad config encountered during initialization:
 [SageTerminalApp] Unrecognized flag: '--all'
 }}}
 I'm not really sure what's happening here: perhaps it's related to the
 IPython upgrade?
 * The same thing happens in `sage.misc.sage_extension` at line 22.
 * There's still an error in `sagenb.notebook.worksheet` line 196:
 `sagenb.notebook` has no attribute `misc`.  I've seen this before and it
 should be easy to fix.
 * There are problems running some tests in `sage.doctest.control` that may
 go away with your new code.  If not I have fixes for them.

 How is [attachment:12415_review.patch] coming along?

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


Reply via email to