Don't resolve passed parameters

kink/inject.py

@@ -123,7 +123,10 @@ def _resolve_kwargs(args, kwargs) -> dict:
         if len(passed_kwargs) == len(parameters_name):
             return passed_kwargs
 
-        resolved_kwargs = _resolve_function_kwargs(binding, parameters_name, parameters, container)
+        # do not resolve for passed kwargs and args
+        parameters_name_to_resolve = tuple(parameters_name - passed_kwargs.keys())
+
+        resolved_kwargs = _resolve_function_kwargs(binding, parameters_name_to_resolve, parameters, container)
 
         all_kwargs = {**resolved_kwargs, **passed_kwargs}
 

tests/test_issue_passed_parameters.py

@@ -0,0 +1,37 @@
+from unittest.mock import MagicMock
+
+from kink import di, inject
+
+def test_do_not_resolve_passed_kwargs() -> None:
+    di["name"] = "Bob"
+    @inject
+    class ExpensiveObject:
+        def __init__(self, name: str):
+            self.name = name
+            raise Exception("Constructing expensive object")
+
+    @inject
+    def greet(namer: ExpensiveObject, greeting: str = "Hello %s") -> str:
+        return greeting % namer.name
+
+    mock_expensive_object = MagicMock(spec=ExpensiveObject)
+    mock_expensive_object.name = "Bill"
+
+    assert greet(namer=mock_expensive_object) == "Hello Bill"
+
+def test_do_not_resolve_passed_args() -> None:
+    di["name"] = "Bob"
+    @inject
+    class ExpensiveObject:
+        def __init__(self, name: str):
+            self.name = name
+            raise Exception("Constructing expensive object")
+
+    @inject
+    def greet(namer: ExpensiveObject, greeting: str = "Hello %s") -> str:
+        return greeting % namer.name
+
+    mock_expensive_object = MagicMock(spec=ExpensiveObject)
+    mock_expensive_object.name = "Bill"
+
+    assert greet(mock_expensive_object) == "Hello Bill"