Test is failing for some reason.

Author: kennethlove

braces/views/_access.py

@@ -198,17 +198,11 @@ def check_permissions(self, request):
 
         if self.object_level_permissions:
             if hasattr(self, "object") and self.object is not None:
-                has_permission = request.user.has_perm(
-                    self.get_permission_required(request), self.object
-                )
+                has_permission = request.user.has_perm(perms, self.object)
             elif hasattr(self, "get_object") and callable(self.get_object):
-                has_permission = request.user.has_perm(
-                    self.get_permission_required(request), self.get_object()
-                )
+                has_permission = request.user.has_perm(perms, self.get_object())
         else:
-            has_permission = request.user.has_perm(
-                self.get_permission_required(request)
-            )
+            has_permission = request.user.has_perm(perms)
         return has_permission
 
     def dispatch(self, request, *args, **kwargs):

tests/test_access_mixins.py

@@ -25,7 +25,7 @@
     AnonymousRequiredView,
     SSLRequiredView,
     RecentLoginRequiredView,
-    UserPassesTestLoginRequiredView,
+    UserPassesTestLoginRequiredView, ObjectPermissionRequiredView,
 )
 
 
@@ -414,6 +414,22 @@ def test_invalid_permission(self):
             self.dispatch_view(self.build_request(), permission_required=None)
 
 
+@pytest.mark.django_db
+class TestObjectPermissionRequiredMixin(_TestAccessBasicsMixin, test.TestCase):
+    """Scenarios around requiring an object permission"""
+
+    view_class = ObjectPermissionRequiredView
+    view_url = "/object_permission_required/1/"
+
+    def build_authorized_user(self):
+        """Create a user with permissions"""
+        return UserFactory(permissions=["auth.add_user"])
+
+    def build_unauthorized_user(self):
+        """Create a user without permissions"""
+        return UserFactory()
+
+
 @pytest.mark.django_db
 class TestMultiplePermissionsRequiredMixin(
     _TestAccessBasicsMixin, test.TestCase

tests/urls.py

@@ -70,6 +70,7 @@
     path("context/", views.ContextView.as_view(), name="context"),
     # PermissionRequiredMixin tests
     path("permission_required/", views.PermissionRequiredView.as_view()),
+    re_path(r"^object_permission_required/(?P<pk>[\w-]+)/$", views.ObjectPermissionRequiredView.as_view()),
     # MultiplePermissionsRequiredMixin tests
     path(
         "multiple_permissions_required/",

tests/views.py

@@ -293,6 +293,20 @@ class PermissionRequiredView(views.PermissionRequiredMixin, OkView):
     permission_required = "auth.add_user"
 
 
+class ObjectPermissionRequiredView(views.PermissionRequiredMixin, DetailView):
+    """
+    View for testing PermissionRequiredMixin.
+    """
+
+    model = User
+    permission_required = "auth.add_user"
+    template_name = "blank.html"
+    object_level_permissions = True
+
+    def get_object(self, **kwargs):
+        return User.objects.first()
+
+
 class MultiplePermissionsRequiredView(
     views.MultiplePermissionsRequiredMixin, OkView
 ):