BRNSystems/minetest
1
0
Fork 0
forked from Mirrorlandia_minetest/minetest
Code Pull Requests 1 Activity

Add unit tests for fs::CopyFileContents

Browse Source
This commit is contained in:
sfan5 2024-01-20 17:16:08 +01:00
parent 93381014a0
commit 714c9361ea
1 changed files with 36 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
src/unittest/test_filesys.cpp
View File

@ -40,6 +40,7 @@ public:
void testRemoveLastPathComponentWithTrailingDelimiter(); void testRemoveLastPathComponentWithTrailingDelimiter();
void testRemoveRelativePathComponent(); void testRemoveRelativePathComponent();
void testSafeWriteToFile(); void testSafeWriteToFile();
void testCopyFileContents();
}; };
static TestFileSys g_test_instance; static TestFileSys g_test_instance;
@ -52,6 +53,7 @@ void TestFileSys::runTests(IGameDef *gamedef)
TEST(testRemoveLastPathComponentWithTrailingDelimiter); TEST(testRemoveLastPathComponentWithTrailingDelimiter);
TEST(testRemoveRelativePathComponent); TEST(testRemoveRelativePathComponent);
TEST(testSafeWriteToFile); TEST(testSafeWriteToFile);
TEST(testCopyFileContents);
} }
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
@ -59,7 +61,7 @@ void TestFileSys::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] == '/') {
@ -275,5 +277,37 @@ void TestFileSys::testSafeWriteToFile()
UASSERT(fs::PathExists(dest_path)); UASSERT(fs::PathExists(dest_path));
std::string contents_actual; std::string contents_actual;
UASSERT(fs::ReadFile(dest_path, contents_actual)); UASSERT(fs::ReadFile(dest_path, contents_actual));
UASSERT(contents_actual == test_data); UASSERTEQ(auto, contents_actual, test_data);
}
void TestFileSys::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);
} }