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

Improve rename tests

Browse Source
This commit is contained in:
Sebastian Messmer 2022-12-18 17:35:08 +01:00
parent 45895d8d4f
commit 8b02ff7cc5
1 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
test/cryfs/impl/filesystem/CryNodeTest_RenameNested.cpp
View File

@ -83,7 +83,12 @@ public:
void expect_rename_succeeds(const bf::path& source_path, const bf::path& dest_path) { void expect_rename_succeeds(const bf::path& source_path, const bf::path& dest_path) {
auto source = device().Load(source_path).value(); auto source = device().Load(source_path).value();
source->rename(dest_path); source->rename(dest_path);
// TODO Test that rename was successful
// Test that rename was successful
if (source_path != dest_path && source_path != "/") {
ASSERT_FALSE(Exists(source_path));
}
ASSERT_TRUE(Exists(dest_path));
} }
void expect_rename_fails(const bf::path& source_path, const bf::path& dest_path, int expected_errno) { void expect_rename_fails(const bf::path& source_path, const bf::path& dest_path, int expected_errno) {
@ -94,7 +99,14 @@ public:
} catch (const FuseErrnoException& e) { } catch (const FuseErrnoException& e) {
ASSERT_EQ(expected_errno, e.getErrno()); ASSERT_EQ(expected_errno, e.getErrno());
} }
// TODO Test rename wasn't successful // Test rename wasn't successful
if (dest_path != "/") {
ASSERT_TRUE(Exists(source_path));
}
auto source_dirs = SourceDirs();
if (source_path != dest_path && source_path != "/" && std::find(source_dirs.begin(), source_dirs.end(), dest_path) == source_dirs.end()) {
ASSERT_FALSE(Exists(dest_path));
}
} }
}; };