fix for reverse_lazy on access mixins

braces/views.py

@@ -10,6 +10,7 @@
 from django.core.urlresolvers import reverse
 from django.http import HttpResponse
 from django.utils.decorators import method_decorator
+from django.utils.encoding import force_text
 from django.views.generic import CreateView
 from django.views.decorators.csrf import csrf_exempt
 
@@ -67,7 +68,7 @@ def get_login_url(self):
                 "Define %(cls)s.login_url or override "
                 "%(cls)s.get_login_url()." % {"cls": self.__class__.__name__})
 
-        return self.login_url
+        return force_text(self.login_url)
 
     def get_redirect_field_name(self):
         """

tests/test_access_mixins.py

@@ -1,5 +1,6 @@
 from django import test
 from django.core.exceptions import ImproperlyConfigured, PermissionDenied
+from django.core.urlresolvers import reverse_lazy
 from .compat import force_text
 from .factories import make_group, make_user
 from .helpers import TestViewHelper
@@ -66,6 +67,11 @@ def test_custom_login_url(self):
         resp = self.dispatch_view(req, login_url='/login/')
         self.assertEqual('/login/?next=%s' % self.view_url, resp['Location'])
 
+        # Test with reverse_lazy
+        resp = self.dispatch_view(req, login_url=reverse_lazy('headline'))
+        self.assertEqual('/headline/?next={}'.format(
+            self.view_url), resp['Location'])
+
     def test_custom_redirect_field_name(self):
         """
         Redirect field name should be customizable.

tests/urls.py

@@ -26,7 +26,7 @@
     url(r'^form_with_user_kwarg/$', views.FormWithUserKwargView.as_view()),
 
     # SetHeadlineMixin tests
-    url(r'^headline/$', views.HeadlineView.as_view()),
+    url(r'^headline/$', views.HeadlineView.as_view(), name='headline'),
     url(r'^headline/(?P<s>[\w-]+)/$', views.DynamicHeadlineView.as_view()),
 
     # PermissionRequiredMixin tests