wagtail 7.2.1__py3-none-any.whl → 7.3__py3-none-any.whl

wagtail/images/tests/test_admin_views.py

@@ -21,6 +21,7 @@ from willow.optimizers.base import OptimizerBase
 from willow.registry import registry
 
 from wagtail.admin.admin_url_finder import AdminURLFinder
+from wagtail.admin.staticfiles import versioned_static
 from wagtail.admin.ui.tables import Table
 from wagtail.images import get_image_model
 from wagtail.images.utils import generate_signature
@@ -60,7 +61,7 @@ class TestImageIndexView(WagtailTestUtils, TestCase):
             file=get_test_image_file(size=(1, 1)),
         )
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages:index"), params)
 
     def test_simple(self):
@@ -656,6 +657,20 @@ class TestImageIndexView(WagtailTestUtils, TestCase):
             [unused_image],
         )
 
+    def test_bulk_action_rendered(self):
+        response = self.get()
+        self.assertEqual(response.status_code, 200)
+        # Should render bulk actions markup
+        bulk_actions_js = versioned_static("wagtailadmin/js/bulk-actions.js")
+        soup = self.get_soup(response.content)
+        script = soup.select_one(f"script[src='{bulk_actions_js}']")
+        self.assertIsNotNone(script)
+        bulk_actions = soup.select("[data-bulk-action-button]")
+        self.assertTrue(bulk_actions)
+        # 'next' parameter is constructed client-side later based on filters state
+        for action in bulk_actions:
+            self.assertNotIn("next=", action["href"])
+
 
 class TestBulkActionsColumn(WagtailTestUtils, TestCase):
     def setUp(self):
@@ -710,7 +725,7 @@ class TestImageIndexViewSearch(WagtailTestUtils, TransactionTestCase):
         self.puppy_image.created_at = local_datetime(2022, 2, 2)
         self.puppy_image.save()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages:index"), params)
 
     def test_search(self):
@@ -876,7 +891,7 @@ class TestImageListingResultsView(WagtailTestUtils, TransactionTestCase):
     def setUp(self):
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages:index_results"), params)
 
     def test_search(self):
@@ -898,10 +913,10 @@ class TestImageAddView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
     def setUp(self):
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages:add"), params)
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(reverse("wagtailimages:add"), post_data)
 
     def test_get(self):
@@ -1280,10 +1295,10 @@ class TestImageAddViewWithLimitedCollectionPermissions(WagtailTestUtils, TestCas
 
         self.login(username="moriarty", password="password")
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages:add"), params)
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(reverse("wagtailimages:add"), post_data)
 
     def test_get(self):
@@ -1348,12 +1363,12 @@ class TestImageEditView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
     def update_from_db(self):
         self.image = Image.objects.get(pk=self.image.pk)
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailimages:edit", args=(self.image.id,)), params
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse("wagtailimages:edit", args=(self.image.id,)), post_data
         )
@@ -1739,7 +1754,7 @@ class TestImageEditViewWithCustomImageModel(WagtailTestUtils, TestCase):
 
         self.storage = self.image.file.storage
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailimages:edit", args=(self.image.id,)), params
         )
@@ -1768,10 +1783,10 @@ class TestImageDeleteView(WagtailTestUtils, TestCase):
 
         self.delete_url = reverse("wagtailimages:delete", args=(self.image.id,))
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(self.delete_url, params)
 
-    def post(self, post_data={}, **kwargs):
+    def post(self, post_data=None, **kwargs):
         return self.client.post(self.delete_url, post_data, **kwargs)
 
     def test_simple(self):
@@ -1987,7 +2002,7 @@ class TestImageChooserView(WagtailTestUtils, TestCase):
     def setUp(self):
         self.user = self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages_chooser:choose"), params)
 
     def test_simple(self):
@@ -2288,7 +2303,7 @@ class TestImageChooserViewSearch(WagtailTestUtils, TransactionTestCase):
     def setUp(self):
         self.user = self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages_chooser:choose"), params)
 
     def test_construct_queryset_hook_search(self):
@@ -2324,7 +2339,7 @@ class TestImageChooserChosenView(WagtailTestUtils, TestCase):
             description="Test description",
         )
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailimages_chooser:chosen", args=(self.image.id,)), params
         )
