eclib: fix doctests after 20240408 update

sagemath / sage

Main repository of SageMath. Now open for Issues and Pull Requests.

https://www.sagemath.org

Other

1.08k stars 394 forks source link

eclib: fix doctests after 20240408 update #37882

Closed tornaria closed 1 week ago

tornaria commented 2 weeks ago

With the update to 20240408 and later, the output of eclib has some extra information that makes some doctests fail. This PR adds some ... in the right spots so the tests still pass after the update, without affecting previous versions of eclib

:memo: Checklist

[x] The title is concise and informative.
[x] The description explains in detail what this PR is about.

tornaria commented 2 weeks ago

Sorry about those dots ..

Nothing to be sorry about. The new mwrank adds output like

Checking saturation at [ 2 3 ]

which is just more diagnostic information. It's unfortunate that sagemath doctests are so sensitive to output changes.

For the record: the three dots ... I added are just a wildcard to match anything, so the doctest works ok on old eclib and on new eclib.

Thanks for the quick review!

github-actions[bot] commented 2 weeks ago

Documentation preview for this PR (built with commit bfcd3aef1d789211d7daaadfd58b6861c0a5e934; changes) is ready! :tada: This preview will update shortly after each push to this PR.