Cleanup: use some more make_shared.
[dcpomatic.git] / test / file_naming_test.cc
index 4c2b0af471ed97c0d135d284d91f70beb2d1b3b4..0b76952a8a72e61e83a34e8febd32823495cd590 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2016-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
 
 */
 
+
+/** @file  test/file_naming_test.cc
+ *  @brief Test how files in DCPs are named.
+ *  @ingroup feature
+ */
+
+
 #include "test.h"
 #include "lib/config.h"
 #include "lib/film.h"
 #include "lib/ffmpeg_content.h"
 #include "lib/dcp_content_type.h"
+#include "lib/video_content.h"
+#ifdef DCPOMATIC_WINDOWS
+#include <boost/locale.hpp>
+#endif
 #include <boost/test/unit_test.hpp>
+#include <boost/regex.hpp>
+
+
+using std::string;
+using std::shared_ptr;
+using std::make_shared;
+
+
+class Keep
+{
+public:
+       Keep ()
+       {
+               _format = Config::instance()->dcp_asset_filename_format ();
+       }
+
+       ~Keep ()
+       {
+               Config::instance()->set_dcp_asset_filename_format (_format);
+       }
+
+private:
+       dcp::NameFormat _format;
+};
 
-using boost::shared_ptr;
 
 BOOST_AUTO_TEST_CASE (file_naming_test)
 {
-       dcp::FilenameFormat nf ("%c");
-       Config::instance()->set_dcp_filename_format (dcp::FilenameFormat ("%c"));
-       shared_ptr<Film> film = new_test_film ("file_naming_test");
+       Keep k;
+       Config::instance()->set_dcp_asset_filename_format (dcp::NameFormat("%c"));
+
+       auto film = new_test_film ("file_naming_test");
        film->set_name ("file_naming_test");
+       film->set_video_frame_rate (24);
+       film->set_dcp_content_type (DCPContentType::from_isdcf_name ("FTR"));
+       auto r = make_shared<FFmpegContent>("test/data/flat_red.png");
+       film->examine_and_add_content (r);
+       auto g = make_shared<FFmpegContent>("test/data/flat_green.png");
+       film->examine_and_add_content (g);
+       auto b = make_shared<FFmpegContent>("test/data/flat_blue.png");
+       film->examine_and_add_content (b);
+       BOOST_REQUIRE (!wait_for_jobs());
+
+       r->set_position (film, dcpomatic::DCPTime::from_seconds(0));
+       r->set_video_frame_rate (24);
+       r->video->set_length (24);
+       g->set_position (film, dcpomatic::DCPTime::from_seconds(1));
+       g->set_video_frame_rate (24);
+       g->video->set_length (24);
+       b->set_position (film, dcpomatic::DCPTime::from_seconds(2));
+       b->set_video_frame_rate (24);
+       b->video->set_length (24);
+
+       film->set_reel_type (ReelType::BY_VIDEO_CONTENT);
+       film->write_metadata ();
+       make_and_verify_dcp (
+               film,
+               {
+                       dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE,
+                       dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE
+               });
+
+       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"))) {
+                       ++got[0];
+               } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_green\\.png_.*\\.mxf"))) {
+                       ++got[1];
+               } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_blue\\.png_.*\\.mxf"))) {
+                       ++got[2];
+               }
+       }
+
+       for (int i = 0; i < 3; ++i) {
+               BOOST_CHECK (got[i] == 2);
+       }
+}
+
+
+BOOST_AUTO_TEST_CASE (file_naming_test2)
+{
+       Keep k;
+       Config::instance()->set_dcp_asset_filename_format (dcp::NameFormat ("%c"));
+
+       auto film = new_test_film ("file_naming_test2");
+       film->set_name ("file_naming_test2");
        film->set_dcp_content_type (DCPContentType::from_isdcf_name ("FTR"));
-       shared_ptr<FFmpegContent> r (new FFmpegContent (film, "test/data/flat_red.png"));
+
+#ifdef DCPOMATIC_WINDOWS
+       /* This is necessary so that the UTF8 string constant below gets converted properly */
+       std::locale::global(boost::locale::generator().generate(""));
+       boost::filesystem::path::imbue(std::locale());
+#endif
+
+       auto r = make_shared<FFmpegContent>("test/data/flät_red.png");
        film->examine_and_add_content (r);
-       shared_ptr<FFmpegContent> g (new FFmpegContent (film, "test/data/flat_green.png"));
+       auto g = make_shared<FFmpegContent>("test/data/flat_green.png");
        film->examine_and_add_content (g);
-       shared_ptr<FFmpegContent> b (new FFmpegContent (film, "test/data/flat_blue.png"));
+       auto b = make_shared<FFmpegContent>("test/data/flat_blue.png");
        film->examine_and_add_content (b);
-       wait_for_jobs ();
+       BOOST_REQUIRE (!wait_for_jobs());
+
+       r->set_position (film, dcpomatic::DCPTime::from_seconds(0));
+       r->set_video_frame_rate (24);
+       r->video->set_length (24);
+       g->set_position (film, dcpomatic::DCPTime::from_seconds(1));
+       g->set_video_frame_rate (24);
+       g->video->set_length (24);
+       b->set_position (film, dcpomatic::DCPTime::from_seconds(2));
+       b->set_video_frame_rate (24);
+       b->video->set_length (24);
+
+       film->set_reel_type (ReelType::BY_VIDEO_CONTENT);
+       make_and_verify_dcp (
+               film,
+               {
+                       dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE,
+                       dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE
+               });
 
-       film->set_reel_type (REELTYPE_BY_VIDEO_CONTENT);
-       film->make_dcp ();
-       wait_for_jobs ();
+       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"))) {
+                       ++got[0];
+               } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_green\\.png_.*\\.mxf"))) {
+                       ++got[1];
+               } else if (boost::regex_match(i.path().string(), boost::regex(".*flat_blue\\.png_.*\\.mxf"))) {
+                       ++got[2];
+               }
+       }
 
-       BOOST_CHECK (boost::filesystem::exists (film->file (film->dcp_name() + "/flat_red.png.mxf")));
-       BOOST_CHECK (boost::filesystem::exists (film->file (film->dcp_name() + "/flat_green.png.mxf")));
-       BOOST_CHECK (boost::filesystem::exists (film->file (film->dcp_name() + "/flat_blue.png.mxf")));
+       for (int i = 0; i < 3; ++i) {
+               BOOST_CHECK (got[i] == 2);
+       }
 }