core,node,server: improve graphman reassign response type

Author: shiyasmohd

core/graphman/src/commands/deployment/reassign.rs

@@ -49,7 +49,7 @@ pub enum ReassignDeploymentError {
 
 #[derive(Clone, Debug)]
 pub enum ReassignResult {
-    EmptyResponse,
+    Ok,
     CompletedWithWarnings(Vec<String>),
 }
 
@@ -121,6 +121,6 @@ pub fn reassign_deployment(
         let warning_msg = format!("This is the only deployment assigned to '{}'. Please make sure that the node ID is spelled correctly.",node.as_str());
         Ok(ReassignResult::CompletedWithWarnings(vec![warning_msg]))
     } else {
-        Ok(ReassignResult::EmptyResponse)
+        Ok(ReassignResult::Ok)
     }
 }

node/src/manager/commands/deployment/reassign.rs

@@ -36,7 +36,7 @@ pub fn run(
     )?;
 
     match reassign_result {
-        ReassignResult::EmptyResponse => {
+        ReassignResult::Ok => {
             println!(
                 "Deployment {} assigned to node {}",
                 deployment.locator(),

server/graphman/src/resolvers/deployment_mutation.rs

@@ -27,7 +27,7 @@ pub struct DeploymentMutation;
 
 #[derive(Clone, Debug, Union)]
 pub enum ReassignResponse {
-    EmptyResponse(EmptyResponse),
+    Ok(EmptyResponse),
     CompletedWithWarnings(CompletedWithWarnings),
 }
 
@@ -124,9 +124,7 @@ impl DeploymentMutation {
             ReassignResult::CompletedWithWarnings(warnings) => Ok(
                 ReassignResponse::CompletedWithWarnings(CompletedWithWarnings::new(warnings)),
             ),
-            ReassignResult::EmptyResponse => {
-                Ok(ReassignResponse::EmptyResponse(EmptyResponse::new()))
-            }
+            ReassignResult::Ok => Ok(ReassignResponse::Ok(EmptyResponse::new())),
         }
     }
 }