[Lldb-commits] [PATCH] D54617: [Reproducers] Add file provider
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Jan 29 02:54:56 PST 2019
labath accepted this revision.
labath added a comment.
Looks good, thanks for bearing with me.
I don't want to hold this up further, but ideally, I'd change that last test to use positive assertions instead of negative ones. E.g., right now this doesn't really fulfill my goal of documenting how symlinks are handled. Something like `EXPECT_THAT(mapping, testing::Contains(YAMLVFSEntry(file_root + "/bar", reproducer_root + "/foo")));` would (as well as giving better error messages).
PS: I believe the symlink test will have to be `#ifndef WINDOWS`.
CHANGES SINCE LAST ACTION
More information about the lldb-commits