[#2251]Fix test test_pages & test_models

maykinmedia · May 12, 2023 · afa9baf · afa9baf
1 parent 2add5bd
commit afa9baf
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 5 deletions.
diff --git a/tests/__init__.py b/tests/__init__.py
diff --git a/tests/templates/test.html b/tests/templates/test.html
@@ -0,0 +1,10 @@
+
+{% load cms_tags %}
+
+
+{% block content %}
+
+    {% placeholder 'test' %}
+
+{% endblock content %}
+
diff --git a/tests/test_models.py b/tests/test_models.py
@@ -2,10 +2,10 @@
 
 from djangocms_export_page.export.common import PageExport
 from djangocms_export_page.export.docx import DocxPageExport
-from tests.factories import BlogFactory
+from annefrank.blog.tests.factories import BlogFactory
 
 
-@override_settings(ROOT_URLCONF='test.blog.tests.urls_tests')
+@override_settings(ROOT_URLCONF='annefrank.blog.tests.urls_tests')
 class ExportModelTests(TestCase):
     def setUp(self):
         self.object = BlogFactory()

diff --git a/tests/test_pages.py b/tests/test_pages.py
@@ -10,10 +10,14 @@
 
 
 class ExportPageTests(TestCase):
+    """
+    Needs test template setup in settings TEMPLATES and CMS_TEMPLATES
+
+    """
+
     def setUp(self):
-        self.placeholder = Placeholder.objects.create(slot='test')
         self.page = create_page('test', 'test.html', 'nl')
-        self.page.placeholders.add(self.placeholder)
+        self.placeholder = self.page.placeholders.get(slot='test')
         self.language = 'nl'
         self.request = RequestFactory().get('/nl/')
 
@@ -27,7 +31,7 @@ def test_base_url(self):
 
     def test_page_url(self):
         export = PageExport(self.request, self.page, language=self.language)
-        self.assertEqual(export.page_url, 'http://example.com/nl/')
+        self.assertEqual(export.page_url, 'http://example.com/nl/test/')
 
     @patch('djangocms_export_page.export.common.get_page_meta')
     def test_meta_extra_custom_props(self, mock):