Fix some test issues in VirtualFileSystemTest

Use camel-case names, remove some dead code, and fix a copy-and-pasted test.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@201691 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/unittests/Basic/VirtualFileSystemTest.cpp b/unittests/Basic/VirtualFileSystemTest.cpp
index be3060ce7f3f0171f2b7cdfc541fbbd103b1e598..e545e9c0597267e8ec9de7dea138f493799d4b67 100644 (file)
--- a/unittests/Basic/VirtualFileSystemTest.cpp
+++ b/unittests/Basic/VirtualFileSystemTest.cpp
@@ -70,7 +70,7 @@ public:
 };
 } // end anonymous namespace
 
-TEST(VirtualFileSystemTest, status_queries) {
+TEST(VirtualFileSystemTest, statusQueries) {
   IntrusiveRefCntPtr<DummyFileSystem> D(new DummyFileSystem());
   ErrorOr<vfs::Status> Status((error_code()));
 
@@ -110,7 +110,7 @@ TEST(VirtualFileSystemTest, status_queries) {
   EXPECT_FALSE(Status->equivalent(*Status2));
 }
 
-TEST(VirtualFileSystemTest, base_only_overlay) {
+TEST(VirtualFileSystemTest, baseOnlyOverlay) {
   IntrusiveRefCntPtr<DummyFileSystem> D(new DummyFileSystem());
   ErrorOr<vfs::Status> Status((error_code()));
   EXPECT_FALSE(Status = D->status("/foo"));
@@ -128,7 +128,7 @@ TEST(VirtualFileSystemTest, base_only_overlay) {
   EXPECT_TRUE(Status->equivalent(*Status2));
 }
 
-TEST(VirtualFileSystemTest, overlay_files) {
+TEST(VirtualFileSystemTest, overlayFiles) {
   IntrusiveRefCntPtr<DummyFileSystem> Base(new DummyFileSystem());
   IntrusiveRefCntPtr<DummyFileSystem> Middle(new DummyFileSystem());
   IntrusiveRefCntPtr<DummyFileSystem> Top(new DummyFileSystem());
@@ -165,36 +165,31 @@ TEST(VirtualFileSystemTest, overlay_files) {
   EXPECT_FALSE(Status1->equivalent(*Status3));
 }
 
-TEST(VirtualFileSystemTest, overlay_dirs) {
+TEST(VirtualFileSystemTest, overlayDirsNonMerged) {
   IntrusiveRefCntPtr<DummyFileSystem> Lower(new DummyFileSystem());
   IntrusiveRefCntPtr<DummyFileSystem> Upper(new DummyFileSystem());
   IntrusiveRefCntPtr<vfs::OverlayFileSystem>
     O(new vfs::OverlayFileSystem(Lower));
   O->pushOverlay(Upper);
 
-  ErrorOr<vfs::Status> Status1((error_code())), Status2((error_code())),
-                       Status3((error_code()));
-
   Lower->addDirectory("/lower-only");
-  Lower->addDirectory("/both");
-  Upper->addDirectory("/both");
   Upper->addDirectory("/upper-only");
 
   // non-merged paths should be the same
-  Status1 = Lower->status("/lower-only");
+  ErrorOr<vfs::Status> Status1 = Lower->status("/lower-only");
   ASSERT_EQ(errc::success, Status1.getError());
-  Status2 = O->status("/lower-only");
+  ErrorOr<vfs::Status> Status2 = O->status("/lower-only");
   ASSERT_EQ(errc::success, Status2.getError());
   EXPECT_TRUE(Status1->equivalent(*Status2));
 
-  Status1 = Lower->status("/lower-only");
+  Status1 = Upper->status("/upper-only");
   ASSERT_EQ(errc::success, Status1.getError());
-  Status2 = O->status("/lower-only");
+  Status2 = O->status("/upper-only");
   ASSERT_EQ(errc::success, Status2.getError());
   EXPECT_TRUE(Status1->equivalent(*Status2));
 }
 
-TEST(VirtualFileSystemTest, permissions) {
+TEST(VirtualFileSystemTest, mergedDirPermissions) {
   // merged directories get the permissions of the upper dir
   IntrusiveRefCntPtr<DummyFileSystem> Lower(new DummyFileSystem());
   IntrusiveRefCntPtr<DummyFileSystem> Upper(new DummyFileSystem());