fix(centralizedConfiguration): remove the single quotation marks from…

… the return message to avoid failures in updating the command in the database.
wazuh · Dec 5, 2024 · 8776051 · 8776051
1 parent c55f273
commit 8776051
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/agent/centralized_configuration/src/centralized_configuration.cpp b/src/agent/centralized_configuration/src/centralized_configuration.cpp
@@ -141,7 +141,7 @@ namespace centralized_configuration
 
             m_reloadModulesFunction();
 
-            const std::string messageOnSuccess = "CentralizedConfiguration '" + command + "' done.";
+            const std::string messageOnSuccess = "CentralizedConfiguration " + command + " done.";
             co_return module_command::CommandExecutionResult {module_command::Status::SUCCESS, messageOnSuccess};
         }
         catch (const nlohmann::json::exception&)

diff --git a/src/agent/centralized_configuration/tests/centralized_configuration_tests.cpp b/src/agent/centralized_configuration/tests/centralized_configuration_tests.cpp
@@ -159,13 +159,13 @@ TEST(CentralizedConfiguration, ExecuteCommandHandlesRecognizedCommands)
                                         "set-group",
                                         groupsList,
                                         module_command::Status::SUCCESS,
-                                        "CentralizedConfiguration 'set-group' done.");
+                                        "CentralizedConfiguration set-group done.");
 
             co_await TestExecuteCommand(centralizedConfiguration,
                                         "update-group",
                                         {},
                                         module_command::Status::SUCCESS,
-                                        "CentralizedConfiguration 'update-group' done.");
+                                        "CentralizedConfiguration update-group done.");
 
             co_await TestExecuteCommand(centralizedConfiguration,
                                         "unknown-command",
@@ -228,7 +228,7 @@ TEST(CentralizedConfiguration, SetFunctionsAreCalledAndReturnsCorrectResultsForS
                                         "set-group",
                                         groupsList,
                                         module_command::Status::SUCCESS,
-                                        "CentralizedConfiguration 'set-group' done.");
+                                        "CentralizedConfiguration set-group done.");
 
             EXPECT_TRUE(wasSetGroupIdFunctionCalled);
             EXPECT_TRUE(wasDownloadGroupFilesFunctionCalled);
@@ -286,7 +286,7 @@ TEST(CentralizedConfiguration, SetFunctionsAreCalledAndReturnsCorrectResultsForU
                                         "update-group",
                                         {},
                                         module_command::Status::SUCCESS,
-                                        "CentralizedConfiguration 'update-group' done.");
+                                        "CentralizedConfiguration update-group done.");
 
             EXPECT_TRUE(wasGetGroupIdFunctionCalled);
             EXPECT_TRUE(wasDownloadGroupFilesFunctionCalled);