refactor(queries): construct FindError instead of returning a strin…

…g error in executor Signed-off-by: ⭐️NINIKA⭐️ <[email protected]>
hyperledger-iroha · Aug 5, 2024 · d3d4763 · d3d4763
1 parent 7fdc428
commit d3d4763
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/smart_contract/executor/src/default.rs b/smart_contract/executor/src/default.rs
@@ -1142,7 +1142,12 @@ pub mod role {
                         crate::data_model::query::builder::SingleQueryError::ExpectedOneGotNone,
                     ) => {
                         // assuming that only a "not found" case is possible here
-                        deny!($executor, "Role not found")
+                        $executor.deny($crate::data_model::ValidationFail::QueryFailed(
+                            $crate::data_model::query::error::QueryExecutionFail::Find(
+                                $crate::data_model::query::error::FindError::Role(role_id.clone()),
+                            ),
+                        ));
+                        return;
                     }
                     Err(_) => {
                         unreachable!();