Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Torsten Kilias <[email protected]>
  • Loading branch information
ckunki and tkilias authored Mar 27, 2024
1 parent 78a9b4c commit 71537ef
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ def id(self):
self._id = pwd.getpwnam(self.name).pw_uid
return self._id

def chown_rec(self, path: Path):
def chown_recursive(self, path: Path):
uid = self.id
gid = self.group.id
os.chown(path, uid, gid)
Expand Down
2 changes: 1 addition & 1 deletion test/integration/docker_socket_and_groups.py
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ def chgrp(self, gid: int, path_on_host: Path):
with self._context_provider(path_on_host, path_in_container) as container:
assert_exec_run(container, f"chgrp {gid} {path_in_container}")

def chown_chmod_rec(
def chown_chmod_recursive(
self,
owner: str,
permissions: str,
Expand Down
2 changes: 1 addition & 1 deletion test/unit/entrypoint/test_user_class.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ def test_uid(mocker, user):
and pwd.getpwnam.call_args == mocker.call("jennifer")


def test_chown_rec(mocker, user, tmp_path):
def test_chown_recursive(mocker, user, tmp_path):
child = tmp_path / "child"
sub = tmp_path / "sub"
grand_child = sub / "grand_child"
Expand Down

0 comments on commit 71537ef

Please sign in to comment.