Merge pull request #25 from interlay/greg/fix/vault-exit

fix(vault): prettier exit response

vault/src/error.rs

@@ -9,6 +9,8 @@ use thiserror::Error;
 
 #[derive(Error, Debug)]
 pub enum Error {
+    #[error("Internal error")]
+    InternalError,
     #[error("Insufficient funds available")]
     InsufficientFunds,
     #[error("Open time inconsistent with chain height")]

vault/src/lib.rs

@@ -476,14 +476,10 @@ pub async fn start<B: BitcoinCoreApi + Send + Sync + 'static>(
         }),
     );
     match result {
-        Ok(_) => {
-            println!("Done");
-        }
+        Ok(_) => Ok(()),
         Err(err) => {
-            println!("Error: {}", err);
-            std::process::exit(1);
+            error!("{:?}", err);
+            Err(Error::InternalError)
         }
-    };
-
-    Ok(())
+    }
 }

vault/src/main.rs

@@ -5,10 +5,9 @@ use runtime::substrate_subxt::PairSigner;
 use runtime::{PolkaBtcProvider, PolkaBtcRuntime};
 use std::sync::Arc;
 use std::time::Duration;
-use vault::{start, Error, Opts};
+use vault::{Error, Opts};
 
-#[tokio::main]
-async fn main() -> Result<(), Error> {
+async fn start() -> Result<(), Error> {
     env_logger::init();
     let opts: Opts = Opts::parse();
     let intact_opts = opts.clone();
@@ -43,5 +42,16 @@ async fn main() -> Result<(), Error> {
         .await?,
     );
 
-    start(intact_opts, provider, btc_rpc).await
+    vault::start(intact_opts, provider, btc_rpc).await
+}
+
+#[tokio::main]
+async fn main() {
+    let exit_code = if let Err(err) = start().await {
+        eprintln!("Error: {}", err);
+        1
+    } else {
+        0
+    };
+    std::process::exit(exit_code);
 }