From 6533246020c9dcc1acfc02b1e5ed74fbbce4ea1f Mon Sep 17 00:00:00 2001
From: HampusM <hampus@hampusmat.com>
Date: Wed, 27 Sep 2023 20:22:46 +0200
Subject: fix: set injectable macro dummies directly after parsing input

---
 macros/src/injectable/dummy.rs          | 58 +++++++++++++++++++++++++++
 macros/src/injectable/implementation.rs | 69 ++-------------------------------
 macros/src/injectable/mod.rs            |  1 +
 3 files changed, 62 insertions(+), 66 deletions(-)
 create mode 100644 macros/src/injectable/dummy.rs

(limited to 'macros/src/injectable')

diff --git a/macros/src/injectable/dummy.rs b/macros/src/injectable/dummy.rs
new file mode 100644
index 0000000..3ace798
--- /dev/null
+++ b/macros/src/injectable/dummy.rs
@@ -0,0 +1,58 @@
+use quote::quote;
+use syn::{Generics, Type};
+
+#[cfg(not(tarpaulin_include))]
+pub fn expand_dummy_blocking_impl(
+    generics: &Generics,
+    self_type: &Type,
+) -> proc_macro2::TokenStream
+{
+    quote! {
+        impl #generics syrette::interfaces::injectable::Injectable<
+            syrette::di_container::blocking::DIContainer,
+        > for #self_type
+        {
+            fn resolve(
+                _: &syrette::di_container::blocking::DIContainer,
+                _: syrette::dependency_history::DependencyHistory
+            ) -> Result<
+                syrette::ptr::TransientPtr<Self>,
+                syrette::errors::injectable::InjectableError>
+            {
+                unimplemented!();
+            }
+        }
+    }
+}
+
+#[cfg(not(tarpaulin_include))]
+#[cfg(feature = "async")]
+pub fn expand_dummy_async_impl(
+    generics: &Generics,
+    self_type: &Type,
+) -> proc_macro2::TokenStream
+{
+    quote! {
+        impl #generics syrette::interfaces::async_injectable::AsyncInjectable<
+            syrette::di_container::asynchronous::AsyncDIContainer,
+        > for #self_type
+        {
+            fn resolve<'di_container, 'fut>(
+                _: &'di_container syrette::di_container::asynchronous::AsyncDIContainer,
+                _: syrette::dependency_history::DependencyHistory
+            ) -> syrette::future::BoxFuture<
+                'fut,
+                Result<
+                    syrette::ptr::TransientPtr<Self>,
+                    syrette::errors::injectable::InjectableError
+                >
+            >
+            where
+                Self: Sized + 'fut,
+                'di_container: 'fut
+            {
+                unimplemented!();
+            }
+        }
+    }
+}
diff --git a/macros/src/injectable/implementation.rs b/macros/src/injectable/implementation.rs
index 98e0abf..34c5437 100644
--- a/macros/src/injectable/implementation.rs
+++ b/macros/src/injectable/implementation.rs
@@ -1,10 +1,9 @@
 use std::error::Error;
 
-use proc_macro2::{Ident, Span, TokenStream};
+use proc_macro2::{Ident, Span};
 use quote::{format_ident, quote, ToTokens};
 use syn::spanned::Spanned;
 use syn::{
-    parse2,
     parse_str,
     ExprMethodCall,
     FnArg,
@@ -37,17 +36,11 @@ pub struct InjectableImpl<Dep: IDependency>
 impl<Dep: IDependency> InjectableImpl<Dep>
 {
     #[cfg(not(tarpaulin_include))]
-    pub fn parse(
-        input: TokenStream,
+    pub fn new(
+        mut item_impl: ItemImpl,
         constructor: &Ident,
     ) -> Result<Self, InjectableImplError>
     {
-        let mut item_impl = parse2::<ItemImpl>(input).map_err(|err| {
-            InjectableImplError::NotAImplementation {
-                err_span: err.span(),
-            }
-        })?;
-
         if let Some((_, trait_path, _)) = item_impl.trait_ {
             return Err(InjectableImplError::TraitImpl {
                 trait_path_span: trait_path.span(),
@@ -313,62 +306,6 @@ impl<Dep: IDependency> InjectableImpl<Dep>
         }
     }
 
-    #[cfg(not(tarpaulin_include))]
-    pub fn expand_dummy_blocking_impl(&self) -> proc_macro2::TokenStream
-    {
-        let generics = &self.generics;
-        let self_type = &self.self_type;
-
-        quote! {
-            impl #generics syrette::interfaces::injectable::Injectable<
-                syrette::di_container::blocking::DIContainer,
-            > for #self_type
-            {
-                fn resolve(
-                    _: &syrette::di_container::blocking::DIContainer,
-                    _: syrette::dependency_history::DependencyHistory
-                ) -> Result<
-                    syrette::ptr::TransientPtr<Self>,
-                    syrette::errors::injectable::InjectableError>
-                {
-                    unimplemented!();
-                }
-            }
-        }
-    }
-
-    #[cfg(not(tarpaulin_include))]
-    pub fn expand_dummy_async_impl(&self) -> proc_macro2::TokenStream
-    {
-        let generics = &self.generics;
-        let self_type = &self.self_type;
-
-        quote! {
-            impl #generics syrette::interfaces::async_injectable::AsyncInjectable<
-                syrette::di_container::asynchronous::AsyncDIContainer,
-            > for #self_type
-            {
-                fn resolve<'di_container, 'fut>(
-                    _: &'di_container
-                        syrette::di_container::asynchronous::AsyncDIContainer,
-                    _: syrette::dependency_history::DependencyHistory
-                ) -> syrette::future::BoxFuture<
-                    'fut,
-                    Result<
-                        syrette::ptr::TransientPtr<Self>,
-                        syrette::errors::injectable::InjectableError
-                    >
-                >
-                where
-                    Self: Sized + 'fut,
-                    'di_container: 'fut
-                {
-                    unimplemented!();
-                }
-            }
-        }
-    }
-
     fn create_get_dep_method_calls(
         dependencies: &[Dep],
         is_async: bool,
diff --git a/macros/src/injectable/mod.rs b/macros/src/injectable/mod.rs
index b713aeb..0e6ddfd 100644
--- a/macros/src/injectable/mod.rs
+++ b/macros/src/injectable/mod.rs
@@ -1,4 +1,5 @@
 pub mod dependency;
+pub mod dummy;
 pub mod implementation;
 pub mod macro_args;
 pub mod named_attr_input;
-- 
cgit v1.2.3-18-g5258