pyramid-fullauth 0.3.2__py2.7.egg → 0.6.0__py2.7.egg

tests/unit/test_social_view.py

@@ -1,7 +1,7 @@
 """Unittest for social login view."""
 import pytest
 import mock
-from velruse import AuthenticationComplete
+from pyramid.compat import text_type
 
 from pyramid_fullauth.views.social import SocialLoginViews
 
@@ -9,57 +9,59 @@ from pyramid_fullauth.views.social import SocialLoginViews
 @pytest.mark.parametrize('profile, email', [
     (
         {
-            'accounts': [{'domain': u'facebook.com', 'userid': u'2343'}],
-            'displayName': u'teddy',
-            'verifiedEmail': 'verified@email.co.uk',
-            'preferredUsername': u'teddy',
-            'emails': [{'value': u'aasd@bwwqwe.pl'}],
-            'name': u'ted'
+            'accounts': [{'domain': text_type('facebook.com'), 'userid': text_type('2343')}],
+            'displayName': text_type('teddy'),
+            'verifiedEmail': text_type('verified@email.co.uk'),
+            'preferredUsername': text_type('teddy'),
+            'emails': [{'value': text_type('aasd@bwwqwe.pl')}],
+            'name': text_type('ted')
         },
         'verified@email.co.uk'
     ), (
         {
-            'accounts': [{'domain': u'facebook.com', 'userid': u'2343'}],
-            'displayName': u'teddy',
-            'preferredUsername': u'teddy',
-            'emails': [{'value': u'aasd@bwwqwe.pl'}],
-            'name': u'ted'
+            'accounts': [{'domain': text_type('facebook.com'), 'userid': text_type('2343')}],
+            'displayName': text_type('teddy'),
+            'preferredUsername': text_type('teddy'),
+            'emails': [{'value': text_type('aasd@bwwqwe.pl')}],
+            'name': text_type('ted')
         },
         'aasd@bwwqwe.pl'
     ), (
         {
-            'accounts': [{'domain': u'facebook.com', 'userid': u'2343'}],
-            'displayName': u'teddy',
-            'preferredUsername': u'teddy',
+            'accounts': [{'domain': text_type('facebook.com'), 'userid': text_type('2343')}],
+            'displayName': text_type('teddy'),
+            'preferredUsername': text_type('teddy'),
             'emails': [{}],
-            'name': u'ted'
+            'name': text_type('ted')
         },
         '2343@facebook.com'
     ), (
         {
-            'accounts': [{'domain': u'facebook.com', 'userid': u'2343'}],
-            'displayName': u'teddy',
-            'preferredUsername': u'teddy',
+            'accounts': [{'domain': text_type('facebook.com'), 'userid': text_type('2343')}],
+            'displayName': text_type('teddy'),
+            'preferredUsername': text_type('teddy'),
             'emails': [],
-            'name': u'ted'
+            'name': text_type('ted')
         },
         '2343@facebook.com'
     ), (
         {
-            'accounts': [{'domain': u'facebook.com', 'userid': u'2343'}],
-            'displayName': u'teddy',
-            'preferredUsername': u'teddy',
-            'name': u'ted'
+            'accounts': [{'domain': text_type('facebook.com'), 'userid': text_type('2343')}],
+            'displayName': text_type('teddy'),
+            'preferredUsername': text_type('teddy'),
+            'name': text_type('ted')
         },
         '2343@facebook.com'
     ),
 ])
 def test_email_from_context(profile, email):
     """Test email_from_context email getting method."""
+    from velruse import AuthenticationComplete
     context = AuthenticationComplete(
         profile,
         {'oauthAccessToken': '7897048593434'},
-        u'facebook',
-        u'facebook')
+        text_type('facebook'),
+        text_type('facebook')
+    )
     view = SocialLoginViews(mock.MagicMock())
     assert view._email_from_context(context) == email

tests/unit/test_user_password.py

@@ -45,7 +45,7 @@ def test_password_empty(db_session, user):
 
 
 def test_set_reset(db_session, user):
-    """User.set_reset - generates reset key for those who forgot their password. """
+    """User.set_reset - generates reset key for those who forgot their password."""
     user = db_session.merge(user)
     assert not user.reset_key
     user.set_reset()

