Fix compilation errors and add tests.

lighthouse/environment/src/lib.rs

@@ -141,11 +141,12 @@ impl<E: EthSpec> EnvironmentBuilder<E> {
             }
         } else {
             let stdout_decorator_builder = slog_term::TermDecorator::new();
-            let stdout_decorator = if config.log_color { 
+            let stdout_decorator = if config.log_color {
                 stdout_decorator_builder.force_color()
-            } else { 
-                stdout_decorator_builder 
-            }.build();
+            } else {
+                stdout_decorator_builder
+            }
+            .build();
             let stdout_decorator =
                 logging::AlignedTermDecorator::new(stdout_decorator, logging::MAX_MESSAGE_WIDTH);
             let stdout_drain = slog_term::FullFormat::new(stdout_decorator).build().fuse();

lighthouse/tests/beacon_node.rs

@@ -1354,6 +1354,32 @@ pub fn malloc_tuning_flag() {
         });
 }
 #[test]
+pub fn log_color_flag() {
+    CommandLineTest::new()
+        .flag("log-color")
+        .run_with_zero_port()
+        .with_config(|config| {
+            assert!(config.log_color);
+        });
+}
+#[test]
+pub fn log_colour_flag() {
+    CommandLineTest::new()
+        .flag("log-colour")
+        .run_with_zero_port()
+        .with_config(|config| {
+            assert!(config.log_color);
+        });
+}
+#[test]
+pub fn abscence_of_log_color_flag() {
+    CommandLineTest::new()
+        .run_with_zero_port()
+        .with_config(|config| {
+            assert!(!config.log_color);
+        });
+}
+#[test]
 #[should_panic]
 fn ensure_panic_on_failed_launch() {
     CommandLineTest::new()

testing/simulator/src/eth1_sim.rs

@@ -65,6 +65,7 @@ pub fn run_eth1_sim(matches: &ArgMatches) -> Result<(), String> {
             debug_level: log_level,
             logfile_debug_level: "debug",
             log_format,
+            log_color: false,
             max_log_size: 0,
             max_log_number: 0,
             compression: false,

testing/simulator/src/no_eth1_sim.rs

@@ -50,6 +50,7 @@ pub fn run_no_eth1_sim(matches: &ArgMatches) -> Result<(), String> {
             debug_level: log_level,
             logfile_debug_level: "debug",
             log_format,
+            log_color: false,
             max_log_size: 0,
             max_log_number: 0,
             compression: false,

testing/simulator/src/sync_sim.rs

@@ -51,6 +51,7 @@ fn syncing_sim(
             debug_level: log_level,
             logfile_debug_level: "debug",
             log_format,
+            log_color: false,
             max_log_size: 0,
             max_log_number: 0,
             compression: false,