================
@@ -967,6 +967,7 @@ TEST(ConfigParseTest, ParsesConfiguration) {
               std::vector<std::string>({"QUNUSED", "QT_REQUIRE_VERSION"}));
 
   CHECK_PARSE_LIST(JavaImportGroups);
+  Style.Macros.clear();
----------------
ilya-biryukov wrote:

Done. I've tried to keep it similar to other tests that have the defaults and 
to follow your suggestions.
I also introduced a simple helper to allow passing a matcher rather than a 
specific value (I did not want to include the whole contents of the macro 
definition, just the prefix of it; it looks clearer).


Let me know if this matches what you wanted, my intuition wrt to how such tests 
should look like is still developing.

https://github.com/llvm/llvm-project/pull/169037
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to