Use anonymous_user fixture in other tests
This commit is contained in:
parent
122c39075a
commit
1f2e14b20e
|
@ -2,7 +2,6 @@ import pytest
|
||||||
|
|
||||||
from rest_framework.views import APIView
|
from rest_framework.views import APIView
|
||||||
|
|
||||||
from django.contrib.auth.models import AnonymousUser
|
|
||||||
from django.http import Http404
|
from django.http import Http404
|
||||||
|
|
||||||
from funkwhale_api.common import permissions
|
from funkwhale_api.common import permissions
|
||||||
|
@ -19,24 +18,26 @@ def test_owner_permission_owner_field_ok(nodb_factories, api_request):
|
||||||
assert check is True
|
assert check is True
|
||||||
|
|
||||||
|
|
||||||
def test_owner_permission_owner_field_not_ok(nodb_factories, api_request):
|
def test_owner_permission_owner_field_not_ok(
|
||||||
|
anonymous_user, nodb_factories, api_request):
|
||||||
playlist = nodb_factories['playlists.Playlist']()
|
playlist = nodb_factories['playlists.Playlist']()
|
||||||
view = APIView.as_view()
|
view = APIView.as_view()
|
||||||
permission = permissions.OwnerPermission()
|
permission = permissions.OwnerPermission()
|
||||||
request = api_request.get('/')
|
request = api_request.get('/')
|
||||||
setattr(request, 'user', AnonymousUser())
|
setattr(request, 'user', anonymous_user)
|
||||||
|
|
||||||
with pytest.raises(Http404):
|
with pytest.raises(Http404):
|
||||||
permission.has_object_permission(request, view, playlist)
|
permission.has_object_permission(request, view, playlist)
|
||||||
|
|
||||||
|
|
||||||
def test_owner_permission_read_only(nodb_factories, api_request):
|
def test_owner_permission_read_only(
|
||||||
|
anonymous_user, nodb_factories, api_request):
|
||||||
playlist = nodb_factories['playlists.Playlist']()
|
playlist = nodb_factories['playlists.Playlist']()
|
||||||
view = APIView.as_view()
|
view = APIView.as_view()
|
||||||
setattr(view, 'owner_checks', ['write'])
|
setattr(view, 'owner_checks', ['write'])
|
||||||
permission = permissions.OwnerPermission()
|
permission = permissions.OwnerPermission()
|
||||||
request = api_request.get('/')
|
request = api_request.get('/')
|
||||||
setattr(request, 'user', AnonymousUser())
|
setattr(request, 'user', anonymous_user)
|
||||||
check = permission.has_object_permission(request, view, playlist)
|
check = permission.has_object_permission(request, view, playlist)
|
||||||
|
|
||||||
assert check is True
|
assert check is True
|
||||||
|
|
Loading…
Reference in New Issue