From fa0ef1773773c58b5708abad0e90a44fc9a308f8 Mon Sep 17 00:00:00 2001
From: Carlton Gibson
Date: Tue, 2 Sep 2014 14:53:37 +0200
Subject: Remove Login Dropdown when Auth Views are not registered.
Fixes #1738
---
tests/browsable_api/__init__.py | 0
tests/browsable_api/auth_urls.py | 10 +++++
tests/browsable_api/no_auth_urls.py | 9 +++++
tests/browsable_api/test_browsable_api.py | 65 +++++++++++++++++++++++++++++++
tests/browsable_api/views.py | 15 +++++++
5 files changed, 99 insertions(+)
create mode 100644 tests/browsable_api/__init__.py
create mode 100644 tests/browsable_api/auth_urls.py
create mode 100644 tests/browsable_api/no_auth_urls.py
create mode 100644 tests/browsable_api/test_browsable_api.py
create mode 100644 tests/browsable_api/views.py
(limited to 'tests')
diff --git a/tests/browsable_api/__init__.py b/tests/browsable_api/__init__.py
new file mode 100644
index 00000000..e69de29b
diff --git a/tests/browsable_api/auth_urls.py b/tests/browsable_api/auth_urls.py
new file mode 100644
index 00000000..3cb2d8da
--- /dev/null
+++ b/tests/browsable_api/auth_urls.py
@@ -0,0 +1,10 @@
+from __future__ import unicode_literals
+from django.conf.urls import patterns, url, include
+
+from views import MockView
+
+urlpatterns = patterns(
+ '',
+ (r'^$', MockView.as_view()),
+ url(r'^auth/', include('rest_framework.urls', namespace='rest_framework')),
+)
diff --git a/tests/browsable_api/no_auth_urls.py b/tests/browsable_api/no_auth_urls.py
new file mode 100644
index 00000000..d9b9bc43
--- /dev/null
+++ b/tests/browsable_api/no_auth_urls.py
@@ -0,0 +1,9 @@
+from __future__ import unicode_literals
+from django.conf.urls import patterns
+
+from views import MockView
+
+urlpatterns = patterns(
+ '',
+ (r'^$', MockView.as_view()),
+)
diff --git a/tests/browsable_api/test_browsable_api.py b/tests/browsable_api/test_browsable_api.py
new file mode 100644
index 00000000..61c76b4f
--- /dev/null
+++ b/tests/browsable_api/test_browsable_api.py
@@ -0,0 +1,65 @@
+from __future__ import unicode_literals
+from django.contrib.auth.models import User
+from django.test import TestCase
+
+from rest_framework.test import APIClient
+
+
+class DropdownWithAuthTests(TestCase):
+ """Tests correct dropdown behaviour with Auth views enabled."""
+
+ urls = 'tests.browsable_api.auth_urls'
+
+ def setUp(self):
+ self.client = APIClient(enforce_csrf_checks=True)
+ self.username = 'john'
+ self.email = 'lennon@thebeatles.com'
+ self.password = 'password'
+ self.user = User.objects.create_user(self.username, self.email, self.password)
+
+ def tearDown(self):
+ self.client.logout()
+
+ def test_name_shown_when_logged_in(self):
+ self.client.login(username=self.username, password=self.password)
+ response = self.client.get('/')
+ self.assertContains(response, 'john')
+
+ def test_logout_shown_when_logged_in(self):
+ self.client.login(username=self.username, password=self.password)
+ response = self.client.get('/')
+ self.assertContains(response, '>Log out<')
+
+ def test_login_shown_when_logged_out(self):
+ response = self.client.get('/')
+ self.assertContains(response, '>Log in<')
+
+
+class NoDropdownWithoutAuthTests(TestCase):
+ """Tests correct dropdown behaviour with Auth views enabled."""
+
+ urls = 'tests.browsable_api.no_auth_urls'
+
+ def setUp(self):
+ self.client = APIClient(enforce_csrf_checks=True)
+ self.username = 'john'
+ self.email = 'lennon@thebeatles.com'
+ self.password = 'password'
+ self.user = User.objects.create_user(self.username, self.email, self.password)
+
+ def tearDown(self):
+ self.client.logout()
+
+ def test_name_shown_when_logged_in(self):
+ self.client.login(username=self.username, password=self.password)
+ response = self.client.get('/')
+ self.assertContains(response, 'john')
+
+ def test_dropdown_not_shown_when_logged_in(self):
+ self.client.login(username=self.username, password=self.password)
+ response = self.client.get('/')
+ self.assertNotContains(response, '')
+
+ def test_dropdown_not_shown_when_logged_out(self):
+ response = self.client.get('/')
+ self.assertNotContains(response, '')
diff --git a/tests/browsable_api/views.py b/tests/browsable_api/views.py
new file mode 100644
index 00000000..000f4e80
--- /dev/null
+++ b/tests/browsable_api/views.py
@@ -0,0 +1,15 @@
+from __future__ import unicode_literals
+
+from rest_framework.views import APIView
+from rest_framework import authentication
+from rest_framework import renderers
+from rest_framework.response import Response
+
+
+class MockView(APIView):
+
+ authentication_classes = (authentication.SessionAuthentication,)
+ renderer_classes = (renderers.BrowsableAPIRenderer,)
+
+ def get(self, request):
+ return Response({'a': 1, 'b': 2, 'c': 3})
--
cgit v1.2.3
From 4836afa89f045263ded91946d43403e807880181 Mon Sep 17 00:00:00 2001
From: Carlton Gibson
Date: Tue, 2 Sep 2014 15:11:23 +0200
Subject: Fix docstring
---
tests/browsable_api/test_browsable_api.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
(limited to 'tests')
diff --git a/tests/browsable_api/test_browsable_api.py b/tests/browsable_api/test_browsable_api.py
index 61c76b4f..5f264783 100644
--- a/tests/browsable_api/test_browsable_api.py
+++ b/tests/browsable_api/test_browsable_api.py
@@ -36,7 +36,7 @@ class DropdownWithAuthTests(TestCase):
class NoDropdownWithoutAuthTests(TestCase):
- """Tests correct dropdown behaviour with Auth views enabled."""
+ """Tests correct dropdown behaviour with Auth views NOT enabled."""
urls = 'tests.browsable_api.no_auth_urls'
--
cgit v1.2.3
From 1e491c82e6ddf2941d1f8639920c6b5dfd9378f8 Mon Sep 17 00:00:00 2001
From: Carlton Gibson
Date: Tue, 2 Sep 2014 15:29:21 +0200
Subject: Fix imports
---
tests/browsable_api/auth_urls.py | 2 +-
tests/browsable_api/no_auth_urls.py | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
(limited to 'tests')
diff --git a/tests/browsable_api/auth_urls.py b/tests/browsable_api/auth_urls.py
index 3cb2d8da..bce7dcf9 100644
--- a/tests/browsable_api/auth_urls.py
+++ b/tests/browsable_api/auth_urls.py
@@ -1,7 +1,7 @@
from __future__ import unicode_literals
from django.conf.urls import patterns, url, include
-from views import MockView
+from .views import MockView
urlpatterns = patterns(
'',
diff --git a/tests/browsable_api/no_auth_urls.py b/tests/browsable_api/no_auth_urls.py
index d9b9bc43..5e3604a6 100644
--- a/tests/browsable_api/no_auth_urls.py
+++ b/tests/browsable_api/no_auth_urls.py
@@ -1,7 +1,7 @@
from __future__ import unicode_literals
from django.conf.urls import patterns
-from views import MockView
+from .views import MockView
urlpatterns = patterns(
'',
--
cgit v1.2.3