test/hints_test.cc

   1 /*
   2     Copyright (C) 2020-2022 Carl Hetherington <cth@carlh.net>
   3
   4     This file is part of DCP-o-matic.
   5
   6     DCP-o-matic is free software; you can redistribute it and/or modify
   7     it under the terms of the GNU General Public License as published by
   8     the Free Software Foundation; either version 2 of the License, or
   9     (at your option) any later version.
  10
  11     DCP-o-matic is distributed in the hope that it will be useful,
  12     but WITHOUT ANY WARRANTY; without even the implied warranty of
  13     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  14     GNU General Public License for more details.
  15
  16     You should have received a copy of the GNU General Public License
  17     along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
  18
  19 */
  20
  21
  22 #include "lib/audio_content.h"
  23 #include "lib/config.h"
  24 #include "lib/content.h"
  25 #include "lib/content_factory.h"
  26 #include "lib/cross.h"
  27 #include "lib/film.h"
  28 #include "lib/font.h"
  29 #include "lib/hints.h"
  30 #include "lib/text_content.h"
  31 #include "lib/util.h"
  32 #include "test.h"
  33 #include <boost/test/unit_test.hpp>
  34
  35
  36 using std::make_shared;
  37 using std::shared_ptr;
  38 using std::string;
  39 using std::vector;
  40 using boost::optional;
  41
  42
  43 vector<string> current_hints;
  44
  45
  46 static
  47 void
  48 collect_hint (string hint)
  49 {
  50         current_hints.push_back (hint);
  51 }
  52
  53
  54 static
  55 vector<string>
  56 get_hints (shared_ptr<Film> film)
  57 {
  58         current_hints.clear ();
  59         Hints hints (film);
  60         /* None of our tests need the audio analysis, and it is quite time-consuming */
  61         hints.disable_audio_analysis ();
  62         hints.Hint.connect (collect_hint);
  63         hints.start ();
  64         hints.join ();
  65         while (signal_manager->ui_idle()) {}
  66         hints.rethrow();
  67         return current_hints;
  68 }
  69
  70
  71 static
  72 void
  73 check (TextType type, string name, optional<string> expected_hint = optional<string>())
  74 {
  75         auto film = new_test_film2 (name);
  76         auto content = content_factory("test/data/" + name + ".srt").front();
  77         content->text.front()->set_type (type);
  78         content->text.front()->set_language (dcp::LanguageTag("en-US"));
  79         film->examine_and_add_content (content);
  80         BOOST_REQUIRE (!wait_for_jobs());
  81         auto hints = get_hints (film);
  82
  83         if (expected_hint) {
  84                 BOOST_REQUIRE_EQUAL (hints.size(), 1U);
  85                 BOOST_CHECK_EQUAL (hints[0], *expected_hint);
  86         } else {
  87                 BOOST_CHECK (hints.empty());
  88         }
  89 }
  90
  91
  92 BOOST_AUTO_TEST_CASE (hint_closed_caption_too_long)
  93 {
  94         check (
  95                 TextType::CLOSED_CAPTION,
  96                 "hint_closed_caption_too_long",
  97                 String::compose("At least one of your closed caption lines has more than %1 characters.  It is advisable to make each line %1 characters at most in length.", MAX_CLOSED_CAPTION_LENGTH, MAX_CLOSED_CAPTION_LENGTH)
  98               );
  99 }
 100
 101
 102 BOOST_AUTO_TEST_CASE (hint_many_closed_caption_lines)
 103 {
 104         check (
 105                 TextType::CLOSED_CAPTION,
 106                 "hint_many_closed_caption_lines",
 107                 String::compose("Some of your closed captions span more than %1 lines, so they will be truncated.", MAX_CLOSED_CAPTION_LINES)
 108               );
 109 }
 110
 111
 112 BOOST_AUTO_TEST_CASE (hint_subtitle_too_early)
 113 {
 114         check (
 115                 TextType::OPEN_SUBTITLE,
 116                 "hint_subtitle_too_early",
 117                 string("It is advisable to put your first subtitle at least 4 seconds after the start of the DCP to make sure it is seen.")
 118                 );
 119 }
 120
 121
 122 BOOST_AUTO_TEST_CASE (hint_short_subtitles)
 123 {
 124         check (
 125                 TextType::OPEN_SUBTITLE,
 126                 "hint_short_subtitles",
 127                 string("At least one of your subtitles lasts less than 15 frames.  It is advisable to make each subtitle at least 15 frames long.")
 128                 );
 129 }
 130
 131
 132 BOOST_AUTO_TEST_CASE (hint_subtitles_too_close)
 133 {
 134         check (
 135                 TextType::OPEN_SUBTITLE,
 136                 "hint_subtitles_too_close",
 137                 string("At least one of your subtitles starts less than 2 frames after the previous one.  It is advisable to make the gap between subtitles at least 2 frames.")
 138               );
 139 }
 140
 141
 142 BOOST_AUTO_TEST_CASE (hint_many_subtitle_lines)
 143 {
 144         check (
 145                 TextType::OPEN_SUBTITLE,
 146                 "hint_many_subtitle_lines",
 147                 string("At least one of your subtitles has more than 3 lines.  It is advisable to use no more than 3 lines.")
 148               );
 149 }
 150
 151
 152 BOOST_AUTO_TEST_CASE (hint_subtitle_too_long)
 153 {
 154         check (
 155                 TextType::OPEN_SUBTITLE,
 156                 "hint_subtitle_too_long",
 157                 string("At least one of your subtitle lines has more than 52 characters.  It is recommended to make each line 52 characters at most in length.")
 158               );
 159 }
 160
 161
 162 BOOST_AUTO_TEST_CASE (hint_subtitle_much_too_long)
 163 {
 164         check (
 165                 TextType::OPEN_SUBTITLE,
 166                 "hint_subtitle_much_too_long",
 167                 string("At least one of your subtitle lines has more than 79 characters.  You should make each line 79 characters at most in length.")
 168               );
 169 }
 170
 171
 172 BOOST_AUTO_TEST_CASE (hint_subtitle_mxf_too_big)
 173 {
 174         string const name = "hint_subtitle_mxf_too_big";
 175
 176         auto film = new_test_film2 (name);
 177         auto content = content_factory("test/data/" + name + ".srt").front();
 178         content->text.front()->set_type (TextType::OPEN_SUBTITLE);
 179         content->text.front()->set_language (dcp::LanguageTag("en-US"));
 180         for (int i = 1; i < 512; ++i) {
 181                 auto font = make_shared<dcpomatic::Font>(String::compose("font_%1", i));
 182                 font->set_file ("test/data/LiberationSans-Regular.ttf");
 183                 content->text.front()->add_font(font);
 184         }
 185         film->examine_and_add_content (content);
 186         BOOST_REQUIRE (!wait_for_jobs());
 187         auto hints = get_hints (film);
 188
 189         BOOST_REQUIRE_EQUAL (hints.size(), 1U);
 190         BOOST_CHECK_EQUAL (
 191                 hints[0],
 192                 "At least one of your subtitle files is larger than " MAX_TEXT_MXF_SIZE_TEXT " in total.  "
 193                 "You should divide the DCP into shorter reels."
 194                 );
 195 }
 196
 197
 198 BOOST_AUTO_TEST_CASE (hint_closed_caption_xml_too_big)
 199 {
 200         string const name = "hint_closed_caption_xml_too_big";
 201
 202         auto film = new_test_film2 (name);
 203
 204         dcp::File ccap(String::compose("build/test/%1.srt", name), "w");
 205         BOOST_REQUIRE (ccap);
 206         for (int i = 0; i < 2048; ++i) {
 207                 fprintf(ccap.get(), "%d\n", i + 1);
 208                 int second = i * 2;
 209                 int minute = second % 60;
 210                 fprintf(ccap.get(), "00:%02d:%02d,000 --> 00:%02d:%02d,000\n", minute, second, minute, second + 1);
 211                 fprintf(ccap.get(), "Here are some closed captions.\n\n");
 212         }
 213         ccap.close();
 214
 215         auto content = content_factory("build/test/" + name + ".srt").front();
 216         content->text.front()->set_type (TextType::CLOSED_CAPTION);
 217         content->text.front()->set_language (dcp::LanguageTag("en-US"));
 218         film->examine_and_add_content (content);
 219         BOOST_REQUIRE (!wait_for_jobs());
 220         auto hints = get_hints (film);
 221
 222         BOOST_REQUIRE_EQUAL (hints.size(), 1U);
 223         BOOST_CHECK_EQUAL (
 224                 hints[0],
 225                 "At least one of your closed caption files' XML part is larger than " MAX_CLOSED_CAPTION_XML_SIZE_TEXT ".  "
 226                 "You should divide the DCP into shorter reels."
 227                 );
 228 }
 229
 230
 231 BOOST_AUTO_TEST_CASE (hints_destroyed_while_running)
 232 {
 233         auto film = new_test_film2 ("hints_destroyed_while_running");
 234         auto content = content_factory(TestPaths::private_data() / "boon_telly.mkv").front();
 235         film->examine_and_add_content (content);
 236         BOOST_REQUIRE (!wait_for_jobs());
 237
 238         auto hints = make_shared<Hints>(film);
 239         hints->start ();
 240         dcpomatic_sleep_seconds (1);
 241         hints.reset ();
 242         dcpomatic_sleep_seconds (1);
 243 }
 244
 245
 246 BOOST_AUTO_TEST_CASE (hints_audio_with_no_language)
 247 {
 248         auto content = content_factory("test/data/sine_440.wav").front();
 249         auto film = new_test_film2 ("hints_audio_with_no_language", { content });
 250         content->audio->set_gain (-6);
 251
 252         auto hints = get_hints (film);
 253         BOOST_REQUIRE_EQUAL (hints.size(), 1U);
 254         BOOST_CHECK_EQUAL (
 255                 hints[0],
 256                 "Some of your content has audio but you have not set the audio language.  It is advisable to set the audio language "
 257                 "in the \"DCP\" tab unless your audio has no spoken parts."
 258                 );
 259 }
 260
 261
 262 BOOST_AUTO_TEST_CASE (hints_certificate_validity)
 263 {
 264         ConfigRestorer cr;
 265
 266         Config::instance()->set_signer_chain(make_shared<dcp::CertificateChain>(openssl_path(), 40 * 365));
 267
 268         auto film = new_test_film2 ("hints_certificate_validity");
 269         auto hints = get_hints (film);
 270         BOOST_REQUIRE_EQUAL (hints.size(), 1U);
 271         BOOST_CHECK_EQUAL (
 272                 hints[0],
 273                 "The certificate chain that DCP-o-matic uses for signing DCPs and KDMs has a validity period "
 274                 "that is too long.  This will cause problems playing back DCPs on some systems. "
 275                 "You are advised to re-create the signing certificate chain by clicking the "
 276                 "\"Re-make certificates and key...\" button in the Keys page of Preferences."
 277                 );
 278 }
 279