target: moved target command to own file

Author: P-Storm

src/bin/bmputil-cli/cli_commands/mod.rs

@@ -10,17 +10,20 @@ use directories::ProjectDirs;
 use log::error;
 
 use crate::cli_commands::probe::ProbeArguments;
+use crate::cli_commands::target::TargetCommmands;
 use crate::{CliArguments, CompletionArguments};
 
 pub mod probe;
+mod target;
 
 #[derive(Subcommand)]
 pub enum ToplevelCommmands
 {
 	/// Actions to be performed against a probe
 	Probe(ProbeArguments),
 	/// Actions to be performed against a target connected to a probe
-	Target,
+	#[command(subcommand)]
+	Target(TargetCommmands),
 	/// Actions that run the tool as a debug/tracing server
 	Server,
 	/// Actions that run debugging commands against a target connected to a probe

src/bin/bmputil-cli/cli_commands/target.rs

@@ -0,0 +1,41 @@
+use bmputil::bmp::BmpMatcher;
+use clap::Subcommand;
+use color_eyre::eyre::Result;
+use log::info;
+
+use crate::CliArguments;
+
+#[derive(Subcommand)]
+#[command(arg_required_else_help(true))]
+pub enum TargetCommmands
+{
+	/// Print information about the target power control state
+	Power,
+}
+
+impl TargetCommmands
+{
+	pub fn subcommand(&self, cli_args: &CliArguments) -> Result<()>
+	{
+		match self {
+			TargetCommmands::Power => power_command(&cli_args),
+		}
+	}
+}
+
+fn power_command(cli_args: &CliArguments) -> Result<()>
+{
+	// Try and identify all the probes on the system that are allowed by the invocation
+	let matcher = BmpMatcher::from_params(cli_args);
+	let mut results = matcher.find_matching_probes();
+
+	// Otherwise, turn the result set into a list and go through them displaying them
+	let device = results.pop_single("power").map_err(|kind| kind.error())?;
+	let remote = device.bmd_serial_interface()?.remote()?;
+
+	let power = remote.get_target_power_state()?;
+
+	info!("Device target power state: {}", power);
+
+	Ok(())
+}

src/bin/bmputil-cli/main.rs

@@ -244,10 +244,7 @@ fn main() -> Result<()>
 
 	match &cli_args.subcommand {
 		ToplevelCommmands::Probe(probe_args) => probe_args.subcommand(&cli_args),
-		ToplevelCommmands::Target => {
-			warn!("Command space reserved for future tool version");
-			Ok(())
-		},
+		ToplevelCommmands::Target(target_commands) => target_commands.subcommand(&cli_args),
 		ToplevelCommmands::Server => {
 			warn!("Command space reserved for future tool version");
 			Ok(())