tests/unit/test_user_repr.py

@@ -5,7 +5,7 @@ from pyramid_fullauth.models import User
 
 
 def test_introduce_email():
-    """user gets introduced by e-mail only."""
+    """User gets introduced by e-mail only."""
     user = User()
     user.email = text_type('test@test.pl')
     # To string should return concatenated email
@@ -13,7 +13,7 @@ def test_introduce_email():
 
 
 def test_introduce_username():
-    """user gets introduced by username."""
+    """User gets introduced by username."""
     user = User()
     # 'User not saved should be represented by \'None\'')
     assert str(user) == 'None'
@@ -25,5 +25,5 @@ def test_introduce_username():
     user.username = text_type('testowy')
     # To string should return username
     assert str(user) == 'testowy'
-    # User should be represented by "<User (\'1\', \'testowy\')>"
-    assert user.__repr__() == "<User ('1', 'testowy')>"
+    # User should be represented by "<User (\'1\')>"
+    assert user.__repr__() == "<User ('1')>"

tests/unit/test_validators.py

@@ -41,4 +41,4 @@ def test_email_invalid_formats(db_session, user, invalid_email):
 
     with pytest.raises(EmailValidationError):
         user.email = invalid_email
-        db_session.commit()
+        transaction.commit()

tests/views/conftest.py

@@ -5,6 +5,11 @@ import transaction
 from pyramid_fullauth.models import AuthenticationProvider
 
 
+def mock_translate(msg, *args, **kwargs):
+    """Mock translate function (simply returns message)."""
+    return msg
+
+
 @pytest.fixture
 def facebook_user(active_user, db_session):
     """Facebook user."""

tests/views/test_activation.py

@@ -1,6 +1,10 @@
 # -*- coding: utf-8 -*-
 """Account activation related tests."""
-from urllib import quote
+try:
+    from urllib import quote
+    # python3
+except ImportError:
+    from urllib.parse import quote
 
 import transaction
 
@@ -31,7 +35,7 @@ def test_account_activation_wrong_key(user, db_session, default_app):
     res = default_app.get('/register/activate/' + activate_key[:-5], status=200)
     transaction.commit()
 
-    assert 'Invalid activation code' in res.body
+    assert 'Invalid activation code' in res.body.decode('unicode_escape')
 
     user = db_session.query(User).filter(User.email == user.email).one()
     assert user.activate_key == activate_key
@@ -46,13 +50,16 @@ def test_account_activation_key_with_trash_chars(user, db_session, default_app):
     user = db_session.merge(user)
 
     activate_key = user.activate_key
