hardcoresushi/libcryfs
1
0
Fork 0
Code Issues Pull Requests Releases Activity

Extend test cases for trying to rename a file to an existing directory or the other way round

Browse Source
This commit is contained in:
Sebastian Messmer 2016-04-28 23:14:33 -07:00
parent bfba538818
commit 33644af64d
1 changed files with 36 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
src/fspp/fstest/FsppNodeTest_Rename.h
View File

@ -131,7 +131,7 @@ public:
EXPECT_EQ(3u, this->LoadDir("/")->children()->size()); // 3, because of '.' and '..' EXPECT_EQ(3u, this->LoadDir("/")->children()->size()); // 3, because of '.' and '..'
} }
void Test_Overwrite_DirWithFile() { void Test_Overwrite_DirWithFile_InSameDir() {
auto file = this->CreateFile("/oldname"); auto file = this->CreateFile("/oldname");
this->CreateDir("/newname"); this->CreateDir("/newname");
try { try {
@ -144,7 +144,22 @@ public:
EXPECT_NE(boost::none, this->device->Load("/newname")); EXPECT_NE(boost::none, this->device->Load("/newname"));
} }
void Test_Overwrite_FileWithDir() { void Test_Overwrite_DirWithFile_InDifferentDir() {
this->CreateDir("/parent1");
this->CreateDir("/parent2");
auto file = this->CreateFile("/parent1/oldname");
this->CreateDir("/parent2/newname");
try {
file->rename("/parent2/newname");
EXPECT_TRUE(false); // expect throw
} catch (const fspp::fuse::FuseErrnoException &e) {
EXPECT_EQ(EISDIR, e.getErrno());
}
EXPECT_NE(boost::none, this->device->Load("/parent1/oldname"));
EXPECT_NE(boost::none, this->device->Load("/parent2/newname"));
}
void Test_Overwrite_FileWithDir_InSameDir() {
auto dir = this->CreateDir("/oldname"); auto dir = this->CreateDir("/oldname");
this->CreateFile("/newname"); this->CreateFile("/newname");
try { try {
@ -156,6 +171,21 @@ public:
EXPECT_NE(boost::none, this->device->Load("/oldname")); EXPECT_NE(boost::none, this->device->Load("/oldname"));
EXPECT_NE(boost::none, this->device->Load("/newname")); EXPECT_NE(boost::none, this->device->Load("/newname"));
} }
void Test_Overwrite_FileWithDir_InDifferentDir() {
this->CreateDir("/parent1");
this->CreateDir("/parent2");
auto dir = this->CreateDir("/parent1/oldname");
this->CreateFile("/parent2/newname");
try {
dir->rename("/parent2/newname");
EXPECT_TRUE(false); // expect throw
} catch (const fspp::fuse::FuseErrnoException &e) {
EXPECT_EQ(ENOTDIR, e.getErrno());
}
EXPECT_NE(boost::none, this->device->Load("/parent1/oldname"));
EXPECT_NE(boost::none, this->device->Load("/parent2/newname"));
}
}; };
REGISTER_NODE_TEST_CASE(FsppNodeTest_Rename, REGISTER_NODE_TEST_CASE(FsppNodeTest_Rename,
@ -173,8 +203,10 @@ REGISTER_NODE_TEST_CASE(FsppNodeTest_Rename,
RootDir, RootDir,
Overwrite, Overwrite,
Overwrite_DoesntHaveSameEntryTwice, Overwrite_DoesntHaveSameEntryTwice,
Overwrite_DirWithFile, Overwrite_DirWithFile_InSameDir,
Overwrite_FileWithDir Overwrite_DirWithFile_InDifferentDir,
Overwrite_FileWithDir_InSameDir,
Overwrite_FileWithDir_InDifferentDir
); );
#endif #endif