Updates:
        Labels: -Priority-High Priority-Critical NeedsReview asmeurer

Comment #5 on issue 1947 by asmeurer: @property methods are not doctested
http://code.google.com/p/sympy/issues/detail?id=1947

I figured it out.

From the commit message:

    The problem is that we are sorting our doctests by line number (the
    Python doctest.py sorts by name) so when the _find_lineno function
    couldn't find the line number for the docstring, we were just skipping
    the docstring.

    But @property docstrings are not found by this function.  For now, we
    just replace the lineno with a dummy value in this case, but eventually
    we will need to write our own version that is smarter, because it prints
    an incorrect line number whenever a doctest fails in a @property
    method's docstring.

    This also fixes a handful of docstrings that are now caught.
    Fortunately, there wasn't really anything that was incorrect, just
    things that weren't imported in the doctest.

So I have a fix at https://github.com/sympy/sympy/pull/68.

Whenever this gets in, this issue should remain open (but the priority lowered), because I didn't fix the issue where failing @property doctests' line numbers are given incorrectly (we need to write or own _find_lineno).

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" 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/sympy-issues?hl=en.

Reply via email to