From b64b7e739d56ae8cb8bb1f5f9204a4c4d810977b Mon Sep 17 00:00:00 2001 From: HampusM Date: Wed, 31 Aug 2022 21:44:10 +0200 Subject: refactor: improve async DI container cast errors --- src/async_di_container.rs | 68 ++++++++++++++++++++++++---------------- src/errors/async_di_container.rs | 13 ++++++-- 2 files changed, 52 insertions(+), 29 deletions(-) diff --git a/src/async_di_container.rs b/src/async_di_container.rs index ecf3a41..23eb2eb 100644 --- a/src/async_di_container.rs +++ b/src/async_di_container.rs @@ -406,48 +406,62 @@ impl AsyncDIContainer AsyncProvidable::Transient(transient_binding) => { Ok(SomeThreadsafePtr::Transient( transient_binding.cast::().map_err(|_| { - AsyncDIContainerError::CastFailed(type_name::()) + AsyncDIContainerError::CastFailed { + interface: type_name::(), + binding_kind: "transient", + } })?, )) } AsyncProvidable::Singleton(singleton_binding) => { - Ok( - SomeThreadsafePtr::ThreadsafeSingleton( - singleton_binding.cast::().map_err( - |err| match err { - CastError::NotArcCastable(_) => { - AsyncDIContainerError::InterfaceNotAsync(type_name::< - Interface, - >( - )) + Ok(SomeThreadsafePtr::ThreadsafeSingleton( + singleton_binding + .cast::() + .map_err(|err| match err { + CastError::NotArcCastable(_) => { + AsyncDIContainerError::InterfaceNotAsync(type_name::< + Interface, + >( + )) + } + CastError::CastFailed { from: _, to: _ } => { + AsyncDIContainerError::CastFailed { + interface: type_name::(), + binding_kind: "singleton", } - CastError::CastFailed { from: _, to: _ } => { - AsyncDIContainerError::CastFailed(type_name::< - Interface, - >( - )) - } - }, - )?, - ), - ) + } + })?, + )) } #[cfg(feature = "factory")] AsyncProvidable::Factory(factory_binding) => { match factory_binding.clone().cast::() { Ok(factory) => Ok(SomeThreadsafePtr::ThreadsafeFactory(factory)), - Err(_err) => { + Err(first_err) => { use crate::interfaces::factory::IFactory; - let default_factory = - factory_binding - .cast::>() - .map_err(|_| { - AsyncDIContainerError::CastFailed(type_name::< + if let CastError::NotArcCastable(_) = first_err { + return Err(AsyncDIContainerError::InterfaceNotAsync( + type_name::(), + )); + } + + let default_factory = factory_binding + .cast::>() + .map_err(|err| match err { + CastError::NotArcCastable(_) => { + AsyncDIContainerError::InterfaceNotAsync(type_name::< Interface, >( )) - })?; + } + CastError::CastFailed { from: _, to: _ } => { + AsyncDIContainerError::CastFailed { + interface: type_name::(), + binding_kind: "factory", + } + } + })?; Ok(SomeThreadsafePtr::Transient(default_factory())) } diff --git a/src/errors/async_di_container.rs b/src/errors/async_di_container.rs index bdb6fa0..22d8fdd 100644 --- a/src/errors/async_di_container.rs +++ b/src/errors/async_di_container.rs @@ -15,8 +15,17 @@ use crate::errors::injectable::InjectableError; pub enum AsyncDIContainerError { /// Unable to cast a binding for a interface. - #[error("Unable to cast binding for interface '{0}'")] - CastFailed(&'static str), + #[error( + "Unable to cast binding for interface '{interface} with kind '{binding_kind}'" + )] + CastFailed + { + /// The interface. + interface: &'static str, + + /// The kind of the found binding. + binding_kind: &'static str, + }, /// Failed to resolve a binding for a interface. #[error("Failed to resolve binding for interface '{interface}'")] -- cgit v1.2.3-18-g5258