Opened 12 years ago

Last modified 11 years ago

#7384 closed defect

SageNB -- Fix Sphinxify doctests — at Version 4

Reported by: timdumol Owned by: boothby
Priority: major Milestone: sage-4.3
Component: notebook Keywords:
Cc: mpatel Merged in:
Authors: Tim Dumol Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by mpatel)'s doctests currently fail. Fix this.

See also: #7367.

Change History (6)

comment:1 Changed 12 years ago by timdumol

  • Status changed from new to needs_review

Changed 12 years ago by timdumol

Fixed the doctests

comment:2 Changed 12 years ago by mpatel

I got two test failures:

sage -t  "4.2/devel/sage-main/sage/"            
File "/home/apps/sage-4.2/devel/sage-main/sage/", line 51:
    sage: sphinxify('A test')
    '\n<div class="docstring">\n    \n  <p>A test</p>\n\n\n</div>'
    '<div class="docstring">\n    \n  <p>A test</p>\n\n\n</div>'
File "/home/apps/sage-4.2/devel/sage-main/sage/", line 53:
    sage: sphinxify('**Testing**\n`monospace`')
    '\n<div class="docstring"...<strong>Testing</strong>\n<span class="math"...</p>\n\n\n</div>'
    '<div class="docstring">\n    \n  <p><strong>Testing</strong>\n<span class="math">monospace</span></p>\n\n\n</div>'
1 items had failures:
   2 of   5 in __main__.example_2
***Test Failed*** 2 failures.

But it could be my setup. If not, please see version 2 of the patch.

Changed 12 years ago by mpatel

Update doctest outputs. Apply only this patch.

comment:3 Changed 12 years ago by mpatel

To the extent it counts, my review is positive.

comment:4 Changed 11 years ago by mpatel

  • Description modified (diff)
Note: See TracTickets for help on using tickets.