From 0755722ecc0777a618a3b2eec5e1d94916bb2b8f Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Fri, 8 Sep 2023 07:47:41 +0200 Subject: [PATCH] Fix hints test now that subtitle reader gives an error on large fonts. --- test/hints_test.cc | 28 ++++++++++++++++------------ 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/test/hints_test.cc b/test/hints_test.cc index 5e59ef924..75a5aff04 100644 --- a/test/hints_test.cc +++ b/test/hints_test.cc @@ -183,20 +183,24 @@ BOOST_AUTO_TEST_CASE (hint_subtitle_mxf_too_big) { string const name = "hint_subtitle_mxf_too_big"; - dcp::File fake_font("build/test/hint_subtitle_mxf_too_big.ttf", "w"); - for (int i = 0; i < 4096; ++i) { - std::vector rubbish(65536); - fake_font.write(rubbish.data(), 1, rubbish.size()); + auto film = new_test_film2 (name); + + for (int i = 0; i < 4; ++i) { + dcp::File fake_font("build/test/hint_subtitle_mxf_too_big.ttf", "w"); + for (int i = 0; i < 512; ++i) { + std::vector rubbish(65536); + fake_font.write(rubbish.data(), 1, rubbish.size()); + } + fake_font.close(); + + auto content = content_factory("test/data/" + name + ".srt")[0]; + content->text[0]->set_type(TextType::OPEN_SUBTITLE); + content->text[0]->set_language(dcp::LanguageTag("en-US")); + film->examine_and_add_content(content); + BOOST_REQUIRE (!wait_for_jobs()); + content->text[0]->get_font("")->set_file("build/test/hint_subtitle_mxf_too_big.ttf"); } - fake_font.close(); - auto film = new_test_film2 (name); - auto content = content_factory("test/data/" + name + ".srt")[0]; - content->text.front()->set_type (TextType::OPEN_SUBTITLE); - content->text.front()->set_language (dcp::LanguageTag("en-US")); - film->examine_and_add_content (content); - BOOST_REQUIRE (!wait_for_jobs()); - content->text.front()->get_font("")->set_file("build/test/hint_subtitle_mxf_too_big.ttf"); auto hints = get_hints (film); BOOST_REQUIRE_EQUAL (hints.size(), 1U); -- 2.30.2