From 402ab29a73e57baedbbf717115e1b60704172225 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?emrear=C4=B1y=C3=BCrek?= Date: Tue, 2 Sep 2025 13:14:37 +0300 Subject: [PATCH] fix: Change error style in Config (#2314) Fix: #1886 Co-authored-by: Ayaz Salikhov Co-authored-by: Alex Kremer Co-authored-by: Sergey Kuznetsov --- src/util/config/Error.hpp | 2 +- tests/unit/util/config/ClioConfigDefinitionTests.cpp | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/util/config/Error.hpp b/src/util/config/Error.hpp index d106903f..3dd6bf52 100644 --- a/src/util/config/Error.hpp +++ b/src/util/config/Error.hpp @@ -46,7 +46,7 @@ struct Error { */ Error(std::string_view key, std::string_view err) : error{ - fmt::format("{} {}", key, err), + fmt::format("The value of {} {}", key, err), } { } diff --git a/tests/unit/util/config/ClioConfigDefinitionTests.cpp b/tests/unit/util/config/ClioConfigDefinitionTests.cpp index 3993836a..aa560c45 100644 --- a/tests/unit/util/config/ClioConfigDefinitionTests.cpp +++ b/tests/unit/util/config/ClioConfigDefinitionTests.cpp @@ -446,7 +446,7 @@ TEST_F(ClioConfigDefinitionParseArrayTest, missingRequiredFields) auto const result = config.parse(configFile); ASSERT_TRUE(result.has_value()); EXPECT_EQ(result->size(), 1); - EXPECT_THAT(result->at(0).error, testing::StartsWith("array.[].int")); + EXPECT_THAT(result->at(0).error, testing::StartsWith("The value of array.[].int")); } TEST_F(ClioConfigDefinitionParseArrayTest, missingAllRequiredFields) @@ -463,7 +463,7 @@ TEST_F(ClioConfigDefinitionParseArrayTest, missingAllRequiredFields) auto const result = config.parse(configFile); ASSERT_TRUE(result.has_value()); EXPECT_EQ(result->size(), 1); - EXPECT_THAT(result->at(0).error, testing::StartsWith("array.[].int")); + EXPECT_THAT(result->at(0).error, testing::StartsWith("The value of array.[].int")); } TEST(ClioConfigDefinitionParse, unexpectedFields)