Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test_pprint relies on set/dictionary repr() ordering #58115

Closed
markshannon opened this issue Jan 29, 2012 · 3 comments
Closed

test_pprint relies on set/dictionary repr() ordering #58115

markshannon opened this issue Jan 29, 2012 · 3 comments
Labels
3.8 (EOL) end of life 3.9 only security fixes 3.10 only security fixes tests Tests in the Lib/test dir

Comments

@markshannon
Copy link
Member

BPO 13907
Nosy @markshannon, @swarmer, @phmc, @iritkatriel

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2020-11-22.18:41:54.123>
created_at = <Date 2012-01-29.22:45:26.925>
labels = ['3.8', 'tests', '3.9', '3.10']
title = 'test_pprint relies on set/dictionary repr() ordering'
updated_at = <Date 2020-11-22.18:41:54.122>
user = 'https://github.com/markshannon'

bugs.python.org fields:

activity = <Date 2020-11-22.18:41:54.122>
actor = 'rhettinger'
assignee = 'none'
closed = True
closed_date = <Date 2020-11-22.18:41:54.123>
closer = 'rhettinger'
components = ['Tests']
creation = <Date 2012-01-29.22:45:26.925>
creator = 'Mark.Shannon'
dependencies = []
files = []
hgrepos = []
issue_num = 13907
keywords = []
message_count = 3.0
messages = ['152272', '165837', '381611']
nosy_count = 4.0
nosy_names = ['Mark.Shannon', 'anton.barkovsky', 'pconnell', 'iritkatriel']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue13907'
versions = ['Python 3.8', 'Python 3.9', 'Python 3.10']

@markshannon
Copy link
Member Author

The function test_set_reprs() includes the comment:
"Consequently, this test is fragile and implementation-dependent"
Changing the dictionary iteration ordering breaks it.

@markshannon markshannon added the tests Tests in the Lib/test dir label Jan 29, 2012
@swarmer
Copy link
Mannequin

swarmer mannequin commented Jul 19, 2012

This test breaks now even in CPython. Remove it and be done with it?

@iritkatriel
Copy link
Member

This test was since renamed to test_set_of_sets_reprs and is currently marked as @expectedfailure:

5184438

@iritkatriel iritkatriel added 3.8 (EOL) end of life 3.9 only security fixes 3.10 only security fixes labels Nov 22, 2020
@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.8 (EOL) end of life 3.9 only security fixes 3.10 only security fixes tests Tests in the Lib/test dir
Projects
None yet
Development

No branches or pull requests

3 participants