summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorRonald Oussoren <ronaldoussoren@mac.com>2010-07-24 14:15:19 (GMT)
committerRonald Oussoren <ronaldoussoren@mac.com>2010-07-24 14:15:19 (GMT)
commitcb615e6abe1ae6523aacddbf108fbf92f26e9eb8 (patch)
tree7fcff6f0659d776c468bb53d32897f2f002a2980 /Lib
parent9c71f90329b842c8b9c6b3b2726f5791b00a5c3b (diff)
downloadcpython-cb615e6abe1ae6523aacddbf108fbf92f26e9eb8.zip
cpython-cb615e6abe1ae6523aacddbf108fbf92f26e9eb8.tar.gz
cpython-cb615e6abe1ae6523aacddbf108fbf92f26e9eb8.tar.bz2
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.
Diffstat (limited to 'Lib')
-rw-r--r--Lib/test/test_posix.py6
1 files changed, 5 insertions, 1 deletions
diff --git a/Lib/test/test_posix.py b/Lib/test/test_posix.py
index 6708aeb..b6322b4 100644
--- a/Lib/test/test_posix.py
+++ b/Lib/test/test_posix.py
@@ -357,7 +357,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):