Commit d59240de authored by Charles-François Natali's avatar Charles-François Natali

Issue #14698: Make test_posix more robust when the current UID doesn't have an

associated pwd entry.
parents 67880cc9 e8a255a5
......@@ -108,7 +108,11 @@ class PosixTester(unittest.TestCase):
# If a non-privileged user invokes it, it should fail with OSError
# EPERM.
if os.getuid() != 0:
try:
name = pwd.getpwuid(posix.getuid()).pw_name
except KeyError:
# the current UID may not have a pwd entry
raise unittest.SkipTest("need a pwd entry")
try:
posix.initgroups(name, 13)
except OSError as e:
......@@ -624,8 +628,9 @@ class PosixTester(unittest.TestCase):
def test_getgrouplist(self):
with os.popen('id -G') as idg:
groups = idg.read().strip()
ret = idg.close()
if not groups:
if ret != 0 or not groups:
raise unittest.SkipTest("need working 'id -G'")
self.assertEqual(
......@@ -637,8 +642,9 @@ class PosixTester(unittest.TestCase):
def test_getgroups(self):
with os.popen('id -G') as idg:
groups = idg.read().strip()
ret = idg.close()
if not groups:
if ret != 0 or not groups:
raise unittest.SkipTest("need working 'id -G'")
# 'id -G' and 'os.getgroups()' should return the same
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment