chore: Refactoring: remove code coverage exclusions

CosmWasm · Feb 27, 2024 · d7042f6 · d7042f6
1 parent 53d4af7
commit d7042f6
Show file tree

Hide file tree

Showing 8 changed files with 0 additions and 19 deletions.
diff --git a/sylvia-derive/src/parser/attributes/custom.rs b/sylvia-derive/src/parser/attributes/custom.rs
@@ -47,8 +47,6 @@ impl Custom {
     }
 }
 
-#[cfg(not(tarpaulin_include))]
-// False negative. It is being called in closure
 impl Parse for Custom {
     fn parse(input: ParseStream) -> Result<Self> {
         let mut custom = Self::default();

diff --git a/sylvia-derive/src/parser/attributes/error.rs b/sylvia-derive/src/parser/attributes/error.rs
@@ -34,8 +34,6 @@ impl ContractErrorAttr {
     }
 }
 
-#[cfg(not(tarpaulin_include))]
-// False negative. It is being called in closure
 impl Parse for ContractErrorAttr {
     fn parse(input: ParseStream) -> Result<Self> {
         input.parse().map(|error| Self { error })

diff --git a/sylvia-derive/src/parser/attributes/messages.rs b/sylvia-derive/src/parser/attributes/messages.rs
@@ -78,8 +78,6 @@ fn interface_has_custom(content: ParseStream) -> Result<Customs> {
     Ok(customs)
 }
 
-#[cfg(not(tarpaulin_include))]
-// False negative. It is being called in closure
 impl Parse for ContractMessageAttr {
     fn parse(input: ParseStream) -> Result<Self> {
         let module = input.parse()?;

diff --git a/sylvia/src/into_response.rs b/sylvia/src/into_response.rs
@@ -8,7 +8,6 @@ pub trait IntoMsg<C> {
 /// `SubMsg<Empty>` can be made into any `SubMsg<C>`
 impl<C> IntoMsg<C> for SubMsg<Empty> {
     fn into_msg(self) -> StdResult<SubMsg<C>> {
-        #[cfg(not(tarpaulin_include))]
         let msg = match self.msg {
             CosmosMsg::Wasm(wasm) => CosmosMsg::Wasm(wasm),
             CosmosMsg::Bank(bank) => CosmosMsg::Bank(bank),

diff --git a/sylvia/tests/custom_query.rs b/sylvia/tests/custom_query.rs
@@ -31,11 +31,9 @@ mod interface {
         type Error: From<StdError>;
         type QueryC: CustomQuery;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(query)]
         fn interface_query(&self, ctx: QueryCtx<MyQuery>) -> StdResult<SomeResponse>;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(exec)]
         fn interface_exec(&self, ctx: ExecCtx<MyQuery>) -> StdResult<Response>;
     }
@@ -79,11 +77,9 @@ mod some_interface {
     pub trait SomeInterface {
         type Error: From<StdError>;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(query)]
         fn some_interface_query(&self, ctx: QueryCtx<MyQuery>) -> StdResult<SomeResponse>;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(exec)]
         fn some_interface_exec(&self, ctx: ExecCtx<MyQuery>) -> StdResult<Response>;
     }
@@ -126,11 +122,9 @@ mod associated_type_interface {
         type Error: From<StdError>;
         type QueryC: CustomQuery;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(query)]
         fn associated_query(&self, ctx: QueryCtx<Self::QueryC>) -> StdResult<SomeResponse>;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(exec)]
         fn associated_exec(&self, ctx: ExecCtx<Self::QueryC>) -> StdResult<Response>;
     }
@@ -171,11 +165,9 @@ mod default_query_interface {
     pub trait DefaultQueryInterface {
         type Error: From<StdError>;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(query)]
         fn default_query(&self, ctx: QueryCtx) -> StdResult<SomeResponse>;
 
-        #[cfg(not(tarpaulin_include))]
         #[sv::msg(exec)]
         fn default_exec(&self, ctx: ExecCtx) -> StdResult<Response>;
     }
@@ -232,7 +224,6 @@ impl MyContract {
         Ok(SomeResponse)
     }
 
-    #[cfg(not(tarpaulin_include))]
     #[sv::msg(migrate)]
     pub fn some_migrate(&self, _ctx: MigrateCtx<MyQuery>) -> StdResult<Response> {
         Ok(Response::default())

diff --git a/sylvia/tests/dispatching.rs b/sylvia/tests/dispatching.rs
@@ -113,7 +113,6 @@ mod contract {
     }
 
     #[allow(dead_code)]
-    #[cfg(not(tarpaulin_include))]
     #[contract]
     #[sv::messages(crate::interface)]
     impl Contract {

diff --git a/sylvia/tests/entry_points.rs b/sylvia/tests/entry_points.rs
@@ -99,7 +99,6 @@ mod contract {
 
     pub struct Contract {}
 
-    #[cfg(not(tarpaulin_include))]
     #[contract]
     #[sv::override_entry_point(sudo=crate::entry_points::sudo(crate::SudoMsg))]
     #[sv::override_entry_point(migrate=crate::entry_points::migrate(crate::migrate::MigrateMsg))]

diff --git a/sylvia/tests/messages_generation.rs b/sylvia/tests/messages_generation.rs
@@ -55,7 +55,6 @@ mod contract {
 
     pub struct Contract {}
 
-    #[cfg(not(tarpaulin_include))]
     #[entry_points]
     #[contract]
     #[allow(dead_code)]