#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)
Change History (9)
comment:1 Changed 7 years ago by
- Component changed from doctest to doctest framework
- Owner changed from mvngu to roed
comment:2 Changed 7 years ago by
- Status changed from new to needs_review
Changed 7 years ago by
comment:3 Changed 7 years ago by
- Reviewers set to David Roe
- Status changed from needs_review to positive_review
comment:4 Changed 7 years ago by
- Merged in set to sage-5.9.beta3
- Resolution set to fixed
- Status changed from positive_review to closed
comment:5 Changed 7 years ago by
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: ↓ 8 Changed 7 years ago by
Obviously, you need to post the .py
file that you're testing.
comment:7 Changed 7 years ago by
I guess running sage -t -optional=all -long ...
might be a solution.
Looks good to me.