add warn, re-add tests

beacon_node/src/cli.rs

@@ -383,7 +383,7 @@ pub fn cli_app<'a, 'b>() -> App<'a, 'b> {
                 .requires("enable_http")
                 .value_name("ORIGIN")
                 .help("Set the value of the Access-Control-Allow-Origin response HTTP header. \
-                    Use * to allow any origin (not recommended in production). \
+                    Use * to allow any origin (not recommended in production). 
                     If no value is supplied, the CORS allowed origin is set to the listen \
                     address of this server (e.g., http://localhost:5052).")
                 .takes_value(true),

beacon_node/src/config.rs

@@ -127,6 +127,14 @@ pub fn get_config<E: EthSpec>(
             client_config.http_api.allow_origin = Some(allow_origin.to_string());
         }
 
+        if cli_args.is_present("http-spec-fork") {
+            warn!(
+                log,
+                "Ignoring --http-spec-fork";
+                "info" => "this flag is deprecated and will be removed"
+            );
+        }
+
         if cli_args.is_present("http-enable-tls") {
             client_config.http_api.tls_config = Some(TlsConfig {
                 cert: cli_args
@@ -142,6 +150,14 @@ pub fn get_config<E: EthSpec>(
             });
         }
 
+        if cli_args.is_present("http-allow-sync-stalled") {
+            warn!(
+                log,
+                "Ignoring --http-allow-sync-stalled";
+                "info" => "this flag is deprecated and will be removed"
+            );
+        }
+
         client_config.http_api.sse_capacity_multiplier =
             parse_required(cli_args, "http-sse-capacity-multiplier")?;
 

lighthouse/tests/beacon_node.rs

@@ -1587,6 +1587,14 @@ fn http_allow_origin_all_flag() {
         .with_config(|config| assert_eq!(config.http_api.allow_origin, Some("*".to_string())));
 }
 
+#[test]
+fn http_allow_sync_stalled_flag() {
+    CommandLineTest::new()
+        .flag("http", None)
+        .flag("http-allow-sync-stalled", None)
+        .run_with_zero_port();
+}
+
 #[test]
 fn http_enable_beacon_processor() {
     CommandLineTest::new()
@@ -1633,6 +1641,14 @@ fn http_spec_fork_default() {
         .with_config(|config| assert_eq!(config.http_api.spec_fork_name, None));
 }
 
+#[test]
+fn http_spec_fork_override() {
+    CommandLineTest::new()
+        .flag("http", None)
+        .flag("http-spec-fork", Some("altair"))
+        .run_with_zero_port();
+}
+
 // Tests for Metrics flags.
 #[test]
 fn metrics_flag() {