diff --git a/fastapi_users/authentication/authenticator.py b/fastapi_users/authentication/authenticator.py index 6117765f..897228a2 100644 --- a/fastapi_users/authentication/authenticator.py +++ b/fastapi_users/authentication/authenticator.py @@ -31,7 +31,9 @@ class DuplicateBackendNamesError(Exception): pass -EnabledBackendsDependency = DependencyCallable[Sequence[AuthenticationBackend[models.UP, models.ID]]] +EnabledBackendsDependency = DependencyCallable[ + Sequence[AuthenticationBackend[models.UP, models.ID]] +] class Authenticator(Generic[models.UP, models.ID]): @@ -62,7 +64,9 @@ class Authenticator(Generic[models.UP, models.ID]): active: bool = False, verified: bool = False, superuser: bool = False, - get_enabled_backends: Optional[EnabledBackendsDependency[models.UP, models.ID]] = None, + get_enabled_backends: Optional[ + EnabledBackendsDependency[models.UP, models.ID] + ] = None, ): """ Return a dependency callable to retrieve currently authenticated user and token. @@ -106,7 +110,9 @@ class Authenticator(Generic[models.UP, models.ID]): active: bool = False, verified: bool = False, superuser: bool = False, - get_enabled_backends: Optional[EnabledBackendsDependency[models.UP, models.ID]] = None, + get_enabled_backends: Optional[ + EnabledBackendsDependency[models.UP, models.ID] + ] = None, ): """ Return a dependency callable to retrieve currently authenticated user. @@ -157,8 +163,8 @@ class Authenticator(Generic[models.UP, models.ID]): ) -> Tuple[Optional[models.UP], Optional[str]]: user: Optional[models.UP] = None token: Optional[str] = None - enabled_backends: Sequence[AuthenticationBackend[models.UP, models.ID]] = kwargs.get( - "enabled_backends", self.backends + enabled_backends: Sequence[AuthenticationBackend[models.UP, models.ID]] = ( + kwargs.get("enabled_backends", self.backends) ) for backend in self.backends: if backend in enabled_backends: diff --git a/fastapi_users/fastapi_users.py b/fastapi_users/fastapi_users.py index 85edb1bd..980b674c 100644 --- a/fastapi_users/fastapi_users.py +++ b/fastapi_users/fastapi_users.py @@ -72,7 +72,9 @@ class FastAPIUsers(Generic[models.UP, models.ID]): return get_reset_password_router(self.get_user_manager) def get_auth_router( - self, backend: AuthenticationBackend[models.UP, models.ID], requires_verification: bool = False + self, + backend: AuthenticationBackend[models.UP, models.ID], + requires_verification: bool = False, ) -> APIRouter: """ Return an auth router for a given authentication backend. diff --git a/tests/test_manager.py b/tests/test_manager.py index 30fa5a5e..2eab6c96 100644 --- a/tests/test_manager.py +++ b/tests/test_manager.py @@ -139,7 +139,7 @@ class TestCreateUser: ): user = UserCreate(email=email, password="guinevere") created_user = await user_manager.create(user) - assert type(created_user) == UserModel + assert isinstance(created_user, UserModel) assert user_manager.on_after_register.called is True @@ -151,7 +151,7 @@ class TestCreateUser: email="lancelot@camelot.b", password="guinevere", is_superuser=True ) created_user = await user_manager.create(user, safe) - assert type(created_user) == UserModel + assert isinstance(created_user, UserModel) assert created_user.is_superuser is result assert user_manager.on_after_register.called is True @@ -164,7 +164,7 @@ class TestCreateUser: email="lancelot@camelot.b", password="guinevere", is_active=False ) created_user = await user_manager.create(user, safe) - assert type(created_user) == UserModel + assert isinstance(created_user, UserModel) assert created_user.is_active is result assert user_manager.on_after_register.called is True