From 4829f4bcec37e336d010dd67b9810985a95526d5 Mon Sep 17 00:00:00 2001
From: Sebastian Messmer <messmer@cryfs.org>
Date: Fri, 29 Apr 2016 11:07:47 -0700
Subject: [PATCH] Extend test cases for rename with overwrite

---
 src/fspp/fstest/FsppNodeTest_Rename.h | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/fspp/fstest/FsppNodeTest_Rename.h b/src/fspp/fstest/FsppNodeTest_Rename.h
index 9fb9a98f..39f3628e 100644
--- a/src/fspp/fstest/FsppNodeTest_Rename.h
+++ b/src/fspp/fstest/FsppNodeTest_Rename.h
@@ -115,7 +115,7 @@ public:
         }
     }
 
-    void Test_Overwrite() {
+    void Test_Overwrite_InSameDir() {
         auto node = this->CreateNode("/oldname");
         this->CreateNode("/newname");
         node->rename("/newname");
@@ -123,6 +123,16 @@ public:
         EXPECT_NE(boost::none, this->device->Load("/newname"));
     }
 
+    void Test_Overwrite_InDifferentDir() {
+        this->CreateDir("/parent1");
+        this->CreateDir("/parent2");
+        auto node = this->CreateNode("/parent1/oldname");
+        this->CreateNode("/parent2/newname");
+        node->rename("/parent2/newname");
+        EXPECT_EQ(boost::none, this->device->Load("/parent1/oldname"));
+        EXPECT_NE(boost::none, this->device->Load("/parent2/newname"));
+    }
+
     void Test_Overwrite_DoesntHaveSameEntryTwice() {
         auto node = this->CreateNode("/oldname");
         this->CreateNode("/newname");
@@ -201,7 +211,8 @@ REGISTER_NODE_TEST_CASE(FsppNodeTest_Rename,
     NestedToNested_NewName,
     ToItself,
     RootDir,
-    Overwrite,
+    Overwrite_InSameDir,
+    Overwrite_InDifferentDir,
     Overwrite_DoesntHaveSameEntryTwice,
     Overwrite_DirWithFile_InSameDir,
     Overwrite_DirWithFile_InDifferentDir,