diff options
author | NavaL <ayoyo@thoughtworks.com> | 2016-10-28 18:30:19 +0200 |
---|---|---|
committer | NavaL <ayoyo@thoughtworks.com> | 2016-10-28 18:30:19 +0200 |
commit | 7eeaf8a5f693578bc78a538e3a009fc87578348f (patch) | |
tree | a5c699678e28d9333a120574048bc429c3976d39 /service/test/unit | |
parent | 9209525c5e88e4314711359b4e9fa42d6958403d (diff) |
adding custom messages per types of auth errors #795
Diffstat (limited to 'service/test/unit')
-rw-r--r-- | service/test/unit/test_authentication.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/service/test/unit/test_authentication.py b/service/test/unit/test_authentication.py index 90ef0a7c..3e1b4a10 100644 --- a/service/test/unit/test_authentication.py +++ b/service/test/unit/test_authentication.py @@ -32,7 +32,11 @@ class AuthenticatorTest(unittest.TestCase): mock_bonafide_session.authenticate = Mock(side_effect=SRPAuthError()) with patch('pixelated.authentication.Session', return_value=mock_bonafide_session): with self.assertRaises(UnauthorizedLogin): - yield auth.authenticate('username', 'password') + try: + yield auth.authenticate('username', 'password') + except UnauthorizedLogin as e: + self.assertEqual("User typed wrong password/username combination.", e.message) + raise @inlineCallbacks def test_domain_name_is_stripped_before_making_bonafide_srp_auth(self): @@ -74,7 +78,11 @@ class AuthenticatorTest(unittest.TestCase): username_with_wrong_domain = '%s@%s' % (username_without_domain, 'wrongdomain.org') auth = Authenticator(self._leap_provider) with self.assertRaises(UnauthorizedLogin): - auth.clean_username(username_with_wrong_domain) + try: + auth.clean_username(username_with_wrong_domain) + except UnauthorizedLogin as e: + self.assertEqual("User typed a wrong domain.", e.message) + raise def test_username_with_domain(self): auth = Authenticator(self._leap_provider) |