remove some warnings

Author: namchuai

engine/cli/commands/cortex_upd_cmd.cc

@@ -175,8 +175,12 @@ std::optional<std::string> CheckNewUpdate(
         CTL_INF("Got the latest release, update to the config file: "
                 << latest_version)
         config.latestRelease = latest_version;
-        config_yaml_utils::DumpYamlConfig(
+        auto result = config_yaml_utils::DumpYamlConfig(
             config, file_manager_utils::GetConfigurationPath().string());
+        if (result.has_error()) {
+          CTL_ERR("Error update " << file_manager_utils::GetConfigurationPath()
+                                  << result.error());
+        }
         if (current_version != latest_version) {
           return latest_version;
         }

engine/test/components/test_cortex_config.cc

@@ -41,7 +41,8 @@ TEST_F(CortexConfigTest, DumpYamlConfig_WritesCorrectly) {
                          123456789,
                          "v1.0.0"};
 
-  DumpYamlConfig(config, test_file_path);
+  auto result = DumpYamlConfig(config, test_file_path);
+  EXPECT_FALSE(result.has_error());
 
   // Verify that the file was created and contains the expected data
   YAML::Node node = YAML::LoadFile(test_file_path);
@@ -68,7 +69,8 @@ TEST_F(CortexConfigTest, FromYaml_ReadsCorrectly) {
                          123456789,
                          "v1.0.0"};
 
-  DumpYamlConfig(config, test_file_path);
+  auto result = DumpYamlConfig(config, test_file_path);
+  EXPECT_FALSE(result.has_error());
 
   // Now read from the YAML file
   CortexConfig loaded_config = FromYaml(test_file_path, default_config);
@@ -115,4 +117,4 @@ TEST_F(CortexConfigTest, FromYaml_IncompleteConfigUsesDefaults) {
             default_config.latestRelease);  // Default value
 }
 
-}  // namespace config_yaml_utils
+}  // namespace config_yaml_utils

engine/test/components/test_file_manager_config_yaml_utils.cc

@@ -37,14 +37,16 @@ TEST_F(FileManagerConfigTest, GetDefaultDataFolderName) {
 
 TEST_F(FileManagerConfigTest, CreateConfigFileIfNotExist) {
 
-  file_manager_utils::CreateConfigFileIfNotExist();
+  auto result = file_manager_utils::CreateConfigFileIfNotExist();
+  EXPECT_FALSE(result.has_error());
   EXPECT_TRUE(
       std::filesystem::exists(file_manager_utils::GetConfigurationPath()));
   std::filesystem::remove(file_manager_utils::GetConfigurationPath());
 }
 
 TEST_F(FileManagerConfigTest, GetCortexConfig) {
-  file_manager_utils::CreateConfigFileIfNotExist();
+  auto result = file_manager_utils::CreateConfigFileIfNotExist();
+  EXPECT_FALSE(result.has_error());
   auto config = file_manager_utils::GetCortexConfig();
   EXPECT_FALSE(config.dataFolderPath.empty());
   EXPECT_FALSE(config.logFolderPath.empty());
@@ -61,8 +63,8 @@ TEST_F(FileManagerConfigTest, DumpYamlConfig) {
                                          .apiServerPort = "8080"};
 
   std::string test_file = "test_config.yaml";
-  config_yaml_utils::DumpYamlConfig(config, test_file);
-
+  auto result = config_yaml_utils::DumpYamlConfig(config, test_file);
+  EXPECT_FALSE(result.has_error());
   EXPECT_TRUE(std::filesystem::exists(test_file));
 
   // Clean up
@@ -91,4 +93,4 @@ TEST_F(FileManagerConfigTest, FromYaml) {
 
   // Clean up
   std::filesystem::remove(test_file);
-}
+}