X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=test%2Fsubtitle_font_id_test.cc;h=42ed8858f022aadc4128af149c382b5b459c8607;hb=0c66eff01358e75ff524f395aeb634ec34a47313;hp=51bccc70a78a3c9ed724e6e6444eb0a1e2db6592;hpb=ebf89c5e6200b65e5cc48bb7b3af23ab68ede749;p=dcpomatic.git

diff --git a/test/subtitle_font_id_test.cc b/test/subtitle_font_id_test.cc
index 51bccc70a..42ed8858f 100644
--- a/test/subtitle_font_id_test.cc
+++ b/test/subtitle_font_id_test.cc
@@ -24,6 +24,12 @@
 #include "lib/film.h"
 #include "lib/font.h"
 #include "lib/text_content.h"
+#include "lib/util.h"
+#include <dcp/cpl.h>
+#include <dcp/dcp.h>
+#include <dcp/reel.h>
+#include <dcp/reel_subtitle_asset.h>
+#include <dcp/smpte_subtitle_asset.h>
 #include "test.h"
 #include <boost/test/unit_test.hpp>
 
@@ -100,3 +106,84 @@ BOOST_AUTO_TEST_CASE(make_dcp_with_subs_from_mkv)
 	make_and_verify_dcp(film, { dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_2K });
 }
 
+
+BOOST_AUTO_TEST_CASE(make_dcp_with_subs_without_font_tag)
+{
+	auto subs = content_factory("test/data/no_font.xml");
+	auto film = new_test_film2("make_dcp_with_subs_without_font_tag", { subs });
+	subs[0]->text.front()->set_use(true);
+	make_and_verify_dcp(
+		film,
+		{
+			dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE,
+			dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME,
+			dcp::VerificationNote::Code::MISSING_CPL_METADATA
+		});
+
+	auto check_file = subtitle_file(film);
+	dcp::SMPTESubtitleAsset check_asset(check_file);
+	BOOST_CHECK_EQUAL(check_asset.load_font_nodes().size(), 1U);
+	auto check_font_data = check_asset.font_data();
+	BOOST_CHECK_EQUAL(check_font_data.size(), 1U);
+	BOOST_CHECK(check_font_data.begin()->second == dcp::ArrayData(default_font_file()));
+}
+
+
+BOOST_AUTO_TEST_CASE(make_dcp_with_subs_in_dcp_without_font_tag)
+{
+	/* Make a DCP with some subs in */
+	auto source_subs = content_factory("test/data/short.srt");
+	auto source = new_test_film2("make_dcp_with_subs_in_dcp_without_font_tag_source", { source_subs });
+	source->set_interop(true);
+	make_and_verify_dcp(
+		source,
+		{
+			dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE,
+			dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME,
+			dcp::VerificationNote::Code::MISSING_CPL_METADATA,
+			dcp::VerificationNote::Code::INVALID_STANDARD
+		});
+
+	/* Find the ID of the subs */
+	dcp::DCP source_dcp(source->dir(source->dcp_name()));
+	source_dcp.read();
+	BOOST_REQUIRE(!source_dcp.cpls().empty());
+	BOOST_REQUIRE(!source_dcp.cpls()[0]->reels().empty());
+	BOOST_REQUIRE(source_dcp.cpls()[0]->reels()[0]->main_subtitle());
+	auto const id = source_dcp.cpls()[0]->reels()[0]->main_subtitle()->asset()->id();
+
+	/* Graft in some bad subs with no <Font> tag */
+	auto source_subtitle_file = subtitle_file(source);
+#if BOOST_VERSION >= 107400
+	boost::filesystem::copy_file("test/data/no_font.xml", source_subtitle_file, boost::filesystem::copy_options::overwrite_existing);
+#else
+	boost::filesystem::copy_file("test/data/no_font.xml", source_subtitle_file, boost::filesystem::copy_option::overwrite_if_exists);
+#endif
+
+	/* Fix the <Id> tag */
+	{
+		Editor editor(source_subtitle_file);
+		editor.replace("4dd8ee05-5986-4c67-a6f8-bbeac62e21db", id);
+	}
+
+	/* Now make a project which imports that DCP and makes another DCP from it */
+	auto dcp_content = make_shared<DCPContent>(source->dir(source->dcp_name()));
+	auto film = new_test_film2("make_dcp_with_subs_without_font_tag", { dcp_content });
+	BOOST_REQUIRE(!dcp_content->text.empty());
+	dcp_content->text.front()->set_use(true);
+	make_and_verify_dcp(
+		film,
+		{
+			dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE,
+			dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME,
+			dcp::VerificationNote::Code::MISSING_CPL_METADATA
+		});
+
+	auto check_file = subtitle_file(film);
+	dcp::SMPTESubtitleAsset check_asset(check_file);
+	BOOST_CHECK_EQUAL(check_asset.load_font_nodes().size(), 1U);
+	auto check_font_data = check_asset.font_data();
+	BOOST_CHECK_EQUAL(check_font_data.size(), 1U);
+	BOOST_CHECK(check_font_data.begin()->second == dcp::ArrayData(default_font_file()));
+}
+