X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=test%2Fconfig_test.cc;h=6b35c595d0b60792b849056f7b66597ae1592c51;hb=25eb7aa723af004162fc7f95995df250c4e4571b;hp=48dec27d76543e9b13566ae1f614f3f9c75d62c8;hpb=e1555a8837da05f135a3705112469206fc17ec80;p=dcpomatic.git diff --git a/test/config_test.cc b/test/config_test.cc index 48dec27d7..6b35c595d 100644 --- a/test/config_test.cc +++ b/test/config_test.cc @@ -19,29 +19,41 @@ */ +#include "lib/cinema.h" #include "lib/config.h" #include "test.h" #include #include +using std::list; using std::ofstream; +using std::make_shared; +using std::string; +using boost::optional; -static void -rewrite_bad_config () +static string +rewrite_bad_config (string filename, string extra_line) { + using namespace boost::filesystem; + + auto base = path("build/test/bad_config/2.16"); + auto file = base / filename; + boost::system::error_code ec; - boost::filesystem::remove ("build/test/bad_config/2.16/config.xml", ec); + remove (file, ec); - Config::override_path = "build/test/bad_config"; - boost::filesystem::create_directories ("build/test/bad_config/2.16"); - ofstream f ("build/test/bad_config/2.16/config.xml"); + boost::filesystem::create_directories (base); + std::ofstream f (file.string().c_str()); f << "\n" << "\n" << "\n" + << extra_line << "\n" << "\n"; f.close (); + + return dcp::file_to_string (file); } @@ -50,46 +62,90 @@ BOOST_AUTO_TEST_CASE (config_backup_test) ConfigRestorer cr; Config::override_path = "build/test/bad_config"; - Config::drop(); - boost::filesystem::remove_all ("build/test/bad_config"); - rewrite_bad_config(); + /* Write an invalid config file to config.xml */ + auto const first_write_xml = rewrite_bad_config("config.xml", "first write"); + /* Load the config; this should fail, causing the bad config to be copied to config.xml.1 + * and a new config.xml created in its place. + */ Config::instance(); BOOST_CHECK ( boost::filesystem::exists("build/test/bad_config/2.16/config.xml.1")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.1") == first_write_xml); BOOST_CHECK (!boost::filesystem::exists("build/test/bad_config/2.16/config.xml.2")); BOOST_CHECK (!boost::filesystem::exists("build/test/bad_config/2.16/config.xml.3")); BOOST_CHECK (!boost::filesystem::exists("build/test/bad_config/2.16/config.xml.4")); Config::drop(); - rewrite_bad_config(); + auto const second_write_xml = rewrite_bad_config("config.xml", "second write"); Config::instance(); BOOST_CHECK ( boost::filesystem::exists("build/test/bad_config/2.16/config.xml.1")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.1") == first_write_xml); BOOST_CHECK ( boost::filesystem::exists("build/test/bad_config/2.16/config.xml.2")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.2") == second_write_xml); BOOST_CHECK (!boost::filesystem::exists("build/test/bad_config/2.16/config.xml.3")); BOOST_CHECK (!boost::filesystem::exists("build/test/bad_config/2.16/config.xml.4")); Config::drop(); - rewrite_bad_config(); + auto const third_write_xml = rewrite_bad_config("config.xml", "third write"); Config::instance(); BOOST_CHECK ( boost::filesystem::exists("build/test/bad_config/2.16/config.xml.1")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.1") == first_write_xml); BOOST_CHECK ( boost::filesystem::exists("build/test/bad_config/2.16/config.xml.2")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.2") == second_write_xml); BOOST_CHECK ( boost::filesystem::exists("build/test/bad_config/2.16/config.xml.3")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.3") == third_write_xml); BOOST_CHECK (!boost::filesystem::exists("build/test/bad_config/2.16/config.xml.4")); Config::drop(); - rewrite_bad_config(); + auto const fourth_write_xml = rewrite_bad_config("config.xml", "fourth write"); Config::instance(); BOOST_CHECK (boost::filesystem::exists("build/test/bad_config/2.16/config.xml.1")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.1") == first_write_xml); BOOST_CHECK (boost::filesystem::exists("build/test/bad_config/2.16/config.xml.2")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.2") == second_write_xml); BOOST_CHECK (boost::filesystem::exists("build/test/bad_config/2.16/config.xml.3")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.3") == third_write_xml); BOOST_CHECK (boost::filesystem::exists("build/test/bad_config/2.16/config.xml.4")); + BOOST_CHECK (dcp::file_to_string("build/test/bad_config/2.16/config.xml.4") == fourth_write_xml); +} + + +BOOST_AUTO_TEST_CASE (config_backup_with_link_test) +{ + using namespace boost::filesystem; + + ConfigRestorer cr; + + auto base = path("build/test/bad_config"); + auto version = base / "2.16"; + + Config::override_path = base; + Config::drop(); + + boost::filesystem::remove_all (base); + + boost::filesystem::create_directories (version); + std::ofstream f (path(version / "config.xml").string().c_str()); + f << "\n" + << "\n" + << "" << path(version / "actual.xml").string() << "\n" + << "\n"; + f.close (); + + Config::drop (); + /* Cause actual.xml to be backed up */ + rewrite_bad_config ("actual.xml", "first write"); + Config::instance (); + + /* Make sure actual.xml was backed up to the right place */ + BOOST_CHECK (boost::filesystem::exists(version / "actual.xml.1")); } @@ -137,3 +193,28 @@ BOOST_AUTO_TEST_CASE (config_upgrade_test) BOOST_REQUIRE (!boost::filesystem::exists(dir / "2.16" / "cinemas.xml")); } + +BOOST_AUTO_TEST_CASE (config_keep_cinemas_if_making_new_config) +{ + ConfigRestorer cr; + + boost::filesystem::path dir = "build/test/config_keep_cinemas_if_making_new_config"; + Config::override_path = dir; + Config::drop (); + boost::filesystem::remove_all (dir); + boost::filesystem::create_directories (dir); + + Config::instance()->write(); + + Config::instance()->add_cinema(make_shared("My Great Cinema", list(), "", 0, 0)); + Config::instance()->write(); + + boost::filesystem::copy_file (dir / "cinemas.xml", dir / "backup_for_test.xml"); + + Config::drop (); + boost::filesystem::remove (dir / "2.16" / "config.xml"); + Config::instance(); + + check_text_file (dir / "backup_for_test.xml", dir / "cinemas.xml.1"); +} +