#26997 closed defect (fixed)
5 internet doctests failing in findstat.py
Reported by:  slabbe  Owned by:  

Priority:  major  Milestone:  sage8.7 
Component:  doctest coverage  Keywords:  
Cc:  vklein  Merged in:  
Authors:  Martin Rubey  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  6079bf0 (Commits)  Commit:  
Dependencies:  Stopgaps: 
Description
On Ubuntu 16.04, running sage8.5.beta1, the command
sage t long optional=sage,internet src/sage/databases/findstat.py
gives
... 3 items had failures: 3 of 16 in sage.databases.findstat 1 of 4 in sage.databases.findstat.FindStatStatistic.__repr__ 1 of 9 in sage.databases.findstat.FindStatStatistic._find_by_values [247 tests, 5 failures, 107.27 s]  sage t long src/sage/databases/findstat.py # 5 doctests failed 
Full failure log is long and is pasted here: https://framabin.org/p/?b9dacda8324c7295#71YTYesLKEKBCNg1LtnDUhKkWDrcm+xiZGDI9a8bjfg=
Change History (9)
comment:1 Changed 2 years ago by
comment:2 Changed 2 years ago by
 Branch set to u/mantepse/5_internet_doctests_failing_in_findstat_py
comment:3 Changed 2 years ago by
 Commit set to 6079bf06704d5e6080779d4140ad46c6943a1d23
easy fix, according to the specification of FancyTuple.__getitem__
.
New commits:
6079bf0  fix __getitem__ of FancyTuple to preserve tupleitems

comment:4 Changed 2 years ago by
 Status changed from new to needs_review
comment:5 Changed 2 years ago by
 Cc vklein added
comment:6 Changed 2 years ago by
 Reviewers set to Frédéric Chapoton
 Status changed from needs_review to positive_review
seems to be ok on python3
comment:7 Changed 23 months ago by
 Milestone changed from sage8.6 to sage8.7
Retarging tickets optimistically to the next milestone. If you are responsible for this ticket (either its reporter or owner) and don't believe you are likely to complete this ticket before the next release (8.7) please retarget this ticket's milestone to sagepending or sagewishlist.
comment:8 Changed 23 months ago by
 Branch changed from u/mantepse/5_internet_doctests_failing_in_findstat_py to 6079bf06704d5e6080779d4140ad46c6943a1d23
 Resolution set to fixed
 Status changed from positive_review to closed
comment:9 Changed 19 months ago by
 Commit 6079bf06704d5e6080779d4140ad46c6943a1d23 deleted
5 new other failing internet doctests in findstat at #27838
The problem is that the semantics of
FancyTuple
changed, and I think in a bad way:I think this comes from #26704 but I didn't check.