fix: address review comments

Author: Stanislav Angelovic

src/Error.cpp

@@ -29,18 +29,17 @@
 #include "ScopeGuard.h"
 
 #include SDBUS_HEADER
-#include <cassert>
 
 namespace sdbus
 {
-    sdbus::Error createError(int errNo, std::string customMsg)
+    Error createError(int errNo, std::string customMsg)
     {
         sd_bus_error sdbusError = SD_BUS_ERROR_NULL;
         sd_bus_error_set_errno(&sdbusError, errNo);
         SCOPE_EXIT{ sd_bus_error_free(&sdbusError); };
 
         Error::Name name(sd_bus_error_is_set(&sdbusError) ? sdbusError.name : "");
-        std::string message(sd_bus_error_is_set(&sdbusError)? std::move(customMsg) : "");
+        std::string message(std::move(customMsg));
         if (!message.empty() && sdbusError.message != nullptr)
         {
             message.append(" (");

tests/unittests/Types_test.cpp

@@ -479,6 +479,6 @@ TEST(AnErrorFactory, CreatesEmptyInvalidErrorWhenErrnoIsZero)
     auto error = sdbus::createError(0, "custom message");
 
     EXPECT_TRUE(error.getName().empty());
-    EXPECT_TRUE(error.getMessage().empty());
+    EXPECT_THAT(error.getMessage(), Eq<std::string>("custom message"));
     EXPECT_FALSE(error.isValid());
 }