From e5268ba6b5271022e03572f62a014e6678b7f253 Mon Sep 17 00:00:00 2001 From: dettlaff Date: Fri, 13 Dec 2024 18:08:59 +0400 Subject: [PATCH] fix: if OR in _check_response_type_and_not_empty --- .../repositories/users/kanidm_user_repository.py | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/selfprivacy_api/repositories/users/kanidm_user_repository.py b/selfprivacy_api/repositories/users/kanidm_user_repository.py index 5ce55f6..6eb99dd 100644 --- a/selfprivacy_api/repositories/users/kanidm_user_repository.py +++ b/selfprivacy_api/repositories/users/kanidm_user_repository.py @@ -188,7 +188,7 @@ class KanidmUserRepository(AbstractUserRepository): KanidmReturnUnknownResponseType: If the response data is not of the expected type. """ - if not response_data and response_data is None: + if not response_data or response_data is None: raise KanidmReturnEmptyResponse if data_type == "list": @@ -523,6 +523,19 @@ class KanidmUserRepository(AbstractUserRepository): @staticmethod def get_groups() -> list[Group]: + """ + Return Kanidm groups. + ! Not implemented in JsonUserRepository ! + + Returns: + list[Group] + + Raises: + KanidmReturnEmptyResponse: If the response from Kanidm is empty. + KanidmQueryError: If an error occurs while generating the link. + KanidmReturnUnknownResponseType: If response type is unknown. + """ + groups_list_data = KanidmUserRepository._send_query( endpoint="group", method="GET",