From 5719c2f97e986bfe2c63f758d662b86472fae29c Mon Sep 17 00:00:00 2001 From: Ronald Oussoren Date: Sat, 24 Jul 2010 14:21:29 +0000 Subject: [PATCH] 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. ........ --- Lib/test/test_posix.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Lib/test/test_posix.py b/Lib/test/test_posix.py index 3f6e11bb363..e069bf8f51a 100644 --- a/Lib/test/test_posix.py +++ b/Lib/test/test_posix.py @@ -371,7 +371,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):