-    res = default_app.get('/register/activate/' + quote(
-        u'ąśðłĸ곶→łęóħó³→←śðđ[]}³½ĸżćŋðń→↓ŧ¶ħ→ĸł¼²³↓←ħ@ĸđśðĸ@ł¼ęłśħđó[³²½łðśđħ'.encode('utf-8')),
+    res = default_app.get(
+        '/register/activate/' + quote(
+            # ąśðłĸ곶→łęóħó³→←śðđ[]}³½ĸżćŋðń→↓ŧ¶ħ→ĸł¼²³↓←ħ@ĸđśðĸ@ł¼ęłśħđó[³²½łðśđħ - already decoded
+            '\xc4\x85\xc5\x9b\xc3\xb0\xc5\x82\xc4\xb8\xc4\x99\xc5\x82\xc2\xb6\xe2\x86\x92\xc5\x82\xc4\x99\xc3\xb3\xc4\xa7\xc3\xb3\xc2\xb3\xe2\x86\x92\xe2\x86\x90\xc5\x9b\xc3\xb0\xc4\x91[]}\xc2\xb3\xc2\xbd\xc4\xb8\xc5\xbc\xc4\x87\xc5\x8b\xc3\xb0\xc5\x84\xe2\x86\x92\xe2\x86\x93\xc5\xa7\xc2\xb6\xc4\xa7\xe2\x86\x92\xc4\xb8\xc5\x82\xc2\xbc\xc2\xb2\xc2\xb3\xe2\x86\x93\xe2\x86\x90\xc4\xa7@\xc4\xb8\xc4\x91\xc5\x9b\xc3\xb0\xc4\xb8@\xc5\x82\xc2\xbc\xc4\x99\xc5\x82\xc5\x9b\xc4\xa7\xc4\x91\xc3\xb3[\xc2\xb3\xc2\xb2\xc2\xbd\xc5\x82\xc3\xb0\xc5\x9b\xc4\x91\xc4\xa7'  #noqa
+        ),
         status=200
     )
     transaction.commit()
 
-    assert 'Invalid activation code' in res.body
+    assert 'Invalid activation code' in res.body.decode('unicode_escape')
     user = db_session.query(User).filter(User.email == user.email).one()
 
     assert user.activate_key == activate_key
@@ -76,4 +83,4 @@ def test_account_activation_twice(user, db_session, default_app):
     assert user.activated_at
 
     res = default_app.get('/register/activate/' + activate_key, status=200)
-    assert 'Invalid activation code' in res.body
+    assert 'Invalid activation code' in res.body.decode('unicode_escape')

tests/views/test_email.py

@@ -140,7 +140,7 @@ def test_email_proceed(db_session, active_user, default_app):
     email = DEFAULT_USER['email']
     user = db_session.query(User).filter(User.email == email).one()
 
-    new_email = u'email2@email.com'
+    new_email = text_type('email2@email.com')
     user.set_new_email(new_email)
     transaction.commit()
 
@@ -165,7 +165,7 @@ def test_email_proceed_wrong_key(db_session, active_user, default_app):
     email = DEFAULT_USER['email']
     user = db_session.query(User).filter(User.email == email).one()
 
-    new_email = u'email2@email.com'
+    new_email = text_type('email2@email.com')
     user.set_new_email(new_email)
     transaction.commit()
 

tests/views/test_events.py

@@ -3,11 +3,12 @@ import pytest
 import transaction
 from mock import MagicMock
 from pyramid import testing
+from pyramid.compat import text_type
 from pyramid.view import view_config
 from pyramid.httpexceptions import HTTPFound
-from velruse import AuthenticationComplete
 from sqlalchemy.orm.exc import NoResultFound
 from pytest_pyramid import factories
+from velruse import AuthenticationComplete
 
 
 from pyramid_fullauth.models import User
@@ -21,11 +22,20 @@ from pyramid_fullauth.events import (
     BeforeRegister, AfterRegister
 )
 from tests.tools import authenticate, is_user_logged, DEFAULT_USER
+from tests.views.conftest import mock_translate
 
 EVENT_PATH = '/event?event={0.__name__}'
 EVENT_URL = 'http://localhost' + EVENT_PATH
 
 
+def unregister_subscriber(config, event):
+    """Unregister subscribers from given events."""
+    for key in config.registry.adapters._subscribers[1].keys():
+        if key.implementedBy(event):
+            del config.registry.adapters._subscribers[1][key]
+            break
+
+
 evented_config = factories.pyramid_config({
     'yml.location': 'tests:config',
     'env': 'login',
@@ -42,7 +52,6 @@ def include_views(config):
     """Dummy pyramid plugin including events."""
     config.add_route('event', '/event')
     config.scan('tests.views.test_events')
-    # config.add_subscriber(raise_attribute_error, BeforeLogIn)
 
 
 @view_config(route_name="event", renderer='json')
@@ -69,7 +78,10 @@ def raise_attribute_error(event):
 def alreadyloggedin_config(evented_config):
     """Add AlreadyLoggedIn event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AlreadyLoggedIn)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AlreadyLoggedIn)
+
 
 alreadyloggedin_app = factories.pyramid_app('alreadyloggedin_config')
 
@@ -85,7 +97,10 @@ def test_default_login_redirect_from_event(active_user, alreadyloggedin_app):
 def beforelogin_config(evented_config):
     """Add BeforeLogIn event that raises AttributeError with event class name."""
     evented_config.add_subscriber(raise_attribute_error, BeforeLogIn)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, BeforeLogIn)
+
 
 beforelogin_app = factories.pyramid_app('beforelogin_config')
 
@@ -107,7 +122,10 @@ def test_error_beforelogin(active_user, beforelogin_app):
 def afterlogin_config(evented_config):
     """Add AfterLogIn event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AfterLogIn)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterLogIn)
+
 
 afterlogin_app = factories.pyramid_app('afterlogin_config')
 
@@ -126,7 +144,10 @@ def test_login_redirect(active_user, afterlogin_app):
 def afterloginerror_config(evented_config):
     """Add AfterLogIn event subscriber that raises AttributeError."""
     evented_config.add_subscriber(raise_attribute_error, AfterLogIn)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterLogIn)
