From 3044e70700ae80b68fbb5b6b9d1d4f2251cf67e7 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Mon, 5 Dec 2022 12:15:30 +0100 Subject: [PATCH] Fix tests on Windows now that we replace . with _ in filenames. --- test/file_naming_test.cc | 25 +++++++++++++++++++------ 1 file changed, 19 insertions(+), 6 deletions(-) diff --git a/test/file_naming_test.cc b/test/file_naming_test.cc index be4856cd1..7175e209a 100644 --- a/test/file_naming_test.cc +++ b/test/file_naming_test.cc @@ -62,6 +62,19 @@ private: }; +static +string +mxf_regex(string part) { +#ifdef DCPOMATIC_WINDOWS + /* Windows replaces . in filenames with _ */ + return String::compose(".*flat_%1_png_.*\\.mxf", part); +#else + return String::compose(".*flat_%1\\.png_.*\\.mxf", part); +#endif +}; + + + BOOST_AUTO_TEST_CASE (file_naming_test) { Keep k; @@ -100,11 +113,11 @@ BOOST_AUTO_TEST_CASE (file_naming_test) int got[3] = { 0, 0, 0 }; for (auto i: boost::filesystem::directory_iterator(film->file(film->dcp_name()))) { - if (boost::regex_match(i.path().string(), boost::regex(".*flat_red\\.png_.*\\.mxf"))) { + if (boost::regex_match(i.path().string(), boost::regex(mxf_regex("red")))) { ++got[0]; - } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_green\\.png_.*\\.mxf"))) { + } else if (boost::regex_match(i.path().string(), boost::regex(mxf_regex("green")))) { ++got[1]; - } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_blue\\.png_.*\\.mxf"))) { + } else if (boost::regex_match(i.path().string(), boost::regex(mxf_regex("blue")))) { ++got[2]; } } @@ -158,11 +171,11 @@ BOOST_AUTO_TEST_CASE (file_naming_test2) int got[3] = { 0, 0, 0 }; for (auto i: boost::filesystem::directory_iterator (film->file(film->dcp_name()))) { - if (boost::regex_match(i.path().string(), boost::regex(".*flat_red\\.png_.*\\.mxf"))) { + if (boost::regex_match(i.path().string(), boost::regex(mxf_regex("red")))) { ++got[0]; - } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_green\\.png_.*\\.mxf"))) { + } else if (boost::regex_match(i.path().string(), boost::regex(mxf_regex("green")))) { ++got[1]; - } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_blue\\.png_.*\\.mxf"))) { + } else if (boost::regex_match(i.path().string(), boost::regex(mxf_regex("blue")))) { ++got[2]; } } -- 2.30.2