Added a test to ensure preview endpoint fields are rendered read only

Author: Bernardvdv

djangocms_snippet/admin.py

@@ -3,6 +3,7 @@
 from django.contrib import admin
 from django.contrib.admin import helpers
 from django.contrib.admin.exceptions import DisallowedModelAdminToField
+from django.contrib.admin.options import IS_POPUP_VAR, TO_FIELD_VAR
 from django.contrib.admin.utils import flatten_fieldsets, unquote
 from django.db import models
 from django.forms import Textarea
@@ -27,9 +28,6 @@
 except ImportError:
     djangocms_versioning_enabled = False
 
-TO_FIELD_VAR = '_to_field'
-IS_POPUP_VAR = '_popup'
-
 
 class SnippetAdmin(*snippet_admin_classes):
     list_display = ('name',)
@@ -144,12 +142,12 @@ def preview_view(self, request, snippet_id=None, form_url='', extra_context=None
     def get_urls(self):
         info = self.model._meta.app_label, self.model._meta.model_name
         return [
-                   url(
-                       r"^(?P<snippet_id>\d+)/preview/$",
-                       self.admin_site.admin_view(self.preview_view),
-                       name="{}_{}_preview".format(*info),
-                   ),
-               ] + super().get_urls()
+           url(
+                   r"^(?P<snippet_id>\d+)/preview/$",
+                   self.admin_site.admin_view(self.preview_view),
+                   name="{}_{}_preview".format(*info),
+               ),
+           ] + super().get_urls()
 
     def has_delete_permission(self, request, obj=None):
         """

tests/test_admin.py

@@ -4,6 +4,7 @@
 from django.shortcuts import reverse
 from django.test import RequestFactory, override_settings
 
+from cms.utils.urlutils import admin_reverse
 from cms.test_utils.testcases import CMSTestCase
 
 from djangocms_versioning.models import Version
@@ -13,6 +14,8 @@
 from djangocms_snippet.forms import SnippetForm
 from djangocms_snippet.models import Snippet, SnippetGrouper
 
+from .utils.factories import SnippetWithVersionFactory
+
 
 class SnippetAdminTestCase(CMSTestCase):
     def setUp(self):
@@ -209,3 +212,23 @@ def test_name_colomn_should_not_be_hyperlinked_with_versioning_enabled(self):
         self.assertContains(response, '<td class="field-name">Test Snippet</td>')
         self.assertNotContains(response, '<th class="field-slug"><a href="/en/admin/djangocms_snippet/'
                                          'snippet/1/change/">test-snippet</a></th>')
+
+    def test_preview_renders_read_only_fields(self):
+        """
+        Check that the preview endpoint is rendered in read only mode
+        """
+        self.snippet_version.publish(user=self.superuser)
+        with self.login_user_context(self.superuser):
+            edit_url = reverse("admin:djangocms_snippet_snippet_preview", args=(self.snippet.id,),)
+            response = self.client.get(edit_url)
+
+        # Snippet name
+        self.assertContains(response, '<div class="readonly">Test Snippet</div>')
+        # Snippet slug
+        self.assertContains(response, '<div class="readonly">test-snippet</div>')
+        # Snippet HTML
+        self.assertContains(response, '<div class="readonly">&lt;h1&gt;This is a test&lt;/h1&gt;</div>')
+        # Snippet template
+        self.assertContains(response, '<div class="readonly"></div>')
+
+