api.user_manager: call UserManager parent with all arguments

Rework the UserManager constructor to call the parent class with all the arguments passed using the sandard idiom. The arguments aren't used in the constructor, so this makes it entirely generic, safer and future-proof. Signed-off-by: Guillaume Tucker <[email protected]>
kernelci · Nov 10, 2023 · 069c0d9 · 069c0d9
1 parent 9c29602
commit 069c0d9
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/api/user_manager.py b/api/user_manager.py
@@ -28,14 +28,12 @@ class UserManager(ObjectIDIDMixin, BaseUserManager[User, PydanticObjectId]):
     reset_password_token_secret = settings.secret_key
     verification_token_secret = settings.secret_key
 
-    def __init__(self, user_db: BaseUserDatabase[User, PydanticObjectId],
-                 password_helper: PasswordHelperProtocol | None = None):
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
         self._email_sender = None
         self._template_env = jinja2.Environment(
-                            loader=jinja2.FileSystemLoader(
-                                "./templates/")
-                        )
-        super().__init__(user_db, password_helper)
+            loader=jinja2.FileSystemLoader("./templates/")
+        )
 
     @property
     def email_sender(self):