Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#14369 closed defect (fixed)

Misc doctesting improvements

Reported by: Jeroen Demeyer Owned by: David Roe
Priority: minor Milestone: sage-5.9
Component: doctest framework Keywords:
Cc: David Roe Merged in: sage-5.9.beta3
Authors: Jeroen Demeyer Reviewers: David Roe
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

A collection of small patches to the doctesting framework.

Attachments (1)

14369_doctest_misc.patch (11.3 KB) - added by Jeroen Demeyer 10 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 10 years ago by David Roe

Component: doctestdoctest framework
Owner: changed from Minh Van Nguyen to David Roe

comment:2 Changed 10 years ago by Jeroen Demeyer

Status: newneeds_review

Changed 10 years ago by Jeroen Demeyer

Attachment: 14369_doctest_misc.patch added

comment:3 Changed 10 years ago by David Roe

Reviewers: David Roe
Status: needs_reviewpositive_review

Looks good to me.

comment:4 Changed 10 years ago by Jeroen Demeyer

Merged in: sage-5.9.beta3
Resolution: fixed
Status: positive_reviewclosed

comment:5 Changed 10 years ago by Simon King

For the record: Running sage -t -optional -long on a .py file does not work anymore, but used to work under sage-5.8. I tried to change this into sage -t --optional --long, but it does not work either.

It would be less disappointing if the changes in the doctesting framework would somehow be documented, or if at least the doctesting framework would give a hint what it is complaining about. All I get is

Running doctests with ID 2013-04-26-16-31-00-2eee1fc7.
Doctesting 1 file.
sage -t /home/simon/.sage/temp/linux-sqwp.site/11900/dir_vpH6Jb/file_5.py
    [0 tests, 0.00 s]
----------------------------------------------------------------------
All tests passed!
----------------------------------------------------------------------
Total time for all tests: 0.0 seconds
    cpu time: 0.0 seconds
    cumulative wall time: 0.0 seconds

i.e., the tests are not even attempted to run, but it is reported that the tests pass.

comment:6 Changed 10 years ago by Jeroen Demeyer

--optional=all works.

Last edited 10 years ago by Jeroen Demeyer (previous) (diff)

comment:7 Changed 10 years ago by John Palmieri

I guess running sage -t -optional=all -long ... might be a solution.

comment:8 in reply to:  6 Changed 10 years ago by Simon King

Replying to jdemeyer:

--optional=all works.

Great, thank you!

Note: See TracTickets for help on using tickets.