+
 
 afterloginerror_app = factories.pyramid_app('afterloginerror_config')
 
@@ -150,7 +171,10 @@ def test_error_afterlogin(active_user, afterloginerror_app):
 def beforeemailchange_config(evented_config):
     """Add BeforeEmailChange event subscriber that raises AttributeError."""
     evented_config.add_subscriber(raise_attribute_error, BeforeEmailChange)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, BeforeEmailChange)
+
 
 beforeemailchange_app = factories.pyramid_app('beforeemailchange_config')
 
@@ -178,7 +202,11 @@ def afteremailchange_config(evented_config):
     """Add AfterEmailChange, AfterEmailChangeActivation event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AfterEmailChange)
     evented_config.add_subscriber(redirect_to_secret, AfterEmailChangeActivation)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterEmailChange)
+    unregister_subscriber(evented_config, AfterEmailChangeActivation)
+
 
 afteremailchange_app = factories.pyramid_app('afteremailchange_config')
 
@@ -244,7 +272,7 @@ def test_afteremailchangeactivation(db_session, active_user, afteremailchange_ap
     email = DEFAULT_USER['email']
     user = db_session.query(User).filter(User.email == email).one()
 
-    new_email = u'email2@email.com'
+    new_email = text_type('email2@email.com')
     user.set_new_email(new_email)
     transaction.commit()
 
@@ -266,7 +294,11 @@ def afterreset_config(evented_config):
     """Add AfterReset, AfterResetRequest event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AfterResetRequest)
     evented_config.add_subscriber(redirect_to_secret, AfterReset)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterResetRequest)
+    unregister_subscriber(evented_config, AfterReset)
+
 
 afterreset_app = factories.pyramid_app('afterreset_config')
 
@@ -310,7 +342,10 @@ def test_afterreset(user, db_session, afterreset_app):
 def beforereset_config(evented_config):
     """Add BeforeReset event subscriber that raises AttributeError."""
     evented_config.add_subscriber(raise_attribute_error, BeforeReset)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, BeforeReset)
+
 
 beforereset_app = factories.pyramid_app('beforereset_config')
 
@@ -327,14 +362,17 @@ def test_beforereset(user, db_session, beforereset_app):
     res.form['password'] = 'YouShallPass'
     res.form['confirm_password'] = 'YouShallPass'
     res = res.form.submit()
-    assert 'Error! BeforeReset' in res.body
+    assert 'Error! BeforeReset' in res.body.decode('unicode_escape')
 
 
 @pytest.fixture
 def aftersocialregister_config(evented_config):
     """Add AfterSocialRegister event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AfterSocialRegister)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterSocialRegister)
+
 
 aftersocialregister_app = factories.pyramid_app('aftersocialregister_config')
 
@@ -342,20 +380,20 @@ aftersocialregister_app = factories.pyramid_app('aftersocialregister_config')
 def test_aftersocialregister(aftersocialregister_config, aftersocialregister_app, db_session):
     """Register fresh user and logs him in and check response if redirect from AfterSocialRegister."""
     profile = {
-        'accounts': [{'domain': u'facebook.com', 'userid': u'2343'}],
-        'displayName': u'teddy',
-        'verifiedEmail': u'we@po.pl',
-        'preferredUsername': u'teddy',
-        'emails': [{'value': u'aasd@bwwqwe.pl'}],
-        'name': u'ted'
+        'accounts': [{'domain': text_type('facebook.com'), 'userid': text_type('2343')}],
+        'displayName': text_type('teddy'),
+        'verifiedEmail': text_type('we@po.pl'),
+        'preferredUsername': text_type('teddy'),
+        'emails': [{'value': text_type('aasd@bwwqwe.pl')}],
+        'name': text_type('ted')
     }
     credentials = {'oauthAccessToken': '7897048593434'}
-    provider_name = u'facebook'
-    provider_type = u'facebook'
+    provider_name = text_type('facebook')
+    provider_type = text_type('facebook')
     request = testing.DummyRequest()
     request.user = None
     request.registry = aftersocialregister_config.registry
-    request.remote_addr = u'127.0.0.123'
+    request.remote_addr = text_type('127.0.0.123')
     request.context = AuthenticationComplete(profile, credentials, provider_name, provider_type)
 
     request.login_perform = MagicMock(name='login_perform')
@@ -375,7 +413,10 @@ def test_aftersocialregister(aftersocialregister_config, aftersocialregister_app
 def aftersociallogin_config(evented_config):
     """Add AfterSocialLogIn event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AfterSocialLogIn)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterSocialLogIn)
