Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#14369 closed defect (fixed)

Misc doctesting improvements

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

Description

A collection of small patches to the doctesting framework.

Attachments (1)

14369_doctest_misc.patch (11.3 KB) - added by jdemeyer 6 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by roed

  • Component changed from doctest to doctest framework
  • Owner changed from mvngu to roed

comment:2 Changed 6 years ago by jdemeyer

  • Status changed from new to needs_review

Changed 6 years ago by jdemeyer

comment:3 Changed 6 years ago by roed

  • Reviewers set to David Roe
  • Status changed from needs_review to positive_review

Looks good to me.

comment:4 Changed 6 years ago by jdemeyer

  • Merged in set to sage-5.9.beta3
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:5 Changed 6 years ago by SimonKing

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 follow-up: Changed 6 years ago by jdemeyer

--optional=all works.

Last edited 6 years ago by jdemeyer (previous) (diff)

comment:7 Changed 6 years ago by jhpalmieri

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

comment:8 in reply to: ↑ 6 Changed 6 years ago by SimonKing

Replying to jdemeyer:

--optional=all works.

Great, thank you!

Note: See TracTickets for help on using tickets.