mirror of
https://github.com/ets-labs/python-dependency-injector.git
synced 2025-02-12 01:20:51 +03:00
Implement lazy initialization and improve copying for Delegate provider
This commit is contained in:
parent
9202ad05a7
commit
3e03b9315e
File diff suppressed because it is too large
Load Diff
|
@ -94,7 +94,7 @@ class Delegate(Provider[Provider]):
|
||||||
def __init__(self, provides: Optional[Provider] = None) -> None: ...
|
def __init__(self, provides: Optional[Provider] = None) -> None: ...
|
||||||
@property
|
@property
|
||||||
def provides(self) -> Optional[Provider]: ...
|
def provides(self) -> Optional[Provider]: ...
|
||||||
def set_provides(self, provides: Optional[Provider]): ...
|
def set_provides(self, provides: Optional[Provider]) -> Delegate: ...
|
||||||
|
|
||||||
|
|
||||||
class Dependency(Provider[T]):
|
class Dependency(Provider[T]):
|
||||||
|
|
|
@ -520,11 +520,7 @@ cdef class Delegate(Provider):
|
||||||
"""
|
"""
|
||||||
|
|
||||||
def __init__(self, provides=None):
|
def __init__(self, provides=None):
|
||||||
"""Initializer.
|
"""Initialize provider."""
|
||||||
|
|
||||||
:param provides: Value that have to be provided.
|
|
||||||
:type provides: :py:class:`Provider`
|
|
||||||
"""
|
|
||||||
self.__provides = None
|
self.__provides = None
|
||||||
self.set_provides(provides)
|
self.set_provides(provides)
|
||||||
super(Delegate, self).__init__()
|
super(Delegate, self).__init__()
|
||||||
|
@ -535,11 +531,13 @@ cdef class Delegate(Provider):
|
||||||
if copied is not None:
|
if copied is not None:
|
||||||
return copied
|
return copied
|
||||||
|
|
||||||
copied = self.__class__()
|
provides = self.provides
|
||||||
memo[id(self)] = copied
|
if provides:
|
||||||
|
provides = deepcopy(self.provides, memo)
|
||||||
|
|
||||||
|
copied = _memorized_duplicate(self, memo)
|
||||||
|
copied.set_provides(provides)
|
||||||
|
|
||||||
if self.provides:
|
|
||||||
copied.set_provides(deepcopy(self.provides, memo))
|
|
||||||
self._copy_overridings(copied, memo)
|
self._copy_overridings(copied, memo)
|
||||||
|
|
||||||
return copied
|
return copied
|
||||||
|
@ -560,12 +558,15 @@ cdef class Delegate(Provider):
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def provides(self):
|
def provides(self):
|
||||||
|
"""Return provider's provides."""
|
||||||
return self.__provides
|
return self.__provides
|
||||||
|
|
||||||
def set_provides(self, provides):
|
def set_provides(self, provides):
|
||||||
|
"""Set provider's provides."""
|
||||||
if provides:
|
if provides:
|
||||||
provides = ensure_is_provider(provides)
|
provides = ensure_is_provider(provides)
|
||||||
self.__provides = provides
|
self.__provides = provides
|
||||||
|
return self
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def related(self):
|
def related(self):
|
||||||
|
|
|
@ -291,6 +291,16 @@ class DelegateTests(unittest.TestCase):
|
||||||
def test_is_provider(self):
|
def test_is_provider(self):
|
||||||
self.assertTrue(providers.is_provider(self.delegate))
|
self.assertTrue(providers.is_provider(self.delegate))
|
||||||
|
|
||||||
|
def test_init_optional_provides(self):
|
||||||
|
provider = providers.Delegate()
|
||||||
|
provider.set_provides(self.delegated)
|
||||||
|
self.assertIs(provider.provides, self.delegated)
|
||||||
|
self.assertIs(provider(), self.delegated)
|
||||||
|
|
||||||
|
def test_set_provides_returns_self(self):
|
||||||
|
provider = providers.Delegate()
|
||||||
|
self.assertIs(provider.set_provides(self.delegated), provider)
|
||||||
|
|
||||||
def test_init_with_not_provider(self):
|
def test_init_with_not_provider(self):
|
||||||
self.assertRaises(errors.Error, providers.Delegate, object())
|
self.assertRaises(errors.Error, providers.Delegate, object())
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user