From 54d82f59ed8a5d2ad4c679680dc52b8a94831d50 Mon Sep 17 00:00:00 2001
From: Tom Christie
Date: Mon, 9 Feb 2015 17:19:22 +0000
Subject: Py3 compat fix

---
 tests/test_request.py | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

(limited to 'tests/test_request.py')

diff --git a/tests/test_request.py b/tests/test_request.py
index 02a9b1e2..06ad8e93 100644
--- a/tests/test_request.py
+++ b/tests/test_request.py
@@ -249,6 +249,26 @@ class TestUserSetter(TestCase):
         login(self.request, self.user)
         self.assertEqual(self.wrapped_request.user, self.user)
 
+    def test_calling_user_fails_when_attribute_error_is_raised(self):
+        """
+        This proves that when an AttributeError is raised inside of the request.user
+        property, that we can handle this and report the true, underlying error.
+        """
+        class AuthRaisesAttributeError(object):
+            def authenticate(self, request):
+                import rest_framework
+                rest_framework.MISSPELLED_NAME_THAT_DOESNT_EXIST
+
+        self.request = Request(factory.get('/'), authenticators=(AuthRaisesAttributeError(),))
+        SessionMiddleware().process_request(self.request)
+
+        login(self.request, self.user)
+        try:
+            self.request.user
+        except AttributeError as error:
+            self.assertEqual(str(error), "'module' object has no attribute 'MISSPELLED_NAME_THAT_DOESNT_EXIST'")
+        else:
+            assert False, 'AttributeError not raised'
 
 class TestAuthSetter(TestCase):
 
-- 
cgit v1.2.3


From 0669f507b3a63114cf429f0662b1781f0e1fa5f8 Mon Sep 17 00:00:00 2001
From: Tom Christie
Date: Mon, 9 Feb 2015 17:22:13 +0000
Subject: pep8 fix

---
 tests/test_request.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'tests/test_request.py')

diff --git a/tests/test_request.py b/tests/test_request.py
index 06ad8e93..c274ab69 100644
--- a/tests/test_request.py
+++ b/tests/test_request.py
@@ -270,8 +270,8 @@ class TestUserSetter(TestCase):
         else:
             assert False, 'AttributeError not raised'
 
-class TestAuthSetter(TestCase):
 
+class TestAuthSetter(TestCase):
     def test_auth_can_be_set(self):
         request = Request(factory.get('/'))
         request.auth = 'DUMMY'
-- 
cgit v1.2.3