Removed dynamic container and its documentation. Added safeguards. (#152

)

docs/index.md

@@ -13,7 +13,6 @@
     introduction/inject-factories
     introduction/scopes
     introduction/multiple-containers
-    introduction/dynamic-container
     introduction/application-settings
 
 .. toctree::

tests/test_dynamic_container.py

@@ -1,23 +1,19 @@
 import datetime
 
+import pytest
+
 from tests import container
 from that_depends import BaseContainer, providers
 
 
 class DIContainer(BaseContainer):
     sync_resource: providers.Resource[datetime.datetime]
-    async_resource: providers.Resource[datetime.datetime]
-
-
-DIContainer.sync_resource = providers.Resource(container.create_sync_resource)
-DIContainer.async_resource = providers.Resource(container.create_async_resource)
-
 
-async def test_dynamic_container() -> None:
-    sync_resource = await DIContainer.sync_resource()
-    async_resource = await DIContainer.async_resource()
 
-    assert isinstance(sync_resource, datetime.datetime)
-    assert isinstance(async_resource, datetime.datetime)
+async def test_dynamic_container_not_supported() -> None:
+    new_provider = providers.Resource(container.create_sync_resource)
+    with pytest.raises(AttributeError):
+        DIContainer.sync_resource = new_provider
 
-    await DIContainer.tear_down()
+    with pytest.raises(AttributeError):
+        DIContainer.something_new = new_provider

that_depends/meta.py

@@ -37,6 +37,14 @@ class BaseContainerMeta(abc.ABCMeta):
 
     _instances: typing.ClassVar[list[type["BaseContainer"]]] = []
 
+    _MUTABLE_ATTRS = (
+        "__abstractmethods__",
+        "__parameters__",
+        "_abc_impl",
+        "providers",
+        "containers",
+    )
+
     _lock: Lock = Lock()
 
     @classmethod
@@ -56,3 +64,11 @@ def __new__(cls, name: str, bases: tuple[type, ...], namespace: dict[str, typing
     def get_instances(cls) -> list[type["BaseContainer"]]:
         """Get all instances that inherit from BaseContainer."""
         return cls._instances
+
+    @override
+    def __setattr__(cls, key: str, value: typing.Any) -> None:
+        if key in cls._MUTABLE_ATTRS:  # Allow modification of mutable attributes
+            super().__setattr__(key, value)
+        else:
+            msg = f"Cannot add new attribute '{key}' to class '{cls.__name__}'"
+            raise AttributeError(msg)