#6343: [with patch, needs work] Adds TestSuite(object).run() generic testing
framework
------------------------------------------+---------------------------------
Reporter: nthiery | Owner: nthiery
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-4.1.1
Component: doctest | Keywords: testunit
Reviewer: Mike Hansen, Minh Van Nguyen | Author: Nicolas M. Thiéry
Merged: |
------------------------------------------+---------------------------------
Comment(by nthiery):
Replying to [comment:13 mvngu]:
> The patch {{{trac_6343-reviewer.patch}}} fixes some typos and copy the
docstring for {{{__init__()}}} in the class {{{InstanceTester}}} to the
class itself. This is necessary as docstring in methods whose names begin
with an underscore (i.e. {{{_}}}) would not show up in the reference
manual.
Agreed, that's better (but not for that argument: my opinion is that _*
methods, or at least _*_ and __*__ methods should show up in the manual;
but that's another discussion).
Thanks for your doc fixes! I double checked them.
> Apart from that, here's a small issue. Can you provide examples and/or
tests for the function {{{_test_pickling()}}} in
{{{sage/structure/sage_object.pyx}}}? The doctest coverage for
{{{sage/structure/sage_object.pyx}}} is very short of 50% and we don't
want to make it more difficult to reach 100% coverage.
Thanks also for spotting the missing doctest. The attached patch fixes
this.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6343#comment:14>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
