This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Please sort assertSetEqual's output
Type: enhancement Stage:
Components: Library (Lib) Versions: Python 3.8
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: michael.foord Nosy List: Jess, Windson Yang, ezio.melotti, michael.foord, rbcollins, remi.lapeyre, rhettinger
Priority: normal Keywords:

Created on 2019-03-07 22:00 by Jess, last changed 2022-04-11 14:59 by admin.

Messages (9)
msg337443 - (view) Author: Jess (Jess) * Date: 2019-03-07 22:00
Currently https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertSetEqual returns a random list, but I'd like to see it sorted for ease of reading which running tests.

Should be small, but useful.  Happy to make the edit myself, but have no clue how to send you changes.
msg337445 - (view) Author: Rémi Lapeyre (remi.lapeyre) * Date: 2019-03-07 22:42
Hi Jess, I think this could be added.

I think it should require only a a sort in https://github.com/python/cpython/blob/0f221d09cad46bee38d1b7a7822772df66c53028/Lib/unittest/case.py#L1127-L1138, to update the tests and add a blurb.

You will need to get Python source code at https://github.com/python/cpython and you can get information about how to contribute to Python in the dev guide : https://devguide.python.org/

If you need help, I would be happy to assist you and to review your pull request.

If you don't have time to work on this issue, I think we could keep it for the next mentored sprint.
msg337447 - (view) Author: Jess (Jess) * Date: 2019-03-07 23:06
Wow!  Thank you, very fast and the precise snippet of info I needed.  Will try to send something off today.  Very exciting.
msg337449 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2019-03-08 04:15
> I think it should require only a a sort 

It's possible to have non-sortable elements in the set, so you'll either need to sort on the repr of the elements or have a fallback:

    assertSetEqual({10, None, 'abc'}, {20, 3+4j, 10})
msg337451 - (view) Author: Windson Yang (Windson Yang) * Date: 2019-03-08 05:36
Just to be clear, as Raymond said, when we have two non-sortable objects (for instance, two instances which their class didn't implement the __lt__ and __gt__ methods), we should compare their repr() without sort() like now.
msg337457 - (view) Author: Rémi Lapeyre (remi.lapeyre) * Date: 2019-03-08 08:55
@rhettinger
> It's possible to have non-sortable elements in the set, so you'll either need to sort on the repr of the elements or have a fallback

Yes, it is the repr that is used in the loop and that what's the sorting needs to be done against.


@Windson Yang
> we should compare their repr() without sort() like now.

I'm not sure to understand, can you provide more information about what you are thinking of?

Is there a reason to add 2.7, 3.4, 3.5, 3.6 and 3.7 to versions affected ?

As far as I can tell, this is a new feature and should only go in 3.8.
msg337467 - (view) Author: Windson Yang (Windson Yang) * Date: 2019-03-08 11:05
My point is careful about the non-sortable object. My mistake, this should be an enhancement, not a bug.
msg337539 - (view) Author: Jess (Jess) * Date: 2019-03-08 22:05
Good call on the repr(), hadn't noted the "3+4j" issue - __gt__ and __lt__ do work for compare there, but not sorted().  *shrug*  Will make sure the solution takes that into account in some fashion.

Bit slower as I expected as setting up the windows env has some bits that are not entirely happy.
msg343054 - (view) Author: Rémi Lapeyre (remi.lapeyre) * Date: 2019-05-21 14:42
Hi Jess, are you still working on this?
History
Date User Action Args
2022-04-11 14:59:12adminsetgithub: 80411
2019-05-21 14:42:32remi.lapeyresetmessages: + msg343054
2019-03-08 22:05:55Jesssetmessages: + msg337539
2019-03-08 11:05:09Windson Yangsetmessages: + msg337467
versions: - Python 2.7, Python 3.4, Python 3.5, Python 3.6, Python 3.7
2019-03-08 08:55:45remi.lapeyresetmessages: + msg337457
2019-03-08 05:41:32Windson Yangsetversions: + Python 2.7, Python 3.4, Python 3.5, Python 3.6, Python 3.7
2019-03-08 05:36:01Windson Yangsetnosy: + Windson Yang
messages: + msg337451
2019-03-08 04:15:34rhettingersetassignee: michael.foord

messages: + msg337449
nosy: + rhettinger
2019-03-07 23:39:53xtreaksetnosy: + rbcollins, ezio.melotti, michael.foord
2019-03-07 23:06:14Jesssetmessages: + msg337447
2019-03-07 22:42:14remi.lapeyresetversions: + Python 3.8
nosy: + remi.lapeyre

messages: + msg337445

components: + Library (Lib), - Tests
2019-03-07 22:00:28Jesscreate