[PATCH] D79552: [YAMLVFSWriter][Tests][NFC] Add couple tests
Jan Korous via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 7 11:20:34 PDT 2020
jkorous marked an inline comment as done.
jkorous added inline comments.
================
Comment at: llvm/unittests/Support/VirtualFileSystemTest.cpp:2278
+
+TEST_F(VFSFromYAMLTest, YAMLVFSWriterTestEmbeddedDirs) {
+ ScopedDir TestDirectory("virtual-file-system-test", /*Unique*/ true);
----------------
JDevlieghere wrote:
> What's an "embedded dir"? I'm not sure what makes this test different from the existing writer test. Could you add a small comment about what the test is testing?
You're right, "nested" is probably a better word here.
This originally helped me to understand how the implementation handles empty directories and since position in the mapping mattered I wanted to have a clean, isolated "unit"-test.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79552/new/
https://reviews.llvm.org/D79552
More information about the llvm-commits
mailing list