don't cache component retrieval when DEBUG is true (#71)

* don't cache component getting when `DEBUG` is true

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* update changelog

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: joshuadavidthomas

CHANGELOG.md

@@ -18,6 +18,10 @@ and this project attempts to adhere to [Semantic Versioning](https://semver.org/
 
 ## [Unreleased]
 
+### Changed
+
+- When `DEBUG=True`, the `django_bird.components.Registry` will no longer cache the retrieval of `Component` instances.
+
 ## [0.6.1]
 
 ### Fixed

src/django_bird/components.py

@@ -5,6 +5,7 @@
 from typing import Any
 
 from cachetools import LRUCache
+from django.conf import settings
 from django.template.backends.django import Template as DjangoTemplate
 from django.template.loader import select_template
 
@@ -44,7 +45,8 @@ def get_component(self, name: str) -> Component:
             return self._cache[name]
         except KeyError:
             component = Component.from_name(name)
-            self._cache[name] = component
+            if not settings.DEBUG:
+                self._cache[name] = component
             return component
 
     def clear(self) -> None:

tests/test_components.py

@@ -3,6 +3,7 @@
 import pytest
 from django.template.backends.django import Template
 from django.template.exceptions import TemplateDoesNotExist
+from django.test import override_settings
 
 from django_bird.components import Component
 from django_bird.components import Registry
@@ -99,3 +100,17 @@ def test_lru_cache_behavior(self, registry, create_bird_template):
     def test_component_not_found(self, registry):
         with pytest.raises(TemplateDoesNotExist):
             registry.get_component("nonexistent")
+
+    def test_cache_with_debug(self, registry, create_bird_template):
+        create_bird_template(name="button", content="<button>Click me</button>")
+
+        assert len(registry._cache) == 0
+
+        with override_settings(DEBUG=True):
+            registry.get_component("button")
+
+        assert len(registry._cache) == 0
+
+        registry.get_component("button")
+
+        assert len(registry._cache) == 1