@@ -2383,7 +2398,7 @@ class TestImageChooserChosenMultipleView(WagtailTestUtils, TestCase):
             description="Unchosen test description",
         )
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             "%s?id=%d&id=%d"
             % (
@@ -2416,13 +2431,13 @@ class TestImageChooserSelectFormatView(WagtailTestUtils, TestCase):
             file=get_test_image_file(),
         )
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailimages_chooser:select_format", args=(self.image.id,)),
             params,
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse("wagtailimages_chooser:select_format", args=(self.image.id,)),
             post_data,
@@ -2505,7 +2520,7 @@ class TestImageChooserUploadView(WagtailTestUtils, TestCase):
     def setUp(self):
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailimages_chooser:create"), params)
 
     def test_simple(self):
@@ -3943,8 +3958,46 @@ class TestURLGeneratorView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         # Check response
         self.assertEqual(response.status_code, 200)
         self.assertTemplateUsed(response, "wagtailimages/images/url_generator.html")
+        self.assertTemplateUsed(
+            response, "wagtailimages/images/url_generator_output.html"
+        )
         self.assertTemplateUsed(response, "wagtailadmin/generic/base.html")
 
+        # Check form defaults are based on image with correct defaults are provided to context
+        self.assertEqual(response.context["form"].initial["width"], self.image.width)
+        self.assertEqual(response.context["form"].initial["height"], self.image.height)
+        self.assertEqual(response.context["form"].initial["filter_method"], "original")
+        self.assertEqual(response.context["form"].initial["closeness"], "0")
+
+        # Check the attributes on the root form for swap usage
+        soup = self.get_soup(response.content)
+        form = soup.find("form", {"id": "image-url-generator-form"})
+        self.assertIsNotNone(form)
+        self.assertIn("w-swap", form["data-controller"])
+        self.assertEqual(form["data-w-swap-loading-class"], "loading")
+        self.assertEqual(
+            form["data-w-swap-messages-value"],
+            '{"400": "The filter options you have selected are not valid."}',
+        )
+        self.assertEqual(
+            form["data-w-swap-src-value"],
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,)),
+        )
+        self.assertEqual(
+            form["data-w-swap-target-value"], "output[form='image-url-generator-form']"
+        )
+
+        # Check that the correct URLs are provided to context to show the result and preview on initial load
+        self.assertTrue(
+            response.context["preview_url"].endswith(
+                f"/images/{self.image.id}/preview/original/"
+            )
+        )
+        self.assertTrue(
+            response.context["result_url"].endswith(f"/{self.image.id}/original/")
+        )
+
+        # Check that the breadcrumbs are correct
         self.assertBreadcrumbsItemsRendered(
             [
                 {"url": reverse("wagtailimages:index"), "label": "Images"},
@@ -3964,6 +4017,75 @@ class TestURLGeneratorView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         self.assertEqual(closeness.get("min"), "0")
         self.assertEqual(closeness.get("max"), "100")
 
+    def test_get_with_search_params(self):
+        """
+        This tests that the view responds correctly with support for search params to override
+        the form initial values & generated preview/result URLs when partial params are provided.
+        """
+        # Get
+        response = self.client.get(
+            reverse("wagtailimages:url_generator", args=(self.image.id,))
+            + "?filter_method=max&closeness=5"
+        )
+
+        # Check response
+        self.assertEqual(response.status_code, 200)
+
+        # Check form defaults are based based on the search params & image sizes
+        self.assertEqual(response.context["form"].initial["width"], self.image.width)
+        self.assertEqual(response.context["form"].initial["height"], self.image.height)
+        self.assertEqual(response.context["form"].initial["filter_method"], "max")
+        self.assertEqual(response.context["form"].initial["closeness"], "5")
+
+        # Check that the correct URLs are provided to context to show the result based on the search params
+        # -c5 should not be included as it's not part of that filter spec
+        filter_spec = "max-640x480"
+
+        self.assertTrue(
+            response.context["preview_url"].endswith(
+                f"/images/{self.image.id}/preview/{filter_spec}/"
+            ),
+            f"`{response.context['preview_url']}` does not end with `/images/{self.image.id}/preview/{filter_spec}/`",
+        )
+        self.assertTrue(
+            response.context["result_url"].endswith(f"/{self.image.id}/{filter_spec}/"),
+            f"`{response.context['result_url']}` does not end with `/{self.image.id}/{filter_spec}/`",
+        )
+
+    def test_get_with_all_search_params(self):
+        """
+        This tests that the view responds correctly with support for search params to override
+        the form initial values & generated preview/result URLs.
+        """
+        # Get
+        response = self.client.get(
+            reverse("wagtailimages:url_generator", args=(self.image.id,))
+            + "?filter_method=fill&width=75&height=50&closeness=5"
+        )
+
+        # Check response
+        self.assertEqual(response.status_code, 200)
+
+        # Check form defaults are based based on the search params
+        self.assertEqual(response.context["form"].initial["width"], "75")
+        self.assertEqual(response.context["form"].initial["height"], "50")
+        self.assertEqual(response.context["form"].initial["filter_method"], "fill")
+        self.assertEqual(response.context["form"].initial["closeness"], "5")
+
+        # Check that the correct URLs are provided to context to show the result based on the search params
+        filter_spec = "fill-75x50-c5"
+
+        self.assertTrue(
+            response.context["preview_url"].endswith(
+                f"/images/{self.image.id}/preview/{filter_spec}/"
+            ),
+            f"`{response.context['preview_url']}` does not end with `/images/{self.image.id}/preview/{filter_spec}/`",
+        )
+        self.assertTrue(
+            response.context["result_url"].endswith(f"/{self.image.id}/{filter_spec}/"),
+            f"`{response.context['result_url']}` does not end with `/{self.image.id}/{filter_spec}/`",
+        )
+
     def test_get_bad_permissions(self):
         """
         This tests that the view returns a "permission denied" redirect if a user without correct
@@ -3986,8 +4108,33 @@ class TestURLGeneratorView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         # Check response
         self.assertRedirects(response, reverse("wagtailadmin_home"))
 
+    def test_get_with_bad_search_params(self):
+        """
+        This tests how the view responds when the search params
+        result in an invalid filter spec.
+        """
+        # Get
+        response = self.client.get(
+            reverse("wagtailimages:url_generator", args=(self.image.id,))
+            + "?filter_method=bad&width=75&height=50&closeness=5"
+        )
+
+        # Check response
+        self.assertEqual(response.status_code, 200)
+
+        # Check message provided
+        messages = [m.message for m in response.context["messages"]]
+        self.assertEqual(len(messages), 1)
+        self.assertIn(
+            "The filter options you have selected are not valid.", messages[0]
+        )
+
+
+class TestURLGeneratorViewOutput(WagtailTestUtils, TestCase):
+    """
+    Test the output partial HTML response for swapping into the image URL generator page.
+    """
 
-class TestGenerateURLView(WagtailTestUtils, TestCase):
     def setUp(self):
         # Create an image for running tests on
         self.image = Image.objects.create(
@@ -4004,17 +4151,27 @@ class TestGenerateURLView(WagtailTestUtils, TestCase):
         """
         # Get
         response = self.client.get(
-            reverse("wagtailimages:generate_url", args=(self.image.id, "fill-800x600"))
+            reverse(
+                "wagtailimages:url_generator_output",
+                args=(self.image.id,),
+            )
+            + "?filter_method=fill&width=800&height=600"
         )
 
         # Check response
         self.assertEqual(response.status_code, 200)
-        self.assertEqual(response["Content-Type"], "application/json")
 
-        # Check JSON
-        content_json = json.loads(response.content.decode())
+        # should only use the 'output' template
+        self.assertTemplateNotUsed(response, "wagtailimages/images/url_generator.html")
+        self.assertTemplateUsed(
+            response, "wagtailimages/images/url_generator_output.html"
+        )
+        self.assertTemplateNotUsed(response, "wagtailadmin/generic/base.html")
 
-        self.assertEqual(set(content_json.keys()), {"url", "preview_url"})
+        soup = self.get_soup(response.content)
+
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        preview_url = soup.find("img")["src"]
 
         expected_url = (
             "http://localhost/images/%(signature)s/%(image_id)d/fill-800x600/"
@@ -4026,12 +4183,12 @@ class TestGenerateURLView(WagtailTestUtils, TestCase):
                 "image_id": self.image.id,
             }
         )
-        self.assertEqual(content_json["url"], expected_url)
+        self.assertEqual(result_url, expected_url)
 
         expected_preview_url = reverse(
             "wagtailimages:preview", args=(self.image.id, "fill-800x600")
         )
-        self.assertEqual(content_json["preview_url"], expected_preview_url)
+        self.assertEqual(preview_url, expected_preview_url)
 
     def test_get_bad_permissions(self):
         """
@@ -4048,21 +4205,18 @@ class TestGenerateURLView(WagtailTestUtils, TestCase):
 
         # Get
         response = self.client.get(
-            reverse("wagtailimages:generate_url", args=(self.image.id, "fill-800x600"))
+            reverse(
+                "wagtailimages:url_generator_output",
+                args=(self.image.id,),
+            )
+            + "?filter_method=fill&width=800&height=600"
         )
 
         # Check response
         self.assertEqual(response.status_code, 403)
-        self.assertEqual(response["Content-Type"], "application/json")
-
-        # Check JSON
-        self.assertJSONEqual(
+        self.assertEqual(
             response.content.decode(),
-            json.dumps(
-                {
-                    "error": "You do not have permission to generate a URL for this image.",
-                }
-            ),
+            "You do not have permission to generate a URL for this image.",
         )
 
     def test_get_bad_image(self):
@@ -4072,23 +4226,14 @@ class TestGenerateURLView(WagtailTestUtils, TestCase):
         # Get
         response = self.client.get(
             reverse(
-                "wagtailimages:generate_url", args=(self.image.id + 1, "fill-800x600")
+                "wagtailimages:url_generator_output",
+                args=(self.image.id + 1,),
             )
+            + "?filter_method=fill&width=800&height=600"
         )
 
         # Check response
         self.assertEqual(response.status_code, 404)
-        self.assertEqual(response["Content-Type"], "application/json")
-
-        # Check JSON
-        self.assertJSONEqual(
-            response.content.decode(),
-            json.dumps(
-                {
-                    "error": "Cannot find image.",
-                }
-            ),
-        )
 
     def test_get_bad_filter_spec(self):
         """
@@ -4097,23 +4242,251 @@ class TestGenerateURLView(WagtailTestUtils, TestCase):
         # Get
         response = self.client.get(
             reverse(
-                "wagtailimages:generate_url", args=(self.image.id, "bad-filter-spec")
+                "wagtailimages:url_generator_output",
+                args=(self.image.id,),
             )
+            + "?filter_method=bad-filter-spec"
         )
 
         # Check response
         self.assertEqual(response.status_code, 400)
-        self.assertEqual(response["Content-Type"], "application/json")
-
-        # Check JSON
-        self.assertJSONEqual(
+        self.assertEqual(
             response.content.decode(),
-            json.dumps(
-                {
-                    "error": "Invalid filter spec.",
-                }
-            ),
+            "Invalid filter spec: `bad-filter-spec`. Unrecognised operation: bad.",
+        )
+
+    def test_get_with_default_filter_spec(self):
+        response = self.client.get(
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,))
+        )
+
+        # check response and find key values
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+
+        # check the result URL from the textarea
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        expected_url = (
+            "http://localhost/images/%(signature)s/%(image_id)d/original/"
+            % {
+                "signature": urllib.parse.quote(
+                    generate_signature(self.image.id, "original"),
+                    safe=urlquote_safechars,
+                ),
+                "image_id": self.image.id,
+            }
+        )
+        self.assertEqual(result_url, expected_url)
+
+        # check the preview URL from the image
+        preview_url = soup.find("img")["src"]
+        expected_preview_url = reverse(
+            "wagtailimages:preview", args=(self.image.id, "original")
+        )
+        self.assertEqual(preview_url, expected_preview_url)
+
+    def test_get_with_width_filter_spec(self):
+        response = self.client.get(
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,))
+            + "?filter_method=width&width=800&height=300"
+            # height should be ignored as it's not part of the width filter spec values
+        )
+
+        # check response and find key values
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+
+        # check the result URL from the textarea
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        expected_url = (
+            "http://localhost/images/%(signature)s/%(image_id)d/width-800/"
+            % {
+                "signature": urllib.parse.quote(
+                    generate_signature(self.image.id, "width-800"),
+                    safe=urlquote_safechars,
+                ),
+                "image_id": self.image.id,
+            }
+        )
+        self.assertEqual(result_url, expected_url)
+
+        # check the preview URL from the image
+        preview_url = soup.find("img")["src"]
+        expected_preview_url = reverse(
+            "wagtailimages:preview", args=(self.image.id, "width-800")
+        )
+        self.assertEqual(preview_url, expected_preview_url)
+
+    def test_get_with_height_filter_spec(self):
+        response = self.client.get(
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,))
+            + "?filter_method=height&width=800&height=320&closeness=20"
+            # width & closeness should be ignored as it's not part of the height filter spec values
+        )
+
+        # check response and find key values
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+
+        # check the result URL from the textarea
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        expected_url = (
+            "http://localhost/images/%(signature)s/%(image_id)d/height-320/"
+            % {
+                "signature": urllib.parse.quote(
+                    generate_signature(self.image.id, "height-320"),
+                    safe=urlquote_safechars,
+                ),
+                "image_id": self.image.id,
+            }
+        )
+        self.assertEqual(result_url, expected_url)
+
+        # check the preview URL from the image
+        preview_url = soup.find("img")["src"]
+        expected_preview_url = reverse(
+            "wagtailimages:preview", args=(self.image.id, "height-320")
+        )
+        self.assertEqual(preview_url, expected_preview_url)
+
+    def test_get_with_fill_with_defaults_filter_spec(self):
+        """
+        When only the `fill` filter spec is provided on its own, it should
+        resolve default width & height values from the image size.
+        """
+
+        response = self.client.get(
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,))
+            + "?filter_method=fill"
+        )
+
+        # check response and find key values
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+
+        # check the result URL from the textarea
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        expected_url = (
+            "http://localhost/images/%(signature)s/%(image_id)d/fill-640x480/"
+            % {
+                "signature": urllib.parse.quote(
+                    generate_signature(self.image.id, "fill-640x480"),
+                    safe=urlquote_safechars,
+                ),
+                "image_id": self.image.id,
+            }
+        )
+        self.assertEqual(result_url, expected_url)
+
+        # check the preview URL from the image
+        preview_url = soup.find("img")["src"]
+        expected_preview_url = reverse(
+            "wagtailimages:preview", args=(self.image.id, "fill-640x480")
+        )
+        self.assertEqual(preview_url, expected_preview_url)
+
+    def test_get_with_min_with_defaults_filter_spec(self):
+        """
+        When only the `min` filter spec is provided on its own, it should
+        resolve default width & height values from the image size.
+        """
+
+        response = self.client.get(
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,))
+            + "?filter_method=min"
+        )
+
+        # check response and find key values
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+
+        # check the result URL from the textarea
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        expected_url = (
+            "http://localhost/images/%(signature)s/%(image_id)d/min-640x480/"
+            % {
+                "signature": urllib.parse.quote(
+                    generate_signature(self.image.id, "min-640x480"),
+                    safe=urlquote_safechars,
+                ),
+                "image_id": self.image.id,
+            }
+        )
+        self.assertEqual(result_url, expected_url)
+
+        # check the preview URL from the image
+        preview_url = soup.find("img")["src"]
+        expected_preview_url = reverse(
+            "wagtailimages:preview", args=(self.image.id, "min-640x480")
+        )
+        self.assertEqual(preview_url, expected_preview_url)
+
+    def test_get_with_max_with_defaults_filter_spec(self):
+        """
+        When only the `max` filter spec is provided on its own, it should
+        resolve default width & height values from the image size.
+        """
+
+        response = self.client.get(
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,))
+            + "?filter_method=max"
+        )
+
+        # check response and find key values
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+
+        # check the result URL from the textarea
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        expected_url = (
+            "http://localhost/images/%(signature)s/%(image_id)d/max-640x480/"
+            % {
+                "signature": urllib.parse.quote(
+                    generate_signature(self.image.id, "max-640x480"),
+                    safe=urlquote_safechars,
+                ),
+                "image_id": self.image.id,
+            }
+        )
+        self.assertEqual(result_url, expected_url)
+
+        # check the preview URL from the image
+        preview_url = soup.find("img")["src"]
+        expected_preview_url = reverse(
+            "wagtailimages:preview", args=(self.image.id, "max-640x480")
+        )
+        self.assertEqual(preview_url, expected_preview_url)
+
+    def test_get_with_fill_with_all_values_filter_spec(self):
+        response = self.client.get(
+            reverse("wagtailimages:url_generator_output", args=(self.image.id,))
+            + "?filter_method=fill&width=320&height=120&closeness=20"
+        )
+
+        # check response and find key values
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+
+        # check the result URL from the textarea
+        result_url = soup.find("textarea", {"id": "result-url"}).get_text()
+        expected_url = (
+            "http://localhost/images/%(signature)s/%(image_id)d/fill-320x120-c20/"
+            % {
+                "signature": urllib.parse.quote(
+                    generate_signature(self.image.id, "fill-320x120-c20"),
+                    safe=urlquote_safechars,
+                ),
+                "image_id": self.image.id,
+            }
+        )
+        self.assertEqual(result_url, expected_url)
+
+        # check the preview URL from the image
+        preview_url = soup.find("img")["src"]
+        expected_preview_url = reverse(
+            "wagtailimages:preview", args=(self.image.id, "fill-320x120-c20")
         )
+        self.assertEqual(preview_url, expected_preview_url)
 
 
 class TestPreviewView(WagtailTestUtils, TestCase):