Ticket #4389 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch, positive review] Sage 3.1.4: optional doctest failure in sage/groups/perm_gps/permgroup.py

Reported by: mabshoff Owned by: mabshoff
Priority: major Milestone: sage-3.2
Component: doctest coverage Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description 

sage -t -long -optional devel/sage/sage/groups/perm_gps/permgroup.py
**********************************************************************
File "/scratch/mabshoff/release-cycle/sage-3.1.3.final/tmp/permgroup.py", line 179:
    sage: H.gens()                            # requires optional database_gap
Expected:
    ((1,2,3,4), (1,3))
Got:
    [(1,2,3,4), (1,3)]
**********************************************************************

The above is caused by changing the printing of permutations that went into Sage a while ago. The fix is obvious.

Cheers,

Michael

Attachments

trac_4389.patch Download (868 bytes) - added by mabshoff 5 years ago.

Change History

Changed 5 years ago by mabshoff

comment:1 Changed 5 years ago by mhansen

  • Summary changed from [with patch, needs review] Sage 3.1.4: optional doctest failure in sage/groups/perm_gps/permgroup.py to [with patch, positive review] Sage 3.1.4: optional doctest failure in sage/groups/perm_gps/permgroup.py

+1

comment:2 Changed 5 years ago by ddrake

+1 here too

comment:3 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in Sage 3.2.alpha2

Note: See TracTickets for help on using tickets.