summaryrefslogtreecommitdiff
path: root/tests/decorators
diff options
context:
space:
mode:
authorBen Lomax <lomax.on.the.run@gmail.com>2023-04-26 07:08:33 +0100
committerMariusz Felisiak <felisiak.mariusz@gmail.com>2023-05-20 15:52:00 +0200
commit00f5d2d110712af84fae2c5f9183a2ea48ce0a4a (patch)
treefcfce5199b8e76bbc9b24d4d2db05f677de6da9e /tests/decorators
parentb43936f2ec37f47277899a91a5464091bd80ad3a (diff)
Refs #31949 -- Made @xframe_options_(deny/sameorigin/exempt) decorators to work with async functions.
Diffstat (limited to 'tests/decorators')
-rw-r--r--tests/decorators/test_clickjacking.py75
1 files changed, 75 insertions, 0 deletions
diff --git a/tests/decorators/test_clickjacking.py b/tests/decorators/test_clickjacking.py
index 278950082d..fe4c551648 100644
--- a/tests/decorators/test_clickjacking.py
+++ b/tests/decorators/test_clickjacking.py
@@ -1,3 +1,5 @@
+from asgiref.sync import iscoroutinefunction
+
from django.http import HttpRequest, HttpResponse
from django.middleware.clickjacking import XFrameOptionsMiddleware
from django.test import SimpleTestCase
@@ -9,6 +11,20 @@ from django.views.decorators.clickjacking import (
class XFrameOptionsDenyTests(SimpleTestCase):
+ def test_wrapped_sync_function_is_not_coroutine_function(self):
+ def sync_view(request):
+ return HttpResponse()
+
+ wrapped_view = xframe_options_deny(sync_view)
+ self.assertIs(iscoroutinefunction(wrapped_view), False)
+
+ def test_wrapped_async_function_is_coroutine_function(self):
+ async def async_view(request):
+ return HttpResponse()
+
+ wrapped_view = xframe_options_deny(async_view)
+ self.assertIs(iscoroutinefunction(wrapped_view), True)
+
def test_decorator_sets_x_frame_options_to_deny(self):
@xframe_options_deny
def a_view(request):
@@ -17,8 +33,30 @@ class XFrameOptionsDenyTests(SimpleTestCase):
response = a_view(HttpRequest())
self.assertEqual(response.headers["X-Frame-Options"], "DENY")
+ async def test_decorator_sets_x_frame_options_to_deny_async_view(self):
+ @xframe_options_deny
+ async def an_async_view(request):
+ return HttpResponse()
+
+ response = await an_async_view(HttpRequest())
+ self.assertEqual(response.headers["X-Frame-Options"], "DENY")
+
class XFrameOptionsSameoriginTests(SimpleTestCase):
+ def test_wrapped_sync_function_is_not_coroutine_function(self):
+ def sync_view(request):
+ return HttpResponse()
+
+ wrapped_view = xframe_options_sameorigin(sync_view)
+ self.assertIs(iscoroutinefunction(wrapped_view), False)
+
+ def test_wrapped_async_function_is_coroutine_function(self):
+ async def async_view(request):
+ return HttpResponse()
+
+ wrapped_view = xframe_options_sameorigin(async_view)
+ self.assertIs(iscoroutinefunction(wrapped_view), True)
+
def test_decorator_sets_x_frame_options_to_sameorigin(self):
@xframe_options_sameorigin
def a_view(request):
@@ -27,8 +65,30 @@ class XFrameOptionsSameoriginTests(SimpleTestCase):
response = a_view(HttpRequest())
self.assertEqual(response.headers["X-Frame-Options"], "SAMEORIGIN")
+ async def test_decorator_sets_x_frame_options_to_sameorigin_async_view(self):
+ @xframe_options_sameorigin
+ async def an_async_view(request):
+ return HttpResponse()
+
+ response = await an_async_view(HttpRequest())
+ self.assertEqual(response.headers["X-Frame-Options"], "SAMEORIGIN")
+
class XFrameOptionsExemptTests(SimpleTestCase):
+ def test_wrapped_sync_function_is_not_coroutine_function(self):
+ def sync_view(request):
+ return HttpResponse()
+
+ wrapped_view = xframe_options_exempt(sync_view)
+ self.assertIs(iscoroutinefunction(wrapped_view), False)
+
+ def test_wrapped_async_function_is_coroutine_function(self):
+ async def async_view(request):
+ return HttpResponse()
+
+ wrapped_view = xframe_options_exempt(async_view)
+ self.assertIs(iscoroutinefunction(wrapped_view), True)
+
def test_decorator_stops_x_frame_options_being_set(self):
"""
@xframe_options_exempt instructs the XFrameOptionsMiddleware to NOT set
@@ -48,3 +108,18 @@ class XFrameOptionsExemptTests(SimpleTestCase):
# middleware's functionality.
middleware_response = XFrameOptionsMiddleware(a_view)(request)
self.assertIsNone(middleware_response.get("X-Frame-Options"))
+
+ async def test_exempt_decorator_async_view(self):
+ @xframe_options_exempt
+ async def an_async_view(request):
+ return HttpResponse()
+
+ request = HttpRequest()
+ response = await an_async_view(request)
+ self.assertIsNone(response.get("X-Frame-Options"))
+ self.assertIs(response.xframe_options_exempt, True)
+
+ # The real purpose of the exempt decorator is to suppress the
+ # middleware's functionality.
+ middleware_response = await XFrameOptionsMiddleware(an_async_view)(request)
+ self.assertIsNone(middleware_response.get("X-Frame-Options"))