Opened 3 years ago

Closed 3 years ago

#25474 closed defect (fixed)

Few internet doctests failed in 6 files

Reported by: slabbe Owned by:
Priority: major Milestone: sage-8.3
Component: doctest coverage Keywords:
Cc: Merged in:
Authors: Sébastien Labbé Reviewers: Julian Rüth
Report Upstream: N/A Work issues:
Branch: 1b13e0a (Commits, GitHub, GitLab) Commit: 1b13e0abd82212345712290c510ab0b1bf8ff910
Dependencies: Stopgaps:

Status badges

Description (last modified by slabbe)

As reported on sage-release 8.3.beta3,

sage -tp --optional=sage,internet --logfile=logs/has_internet.log src/sage/tests/cmdline.py src/sage/combinat/tutorial.py src/sage/combinat/words/paths.py src/sage/combinat/designs/bibd.py src/doc/en/developer/coding_basics.rst src/sage/combinat/integer_lists/invlex.pyx 

gives

----------------------------------------------------------------------
sage -t src/sage/combinat/designs/bibd.py  # 1 doctest failed
sage -t src/doc/en/developer/coding_basics.rst  # 1 doctest failed
sage -t src/sage/combinat/integer_lists/invlex.pyx  # 1 doctest failed
sage -t src/sage/combinat/tutorial.py  # 2 doctests failed
sage -t src/sage/combinat/words/paths.py  # 1 doctest failed
sage -t src/sage/tests/cmdline.py  # 2 doctests failed
----------------------------------------------------------------------

Change History (6)

comment:1 Changed 3 years ago by slabbe

  • Description modified (diff)

Keeping only easy fixes in this ticket. Moving the failures associated to files:

src/sage/symbolic/integration/integral.py 
src/sage/symbolic/integration/external.py
src/sage/repl/load.py 
src/sage/coding/databases.py 

in ticket #25501.

Last edited 3 years ago by slabbe (previous) (diff)

comment:2 Changed 3 years ago by slabbe

  • Summary changed from Few internet doctests failed in 12 files to Few internet doctests failed in 6 files

comment:3 Changed 3 years ago by slabbe

  • Branch set to u/slabbe/25474
  • Commit set to 1b13e0abd82212345712290c510ab0b1bf8ff910
  • Description modified (diff)
  • Status changed from new to needs_review

New commits:

1b13e0a25474: fixing internet doctests in combinat, coding_basics, cmdline

comment:4 Changed 3 years ago by saraedum

  • Reviewers set to Julian Rüth
  • Status changed from needs_review to needs_work

Author is missing. Feel free to set this to positive review once you fixed that.

comment:5 Changed 3 years ago by slabbe

  • Authors set to Sébastien Labbé
  • Status changed from needs_work to positive_review

comment:6 Changed 3 years ago by vbraun

  • Branch changed from u/slabbe/25474 to 1b13e0abd82212345712290c510ab0b1bf8ff910
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.