Add check for current user not authenticated

This commit is contained in:
Imdad Ahad
2016-10-24 17:36:53 +01:00
parent 76a61ff737
commit 2b5894bed9

View File

@@ -1,7 +1,7 @@
import pytest import pytest
from app.utils import user_has_permissions from app.utils import user_has_permissions
from app.main.views.index import index from app.main.views.index import index
from werkzeug.exceptions import Forbidden from werkzeug.exceptions import Forbidden, Unauthorized
from flask import request from flask import request
@@ -9,7 +9,8 @@ def _test_permissions(app_, usr, permissions, service_id, will_succeed, any_=Fal
with app_.test_request_context() as ctx: with app_.test_request_context() as ctx:
request.view_args.update({'service_id': service_id}) request.view_args.update({'service_id': service_id})
with app_.test_client() as client: with app_.test_client() as client:
client.login(usr) if usr:
client.login(usr)
decorator = user_has_permissions(*permissions, any_=any_, admin_override=admin_override) decorator = user_has_permissions(*permissions, any_=any_, admin_override=admin_override)
decorated_index = decorator(index) decorated_index = decorator(index)
if will_succeed: if will_succeed:
@@ -17,8 +18,8 @@ def _test_permissions(app_, usr, permissions, service_id, will_succeed, any_=Fal
else: else:
try: try:
response = decorated_index() response = decorated_index()
pytest.fail("Failed to throw a forbidden exception") pytest.fail("Failed to throw a forbidden or unauthorised exception")
except Forbidden: except (Forbidden, Unauthorized):
pass pass
@@ -107,6 +108,17 @@ def test_platform_admin_user_can_not_access_page(app_,
admin_override=False) admin_override=False)
def test_user_with_permissions_returns_401_unauthenticated_user(app_):
from flask_login import current_user
assert not current_user
_test_permissions(
app_,
None,
[],
'',
will_succeed=False)
def _user_with_permissions(): def _user_with_permissions():
from app.notify_client.user_api_client import User from app.notify_client.user_api_client import User