#12262: Bring Doctest coverage for sage/rings/finite_rings to 100%
------------------------------+---------------------------------------------
Reporter: roed | Owner: mvngu
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.0
Component: doctest | Keywords:
Work_issues: | Upstream: N/A
Reviewer: David Loeffler | Author: David Roe, David Loeffler
Merged: | Dependencies: #12062
------------------------------+---------------------------------------------
Changes (by newvalueoldvalue):
* reviewer: => David Loeffler
* author: David Roe => David Roe, David Loeffler
Comment:
Apply 12262-rebase.patch, 12262-review.patch
This needed some minor rebasing to apply to 5.0.beta8 (I think because of
#11900). Anyway, it's good stuff. I've uploaded my rebased version, and a
reviewer patch that doctests one file that somehow escaped, and makes a
minor, matter-of-taste change to the doctesting of {{{_repr_}}} for
homsets. If David or anyone else can sign off on my reviewer patch, then
we can give this a positive review.
With both patches installed, every file in {{{finite_rings}}} gets 100%
coverage except {{{integer_mod.pyx}}}, cf #12260.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12262#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.
