[issue9131] test_set_reprs in test_pprint is fragile

Daniel Stutzbach added the comment: Committed in r84961 -- stage: patch review -> committed/rejected status: open -> closed type: -> behavior versions: -Python 3.3 ___ Python tracker _

[issue9131] test_set_reprs in test_pprint is fragile

Changes by Raymond Hettinger : -- assignee: rhettinger -> stutzbach ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscrib

[issue9131] test_set_reprs in test_pprint is fragile

Changes by Daniel Stutzbach : Added file: http://bugs.python.org/file18593/issue9131.patch ___ Python tracker ___ ___ Python-bugs-list mailing

[issue9131] test_set_reprs in test_pprint is fragile

Changes by Daniel Stutzbach : Removed file: http://bugs.python.org/file18592/issue9131.patch ___ Python tracker ___ ___ Python-bugs-list mailin

[issue9131] test_set_reprs in test_pprint is fragile

Daniel Stutzbach added the comment: errr... ignore that first patch (now deleted) :-) -- Added file: http://bugs.python.org/file18592/issue9131.patch ___ Python tracker ___

[issue9131] test_set_reprs in test_pprint is fragile

Changes by Daniel Stutzbach : Removed file: http://bugs.python.org/file18591/issue9131.patch ___ Python tracker ___ ___ Python-bugs-list mailin

[issue9131] test_set_reprs in test_pprint is fragile

Changes by Daniel Stutzbach : -- keywords: +needs review, patch resolution: -> accepted stage: needs patch -> patch review Added file: http://bugs.python.org/file18591/issue9131.patch ___ Python tracker ___

[issue9131] test_set_reprs in test_pprint is fragile

Daniel Stutzbach added the comment: Nevermind, I found it: @support.cpython_only I'll work on a patch to add the decorator and a comment about why the test is fragile. -- ___ Python tracker __

[issue9131] test_set_reprs in test_pprint is fragile

Daniel Stutzbach added the comment: How do we mark a test as implementation specific? Is there a decorator for that? -- ___ Python tracker ___ _

[issue9131] test_set_reprs in test_pprint is fragile

Raymond Hettinger added the comment: After discussion on #python-dev, have decided to mark the test as implementation specific and add a comment about why the test is fragile. -- assignee: stutzbach -> rhettinger ___ Python tracker

[issue9131] test_set_reprs in test_pprint is fragile

Raymond Hettinger added the comment: Yes, it's a fragile and crummy test. Feel free to delete it. -- ___ Python tracker ___ ___ Python

[issue9131] test_set_reprs in test_pprint is fragile

Mark Dickinson added the comment: I agree that this looks a bit suspicious. Adding Raymond to the nosy list, since it looks like this is his code (r60264). -- nosy: +mark.dickinson, rhettinger ___ Python tracker _

[issue9131] test_set_reprs in test_pprint is fragile

New submission from Daniel Stutzbach : test_set_reprs in test_pprint creates a complex arrangement of frozensets and tests the pretty-printed repr against a string hard-coded in the test. The hard-coded repr depends on the sort order of frozensets. However, "Since sets only define partial o