Fix update group payload model

All fields in the payload should be optional when updating.

lib/galaxy/managers/groups.py

@@ -18,7 +18,10 @@
 from galaxy.model.base import transaction
 from galaxy.model.scoped_session import galaxy_scoped_session
 from galaxy.schema.fields import Security
-from galaxy.schema.groups import GroupCreatePayload
+from galaxy.schema.groups import (
+    GroupCreatePayload,
+    GroupUpdatePayload,
+)
 from galaxy.structured_app import MinimalManagerApp
 
 
@@ -77,7 +80,7 @@ def show(self, trans: ProvidesAppContext, group_id: int):
         item["roles_url"] = self._url_for(trans, "group_roles", group_id=encoded_id)
         return item
 
-    def update(self, trans: ProvidesAppContext, group_id: int, payload: GroupCreatePayload):
+    def update(self, trans: ProvidesAppContext, group_id: int, payload: GroupUpdatePayload):
         """
         Modifies a group.
         """
@@ -87,9 +90,9 @@ def update(self, trans: ProvidesAppContext, group_id: int, payload: GroupCreateP
             self._check_duplicated_group_name(sa_session, name)
             group.name = name
             sa_session.add(group)
-        user_ids = payload.user_ids
+        user_ids = payload.user_ids or []
         users = get_users_by_ids(sa_session, user_ids)
-        role_ids = payload.role_ids
+        role_ids = payload.role_ids or []
         roles = get_roles_by_ids(sa_session, role_ids)
         self._app.security_agent.set_entity_group_associations(
             groups=[group], roles=roles, users=users, delete_existing_assocs=False

lib/galaxy/schema/groups.py

@@ -9,6 +9,7 @@
 )
 from typing_extensions import Literal
 
+from galaxy.schema import partial_model
 from galaxy.schema.fields import (
     DecodedDatabaseIdField,
     EncodedDatabaseIdField,
@@ -69,3 +70,8 @@ class GroupCreatePayload(Model):
         [],
         title="role IDs",
     )
+
+
+@partial_model()
+class GroupUpdatePayload(GroupCreatePayload):
+    pass

lib/galaxy/webapps/galaxy/api/groups.py

@@ -17,6 +17,7 @@
     GroupCreatePayload,
     GroupListResponse,
     GroupResponse,
+    GroupUpdatePayload,
 )
 from galaxy.webapps.galaxy.api import (
     depends,
@@ -80,8 +81,8 @@ def show(
     def update(
         self,
         group_id: Annotated[DecodedDatabaseIdField, Path(...)],
+        payload: Annotated[GroupUpdatePayload, Body(...)],
         trans: ProvidesAppContext = DependsOnTrans,
-        payload: GroupCreatePayload = Body(...),
     ) -> GroupResponse:
         return self.manager.update(trans, group_id, payload)