Skip to content

Commit 305aacf

Browse files
committed
[clang-format][NFC] Add missing config tests for List of Strings
1 parent 1a0d0ae commit 305aacf

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

clang/unittests/Format/ConfigParseTest.cpp

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,9 @@ TEST(ConfigParseTest, GetsCorrectBasedOnStyle) {
144144
EXPECT_EQ(0, parseConfiguration(TEXT, &Style).value()); \
145145
EXPECT_EQ(VALUE, Style.FIELD) << "Unexpected value after parsing!"
146146

147+
#define CHECK_PARSE_LIST(FIELD) \
148+
CHECK_PARSE(#FIELD ": [foo]", FIELD, std::vector<std::string>{"foo"})
149+
147150
#define CHECK_PARSE_NESTED_VALUE(TEXT, STRUCT, FIELD, VALUE) \
148151
EXPECT_NE(VALUE, Style.STRUCT.FIELD) << "Initial value already the same!"; \
149152
EXPECT_EQ(0, parseConfiguration(#STRUCT ":\n " TEXT, &Style).value()); \
@@ -899,11 +902,15 @@ TEST(ConfigParseTest, ParsesConfiguration) {
899902
CHECK_PARSE("StatementMacros: [QUNUSED, QT_REQUIRE_VERSION]", StatementMacros,
900903
std::vector<std::string>({"QUNUSED", "QT_REQUIRE_VERSION"}));
901904

902-
Style.NamespaceMacros.clear();
903-
CHECK_PARSE("NamespaceMacros: [TESTSUITE]", NamespaceMacros,
904-
std::vector<std::string>{"TESTSUITE"});
905-
CHECK_PARSE("NamespaceMacros: [TESTSUITE, SUITE]", NamespaceMacros,
906-
std::vector<std::string>({"TESTSUITE", "SUITE"}));
905+
CHECK_PARSE_LIST(JavaImportGroups);
906+
CHECK_PARSE_LIST(Macros);
907+
CHECK_PARSE_LIST(NamespaceMacros);
908+
CHECK_PARSE_LIST(ObjCPropertyAttributeOrder);
909+
CHECK_PARSE_LIST(TableGenBreakingDAGArgOperators);
910+
CHECK_PARSE_LIST(TemplateNames);
911+
CHECK_PARSE_LIST(TypeNames);
912+
CHECK_PARSE_LIST(TypenameMacros);
913+
CHECK_PARSE_LIST(VariableTemplates);
907914

908915
Style.WhitespaceSensitiveMacros.clear();
909916
CHECK_PARSE("WhitespaceSensitiveMacros: [STRINGIZE]",

0 commit comments

Comments
 (0)