future Python release, patterns with this ending will match both files
and directories. Add a trailing slash to match only directories.
-.. method:: Path.group()
+.. method:: Path.group(*, follow_symlinks=True)
- Return the name of the group owning the file. :exc:`KeyError` is raised
+ Return the name of the group owning the file. :exc:`KeyError` is raised
if the file's gid isn't found in the system database.
+ This method normally follows symlinks; to get the group of the symlink, add
+ the argument ``follow_symlinks=False``.
+
.. versionchanged:: 3.13
Raises :exc:`UnsupportedOperation` if the :mod:`grp` module is not
available. In previous versions, :exc:`NotImplementedError` was raised.
+ .. versionchanged:: 3.13
+ The *follow_symlinks* parameter was added.
+
.. method:: Path.is_dir(*, follow_symlinks=True)
'#!/usr/bin/env python3\n'
-.. method:: Path.owner()
+.. method:: Path.owner(*, follow_symlinks=True)
- Return the name of the user owning the file. :exc:`KeyError` is raised
+ Return the name of the user owning the file. :exc:`KeyError` is raised
if the file's uid isn't found in the system database.
+ This method normally follows symlinks; to get the owner of the symlink, add
+ the argument ``follow_symlinks=False``.
+
.. versionchanged:: 3.13
Raises :exc:`UnsupportedOperation` if the :mod:`pwd` module is not
available. In previous versions, :exc:`NotImplementedError` was raised.
+ .. versionchanged:: 3.13
+ The *follow_symlinks* parameter was added.
+
.. method:: Path.read_bytes()
"""
self._unsupported("rmdir")
- def owner(self):
+ def owner(self, *, follow_symlinks=True):
"""
Return the login name of the file owner.
"""
self._unsupported("owner")
- def group(self):
+ def group(self, *, follow_symlinks=True):
"""
Return the group name of the file gid.
"""
return self.with_segments(os.path.realpath(self, strict=strict))
if pwd:
- def owner(self):
+ def owner(self, *, follow_symlinks=True):
"""
Return the login name of the file owner.
"""
- return pwd.getpwuid(self.stat().st_uid).pw_name
+ uid = self.stat(follow_symlinks=follow_symlinks).st_uid
+ return pwd.getpwuid(uid).pw_name
if grp:
- def group(self):
+ def group(self, *, follow_symlinks=True):
"""
Return the group name of the file gid.
"""
- return grp.getgrgid(self.stat().st_gid).gr_name
+ gid = self.stat(follow_symlinks=follow_symlinks).st_gid
+ return grp.getgrgid(gid).gr_name
if hasattr(os, "readlink"):
def readlink(self):
only_posix = unittest.skipIf(os.name == 'nt',
'test requires a POSIX-compatible system')
+root_in_posix = False
+if hasattr(os, 'geteuid'):
+ root_in_posix = (os.geteuid() == 0)
#
# Tests for the pure classes.
# XXX also need a test for lchmod.
- @unittest.skipUnless(pwd, "the pwd module is needed for this test")
- def test_owner(self):
- p = self.cls(BASE) / 'fileA'
- uid = p.stat().st_uid
+ def _get_pw_name_or_skip_test(self, uid):
try:
- name = pwd.getpwuid(uid).pw_name
+ return pwd.getpwuid(uid).pw_name
except KeyError:
self.skipTest(
"user %d doesn't have an entry in the system database" % uid)
- self.assertEqual(name, p.owner())
- @unittest.skipUnless(grp, "the grp module is needed for this test")
- def test_group(self):
+ @unittest.skipUnless(pwd, "the pwd module is needed for this test")
+ def test_owner(self):
p = self.cls(BASE) / 'fileA'
- gid = p.stat().st_gid
+ expected_uid = p.stat().st_uid
+ expected_name = self._get_pw_name_or_skip_test(expected_uid)
+
+ self.assertEqual(expected_name, p.owner())
+
+ @unittest.skipUnless(pwd, "the pwd module is needed for this test")
+ @unittest.skipUnless(root_in_posix, "test needs root privilege")
+ def test_owner_no_follow_symlinks(self):
+ all_users = [u.pw_uid for u in pwd.getpwall()]
+ if len(all_users) < 2:
+ self.skipTest("test needs more than one user")
+
+ target = self.cls(BASE) / 'fileA'
+ link = self.cls(BASE) / 'linkA'
+
+ uid_1, uid_2 = all_users[:2]
+ os.chown(target, uid_1, -1)
+ os.chown(link, uid_2, -1, follow_symlinks=False)
+
+ expected_uid = link.stat(follow_symlinks=False).st_uid
+ expected_name = self._get_pw_name_or_skip_test(expected_uid)
+
+ self.assertEqual(expected_uid, uid_2)
+ self.assertEqual(expected_name, link.owner(follow_symlinks=False))
+
+ def _get_gr_name_or_skip_test(self, gid):
try:
- name = grp.getgrgid(gid).gr_name
+ return grp.getgrgid(gid).gr_name
except KeyError:
self.skipTest(
"group %d doesn't have an entry in the system database" % gid)
- self.assertEqual(name, p.group())
+
+ @unittest.skipUnless(grp, "the grp module is needed for this test")
+ def test_group(self):
+ p = self.cls(BASE) / 'fileA'
+ expected_gid = p.stat().st_gid
+ expected_name = self._get_gr_name_or_skip_test(expected_gid)
+
+ self.assertEqual(expected_name, p.group())
+
+ @unittest.skipUnless(grp, "the grp module is needed for this test")
+ @unittest.skipUnless(root_in_posix, "test needs root privilege")
+ def test_group_no_follow_symlinks(self):
+ all_groups = [g.gr_gid for g in grp.getgrall()]
+ if len(all_groups) < 2:
+ self.skipTest("test needs more than one group")
+
+ target = self.cls(BASE) / 'fileA'
+ link = self.cls(BASE) / 'linkA'
+
+ gid_1, gid_2 = all_groups[:2]
+ os.chown(target, -1, gid_1)
+ os.chown(link, -1, gid_2, follow_symlinks=False)
+
+ expected_gid = link.stat(follow_symlinks=False).st_gid
+ expected_name = self._get_pw_name_or_skip_test(expected_gid)
+
+ self.assertEqual(expected_gid, gid_2)
+ self.assertEqual(expected_name, link.group(follow_symlinks=False))
def test_unlink(self):
p = self.cls(BASE) / 'fileA'
projects / thirdparty / Python / cpython.git / commitdiff
