Added integration tests to ensure the CLI commands work

Author: jclapis

Cargo.lock

@@ -19,6 +19,7 @@ alloy = { version = "^1.0.35", features = [
     "ssz",
 ] }
 alloy-primitives = "^1.3.1"
+assert_cmd = "2.1.2"
 async-trait = "0.1.80"
 axum = { version = "0.8.1", features = ["macros"] }
 axum-extra = { version = "0.10.0", features = ["typed-header"] }
@@ -54,6 +55,7 @@ lh_types = { package = "types", git = "https://github.com/sigp/lighthouse", tag
 notify = "8.2.0"
 parking_lot = "0.12.3"
 pbkdf2 = "0.12.2"
+predicates = "3.0.3"
 prometheus = "0.14.0"
 prost = "0.13.4"
 rand = { version = "0.9", features = ["os_rng"] }

Cargo.toml

@@ -18,6 +18,10 @@ tracing.workspace = true
 tree_hash.workspace = true
 tree_hash_derive.workspace = true
 
+[dev-dependencies]
+assert_cmd.workspace = true
+predicates.workspace = true
+
 [[bin]]
 name = "commit-boost"
 path = "commit-boost.rs"

bin/Cargo.toml

@@ -13,15 +13,12 @@ use clap::{Parser, Subcommand};
 use eyre::Result;
 use tracing::{error, info};
 
-/// Version string with a leading 'v'
-const VERSION: &str = concat!("v", env!("CARGO_PKG_VERSION"));
-
 /// Long about string for the CLI
 const LONG_ABOUT: &str = "Commit-Boost allows Ethereum validators to safely run MEV-Boost and community-built commitment protocols";
 
 /// Subcommands and global arguments for the module
 #[derive(Parser, Debug)]
-#[command(name = "Commit-Boost", version = VERSION, about, long_about = LONG_ABOUT)]
+#[command(name = "Commit-Boost", version = commit_boost::VERSION, about, long_about = LONG_ABOUT)]
 struct Cli {
     #[command(subcommand)]
     command: Commands,
@@ -115,6 +112,8 @@ async fn run_init(config_path: PathBuf, output_path: PathBuf) -> Result<()> {
 
 #[cfg(test)]
 mod tests {
+    use commit_boost::VERSION;
+
     use super::*;
 
     #[test]

bin/commit-boost.rs

@@ -24,3 +24,6 @@ pub mod prelude {
     }
     pub use tree_hash_derive::TreeHash;
 }
+
+/// Version string with a leading 'v'
+pub const VERSION: &str = concat!("v", env!("CARGO_PKG_VERSION"));

bin/src/lib.rs

@@ -0,0 +1,36 @@
+use assert_cmd::{Command, cargo};
+
+/// Tests that the binary can be run and returns a version string
+#[test]
+fn test_load_example_config() {
+    let mut cmd = Command::new(cargo::cargo_bin!());
+    let expected_version = format!("Commit-Boost {}\n", commit_boost::VERSION);
+    cmd.arg("--version").assert().success().stdout(expected_version);
+}
+
+/// Tests that the init command can be run and complains about not having
+/// --config set
+#[test]
+fn test_run_init() {
+    let mut cmd = Command::new(cargo::cargo_bin!());
+    cmd.args(["init"]).assert().failure().stderr(predicates::str::contains(
+        "error: the following required arguments were not provided:\n  --config <CONFIG_PATH>",
+    ));
+}
+
+/// Tests that PBS runs without CB_CONFIG being set and complains normally
+#[test]
+fn test_run_pbs_no_config() {
+    let mut cmd = Command::new(cargo::cargo_bin!());
+    cmd.args(["pbs"]).assert().failure().stderr(predicates::str::contains("CB_CONFIG is not set"));
+}
+
+/// Tests that Signer runs without CB_CONFIG being set and complains normally
+#[test]
+fn test_run_signer_no_config() {
+    let mut cmd = Command::new(cargo::cargo_bin!());
+    cmd.args(["signer"])
+        .assert()
+        .failure()
+        .stderr(predicates::str::contains("CB_CONFIG is not set"));
+}