mirror of
https://github.com/encode/django-rest-framework.git
synced 2025-08-04 20:40:14 +03:00
improve coverage by adding testcases for has_object_permission as well
This commit is contained in:
parent
0403e7f59b
commit
2d762fe3f4
|
@ -476,11 +476,17 @@ class AllowPermission(permissions.BasePermission):
|
||||||
def has_permission(self, request, view):
|
def has_permission(self, request, view):
|
||||||
return True
|
return True
|
||||||
|
|
||||||
|
def has_object_permission(self, request, view, obj):
|
||||||
|
return True
|
||||||
|
|
||||||
|
|
||||||
class DenyPermission(permissions.BasePermission):
|
class DenyPermission(permissions.BasePermission):
|
||||||
def has_permission(self, request, view):
|
def has_permission(self, request, view):
|
||||||
return False
|
return False
|
||||||
|
|
||||||
|
def has_object_permission(self, request, view, obj):
|
||||||
|
return False
|
||||||
|
|
||||||
|
|
||||||
class PermissionOperatorTests(TestCase):
|
class PermissionOperatorTests(TestCase):
|
||||||
def test_permission_logical_or(self):
|
def test_permission_logical_or(self):
|
||||||
|
@ -492,3 +498,8 @@ class PermissionOperatorTests(TestCase):
|
||||||
self.assertTrue((AllowPermission | DenyPermission)().has_permission(None, None))
|
self.assertTrue((AllowPermission | DenyPermission)().has_permission(None, None))
|
||||||
self.assertTrue((DenyPermission | AllowPermission)().has_permission(None, None))
|
self.assertTrue((DenyPermission | AllowPermission)().has_permission(None, None))
|
||||||
self.assertTrue((AllowPermission | AllowPermission)().has_permission(None, None))
|
self.assertTrue((AllowPermission | AllowPermission)().has_permission(None, None))
|
||||||
|
|
||||||
|
self.assertFalse((DenyPermission | DenyPermission)().has_object_permission(None, None, None))
|
||||||
|
self.assertTrue((AllowPermission | DenyPermission)().has_object_permission(None, None, None))
|
||||||
|
self.assertTrue((DenyPermission | AllowPermission)().has_object_permission(None, None, None))
|
||||||
|
self.assertTrue((AllowPermission | AllowPermission)().has_object_permission(None, None, None))
|
||||||
|
|
Loading…
Reference in New Issue
Block a user