From 7bed48c852a741df5a14359916faf21d90d39814 Mon Sep 17 00:00:00 2001 From: HampusM Date: Thu, 31 Aug 2023 19:19:06 +0200 Subject: refactor: pass around BindingOptions instead of name --- src/di_container/asynchronous/binding/builder.rs | 59 ++++--- .../asynchronous/binding/scope_configurator.rs | 9 +- .../asynchronous/binding/when_configurator.rs | 9 +- src/di_container/asynchronous/mod.rs | 63 +++++--- src/di_container/binding_storage.rs | 170 +++++++++------------ src/di_container/blocking/binding/builder.rs | 44 +++--- .../blocking/binding/scope_configurator.rs | 13 +- .../blocking/binding/when_configurator.rs | 9 +- src/di_container/blocking/mod.rs | 63 +++++--- src/di_container/mod.rs | 4 +- src/test_utils.rs | 16 +- 11 files changed, 257 insertions(+), 202 deletions(-) (limited to 'src') diff --git a/src/di_container/asynchronous/binding/builder.rs b/src/di_container/asynchronous/binding/builder.rs index 5862d63..83a1efb 100644 --- a/src/di_container/asynchronous/binding/builder.rs +++ b/src/di_container/asynchronous/binding/builder.rs @@ -9,6 +9,7 @@ use crate::di_container::asynchronous::binding::scope_configurator::AsyncBinding #[cfg(feature = "factory")] use crate::di_container::asynchronous::binding::when_configurator::AsyncBindingWhenConfigurator; use crate::di_container::asynchronous::IAsyncDIContainer; +use crate::di_container::BindingOptions; use crate::errors::async_di_container::AsyncBindingBuilderError; use crate::interfaces::async_injectable::AsyncInjectable; use crate::util::use_double; @@ -104,7 +105,11 @@ where where Implementation: AsyncInjectable, { - if self.di_container.has_binding::(None).await { + if self + .di_container + .has_binding::(BindingOptions::new()) + .await + { return Err(AsyncBindingBuilderError::BindingAlreadyExists(type_name::< Interface, >( @@ -189,7 +194,11 @@ where use crate::private::castable_factory::threadsafe::ThreadsafeCastableFactory; use crate::provider::r#async::AsyncFactoryVariant; - if self.di_container.has_binding::(None).await { + if self + .di_container + .has_binding::(BindingOptions::new()) + .await + { return Err(AsyncBindingBuilderError::BindingAlreadyExists(type_name::< Interface, >( @@ -200,7 +209,7 @@ where self.di_container .set_binding::( - None, + BindingOptions::new(), Box::new(crate::provider::r#async::AsyncFactoryProvider::new( crate::ptr::ThreadsafeFactoryPtr::new(factory_impl), AsyncFactoryVariant::Normal, @@ -292,7 +301,11 @@ where use crate::private::castable_factory::threadsafe::ThreadsafeCastableFactory; use crate::provider::r#async::AsyncFactoryVariant; - if self.di_container.has_binding::(None).await { + if self + .di_container + .has_binding::(BindingOptions::new()) + .await + { return Err(AsyncBindingBuilderError::BindingAlreadyExists(type_name::< Interface, >( @@ -303,7 +316,7 @@ where self.di_container .set_binding::( - None, + BindingOptions::new(), Box::new(crate::provider::r#async::AsyncFactoryProvider::new( crate::ptr::ThreadsafeFactoryPtr::new(factory_impl), AsyncFactoryVariant::Normal, @@ -382,7 +395,11 @@ where use crate::private::castable_factory::threadsafe::ThreadsafeCastableFactory; use crate::provider::r#async::AsyncFactoryVariant; - if self.di_container.has_binding::(None).await { + if self + .di_container + .has_binding::(BindingOptions::new()) + .await + { return Err(AsyncBindingBuilderError::BindingAlreadyExists(type_name::< Interface, >( @@ -393,7 +410,7 @@ where self.di_container .set_binding::( - None, + BindingOptions::new(), Box::new(crate::provider::r#async::AsyncFactoryProvider::new( crate::ptr::ThreadsafeFactoryPtr::new(factory_impl), AsyncFactoryVariant::Default, @@ -477,7 +494,11 @@ where use crate::private::castable_factory::threadsafe::ThreadsafeCastableFactory; use crate::provider::r#async::AsyncFactoryVariant; - if self.di_container.has_binding::(None).await { + if self + .di_container + .has_binding::(BindingOptions::new()) + .await + { return Err(AsyncBindingBuilderError::BindingAlreadyExists(type_name::< Interface, >( @@ -488,7 +509,7 @@ where self.di_container .set_binding::( - None, + BindingOptions::new(), Box::new(crate::provider::r#async::AsyncFactoryProvider::new( crate::ptr::ThreadsafeFactoryPtr::new(factory_impl), AsyncFactoryVariant::AsyncDefault, @@ -519,13 +540,13 @@ mod tests di_container_mock .expect_has_binding::() - .with(eq(None)) + .with(eq(BindingOptions::new())) .return_once(|_name| false) .once(); di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|binding_options, _provider| binding_options.name.is_none()) .return_once(|_name, _provider| ()) .once(); @@ -562,13 +583,13 @@ mod tests di_container_mock .expect_has_binding::() - .with(eq(None)) + .with(eq(BindingOptions::new())) .return_once(|_name| false) .once(); di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|binding_options, _provider| binding_options.name.is_none()) .return_once(|_name, _provider| ()) .once(); @@ -613,13 +634,13 @@ mod tests di_container_mock .expect_has_binding::() - .with(eq(None)) + .with(eq(BindingOptions::new())) .return_once(|_name| false) .once(); di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|binding_options, _provider| binding_options.name.is_none()) .return_once(|_name, _provider| ()) .once(); @@ -659,13 +680,13 @@ mod tests di_container_mock .expect_has_binding::() - .with(eq(None)) + .with(eq(BindingOptions::new())) .return_once(|_name| false) .once(); di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|binding_options, _provider| binding_options.name.is_none()) .return_once(|_name, _provider| ()) .once(); @@ -706,13 +727,13 @@ mod tests di_container_mock .expect_has_binding::() - .with(eq(None)) + .with(eq(BindingOptions::new())) .return_once(|_name| false) .once(); di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|binding_options, _provider| binding_options.name.is_none()) .return_once(|_name, _provider| ()) .once(); diff --git a/src/di_container/asynchronous/binding/scope_configurator.rs b/src/di_container/asynchronous/binding/scope_configurator.rs index 0b5bad8..f10bb48 100644 --- a/src/di_container/asynchronous/binding/scope_configurator.rs +++ b/src/di_container/asynchronous/binding/scope_configurator.rs @@ -6,6 +6,7 @@ use std::sync::Arc; use crate::di_container::asynchronous::binding::when_configurator::AsyncBindingWhenConfigurator; use crate::di_container::asynchronous::IAsyncDIContainer; +use crate::di_container::BindingOptions; use crate::errors::async_di_container::AsyncBindingScopeConfiguratorError; use crate::interfaces::async_injectable::AsyncInjectable; use crate::provider::r#async::{AsyncSingletonProvider, AsyncTransientTypeProvider}; @@ -85,7 +86,7 @@ where self.di_container .set_binding::( - None, + BindingOptions::new(), Box::new(AsyncSingletonProvider::new(singleton)), ) .await; @@ -97,7 +98,7 @@ where { self.di_container .set_binding::( - None, + BindingOptions::new(), Box::new( AsyncTransientTypeProvider::::new(), ), @@ -121,7 +122,7 @@ mod tests di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|binding_options, _provider| binding_options.name.is_none()) .return_once(|_name, _provider| ()) .once(); @@ -143,7 +144,7 @@ mod tests di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|binding_options, _provider| binding_options.name.is_none()) .return_once(|_name, _provider| ()) .once(); diff --git a/src/di_container/asynchronous/binding/when_configurator.rs b/src/di_container/asynchronous/binding/when_configurator.rs index d08239e..4521178 100644 --- a/src/di_container/asynchronous/binding/when_configurator.rs +++ b/src/di_container/asynchronous/binding/when_configurator.rs @@ -6,6 +6,7 @@ use std::marker::PhantomData; use std::sync::Arc; use crate::di_container::asynchronous::IAsyncDIContainer; +use crate::di_container::BindingOptions; use crate::errors::async_di_container::AsyncBindingWhenConfiguratorError; /// When configurator for a binding for type `Interface` inside a [`IAsyncDIContainer`]. @@ -45,7 +46,7 @@ where { let binding = self .di_container - .remove_binding::(None) + .remove_binding::(BindingOptions::new()) .await .map_or_else( || { @@ -57,7 +58,7 @@ where )?; self.di_container - .set_binding::(Some(name), binding) + .set_binding::(BindingOptions::new().name(name), binding) .await; Ok(()) @@ -81,13 +82,13 @@ mod tests di_container_mock .expect_remove_binding::() - .with(eq(None)) + .with(eq(BindingOptions::new())) .return_once(|_name| Some(Box::new(MockIAsyncProvider::new()))) .once(); di_container_mock .expect_set_binding::() - .withf(|name, _provider| name == &Some("awesome")) + .withf(|binding_options, _provider| binding_options.name == Some("awesome")) .return_once(|_name, _provider| ()) .once(); diff --git a/src/di_container/asynchronous/mod.rs b/src/di_container/asynchronous/mod.rs index e5f7f5d..8d67b99 100644 --- a/src/di_container/asynchronous/mod.rs +++ b/src/di_container/asynchronous/mod.rs @@ -235,10 +235,7 @@ impl IAsyncDIContainer for AsyncDIContainer { Box::pin(async move { let binding_providable = self - .get_binding_providable::( - binding_options.name, - dependency_history, - ) + .get_binding_providable::(binding_options, dependency_history) .await?; self.handle_binding_providable(binding_providable).await @@ -249,16 +246,22 @@ impl IAsyncDIContainer for AsyncDIContainer #[async_trait] impl details::DIContainerInternals for AsyncDIContainer { - async fn has_binding(self: &Arc, name: Option<&'static str>) -> bool + async fn has_binding( + self: &Arc, + binding_options: BindingOptions<'static>, + ) -> bool where Interface: ?Sized + 'static, { - self.binding_storage.lock().await.has::(name) + self.binding_storage + .lock() + .await + .has::(binding_options) } async fn set_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, provider: Box>, ) where Interface: 'static + ?Sized, @@ -266,17 +269,20 @@ impl details::DIContainerInternals for AsyncDIContainer self.binding_storage .lock() .await - .set::(name, provider); + .set::(binding_options, provider); } async fn remove_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> Option>> where Interface: 'static + ?Sized, { - self.binding_storage.lock().await.remove::(name) + self.binding_storage + .lock() + .await + .remove::(binding_options) } } @@ -411,7 +417,7 @@ impl AsyncDIContainer async fn get_binding_providable( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, dependency_history: DependencyHistory, ) -> Result, AsyncDIContainerError> where @@ -423,12 +429,12 @@ impl AsyncDIContainer let bindings_lock = self.binding_storage.lock().await; provider = bindings_lock - .get::(name) + .get::(binding_options.clone()) .map_or_else( || { Err(AsyncDIContainerError::BindingNotFound { interface: type_name::(), - name, + name: binding_options.name, }) }, Ok, @@ -452,6 +458,7 @@ pub(crate) mod details use async_trait::async_trait; + use crate::di_container::BindingOptions; use crate::provider::r#async::IAsyncProvider; #[async_trait] @@ -459,21 +466,21 @@ pub(crate) mod details { async fn has_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> bool where Interface: ?Sized + 'static; async fn set_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, provider: Box>, ) where Interface: 'static + ?Sized; async fn remove_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> Option>> where Interface: 'static + ?Sized; @@ -514,7 +521,10 @@ mod tests .binding_storage .lock() .await - .set::(None, Box::new(mock_provider)); + .set::( + BindingOptions::new(), + Box::new(mock_provider), + ); } di_container @@ -550,7 +560,7 @@ mod tests .lock() .await .set::( - Some("special"), + BindingOptions::new().name("special"), Box::new(mock_provider), ); } @@ -591,7 +601,10 @@ mod tests .binding_storage .lock() .await - .set::(None, Box::new(mock_provider)); + .set::( + BindingOptions::new(), + Box::new(mock_provider), + ); } let first_number_rc = di_container @@ -640,7 +653,7 @@ mod tests .lock() .await .set::( - Some("cool"), + BindingOptions::new().name("cool"), Box::new(mock_provider), ); } @@ -747,7 +760,10 @@ mod tests .binding_storage .lock() .await - .set::(None, Box::new(mock_provider)); + .set::( + BindingOptions::new(), + Box::new(mock_provider), + ); } di_container @@ -843,7 +859,10 @@ mod tests .binding_storage .lock() .await - .set::(Some("special"), Box::new(mock_provider)); + .set::( + BindingOptions::new().name("special"), + Box::new(mock_provider), + ); } di_container diff --git a/src/di_container/binding_storage.rs b/src/di_container/binding_storage.rs index 3c3c565..2451791 100644 --- a/src/di_container/binding_storage.rs +++ b/src/di_container/binding_storage.rs @@ -2,11 +2,13 @@ use std::any::TypeId; use ahash::AHashMap; +use crate::di_container::BindingOptions; + pub struct DIContainerBindingStorage where Provider: 'static + ?Sized, { - inner: AHashMap, Box>, + inner: AHashMap, Box>, } impl DIContainerBindingStorage @@ -21,69 +23,64 @@ where } #[allow(clippy::borrowed_box)] - pub fn get<'me, Interface>( - &'me self, - name: Option<&'me str>, - ) -> Option<&'me Box> + pub fn get<'this, Interface>( + &'this self, + options: BindingOptions<'this>, + ) -> Option<&'this Box> where Interface: 'static + ?Sized, { - let interface_typeid = TypeId::of::(); - - self.inner.get(&BindingIdentification { - type_id: interface_typeid, - name, - }) + self.inner.get(&BindingId::new::(options)) } - pub fn set(&mut self, name: Option<&'static str>, provider: Box) - where + pub fn set( + &mut self, + options: BindingOptions<'static>, + provider: Box, + ) where Interface: 'static + ?Sized, { - let interface_typeid = TypeId::of::(); - - self.inner.insert( - BindingIdentification { - type_id: interface_typeid, - name, - }, - provider, - ); + self.inner + .insert(BindingId::new::(options), provider); } pub fn remove( &mut self, - name: Option<&'static str>, + options: BindingOptions<'static>, ) -> Option> where Interface: 'static + ?Sized, { - let interface_typeid = TypeId::of::(); - - self.inner.remove(&BindingIdentification { - type_id: interface_typeid, - name, - }) + self.inner.remove(&BindingId::new::(options)) } - pub fn has(&self, name: Option<&'static str>) -> bool + pub fn has(&self, options: BindingOptions) -> bool where Interface: 'static + ?Sized, { - let interface_typeid = TypeId::of::(); - - self.inner.contains_key(&BindingIdentification { - type_id: interface_typeid, - name, - }) + self.inner + .contains_key(&BindingId::new::(options)) } } #[derive(Debug, PartialEq, Eq, Hash)] -struct BindingIdentification<'a> +struct BindingId<'opts> { type_id: TypeId, - name: Option<&'a str>, + options: BindingOptions<'opts>, +} + +impl<'opts> BindingId<'opts> +{ + fn new(options: BindingOptions<'opts>) -> Self + where + Interface: ?Sized + 'static, + { + Self { + type_id: TypeId::of::(), + options, + } + } } #[cfg(test)] @@ -121,15 +118,12 @@ mod tests DIContainerBindingStorage::::new(); binding_map.inner.insert( - BindingIdentification { - type_id: TypeId::of::(), - name: None, - }, + BindingId::new::(BindingOptions::new()), Box::new(subjects::SomeProviderImpl { id: 20 }), ); assert!(binding_map - .get::(None) + .get::(BindingOptions::new()) .map_or_else(|| false, |provider| provider.get_id() == 20)); } @@ -142,18 +136,17 @@ mod tests DIContainerBindingStorage::::new(); binding_map.inner.insert( - BindingIdentification { - type_id: TypeId::of::(), - name: Some("hello"), - }, + BindingId::new::(BindingOptions::new().name("hello")), Box::new(subjects::SomeProviderImpl { id: 11 }), ); assert!(binding_map - .get::(Some("hello")) + .get::(BindingOptions::new().name("hello")) .map_or_else(|| false, |provider| provider.get_id() == 11)); - assert!(binding_map.get::(None).is_none()); + assert!(binding_map + .get::(BindingOptions::new()) + .is_none()); } #[test] @@ -164,17 +157,16 @@ mod tests let mut binding_map = DIContainerBindingStorage::::new(); - binding_map - .set::(None, Box::new(subjects::SomeProviderImpl { id: 65 })); + binding_map.set::( + BindingOptions::new(), + Box::new(subjects::SomeProviderImpl { id: 65 }), + ); - let expected_key = &BindingIdentification { - type_id: TypeId::of::(), - name: None, - }; + let expected_key = BindingId::new::(BindingOptions::new()); - assert!(binding_map.inner.contains_key(expected_key)); + assert!(binding_map.inner.contains_key(&expected_key)); - assert_eq!(binding_map.inner[expected_key].get_id(), 65); + assert_eq!(binding_map.inner[&expected_key].get_id(), 65); } #[test] @@ -186,18 +178,16 @@ mod tests DIContainerBindingStorage::::new(); binding_map.set::( - Some("special"), + BindingOptions::new().name("special"), Box::new(subjects::SomeProviderImpl { id: 3 }), ); - let expected_key = &BindingIdentification { - type_id: TypeId::of::(), - name: Some("special"), - }; + let expected_key = + BindingId::new::(BindingOptions::new().name("special")); - assert!(binding_map.inner.contains_key(expected_key)); + assert!(binding_map.inner.contains_key(&expected_key)); - assert_eq!(binding_map.inner[expected_key].get_id(), 3); + assert_eq!(binding_map.inner[&expected_key].get_id(), 3); } #[test] @@ -209,21 +199,15 @@ mod tests DIContainerBindingStorage::::new(); binding_map.inner.insert( - BindingIdentification { - type_id: TypeId::of::(), - name: None, - }, + BindingId::new::(BindingOptions::new()), Box::new(subjects::SomeProviderImpl { id: 103 }), ); - binding_map.remove::(None); - - let expected_key = &BindingIdentification { - type_id: TypeId::of::(), - name: None, - }; + binding_map.remove::(BindingOptions::new()); - assert!(!binding_map.inner.contains_key(expected_key)); + assert!(!binding_map + .inner + .contains_key(&BindingId::new::(BindingOptions::new()))); } #[test] @@ -235,21 +219,17 @@ mod tests DIContainerBindingStorage::::new(); binding_map.inner.insert( - BindingIdentification { - type_id: TypeId::of::(), - name: Some("cool"), - }, + BindingId::new::(BindingOptions::new().name("cool")), Box::new(subjects::SomeProviderImpl { id: 42 }), ); - binding_map.remove::(Some("cool")); + binding_map.remove::(BindingOptions::new().name("cool")); - let expected_key = &BindingIdentification { - type_id: TypeId::of::(), - name: Some("cool"), - }; - - assert!(!binding_map.inner.contains_key(expected_key)); + assert!( + !binding_map.inner.contains_key(&BindingId::new::( + BindingOptions::new().name("cool") + )) + ); } #[test] @@ -260,17 +240,14 @@ mod tests let mut binding_map = DIContainerBindingStorage::::new(); - assert!(!binding_map.has::(None)); + assert!(!binding_map.has::(BindingOptions::new())); binding_map.inner.insert( - BindingIdentification { - type_id: TypeId::of::(), - name: None, - }, + BindingId::new::(BindingOptions::new()), Box::new(subjects::SomeProviderImpl { id: 103 }), ); - assert!(binding_map.has::(None)); + assert!(binding_map.has::(BindingOptions::new())); } #[test] @@ -281,16 +258,13 @@ mod tests let mut binding_map = DIContainerBindingStorage::::new(); - assert!(!binding_map.has::(Some("awesome"))); + assert!(!binding_map.has::(BindingOptions::new().name("awesome"))); binding_map.inner.insert( - BindingIdentification { - type_id: TypeId::of::(), - name: Some("awesome"), - }, + BindingId::new::(BindingOptions::new().name("awesome")), Box::new(subjects::SomeProviderImpl { id: 101 }), ); - assert!(binding_map.has::(Some("awesome"))); + assert!(binding_map.has::(BindingOptions::new().name("awesome"))); } } diff --git a/src/di_container/blocking/binding/builder.rs b/src/di_container/blocking/binding/builder.rs index 0c323ec..577f034 100644 --- a/src/di_container/blocking/binding/builder.rs +++ b/src/di_container/blocking/binding/builder.rs @@ -9,6 +9,7 @@ use crate::di_container::blocking::binding::scope_configurator::BindingScopeConf #[cfg(feature = "factory")] use crate::di_container::blocking::binding::when_configurator::BindingWhenConfigurator; use crate::di_container::blocking::IDIContainer; +use crate::di_container::BindingOptions; use crate::errors::di_container::BindingBuilderError; use crate::interfaces::injectable::Injectable; use crate::util::use_double; @@ -99,7 +100,10 @@ where Implementation: Injectable, { { - if self.di_container.has_binding::(None) { + if self + .di_container + .has_binding::(BindingOptions::new()) + { return Err(BindingBuilderError::BindingAlreadyExists(type_name::< Interface, >( @@ -197,7 +201,10 @@ where { use crate::private::castable_factory::blocking::CastableFactory; - if self.di_container.has_binding::(None) { + if self + .di_container + .has_binding::(BindingOptions::new()) + { return Err(BindingBuilderError::BindingAlreadyExists(type_name::< Interface, >())); @@ -206,7 +213,7 @@ where let factory_impl = CastableFactory::new(factory_func); self.di_container.set_binding::( - None, + BindingOptions::new(), Box::new(crate::provider::blocking::FactoryProvider::new( crate::ptr::FactoryPtr::new(factory_impl), false, @@ -286,7 +293,10 @@ where { use crate::private::castable_factory::blocking::CastableFactory; - if self.di_container.has_binding::(None) { + if self + .di_container + .has_binding::(BindingOptions::new()) + { return Err(BindingBuilderError::BindingAlreadyExists(type_name::< Interface, >())); @@ -295,7 +305,7 @@ where let factory_impl = CastableFactory::new(factory_func); self.di_container.set_binding::( - None, + BindingOptions::new(), Box::new(crate::provider::blocking::FactoryProvider::new( crate::ptr::FactoryPtr::new(factory_impl), true, @@ -324,14 +334,14 @@ mod tests mock_di_container .expect_has_binding::() - .with(eq(None)) - .return_once(|_name| false) + .with(eq(BindingOptions::new())) + .return_once(|_options| false) .once(); mock_di_container .expect_set_binding::() - .withf(|name, _provider| name.is_none()) - .return_once(|_name, _provider| ()) + .withf(|options, _provider| options.name.is_none()) + .return_once(|_options, _provider| ()) .once(); let binding_builder = @@ -361,14 +371,14 @@ mod tests mock_di_container .expect_has_binding::() - .with(eq(None)) - .return_once(|_name| false) + .with(eq(BindingOptions::new())) + .return_once(|_| false) .once(); mock_di_container .expect_set_binding::() - .withf(|name, _provider| name.is_none()) - .return_once(|_name, _provider| ()) + .withf(|options, _provider| options.name.is_none()) + .return_once(|_, _provider| ()) .once(); let binding_builder = @@ -404,14 +414,14 @@ mod tests mock_di_container .expect_has_binding::() - .with(eq(None)) - .return_once(|_name| false) + .with(eq(BindingOptions::new())) + .return_once(|_| false) .once(); mock_di_container .expect_set_binding::() - .withf(|name, _provider| name.is_none()) - .return_once(|_name, _provider| ()) + .withf(|options, _provider| options.name.is_none()) + .return_once(|_, _provider| ()) .once(); let binding_builder = diff --git a/src/di_container/blocking/binding/scope_configurator.rs b/src/di_container/blocking/binding/scope_configurator.rs index 0aefa93..0fcdfdf 100644 --- a/src/di_container/blocking/binding/scope_configurator.rs +++ b/src/di_container/blocking/binding/scope_configurator.rs @@ -6,6 +6,7 @@ use std::rc::Rc; use crate::di_container::blocking::binding::when_configurator::BindingWhenConfigurator; use crate::di_container::blocking::IDIContainer; +use crate::di_container::BindingOptions; use crate::errors::di_container::BindingScopeConfiguratorError; use crate::interfaces::injectable::Injectable; use crate::provider::blocking::{SingletonProvider, TransientTypeProvider}; @@ -81,8 +82,10 @@ where .map_err(BindingScopeConfiguratorError::SingletonResolveFailed)?, ); - self.di_container - .set_binding::(None, Box::new(SingletonProvider::new(singleton))); + self.di_container.set_binding::( + BindingOptions::new(), + Box::new(SingletonProvider::new(singleton)), + ); Ok(BindingWhenConfigurator::new(self.di_container)) } @@ -90,7 +93,7 @@ where pub(crate) fn set_in_transient_scope(&self) { self.di_container.set_binding::( - None, + BindingOptions::new(), Box::new(TransientTypeProvider::::new()), ); } @@ -110,7 +113,7 @@ mod tests di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|options, _provider| options.name.is_none()) .return_once(|_name, _provider| ()) .once(); @@ -131,7 +134,7 @@ mod tests di_container_mock .expect_set_binding::() - .withf(|name, _provider| name.is_none()) + .withf(|options, _provider| options.name.is_none()) .return_once(|_name, _provider| ()) .once(); diff --git a/src/di_container/blocking/binding/when_configurator.rs b/src/di_container/blocking/binding/when_configurator.rs index fcef377..52b23ff 100644 --- a/src/di_container/blocking/binding/when_configurator.rs +++ b/src/di_container/blocking/binding/when_configurator.rs @@ -6,6 +6,7 @@ use std::marker::PhantomData; use std::rc::Rc; use crate::di_container::blocking::IDIContainer; +use crate::di_container::BindingOptions; use crate::errors::di_container::BindingWhenConfiguratorError; /// When configurator for a binding for type `Interface` inside a [`IDIContainer`]. @@ -45,7 +46,7 @@ where { let binding = self .di_container - .remove_binding::(None) + .remove_binding::(BindingOptions::new()) .map_or_else( || { Err(BindingWhenConfiguratorError::BindingNotFound(type_name::< @@ -57,7 +58,7 @@ where )?; self.di_container - .set_binding::(Some(name), binding); + .set_binding::(BindingOptions::new().name(name), binding); Ok(()) } @@ -79,13 +80,13 @@ mod tests di_container_mock .expect_remove_binding::() - .with(eq(None)) + .with(eq(BindingOptions::new())) .return_once(|_name| Some(Box::new(MockIProvider::new()))) .once(); di_container_mock .expect_set_binding::() - .withf(|name, _provider| name == &Some("cool")) + .withf(|options, _provider| options.name == Some("cool")) .return_once(|_name, _provider| ()) .once(); diff --git a/src/di_container/blocking/mod.rs b/src/di_container/blocking/mod.rs index 5b0acc8..27ea0fb 100644 --- a/src/di_container/blocking/mod.rs +++ b/src/di_container/blocking/mod.rs @@ -210,10 +210,8 @@ impl IDIContainer for DIContainer where Interface: 'static + ?Sized, { - let binding_providable = self.get_binding_providable::( - binding_options.name, - dependency_history, - )?; + let binding_providable = self + .get_binding_providable::(binding_options, dependency_history)?; #[cfg(feature = "factory")] return self.handle_binding_providable(binding_providable); @@ -225,33 +223,37 @@ impl IDIContainer for DIContainer impl details::DIContainerInternals for DIContainer { - fn has_binding(self: &Rc, name: Option<&'static str>) -> bool + fn has_binding(self: &Rc, binding_options: BindingOptions) -> bool where Interface: ?Sized + 'static, { - self.binding_storage.borrow().has::(name) + self.binding_storage + .borrow() + .has::(binding_options) } fn set_binding( self: &Rc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, provider: Box>, ) where Interface: 'static + ?Sized, { self.binding_storage .borrow_mut() - .set::(name, provider); + .set::(binding_options, provider); } fn remove_binding( self: &Rc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> Option>> where Interface: 'static + ?Sized, { - self.binding_storage.borrow_mut().remove::(name) + self.binding_storage + .borrow_mut() + .remove::(binding_options) } } @@ -316,15 +318,17 @@ impl DIContainer fn get_binding_providable( self: &Rc, - name: Option<&str>, + binding_options: BindingOptions, dependency_history: DependencyHistory, ) -> Result, DIContainerError> where Interface: 'static + ?Sized, { + let name = binding_options.name; + self.binding_storage .borrow() - .get::(name) + .get::(binding_options) .map_or_else( || { Err(DIContainerError::BindingNotFound { @@ -346,24 +350,29 @@ pub(crate) mod details { use std::rc::Rc; + use crate::di_container::blocking::BindingOptionsWithLt; + use crate::di_container::BindingOptions; use crate::provider::blocking::IProvider; pub trait DIContainerInternals { - fn has_binding(self: &Rc, name: Option<&'static str>) -> bool + fn has_binding( + self: &Rc, + binding_options: BindingOptionsWithLt, + ) -> bool where Interface: ?Sized + 'static; fn set_binding( self: &Rc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, provider: Box>, ) where Interface: 'static + ?Sized; fn remove_binding( self: &Rc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> Option>> where Interface: 'static + ?Sized; @@ -395,7 +404,10 @@ mod tests di_container .binding_storage .borrow_mut() - .set::(None, Box::new(mock_provider)); + .set::( + BindingOptions::new(), + Box::new(mock_provider), + ); di_container .get::()? @@ -420,7 +432,10 @@ mod tests di_container .binding_storage .borrow_mut() - .set::(Some("special"), Box::new(mock_provider)); + .set::( + BindingOptions::new().name("special"), + Box::new(mock_provider), + ); di_container .get_named::("special")? @@ -447,7 +462,7 @@ mod tests di_container .binding_storage .borrow_mut() - .set::(None, Box::new(mock_provider)); + .set::(BindingOptions::new(), Box::new(mock_provider)); let first_number_rc = di_container.get::()?.singleton()?; @@ -479,7 +494,10 @@ mod tests di_container .binding_storage .borrow_mut() - .set::(Some("cool"), Box::new(mock_provider)); + .set::( + BindingOptions::new().name("cool"), + Box::new(mock_provider), + ); let first_number_rc = di_container .get_named::("cool")? @@ -567,7 +585,7 @@ mod tests di_container .binding_storage .borrow_mut() - .set::(None, Box::new(mock_provider)); + .set::(BindingOptions::new(), Box::new(mock_provider)); di_container.get::()?.factory()?; @@ -645,7 +663,10 @@ mod tests di_container .binding_storage .borrow_mut() - .set::(Some("special"), Box::new(mock_provider)); + .set::( + BindingOptions::new().name("special"), + Box::new(mock_provider), + ); di_container .get_named::("special")? diff --git a/src/di_container/mod.rs b/src/di_container/mod.rs index 63733f5..7e8c11f 100644 --- a/src/di_container/mod.rs +++ b/src/di_container/mod.rs @@ -14,7 +14,7 @@ pub mod blocking; /// # /// BindingOptions::new().name("foo"); /// ``` -#[derive(Debug, Default, Clone)] +#[derive(Debug, Default, Clone, PartialEq, Eq, Hash)] pub struct BindingOptions<'a> { name: Option<&'a str>, @@ -24,7 +24,7 @@ impl<'a> BindingOptions<'a> { /// Returns a new `BindingOptions`. #[must_use] - pub fn new() -> Self + pub const fn new() -> Self { Self { name: None } } diff --git a/src/test_utils.rs b/src/test_utils.rs index 1e0e04d..a304a71 100644 --- a/src/test_utils.rs +++ b/src/test_utils.rs @@ -289,6 +289,7 @@ pub mod mocks use crate::di_container::blocking::binding::builder::BindingBuilder; use crate::di_container::blocking::details::DIContainerInternals; use crate::di_container::blocking::{BindingOptionsWithLt, IDIContainer}; + use crate::di_container::BindingOptions; use crate::errors::di_container::DIContainerError; use crate::provider::blocking::IProvider; use crate::ptr::SomePtr; @@ -328,21 +329,24 @@ pub mod mocks impl DIContainerInternals for DIContainer { - fn has_binding(self: &Rc, name: Option<&'static str>) -> bool + fn has_binding( + self: &Rc, + binding_options: BindingOptionsWithLt + ) -> bool where Interface: ?Sized + 'static; #[doc(hidden)] fn set_binding( self: &Rc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, provider: Box>, ) where Interface: 'static + ?Sized; fn remove_binding( self: &Rc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> Option>> where Interface: 'static + ?Sized; @@ -406,21 +410,21 @@ pub mod mocks { async fn has_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> bool where Interface: ?Sized + 'static; async fn set_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, provider: Box>, ) where Interface: 'static + ?Sized; async fn remove_binding( self: &Arc, - name: Option<&'static str>, + binding_options: BindingOptions<'static>, ) -> Option>> where Interface: 'static + ?Sized; -- cgit v1.2.3-18-g5258