Use correct context when rendering navigation

Author: HiPhish

src/django_simple_nav/jinja2/django_simple_nav.py

@@ -29,8 +29,12 @@ def django_simple_nav(
             template_name = cast(Nav, nav).template_name
         if template_name is None:
             raise TemplateRuntimeError("Navigation object has no template")
-        items = cast(Nav, nav).get_items(context["request"])
+        request = context['request']
+        new_context = {
+            'request': request,
+            **cast(Nav, nav).get_context_data(request)
+        }
     except Exception as err:
         raise TemplateRuntimeError(str(err)) from err
 
-    return loader.load(context.environment, template_name).render(items=items)
+    return loader.load(context.environment, template_name).render(new_context)

tests/jinja2/tests/dummy_nav.html

@@ -2,9 +2,9 @@
   {% for item in items recursive %}
     <li>
       <a href="{{ item.url }}">{{ item.title }}</a>
-      {% if item.items %}
+      {% if item['items'] %}
         <ul>
-          {{ loop(item.items) }}
+          {{ loop(item['items']) }}
         </ul>
       {% endif %}
     </li>

tests/test_jinja.py

@@ -131,3 +131,15 @@ def test_request_not_in_context():
 
     with pytest.raises(TemplateRuntimeError):
         template.render()
+
+
+def test_invalid_request():
+    class InvalidRequest:
+        ...
+
+    template = environment.from_string(
+        "{{ django_simple_nav('tests.navs.DummyNav') }}"
+    )
+
+    with pytest.raises(TemplateRuntimeError):
+        template.render({"request": InvalidRequest()})