sagemath/sage

Make misc doctests ready for random seeds

Closed this issue · 10 comments

kliem commented

This ticket makes

sage -t --long --random-seed=n src/sage/misc/

pass for different values n than just 0.

CC: @DaveWitteMorris

Component: doctest framework

Author: Jonathan Kliem

Branch/Commit: bcd0654

Reviewer: Marc Mezzarobba

Issue created by migration from https://trac.sagemath.org/ticket/29969

kliem commented
comment:1

At least the following need fixes

sage -t --long --random-seed=151058820726654196682836430928254760259 src/sage/misc/misc.py  # 4 doctests failed
sage -t --long --random-seed=151058820726654196682836430928254760259 src/sage/misc/prandom.py  # 35 doctests failed
sage -t --long --random-seed=151058820726654196682836430928254760259 src/sage/misc/randstate.pyx  # 2 doctests failed
sage -t --long --random-seed=151058820726654196682836430928254760259 src/sage/misc/sage_unittest.py  # 2 doctests failed
kliem commented

Commit: bcd0654

kliem commented

Author: Jonathan Kliem

kliem commented

Branch: public/29969

kliem commented

Changed dependencies from #29962 to none

kliem commented

New commits:

bcd0654make sage/misc ready for implicit fuzzuing of doctests
mezzarobba commented
comment:4

Looks good (and very thorough) to me, thank you!

mezzarobba commented

Reviewer: Marc Mezzarobba

kliem commented
comment:5

Thank you for reviewing.

vbraun commented

Changed branch from public/29969 to bcd0654