Apply suggestions from code review

Co-authored-by: Jonathan Keane <[email protected]>

Author: schloerke

src/posit/connect/permissions.py

@@ -152,14 +152,14 @@ def destroy(self, *permissions: str | Group | User | Permission) -> list[Permiss
         ----------
         *permissions : str | Group | User | Permission
             The content item permissions to remove. If a `str` is received, it is compared against
-            the `Permissions`' `principal_guid`. If a `Group` or `User` is received, the associated
+            the `Permissions`'s `principal_guid`. If a `Group` or `User` is received, the associated
             `Permission` will be removed.
 
         Returns
         -------
         list[Permission]
-            The removed permissions. If a permission is not found, it is not included in the
-            returned list.
+            The removed permissions. If a permission is not found, there is nothing to remove and 
+            it is not included in the returned list.
 
         Examples
         --------
@@ -206,9 +206,7 @@ def destroy(self, *permissions: str | Group | User | Permission) -> list[Permiss
         for arg in permissions:
             if isinstance(arg, str):
                 principal_guid = arg
-            elif isinstance(arg, Group):
-                principal_guid: str = arg["guid"]
-            elif isinstance(arg, User):
+            elif isinstance(arg, (Group, User)):
                 principal_guid: str = arg["guid"]
             elif isinstance(arg, Permission):
                 principal_guid: str = arg["principal_guid"]

tests/posit/connect/test_permissions.py

@@ -321,7 +321,6 @@ def test_destroy(self):
         )
 
         # Assert bad input value
-
         with pytest.raises(TypeError):
             permissions.destroy(
                 42  # pyright: ignore[reportArgumentType]