diff --git a/test/config/test_llarp_config_definition.cpp b/test/config/test_llarp_config_definition.cpp index 827d05344..951b88e1a 100644 --- a/test/config/test_llarp_config_definition.cpp +++ b/test/config/test_llarp_config_definition.cpp @@ -275,9 +275,7 @@ TEST_CASE("ConfigDefinition undeclared add/remove test", "[config]") // ...then remove... REQUIRE_NOTHROW(config.removeUndeclaredHandler("foo")); - CHECK_THROWS_WITH( - config.addConfigValue("foo", "bar", "val"), - "no declared section [foo]"); + CHECK_THROWS_WITH(config.addConfigValue("foo", "bar", "val"), "unrecognized section [foo]"); // ...then add again REQUIRE_NOTHROW(config.addUndeclaredHandler("foo", [&](std::string_view, std::string_view, std::string_view) { @@ -308,7 +306,7 @@ TEST_CASE("ConfigDefinition undeclared handler wrong section", "[config]") throw std::runtime_error("FAIL"); }); - REQUIRE_THROWS_WITH(config.addConfigValue("argle", "bar", "val"), "no declared section [argle]"); + REQUIRE_THROWS_WITH(config.addConfigValue("argle", "bar", "val"), "unrecognized section [argle]"); } TEST_CASE("ConfigDefinition undeclared handler duplicate names", "[config]") diff --git a/test/config/test_llarp_config_output.cpp b/test/config/test_llarp_config_output.cpp index e33e3f979..ba8cc438a 100644 --- a/test/config/test_llarp_config_output.cpp +++ b/test/config/test_llarp_config_output.cpp @@ -150,20 +150,3 @@ baz=1 )raw"); } -TEST_CASE("ConfigDefinition should print comments for missing keys") -{ - // TODO: this currently fails: how to implement? - llarp::ConfigDefinition config; - - config.addSectionComments("foo", {"foo section comment"}); - config.addOptionComments("foo", "bar", {"foo bar option comment"}); - - std::string output = config.generateINIConfig(); - - CHECK(output == R"raw(# foo section comment -[foo] - -# foo bar option comment -bar= -)raw"); -}