Parent bindings should not affect multibind bindings

This is more in line with how regular bindings work.

Author: davidparsson

injector/__init__.py

@@ -359,16 +359,7 @@ def append(self, provider: Provider[T], scope: Type['Scope']) -> None:
 
     def get_scoped_providers(self, injector: 'Injector') -> Generator[Provider[T], None, None]:
         for binding in self._multi_bindings:
-            if (
-                isinstance(binding.provider, ClassProvider)
-                and binding.scope is NoScope
-                and self._binder.parent
-                and self._binder.parent.has_explicit_binding_for(binding.provider._cls)
-            ):
-                parent_binding, _ = self._binder.parent.get_binding(binding.provider._cls)
-                scope_binding, _ = self._binder.parent.get_binding(parent_binding.scope)
-            else:
-                scope_binding, _ = self._binder.get_binding(binding.scope)
+            scope_binding, _ = self._binder.get_binding(binding.scope)
             scope_instance: Scope = scope_binding.provider.get(injector)
             provider_instance = scope_instance.get(binding.interface, binding.provider)
             yield provider_instance

injector_test.py

@@ -723,19 +723,46 @@ def configure_dict(binder: Binder):
         Injector([configure_dict])
 
 
-def test_multibind_types_respect_the_bound_type_scope() -> None:
+def test_multibind_types_are_not_affected_by_the_bound_type_scope() -> None:
     def configure(binder: Binder) -> None:
         binder.bind(PluginA, to=PluginA, scope=singleton)
         binder.multibind(List[Plugin], to=PluginA)
 
     injector = Injector([configure])
     first_list = injector.get(List[Plugin])
     second_list = injector.get(List[Plugin])
-    child_injector = injector.create_child_injector()
-    third_list = child_injector.get(List[Plugin])
 
-    assert first_list[0] is second_list[0]
-    assert third_list[0] is second_list[0]
+    assert injector.get(PluginA) is injector.get(PluginA)
+    assert first_list[0] is not injector.get(PluginA)
+    assert first_list[0] is not second_list[0]
+
+
+def test_multibind_types_are_not_affected_by_the_bound_type_provider() -> None:
+    def configure(binder: Binder) -> None:
+        binder.bind(PluginA, to=InstanceProvider(PluginA()))
+        binder.multibind(List[Plugin], to=PluginA)
+
+    injector = Injector([configure])
+    first_list = injector.get(List[Plugin])
+    second_list = injector.get(List[Plugin])
+
+    assert injector.get(PluginA) is injector.get(PluginA)
+    assert first_list[0] is not injector.get(PluginA)
+    assert first_list[0] is not second_list[0]
+
+
+def test_multibind_dict_types_use_their_own_bound_providers_and_scopes() -> None:
+    def configure(binder: Binder) -> None:
+        binder.bind(PluginA, to=InstanceProvider(PluginA()))
+        binder.bind(PluginB, to=PluginB, scope=singleton)
+        binder.multibind(Dict[str, Plugin], to={'a': PluginA, 'b': PluginB})
+
+    injector = Injector([configure])
+
+    dictionary = injector.get(Dict[str, Plugin])
+
+    assert dictionary['a'] is not injector.get(PluginA)
+    assert dictionary['b'] is not injector.get(PluginB)
 
 
 def test_multibind_list_scopes_applies_to_the_bound_items() -> None: