Removed possibility to find groups by group name and user id

Author: kompotkot

bugout/app.py

@@ -211,12 +211,11 @@ def get_group(
     def find_group(
         self,
         token: Union[str, uuid.UUID],
-        group_id: Optional[Union[str, uuid.UUID]] = None,
-        name: Optional[str] = None,
+        group_id: Union[str, uuid.UUID],
         timeout: float = REQUESTS_TIMEOUT,
     ) -> data.BugoutGroup:
         self.user.timeout = timeout
-        return self.group.find_group(token=token, group_id=group_id, name=name)
+        return self.group.find_group(token=token, group_id=group_id)
 
     def get_user_groups(
         self, token: Union[str, uuid.UUID], timeout: float = REQUESTS_TIMEOUT

bugout/data.py

@@ -75,7 +75,7 @@ class BugoutUserTokens(BaseModel):
 
 class BugoutGroup(BaseModel):
     id: uuid.UUID
-    group_name: Optional[str]
+    group_name: Optional[str] = Field(alias="name")
     autogenerated: bool
 
 

bugout/group.py

@@ -50,19 +50,10 @@ def get_group(
     def find_group(
         self,
         token: Union[str, uuid.UUID],
-        group_id: Optional[Union[str, uuid.UUID]] = None,
-        name: Optional[str] = None,
+        group_id: Union[str, uuid.UUID],
     ) -> BugoutGroup:
-        find_group_path = f"group/find"
-        if group_id is None and name is None:
-            raise GroupInvalidParameters(
-                "In order to find group, at least one of name, or id must be specified"
-            )
-        query_params = {}
-        if group_id is not None:
-            query_params.update({"group_id": group_id})
-        if name is not None:
-            query_params.update({"name": name})
+        find_group_path = f"groups/find"
+        query_params = {"group_id": group_id}
         headers = {
             "Authorization": f"Bearer {token}",
         }