allow overriding in injections (#80)

* allow overriding in injections

tests/test_injection.py

@@ -26,15 +26,20 @@ async def test_injection(
     assert fixture_one == 1
 
 
-async def test_wrong_injection() -> None:
+async def test_injection_with_overriding() -> None:
     @inject
     async def inner(
-        _: container.SimpleFactory = Provide[container.DIContainer.simple_factory],
+        arg1: bool,
+        arg2: container.SimpleFactory = Provide[container.DIContainer.simple_factory],
     ) -> None:
-        """Do nothing."""
-
-    with pytest.raises(RuntimeError, match="Injected arguments must not be redefined"):
-        await inner(_=container.SimpleFactory(dep1="1", dep2=2))
+        _ = arg1
+        original_obj = await container.DIContainer.simple_factory()
+        assert arg2.dep1 != original_obj.dep1
+        assert arg2.dep2 != original_obj.dep2
+
+    await inner(arg1=True, arg2=container.SimpleFactory(dep1="1", dep2=2))
+    await inner(True, container.SimpleFactory(dep1="1", dep2=2))
+    await inner(True, arg2=container.SimpleFactory(dep1="1", dep2=2))
 
 
 async def test_empty_injection() -> None:

that_depends/injection.py

@@ -27,13 +27,15 @@ def _inject_to_async(
     @functools.wraps(func)
     async def inner(*args: P.args, **kwargs: P.kwargs) -> T:
         injected = False
-        for field_name, field_value in signature.parameters.items():
+        for i, (field_name, field_value) in enumerate(signature.parameters.items()):
+            if i < len(args):
+                continue
+
             if not isinstance(field_value.default, AbstractProvider):
                 continue
 
             if field_name in kwargs:
-                msg = f"Injected arguments must not be redefined, {field_name=}"
-                raise RuntimeError(msg)
+                continue
 
             kwargs[field_name] = await field_value.default()
             injected = True