================
@@ -1705,15 +1705,19 @@ TEST(ConfigParseTest, RedirectInheritance) {
ASSERT_TRUE(
FS.addFile("/a/.clang-format", 0,
llvm::MemoryBuffer::getMemBuffer("BasedOnStyle: GNU")));
- ASSERT_TRUE(FS.addFile("/a/config/.clang-format", 0,
- llvm::MemoryBuffer::getMemBuffer(Config)));
-
ASSERT_TRUE(FS.addFile("/a/proj/.clang-format", 0,
llvm::MemoryBuffer::getMemBuffer(
- "BasedOnStyle: InheritParentConfig=../config")));
----------------
owenca wrote:
One of the bugs the updated patch fixes was the handling of absolute pathnames,
so chnaging `../config` to `/a/config` gives us test coverage for that. (We
already got some coverage for relative paths with
`InheritParentConfig=config/` below.)
https://github.com/llvm/llvm-project/pull/182791
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
