Mirrorlandia_minetest/minetest
1
0
Fork 1
mirror of https://github.com/minetest/minetest.git synced 2024-11-30 19:43:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

Add unit tests for fs::CopyFileContents

Browse Source
This commit is contained in:
sfan5 2024-01-20 17:16:08 +01:00 committed by grorp
parent 4d024d737c
commit 008310aad9
1 changed files with 49 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

50
src/unittest/test_filepath.cpp
View File

@ -38,6 +38,8 @@ public:
void testRemoveLastPathComponent(); void testRemoveLastPathComponent();
void testRemoveLastPathComponentWithTrailingDelimiter(); void testRemoveLastPathComponentWithTrailingDelimiter();
void testRemoveRelativePathComponent(); void testRemoveRelativePathComponent();
void testSafeWriteToFile();
void testCopyFileContents();
}; };
static TestFilePath g_test_instance; static TestFilePath g_test_instance;
@ -49,6 +51,8 @@ void TestFilePath::runTests(IGameDef *gamedef)
TEST(testRemoveLastPathComponent); TEST(testRemoveLastPathComponent);
TEST(testRemoveLastPathComponentWithTrailingDelimiter); TEST(testRemoveLastPathComponentWithTrailingDelimiter);
TEST(testRemoveRelativePathComponent); TEST(testRemoveRelativePathComponent);
TEST(testSafeWriteToFile);
TEST(testCopyFileContents);
} }
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
@ -56,7 +60,7 @@ void TestFilePath::runTests(IGameDef *gamedef)
// adjusts a POSIX path to system-specific conventions // adjusts a POSIX path to system-specific conventions
// -> changes '/' to DIR_DELIM // -> changes '/' to DIR_DELIM
// -> absolute paths start with "C:\\" on windows // -> absolute paths start with "C:\\" on windows
std::string p(std::string path) static std::string p(std::string path)
{ {
for (size_t i = 0; i < path.size(); ++i) { for (size_t i = 0; i < path.size(); ++i) {
if (path[i] == '/') { if (path[i] == '/') {
@ -262,3 +266,47 @@ void TestFilePath::testRemoveRelativePathComponent()
result = fs::RemoveRelativePathComponents(path); result = fs::RemoveRelativePathComponents(path);
UASSERT(result == p("/a/e")); UASSERT(result == p("/a/e"));
} }
void TestFilePath::testSafeWriteToFile()
{
const std::string dest_path = getTestTempFile();
const std::string test_data("hello\0world", 11);
fs::safeWriteToFile(dest_path, test_data);
UASSERT(fs::PathExists(dest_path));
std::string contents_actual;
UASSERT(fs::ReadFile(dest_path, contents_actual));
UASSERTEQ(auto, contents_actual, test_data);
}
void TestFilePath::testCopyFileContents()
{
const auto dir_path = getTestTempDirectory();
const auto file1 = dir_path + DIR_DELIM "src", file2 = dir_path + DIR_DELIM "dst";
const std::string test_data("hello\0world", 11);
// error case
UASSERT(!fs::CopyFileContents(file1, "somewhere"));
{
std::ofstream ofs(file1);
ofs << test_data;
}
// normal case
UASSERT(fs::CopyFileContents(file1, file2));
std::string contents_actual;
UASSERT(fs::ReadFile(file2, contents_actual));
UASSERTEQ(auto, contents_actual, test_data);
// should overwrite and truncate
{
std::ofstream ofs(file2);
for (int i = 0; i < 10; i++)
ofs << "OH MY GAH";
}
UASSERT(fs::CopyFileContents(file1, file2));
contents_actual.clear();
UASSERT(fs::ReadFile(file2, contents_actual));
UASSERTEQ(auto, contents_actual, test_data);
}