diff options
| author | Ben Lomax <lomax.on.the.run@gmail.com> | 2023-04-26 07:08:33 +0100 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2023-05-20 15:52:00 +0200 |
| commit | 00f5d2d110712af84fae2c5f9183a2ea48ce0a4a (patch) | |
| tree | fcfce5199b8e76bbc9b24d4d2db05f677de6da9e /tests/decorators | |
| parent | b43936f2ec37f47277899a91a5464091bd80ad3a (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.py | 75 |
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")) |
