Fix overriding injections in sync functions. (#110)

Co-authored-by: Alexander <e1634240@student.tuwien.ac.at>

tests/test_injection.py

@@ -64,15 +64,17 @@ def test_sync_injection(
     assert fixture_one == 1
 
 
-def test_wrong_sync_injection() -> None:
+def test_overriden_sync_injection() -> None:
     @inject
     def inner(
         _: container.SimpleFactory = Provide[container.DIContainer.simple_factory],
-    ) -> None:
+    ) -> container.SimpleFactory:
         """Do nothing."""
+        return _
 
-    with pytest.raises(RuntimeError, match="Injected arguments must not be redefined"):
-        inner(_=container.SimpleFactory(dep1="1", dep2=2))
+    factory = container.SimpleFactory(dep1="1", dep2=2)
+    with pytest.warns(RuntimeWarning, match="Expected injection, but nothing found. Remove @inject decorator."):
+        assert inner(_=factory) == factory
 
 
 def test_sync_empty_injection() -> None:

that_depends/injection.py

@@ -56,13 +56,13 @@ def _inject_to_sync(
     @functools.wraps(func)
     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] = field_value.default.sync_resolve()
             injected = True