X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=test%2Fclosed_caption_test.cc;h=313e2a8d3f68e6198d65d5dae8fac14666ffb195;hb=0af9eeb3c1cbf9482a6a98a525f2900190f39d71;hp=2b80c812eba948008ad4cf86310e55c84cf38f55;hpb=ea63ad9560757e56505551db3bf2e1c31be5c76c;p=dcpomatic.git

diff --git a/test/closed_caption_test.cc b/test/closed_caption_test.cc
index 2b80c812e..313e2a8d3 100644
--- a/test/closed_caption_test.cc
+++ b/test/closed_caption_test.cc
@@ -18,19 +18,21 @@
 
 */
 
+
 #include "lib/film.h"
-#include "lib/text_content.h"
 #include "lib/string_text_file_content.h"
+#include "lib/text_content.h"
 #include "test.h"
-#include <dcp/dcp.h>
 #include <dcp/cpl.h>
+#include <dcp/dcp.h>
 #include <dcp/reel.h>
 #include <dcp/reel_closed_caption_asset.h>
 #include <boost/test/unit_test.hpp>
 
+
 using std::list;
 using std::make_shared;
-using std::shared_ptr;
+
 
 /** Basic test that Interop closed captions are written */
 BOOST_AUTO_TEST_CASE (closed_caption_test1)
@@ -75,11 +77,11 @@ BOOST_AUTO_TEST_CASE (closed_caption_test2)
 	auto film = new_test_film2 ("closed_caption_test2", { content1, content2, content3 }, &cl);
 
 	content1->only_text()->set_type (TextType::CLOSED_CAPTION);
-	content1->only_text()->set_dcp_track (DCPTextTrack("First track", "fr-FR"));
+	content1->only_text()->set_dcp_track (DCPTextTrack("First track", dcp::LanguageTag("fr-FR")));
 	content2->only_text()->set_type (TextType::CLOSED_CAPTION);
-	content2->only_text()->set_dcp_track (DCPTextTrack("Second track", "de-DE"));
+	content2->only_text()->set_dcp_track (DCPTextTrack("Second track", dcp::LanguageTag("de-DE")));
 	content3->only_text()->set_type (TextType::CLOSED_CAPTION);
-	content3->only_text()->set_dcp_track (DCPTextTrack("Third track", "it-IT"));
+	content3->only_text()->set_dcp_track (DCPTextTrack("Third track", dcp::LanguageTag("it-IT")));
 
 	make_and_verify_dcp (
 		film,
@@ -88,7 +90,12 @@ BOOST_AUTO_TEST_CASE (closed_caption_test2)
 			dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_LENGTH,
 			dcp::VerificationNote::Code::MISSING_CPL_METADATA,
 			dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME,
-		}
+		},
+		true,
+		/* Clairmeta gives an error about having duplicate ClosedCaption entries,
+		 * which seems wrong.
+		 */
+		false
 		);
 
 	dcp::DCP check (film->dir(film->dcp_name()));
@@ -100,15 +107,15 @@ BOOST_AUTO_TEST_CASE (closed_caption_test2)
 	BOOST_REQUIRE_EQUAL (ccaps.size(), 3U);
 
 	auto i = ccaps.begin ();
-	BOOST_CHECK_EQUAL ((*i)->annotation_text(), "First track");
+	BOOST_CHECK_EQUAL ((*i)->annotation_text().get_value_or(""), "First track");
 	BOOST_REQUIRE (static_cast<bool>((*i)->language()));
 	BOOST_CHECK_EQUAL ((*i)->language().get(), "fr-FR");
 	++i;
-	BOOST_CHECK_EQUAL ((*i)->annotation_text(), "Second track");
+	BOOST_CHECK_EQUAL ((*i)->annotation_text().get_value_or(""), "Second track");
 	BOOST_REQUIRE (static_cast<bool>((*i)->language()));
 	BOOST_CHECK_EQUAL ((*i)->language().get(), "de-DE");
 	++i;
-	BOOST_CHECK_EQUAL ((*i)->annotation_text(), "Third track");
+	BOOST_CHECK_EQUAL ((*i)->annotation_text().get_value_or(""), "Third track");
 	BOOST_REQUIRE (static_cast<bool>((*i)->language()));
 	BOOST_CHECK_EQUAL ((*i)->language().get(), "it-IT");