Fix typing

Author: fsbraun

djangocms_frontend/component_pool.py

@@ -1,6 +1,7 @@
 from collections import defaultdict
 import importlib
 import os
+from collections.abc import Iterator
 import warnings
 
 from django import forms
@@ -71,7 +72,9 @@ def component_factory(module, component: tuple, fields: list[tuple], template: s
             },
         )
 
-    def scan_templates_for_component_declaration(self, templates: list[tuple[str, str]]) -> list[CMSFrontendComponent]:
+    def scan_templates_for_component_declaration(
+        self, templates: list[tuple[str, str]]
+    ) -> Iterator[CMSFrontendComponent]:
         from django.forms import fields
 
         field_context = self.get_field_context()

tests/test_autocomponent.py

@@ -1,5 +1,6 @@
 from cms.api import add_plugin
 from cms.test_utils.testcases import CMSTestCase
+from django.template import Template, TemplateSyntaxError
 
 from tests.fixtures import TestFixture
 
@@ -69,3 +70,23 @@ def test_split_template_tag(self):
         self.assertEqual(split("Mixin1|Mixin2"), ["Mixin1", "Mixin2"])
         self.assertEqual(split("hero.html"), ["hero.html"])
         self.assertEqual(split("hero.html, Mixin1, Mixin2", ", "), ["hero.html", "Mixin1", "Mixin2"])
+
+    def test_invalid_cms_component_usage_missing_required_argument(self):
+        # The {% cms_component %} tag requires a component name.
+        invalid_template = "{% load cms_tags %}{% cms_component %}"
+        with self.assertRaises(TemplateSyntaxError):
+            Template(invalid_template)
+
+    def test_invalid_field_usage_invalid_argument(self):
+        # The {% field %} tag requires valid arguments: a field name and a component instance.
+        # Here we simulate invalid usage by providing an invalid component.
+        invalid_template = "{% load cms_tags %}{% field 'nonexistent_field' component %}"
+        with self.assertRaises(TemplateSyntaxError):
+            Template(invalid_template)
+
+    def test_multiple_cms_component_tags_error(self):
+        # Assuming only one {% cms_component %} tag is allowed per template.
+        # This should raise an error if multiple tags are used.
+        invalid_template = "{% load cms_tags %}{% cms_component 'Hero' %}{% cms_component 'Footer' %}"
+        with self.assertRaises(TemplateSyntaxError):
+            Template(invalid_template)