tweak start_node_without_binary api

test-utils/cli/Cargo.toml

@@ -22,4 +22,5 @@ tokio = { version = "1.22.0", features = ["full"] }
 node-primitives = { path = "../../bin/node/primitives" }
 node-template = { path = "../../bin/node-template/node" }
 sc-cli = { path = "../../client/cli" }
+sc-service = { path = "../../client/service" }
 futures = "0.3.28"

test-utils/cli/src/lib.rs

@@ -51,26 +51,17 @@ use tokio::io::{AsyncBufReadExt, AsyncRead};
 /// 	common::start_node_without_binary();
 /// });
 /// ```
-pub fn start_node_without_binary() {
+pub fn start_node_without_binary() -> Result<(), sc_service::error::Error> {
 	use sc_cli::SubstrateCli;
 
 	let cli = node_template::cli::Cli::from_iter(vec![
-		"node-template",
+		"node-template", // first arg is ignored, this could be anything.
 		"--dev",
 		"--tmp",
 		"--rpc-port=45789",
 	]);
 	let runner = cli.create_runner(&cli.run).unwrap();
-	match runner.run_node_until_exit(|config| async move {
-		node_template::service::new_full(config).map_err(sc_cli::Error::Service)
-	}) {
-		Ok(_) => {
-			println!("Node process exited successfully.");
-		},
-		Err(e) => {
-			println!("Node process exited with error: {:?}", e);
-		},
-	};
+	runner.run_node_until_exit(|config| async move { node_template::service::new_full(config) })
 }
 
 /// Starts a new Substrate node in development mode with a temporary chain.