]> granicus.if.org Git - python/commitdiff
Merged revisions 83133 via svnmerge from
authorRonald Oussoren <ronaldoussoren@mac.com>
Sat, 24 Jul 2010 14:19:26 +0000 (14:19 +0000)
committerRonald Oussoren <ronaldoussoren@mac.com>
Sat, 24 Jul 2010 14:19:26 +0000 (14:19 +0000)
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

index 31459ffd3c864fdb40a2a98fd0208e4b7f24955b..a830c369c2857d342171cab4cdad184d86ef4b2f 100644 (file)
@@ -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):