[PATCH] D137823: [clang-format][NFC] Moved configuration parsing tests in own file

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 12 23:12:15 PST 2022


owenpan added inline comments.


================
Comment at: clang/unittests/Format/ConfigParseTest.cpp:14
+
+#define DEBUG_TYPE "parse-test"
+
----------------
You can delete it.


================
Comment at: clang/unittests/Format/ConfigParseTest.cpp:1006
+
+TEST(FormatStyle, GetStyleWithEmptyFileName) {
+  llvm::vfs::InMemoryFileSystem FS;
----------------
Otherwise, the test will be skipped.


================
Comment at: clang/unittests/Format/ConfigParseTest.cpp:1013
+
+TEST(FormatStyle, GetStyleOfFile) {
+  llvm::vfs::InMemoryFileSystem FS;
----------------
Change `FormatStyle` to `ParseTest` here and on line 1220 below.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137823/new/

https://reviews.llvm.org/D137823



More information about the cfe-commits mailing list