+
 
 aftersociallogin_app = factories.pyramid_app('aftersociallogin_config')
 
@@ -383,20 +424,20 @@ aftersociallogin_app = factories.pyramid_app('aftersociallogin_config')
 def test_aftersociallogin(aftersociallogin_config, aftersociallogin_app, db_session):
     """Register fresh user and logs him in and check response if redirect from AfterSocialLogIn."""
     profile = {
-        'accounts': [{'domain': u'facebook.com', 'userid': u'2343'}],
-        'displayName': u'teddy',
-        'verifiedEmail': u'we@po.pl',
-        'preferredUsername': u'teddy',
-        'emails': [{'value': u'aasd@bwwqwe.pl'}],
-        'name': u'ted'
+        'accounts': [{'domain': text_type('facebook.com'), 'userid': text_type('2343')}],
+        'displayName': text_type('teddy'),
+        'verifiedEmail': text_type('we@po.pl'),
+        'preferredUsername': text_type('teddy'),
+        'emails': [{'value': text_type('aasd@bwwqwe.pl')}],
+        'name': text_type('ted')
     }
     credentials = {'oauthAccessToken': '7897048593434'}
-    provider_name = u'facebook'
-    provider_type = u'facebook'
+    provider_name = text_type('facebook')
+    provider_type = text_type('facebook')
     request = testing.DummyRequest()
     request.user = None
     request.registry = aftersociallogin_config.registry
-    request.remote_addr = u'127.0.0.123'
+    request.remote_addr = text_type('127.0.0.123')
     request.context = AuthenticationComplete(profile, credentials, provider_name, provider_type)
 
     def login_perform(*args, **kwargs):
