From: Ronald Oussoren Date: Sat, 24 Jul 2010 14:19:26 +0000 (+0000) Subject: Merged revisions 83133 via svnmerge from X-Git-Tag: v3.1.3rc1~465 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=55b9da588552c0a3d892a3e9f82db108e4d542bf;p=python Merged revisions 83133 via svnmerge from svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r83133 | ronald.oussoren | 2010-07-24 15:15:19 +0100 (Sat, 24 Jul 2010) | 5 lines Fix for issue 9367: the test code for os.getgroups assumes that the result of getgroups and the output of the id(1) command return groups in the same order. That assumption is both fragile and false. ........ --- diff --git a/Lib/test/test_posix.py b/Lib/test/test_posix.py index 31459ffd3c..a830c369c2 100644 --- a/Lib/test/test_posix.py +++ b/Lib/test/test_posix.py @@ -281,7 +281,11 @@ class PosixTester(unittest.TestCase): if not groups: raise unittest.SkipTest("need working 'id -G'") - self.assertEqual([int(x) for x in groups.split()], posix.getgroups()) + # The order of groups isn't important, hence the calls + # to sorted. + self.assertEqual( + list(sorted([int(x) for x in groups.split()])), + list(sorted(posix.getgroups()))) class PosixGroupsTester(unittest.TestCase):