better exception message for user api

tracim · Jun 7, 2018 · 6084169 · 6084169
1 parent 2f98c67
commit 6084169
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/tracim/lib/core/user.py b/tracim/lib/core/user.py
@@ -73,7 +73,7 @@ def get_current_user(self) -> User:
         Get current_user
         """
         if not self._user:
-            raise UserDoesNotExist()
+            raise UserDoesNotExist('There is no current user')
         return self._user
 
     def get_all(self) -> typing.Iterable[User]:
@@ -102,9 +102,9 @@ def authenticate_user(self, email: str, password: str) -> User:
             if user.validate_password(password):
                 return user
             else:
-                raise WrongUserPassword()
-        except (WrongUserPassword, UserDoesNotExist):
-            raise AuthenticationFailed()
+                raise WrongUserPassword('User "{}" password is incorrect'.format(email))  # nopep8
+        except (WrongUserPassword, UserDoesNotExist) as exc:
+            raise AuthenticationFailed('User "{}" authentication failed'.format(email)) from exc  # nopep8
 
     # Actions