@@ -417,18 +458,26 @@ def test_aftersociallogin(aftersociallogin_config, aftersociallogin_app, db_sess
 def alreadyconnected_config(evented_config):
     """Add SocialAccountAlreadyConnected event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, SocialAccountAlreadyConnected)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, SocialAccountAlreadyConnected)
+
 
 alreadyconnected_app = factories.pyramid_app('alreadyconnected_config')
 
 
 def test_alreadyconnected(alreadyconnected_config, alreadyconnected_app, facebook_user, db_session):
-    """Try to connect facebook account to logged in user used by other user check redirect from SocialAccountAlreadyConnected."""
+    """
+    Try to connect facebook account to logged in user used by other users facebook account.
+
+    Check redirect from SocialAccountAlreadyConnected.
+    """
     # this user will be logged and trying to connect facebook's user account.
     fresh_user = User(
-        email='new@user.pl',
-        password='somepassword',
-        address_ip='127.0.0.1')
+        email=text_type('new@user.pl'),
+        password=text_type('somepassword'),
+        address_ip=text_type('127.0.0.1')
+    )
     db_session.add(fresh_user)
     transaction.commit()
     user = db_session.merge(facebook_user)
@@ -436,21 +485,21 @@ def test_alreadyconnected(alreadyconnected_config, alreadyconnected_app, faceboo
 
     # mock request
     profile = {
-        'accounts': [{'domain': u'facebook.com', 'userid': user.provider_id('facebook')}],
-        'displayName': u'teddy',
-        'preferredUsername': u'teddy',
-        'emails': [{'value': u'aasd@basd.pl'}],
-        'name': u'ted'
+        'accounts': [{'domain': text_type('facebook.com'), 'userid': user.provider_id('facebook')}],
+        'displayName': text_type('teddy'),
+        'preferredUsername': text_type('teddy'),
+        'emails': [{'value': text_type('aasd@basd.pl')}],
+        'name': text_type('ted')
     }
     credentials = {'oauthAccessToken': '7897048593434'}
-    provider_name = u'facebook'
-    provider_type = u'facebook'
+    provider_name = text_type('facebook')
+    provider_type = text_type('facebook')
     request = testing.DummyRequest()
     request.user = fresh_user
     request.registry = alreadyconnected_config.registry
-    request.remote_addr = u'127.0.0.123'
+    request.remote_addr = text_type('127.0.0.123')
     request.context = AuthenticationComplete(profile, credentials, provider_name, provider_type)
-    request._ = lambda msg, *args, **kwargs: msg
+    request._ = mock_translate
 
     request.login_perform = MagicMock(name='login_perform')
     request.login_perform.return_value = {'status': True}
@@ -467,7 +516,10 @@ def test_alreadyconnected(alreadyconnected_config, alreadyconnected_app, faceboo
 def afteractivate_config(evented_config):
     """Add AfterActivate event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AfterActivate)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterActivate)
+
 
 afteractivate_app = factories.pyramid_app('afteractivate_config')
 
@@ -493,7 +545,10 @@ def test_afteractivate(user, db_session, afteractivate_app):
 def beforeregister_config(evented_config):
     """Add BeforeRegister event subscriber that raises AttributeError."""
     evented_config.add_subscriber(raise_attribute_error, BeforeRegister)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, BeforeRegister)
+
 
 beforeregister_app = factories.pyramid_app('beforeregister_config')
 
@@ -520,7 +575,10 @@ def test_beforeregister(db_session, beforeregister_app):
 def afterregister_config(evented_config):
     """Add AfterRegister event subscriber that redirects to event page."""
     evented_config.add_subscriber(redirect_to_secret, AfterRegister)
-    return evented_config
+    evented_config.commit()
+    yield evented_config
+    unregister_subscriber(evented_config, AfterRegister)
+
 
 afteraregister_app = factories.pyramid_app('afterregister_config')
 

tests/views/test_login.py

@@ -1,5 +1,7 @@
 """Log in related test."""
 import pytest
+import transaction
+
 from tests.tools import authenticate, is_user_logged, DEFAULT_USER
 
 
@@ -23,7 +25,11 @@ def test_login_view_secret(extended_app):
     assert 'email' in res.form.fields
 
 
-def test_login(active_user, extended_app):
+@pytest.mark.parametrize('email', (
+    DEFAULT_USER['email'].lower(),
+    DEFAULT_USER['email'].upper()
+))
+def test_login_ok(active_user, extended_app, email):
     """Actually log in test."""
     res = extended_app.get('/secret', status=302)
     res = res.follow()
@@ -31,7 +37,7 @@ def test_login(active_user, extended_app):
 
     assert is_user_logged(extended_app) is False
 
-    res = authenticate(extended_app)
+    res = authenticate(extended_app, email=email)
     assert 'Max-Age=' not in str(res)
 
     assert is_user_logged(extended_app) is True
@@ -149,3 +155,18 @@ def test_default_login_redirectaway(active_user, authable_app):
     authenticate(authable_app)
     res = authable_app.get('/login', status=303)
     assert res.location == 'http://localhost/'
+
+
+def test_login_invalid_cookie(db_session, active_user, extended_app):
+    """Test access login page by deleted user."""
+    res = authenticate(extended_app)
+    assert 'Max-Age=' not in str(res)
+
+    assert is_user_logged(extended_app) is True
+
+    db_session.delete(active_user)
+    transaction.commit()
+
+    # will rise Attribute error
+    res = extended_app.get('/login')
+    assert res.status_code == 200, "Should stay since user is no longer valid!"