From 2d762fe3f43bfd6da939207c6acb39f5deee70ec Mon Sep 17 00:00:00 2001 From: Lorin Werthen Date: Fri, 15 Jul 2016 13:47:36 +0200 Subject: [PATCH] improve coverage by adding testcases for has_object_permission as well --- tests/test_permissions.py | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/tests/test_permissions.py b/tests/test_permissions.py index 0c3f1cf81..f58bbe4f1 100644 --- a/tests/test_permissions.py +++ b/tests/test_permissions.py @@ -476,11 +476,17 @@ class AllowPermission(permissions.BasePermission): def has_permission(self, request, view): return True + def has_object_permission(self, request, view, obj): + return True + class DenyPermission(permissions.BasePermission): def has_permission(self, request, view): return False + def has_object_permission(self, request, view, obj): + return False + class PermissionOperatorTests(TestCase): def test_permission_logical_or(self): @@ -492,3 +498,8 @@ class PermissionOperatorTests(TestCase): self.assertTrue((AllowPermission | DenyPermission)().has_permission(None, None)) self.assertTrue((DenyPermission | 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))