Fix cli options (#4772)

## Issue Addressed Fixes breaking change introduced on #4674 that doesn't allow multiple `http_enabled` `ArgGroup` flags
sigp · Sep 22, 2023 · 0f05499 · 0f05499
1 parent fbb6997
commit 0f05499
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/beacon_node/src/cli.rs b/beacon_node/src/cli.rs
@@ -1273,5 +1273,5 @@ pub fn cli_app<'a, 'b>() -> App<'a, 'b> {
                 .default_value("64")
                 .takes_value(true)
         )
-        .group(ArgGroup::with_name("enable_http").args(&["http", "gui", "staking"]))
+        .group(ArgGroup::with_name("enable_http").args(&["http", "gui", "staking"]).multiple(true))
 }
diff --git a/lighthouse/tests/beacon_node.rs b/lighthouse/tests/beacon_node.rs
@@ -2337,6 +2337,18 @@ fn gui_flag() {
         });
 }
 
+#[test]
+fn multiple_http_enabled_flags() {
+    CommandLineTest::new()
+        .flag("gui", None)
+        .flag("http", None)
+        .flag("staking", None)
+        .run_with_zero_port()
+        .with_config(|config| {
+            assert!(config.http_api.enabled);
+        });
+}
+
 #[test]
 fn optimistic_finalized_sync_default() {
     CommandLineTest::new()