Tests for short and long usernames.
This commit is contained in:
parent
d756206541
commit
bf2f121a83
@ -373,7 +373,7 @@ def test_authentication_views(test_app):
|
|||||||
assert form.username.data == u'andrew'
|
assert form.username.data == u'andrew'
|
||||||
|
|
||||||
# Successful login with short user
|
# Successful login with short user
|
||||||
# ----------------
|
# --------------------------------
|
||||||
short_user = fixture_add_user(username=u'me', password=u'sho')
|
short_user = fixture_add_user(username=u'me', password=u'sho')
|
||||||
template.clear_test_template_context()
|
template.clear_test_template_context()
|
||||||
response = test_app.post(
|
response = test_app.post(
|
||||||
@ -381,11 +381,9 @@ def test_authentication_views(test_app):
|
|||||||
'username': u'me',
|
'username': u'me',
|
||||||
'password': 'sho'})
|
'password': 'sho'})
|
||||||
|
|
||||||
context = template.TEMPLATE_TEST_CONTEXT['mediagoblin/auth/login.html']
|
|
||||||
form = context['login_form']
|
|
||||||
# User should be redirected
|
# User should be redirected
|
||||||
print('errors are', form.username.errors)
|
|
||||||
response.follow()
|
response.follow()
|
||||||
|
|
||||||
assert urlparse.urlsplit(response.location)[2] == '/'
|
assert urlparse.urlsplit(response.location)[2] == '/'
|
||||||
assert 'mediagoblin/root.html' in template.TEMPLATE_TEST_CONTEXT
|
assert 'mediagoblin/root.html' in template.TEMPLATE_TEST_CONTEXT
|
||||||
|
|
||||||
@ -394,6 +392,10 @@ def test_authentication_views(test_app):
|
|||||||
session = context['request'].session
|
session = context['request'].session
|
||||||
assert session['user_id'] == six.text_type(short_user.id)
|
assert session['user_id'] == six.text_type(short_user.id)
|
||||||
|
|
||||||
|
# Must logout
|
||||||
|
template.clear_test_template_context()
|
||||||
|
response = test_app.get('/auth/logout/')
|
||||||
|
|
||||||
# Successful login with long user
|
# Successful login with long user
|
||||||
# ----------------
|
# ----------------
|
||||||
long_user = fixture_add_user(
|
long_user = fixture_add_user(
|
||||||
@ -401,13 +403,10 @@ def test_authentication_views(test_app):
|
|||||||
template.clear_test_template_context()
|
template.clear_test_template_context()
|
||||||
response = test_app.post(
|
response = test_app.post(
|
||||||
'/auth/login/', {
|
'/auth/login/', {
|
||||||
'username': u'me',
|
'username': u'realllylonguser@reallylongdomain.com.co',
|
||||||
'password': 'sho'})
|
'password': 'sho'})
|
||||||
|
|
||||||
context = template.TEMPLATE_TEST_CONTEXT['mediagoblin/auth/login.html']
|
|
||||||
form = context['login_form']
|
|
||||||
# User should be redirected
|
# User should be redirected
|
||||||
print('errors are', form.username.errors)
|
|
||||||
response.follow()
|
response.follow()
|
||||||
assert urlparse.urlsplit(response.location)[2] == '/'
|
assert urlparse.urlsplit(response.location)[2] == '/'
|
||||||
assert 'mediagoblin/root.html' in template.TEMPLATE_TEST_CONTEXT
|
assert 'mediagoblin/root.html' in template.TEMPLATE_TEST_CONTEXT
|
||||||
@ -415,7 +414,10 @@ def test_authentication_views(test_app):
|
|||||||
# Make sure user is in the session
|
# Make sure user is in the session
|
||||||
context = template.TEMPLATE_TEST_CONTEXT['mediagoblin/root.html']
|
context = template.TEMPLATE_TEST_CONTEXT['mediagoblin/root.html']
|
||||||
session = context['request'].session
|
session = context['request'].session
|
||||||
assert session['user_id'] == six.text_type(short_user.id)
|
assert session['user_id'] == six.text_type(long_user.id)
|
||||||
|
|
||||||
|
template.clear_test_template_context()
|
||||||
|
response = test_app.get('/auth/logout/')
|
||||||
|
|
||||||
@pytest.fixture()
|
@pytest.fixture()
|
||||||
def authentication_disabled_app(request):
|
def authentication_disabled_app(request):
|
||||||
|
Loading…
x
Reference in New Issue
Block a user