cleaning the code as per copilot review comments

Author: Saumya-R

feature_integration_tests/LIFECYCLE_TESTS_SUMMARY.md

@@ -91,7 +91,7 @@ def create_launch_manager_config(config_path: Path, components: dict[str, Any],
         },
         "components": components,
         "run_targets": run_targets,
-        "initial_run_target": "Startup",
+        "initial_run_target": "startup",
         "fallback_run_target": {
             "description": "Fallback state",
             "depends_on": [],

feature_integration_tests/README_FIT_LIFECYCLE.md

@@ -96,7 +96,7 @@ def test_run_target_switch(self, results: ScenarioResult, logs_info_level: LogCo
         if version == "cpp":
             assert "Switching from startup to running" in results.stdout, "Run target switch failed"
         else:
-            switch_logs = logs_info_level.get_logs(field="message", pattern="Switching from startup to running")
+            switch_logs = logs_info_level.get_logs(field="message", pattern="Switching run targets")
             assert len(switch_logs) > 0, "Run target switch failed"
 
     def test_process_state_communication(

feature_integration_tests/test_cases/lifecycle_scenario.py

@@ -99,15 +99,15 @@ class ProcessLaunchingSupport : public Scenario {
 
         // Attempt to report execution state - this demonstrates the API usage
         // Note: This requires a running Launch Manager daemon to succeed
+        std::cout << "Lifecycle client API called" << std::endl;
         LifecycleClient client{};
         auto result = client.ReportExecutionState(ExecutionState::kRunning);
 
         if (result.has_value()) {
             std::cout << "Successfully reported execution state as running" << std::endl;
         } else {
             // In a test environment without Launch Manager, this is expected
-            std::cout << "Lifecycle client API called (Launch Manager not available in test env)"
-                      << std::endl;
+            std::cout << "Launch Manager not available in test env" << std::endl;
             std::cout << "In production, this would report state to Launch Manager" << std::endl;
         }
 

feature_integration_tests/test_cases/tests/lifecycle/test_run_targets.py

@@ -13,6 +13,8 @@
 
 #include <scenario.hpp>
 
+#include "scenarios/lifecycle/launch_manager_support.h"
+
 #include <vector>
 
 Scenario::Ptr make_multiple_kvs_per_app_scenario();
@@ -24,24 +26,6 @@ Scenario::Ptr make_multi_instance_isolation_scenario();
 ScenarioGroup::Ptr supported_datatypes_group();
 ScenarioGroup::Ptr default_values_group();
 
-Scenario::Ptr make_process_launching_support_scenario();
-Scenario::Ptr make_dependency_ordering_scenario();
-Scenario::Ptr make_parallel_launching_scenario();
-Scenario::Ptr make_control_interface_support_scenario();
-Scenario::Ptr make_process_arguments_scenario();
-Scenario::Ptr make_process_security_scenario();
-Scenario::Ptr make_process_resources_scenario();
-Scenario::Ptr make_conditional_launching_scenario();
-Scenario::Ptr make_process_management_scenario();
-Scenario::Ptr make_run_targets_scenario();
-Scenario::Ptr make_process_termination_scenario();
-Scenario::Ptr make_monitoring_and_recovery_scenario();
-Scenario::Ptr make_control_interface_commands_scenario();
-Scenario::Ptr make_logging_support_scenario();
-Scenario::Ptr make_configuration_management_scenario();
-Scenario::Ptr make_debug_and_terminal_scenario();
-Scenario::Ptr make_io_and_file_descriptors_scenario();
-
 
 ScenarioGroup::Ptr persistency_scenario_group() {
     return std::make_shared<ScenarioGroupImpl>(

feature_integration_tests/test_scenarios/cpp/src/scenarios/lifecycle/launch_manager_support.cpp

@@ -56,13 +56,14 @@ impl Scenario for ProcessLaunchingSupport {
 
         // Attempt to report execution state - this demonstrates the API usage
         // Note: This requires a running Launch Manager daemon to succeed
+        info!("Lifecycle client API called");
         let result = lifecycle_client_rs::report_execution_state_running();
 
         if result {
             info!("Successfully reported execution state as running");
         } else {
             // In a test environment without Launch Manager, this is expected
-            info!("Lifecycle client API called (Launch Manager not available in test env)");
+            info!("Launch Manager not available in test env");
             info!("In production, this would report state to Launch Manager");
         }
 
@@ -401,7 +402,7 @@ impl Scenario for RunTargets {
         }
 
         info!("Starting run target: {}", initial_target);
-        info!("Switching from startup to running");
+        info!("Switching run targets");
         info!("Process state reported");
 
         Ok(())