Merge pull request #687 from marcospereirampj/fix/pyright

fix: strong and consistent typing across the library

Author: ryshoooo

poetry.lock

@@ -31,13 +31,12 @@ Documentation = "https://python-keycloak.readthedocs.io/en/latest/"
 "Issue tracker" = "https://github.com/marcospereirampj/python-keycloak/issues"
 
 [tool.poetry.dependencies]
-python = ">=3.9,<4.0"
+python = ">=3.9,<=3.14"
 requests = ">=2.20.0"
 requests-toolbelt = ">=0.6.0"
 deprecation = ">=2.1.0"
 jwcrypto = ">=1.5.4"
 httpx = ">=0.23.2"
-async-property = ">=0.2.2"
 aiofiles = ">=24.1.0"
 
 [tool.poetry.group.docs.dependencies]

pyproject.toml

@@ -133,7 +133,7 @@ def logic(self) -> str:
         return self._logic
 
     @logic.setter
-    def logic(self, value: str) -> str:
+    def logic(self, value: str) -> None:
         self._logic = value
 
     @property

src/keycloak/authorization/permission.py

@@ -24,6 +24,9 @@
 
 from keycloak.exceptions import KeycloakAuthorizationConfigError
 
+from .permission import Permission
+from .role import Role
+
 
 class Policy:
     """
@@ -172,7 +175,7 @@ def permissions(self) -> list:
     def permissions(self, value: list) -> None:
         self._permissions = value
 
-    def add_role(self, role: dict) -> None:
+    def add_role(self, role: str | Role) -> None:
         """
         Add keycloak role in policy.
 
@@ -185,7 +188,7 @@ def add_role(self, role: dict) -> None:
             raise KeycloakAuthorizationConfigError(error_msg)
         self._roles.append(role)
 
-    def add_permission(self, permission: dict) -> None:
+    def add_permission(self, permission: str | Permission) -> None:
         """
         Add keycloak permission in policy.
 

src/keycloak/authorization/policy.py

@@ -61,7 +61,7 @@ def get_name(self) -> str:
         """
         return self.name
 
-    def __eq__(self, other: str | Role) -> bool:
+    def __eq__(self, other: object) -> bool:
         """
         Eq method.