refac: remove unused or unnecessary StratusError variants (#1839)

cloudwalk · Nov 1, 2024 · 75fc79a · 75fc79a
1 parent 0daacb7
commit 75fc79a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 10 deletions.
diff --git a/src/eth/primitives/stratus_error.rs b/src/eth/primitives/stratus_error.rs
@@ -123,14 +123,6 @@ pub enum StratusError {
     // -------------------------------------------------------------------------
     // Miner
     // -------------------------------------------------------------------------
-    #[error("Requested miner mode conflicts with current miner mode.")]
-    #[strum(props(kind = "internal"))]
-    MinerModeConflict,
-
-    #[error("Miner mode change to ({miner_mode}) is unsupported.")]
-    #[strum(props(kind = "internal"))]
-    MinerModeChangeUnsupported { miner_mode: &'static str },
-
     #[error("Miner mode param is invalid.")]
     #[strum(props(kind = "internal"))]
     MinerModeParamInvalid,

diff --git a/src/eth/rpc/rpc_server.rs b/src/eth/rpc/rpc_server.rs
@@ -518,8 +518,7 @@ async fn change_miner_mode(new_mode: MinerMode, ctx: &RpcContext) -> Result<Json
             ctx.miner.start_interval_mining(duration).await;
         }
         MinerMode::Automine => {
-            tracing::error!("automine mode is not supported");
-            return Err(StratusError::MinerModeChangeUnsupported { miner_mode: "automine" });
+            return log_and_err!("Miner mode change to 'automine' is unsupported.").map_err(Into::into);
         }
     }