Merge read_smpte_subtitle_test with smpte_subtitle_test.

diff --git a/test/read_smpte_subtitle_test.cc b/test/read_smpte_subtitle_test.cc
deleted file mode 100644 (file)
index 49fd1f6..0000000
--- a/test/read_smpte_subtitle_test.cc
+++ /dev/null
@@ -1,135 +0,0 @@
-/*
-    Copyright (C) 2015-2021 Carl Hetherington <cth@carlh.net>
-
-    This file is part of libdcp.
-
-    libdcp is free software; you can redistribute it and/or modify
-    it under the terms of the GNU General Public License as published by
-    the Free Software Foundation; either version 2 of the License, or
-    (at your option) any later version.
-
-    libdcp is distributed in the hope that it will be useful,
-    but WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU General Public License for more details.
-
-    You should have received a copy of the GNU General Public License
-    along with libdcp.  If not, see <http://www.gnu.org/licenses/>.
-
-    In addition, as a special exception, the copyright holders give
-    permission to link the code of portions of this program with the
-    OpenSSL library under certain conditions as described in each
-    individual source file, and distribute linked combinations
-    including the two.
-
-    You must obey the GNU General Public License in all respects
-    for all of the code used other than OpenSSL.  If you modify
-    file(s) with this exception, you may extend this exception to your
-    version of the file(s), but you are not obligated to do so.  If you
-    do not wish to do so, delete this exception statement from your
-    version.  If you delete this exception statement from all source
-    files in the program, then also delete it here.
-*/
-
-#include "smpte_subtitle_asset.h"
-#include "test.h"
-#include "local_time.h"
-#include "smpte_load_font_node.h"
-#include "subtitle_image.h"
-#include "stream_operators.h"
-#include <boost/test/unit_test.hpp>
-#include <boost/optional.hpp>
-#include <boost/optional/optional_io.hpp>
-
-using std::list;
-using std::shared_ptr;
-using std::dynamic_pointer_cast;
-
-/** Check reading of a SMPTE subtitle file */
-BOOST_AUTO_TEST_CASE (read_smpte_subtitle_test)
-{
-       dcp::SMPTESubtitleAsset sc (
-               private_test /
-               "data" /
-               "JourneyToJah_TLR-1_F_EN-DE-FR_CH_51_2K_LOK_20140225_DGL_SMPTE_OV" /
-               "8b48f6ae-c74b-4b80-b994-a8236bbbad74_sub.mxf"
-               );
-
-       BOOST_CHECK_EQUAL (sc.id(), "8b48f6ae-c74b-4b80-b994-a8236bbbad74");
-       BOOST_CHECK_EQUAL (sc.content_title_text(), "Journey to Jah");
-       BOOST_REQUIRE (sc.annotation_text());
-       BOOST_CHECK_EQUAL (sc.annotation_text().get(), "Journey to Jah");
-       BOOST_CHECK_EQUAL (sc.issue_date(), dcp::LocalTime ("2014-02-25T11:22:48.000-00:00"));
-       BOOST_REQUIRE (sc.reel_number());
-       BOOST_CHECK_EQUAL (sc.reel_number().get(), 1);
-       BOOST_REQUIRE (sc.language ());
-       BOOST_CHECK_EQUAL (sc.language().get (), "de");
-       BOOST_CHECK_EQUAL (sc.edit_rate(), dcp::Fraction (25, 1));
-       BOOST_CHECK_EQUAL (sc.time_code_rate(), 25);
-       BOOST_CHECK_EQUAL (sc.start_time(), dcp::Time (0, 0, 0, 0, 25));
-       auto lfn = sc.load_font_nodes ();
-       BOOST_REQUIRE_EQUAL (lfn.size(), 1);
-       shared_ptr<dcp::SMPTELoadFontNode> smpte_lfn = dynamic_pointer_cast<dcp::SMPTELoadFontNode> (lfn.front ());
-       BOOST_REQUIRE (smpte_lfn);
-       BOOST_CHECK_EQUAL (smpte_lfn->id, "theFontId");
-       BOOST_CHECK_EQUAL (smpte_lfn->urn, "9118bbce-4105-4a05-b37c-a5a6f75e1fea");
-       BOOST_REQUIRE_EQUAL (sc.subtitles().size(), 63);
-       BOOST_REQUIRE (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().front()));
-       BOOST_CHECK_EQUAL (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().front())->text(), "Noch mal.");
-       BOOST_CHECK_EQUAL (sc.subtitles().front()->in(), dcp::Time (0, 0, 25, 12, 25));
-       BOOST_CHECK_EQUAL (sc.subtitles().front()->out(), dcp::Time (0, 0, 26, 4, 25));
-       BOOST_REQUIRE (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().back()));
-       BOOST_CHECK_EQUAL (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().back())->text(), "Prochainement");
-       BOOST_CHECK_EQUAL (sc.subtitles().back()->in(), dcp::Time (0, 1, 57, 17, 25));
-       BOOST_CHECK_EQUAL (sc.subtitles().back()->out(), dcp::Time (0, 1, 58, 12, 25));
-}
-
-/** And another one featuring <Font> within <Text> */
-BOOST_AUTO_TEST_CASE (read_smpte_subtitle_test2)
-{
-       dcp::SMPTESubtitleAsset sc (private_test / "olsson.xml");
-
-       auto subs = sc.subtitles();
-       BOOST_REQUIRE_EQUAL (subs.size(), 6);
-       auto i = 0;
-       auto is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
-       BOOST_REQUIRE (is);
-       BOOST_CHECK_EQUAL (is->text(), "Testing is ");
-       BOOST_CHECK (!is->italic());
-       ++i;
-       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
-       BOOST_REQUIRE (is);
-       BOOST_CHECK_EQUAL (is->text(), "really");
-       BOOST_CHECK (is->italic());
-       ++i;
-       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
-       BOOST_REQUIRE (is);
-       BOOST_CHECK_EQUAL (is->text(), " fun!");
-       BOOST_CHECK (!is->italic());
-       ++i;
-       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
-       BOOST_REQUIRE (is);
-       BOOST_CHECK_EQUAL (is->text(), "This is the ");
-       BOOST_CHECK (!is->italic());
-       ++i;
-       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
-       BOOST_REQUIRE (is);
-       BOOST_CHECK_EQUAL (is->text(), "second");
-       BOOST_CHECK (is->italic());
-       ++i;
-       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
-       BOOST_REQUIRE (is);
-       BOOST_CHECK_EQUAL (is->text(), " line!");
-       BOOST_CHECK (!is->italic());
-}
-
-/** And another one featuring image subtitles */
-BOOST_AUTO_TEST_CASE (read_smpte_subtitle_test3)
-{
-       dcp::SMPTESubtitleAsset subs ("test/data/subs.mxf");
-
-       BOOST_REQUIRE_EQUAL (subs.subtitles().size(), 1);
-       auto si = dynamic_pointer_cast<const dcp::SubtitleImage>(subs.subtitles()[0]);
-       BOOST_REQUIRE (si);
-       BOOST_CHECK (si->png_image() == dcp::ArrayData("test/data/sub.png"));
-}

diff --git a/test/smpte_subtitle_test.cc b/test/smpte_subtitle_test.cc
index 2af7669605ee5a7fb2dd753476bb099c48e30486..2e35ba8dbc4c836c7768c408624e41f8b65060bb 100644 (file)
--- a/test/smpte_subtitle_test.cc
+++ b/test/smpte_subtitle_test.cc
@@ -31,14 +31,24 @@
     files in the program, then also delete it here.
 */
 
-#include <boost/test/unit_test.hpp>
+
+#include "smpte_load_font_node.h"
 #include "smpte_subtitle_asset.h"
+#include "stream_operators.h"
+#include "subtitle_image.h"
+#include "test.h"
+#include "types.h"
+#include <boost/optional/optional_io.hpp>
+#include <boost/test/unit_test.hpp>
+
 
 using std::make_shared;
 using std::string;
 using std::shared_ptr;
+using std::dynamic_pointer_cast;
 using boost::optional;
 
+
 BOOST_AUTO_TEST_CASE (smpte_subtitle_id_test)
 {
        dcp::SMPTESubtitleAsset subs;
@@ -68,3 +78,95 @@ BOOST_AUTO_TEST_CASE (smpte_subtitle_id_test)
        dcp::SMPTESubtitleAsset check("build/test/smpte_subtitle_id_test.mxf");
        BOOST_CHECK(check.id() != check.xml_id());
 }
+
+
+/** Check reading of a SMPTE subtitle file */
+BOOST_AUTO_TEST_CASE (read_smpte_subtitle_test)
+{
+       dcp::SMPTESubtitleAsset sc (
+               private_test /
+               "data" /
+               "JourneyToJah_TLR-1_F_EN-DE-FR_CH_51_2K_LOK_20140225_DGL_SMPTE_OV" /
+               "8b48f6ae-c74b-4b80-b994-a8236bbbad74_sub.mxf"
+               );
+
+       BOOST_CHECK_EQUAL (sc.id(), "8b48f6ae-c74b-4b80-b994-a8236bbbad74");
+       BOOST_CHECK_EQUAL (sc.content_title_text(), "Journey to Jah");
+       BOOST_REQUIRE (sc.annotation_text());
+       BOOST_CHECK_EQUAL (sc.annotation_text().get(), "Journey to Jah");
+       BOOST_CHECK_EQUAL (sc.issue_date(), dcp::LocalTime ("2014-02-25T11:22:48.000-00:00"));
+       BOOST_REQUIRE (sc.reel_number());
+       BOOST_CHECK_EQUAL (sc.reel_number().get(), 1);
+       BOOST_REQUIRE (sc.language ());
+       BOOST_CHECK_EQUAL (sc.language().get (), "de");
+       BOOST_CHECK_EQUAL (sc.edit_rate(), dcp::Fraction (25, 1));
+       BOOST_CHECK_EQUAL (sc.time_code_rate(), 25);
+       BOOST_CHECK_EQUAL (sc.start_time(), dcp::Time (0, 0, 0, 0, 25));
+       auto lfn = sc.load_font_nodes ();
+       BOOST_REQUIRE_EQUAL (lfn.size(), 1);
+       shared_ptr<dcp::SMPTELoadFontNode> smpte_lfn = dynamic_pointer_cast<dcp::SMPTELoadFontNode> (lfn.front ());
+       BOOST_REQUIRE (smpte_lfn);
+       BOOST_CHECK_EQUAL (smpte_lfn->id, "theFontId");
+       BOOST_CHECK_EQUAL (smpte_lfn->urn, "9118bbce-4105-4a05-b37c-a5a6f75e1fea");
+       BOOST_REQUIRE_EQUAL (sc.subtitles().size(), 63);
+       BOOST_REQUIRE (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().front()));
+       BOOST_CHECK_EQUAL (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().front())->text(), "Noch mal.");
+       BOOST_CHECK_EQUAL (sc.subtitles().front()->in(), dcp::Time (0, 0, 25, 12, 25));
+       BOOST_CHECK_EQUAL (sc.subtitles().front()->out(), dcp::Time (0, 0, 26, 4, 25));
+       BOOST_REQUIRE (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().back()));
+       BOOST_CHECK_EQUAL (dynamic_pointer_cast<const dcp::SubtitleString>(sc.subtitles().back())->text(), "Prochainement");
+       BOOST_CHECK_EQUAL (sc.subtitles().back()->in(), dcp::Time (0, 1, 57, 17, 25));
+       BOOST_CHECK_EQUAL (sc.subtitles().back()->out(), dcp::Time (0, 1, 58, 12, 25));
+}
+
+
+/** And another one featuring <Font> within <Text> */
+BOOST_AUTO_TEST_CASE (read_smpte_subtitle_test2)
+{
+       dcp::SMPTESubtitleAsset sc (private_test / "olsson.xml");
+
+       auto subs = sc.subtitles();
+       BOOST_REQUIRE_EQUAL (subs.size(), 6);
+       auto i = 0;
+       auto is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
+       BOOST_REQUIRE (is);
+       BOOST_CHECK_EQUAL (is->text(), "Testing is ");
+       BOOST_CHECK (!is->italic());
+       ++i;
+       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
+       BOOST_REQUIRE (is);
+       BOOST_CHECK_EQUAL (is->text(), "really");
+       BOOST_CHECK (is->italic());
+       ++i;
+       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
+       BOOST_REQUIRE (is);
+       BOOST_CHECK_EQUAL (is->text(), " fun!");
+       BOOST_CHECK (!is->italic());
+       ++i;
+       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
+       BOOST_REQUIRE (is);
+       BOOST_CHECK_EQUAL (is->text(), "This is the ");
+       BOOST_CHECK (!is->italic());
+       ++i;
+       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
+       BOOST_REQUIRE (is);
+       BOOST_CHECK_EQUAL (is->text(), "second");
+       BOOST_CHECK (is->italic());
+       ++i;
+       is = dynamic_pointer_cast<const dcp::SubtitleString>(subs[i]);
+       BOOST_REQUIRE (is);
+       BOOST_CHECK_EQUAL (is->text(), " line!");
+       BOOST_CHECK (!is->italic());
+}
+
+
+/** And another one featuring image subtitles */
+BOOST_AUTO_TEST_CASE (read_smpte_subtitle_test3)
+{
+       dcp::SMPTESubtitleAsset subs ("test/data/subs.mxf");
+
+       BOOST_REQUIRE_EQUAL (subs.subtitles().size(), 1);
+       auto si = dynamic_pointer_cast<const dcp::SubtitleImage>(subs.subtitles()[0]);
+       BOOST_REQUIRE (si);
+       BOOST_CHECK (si->png_image() == dcp::ArrayData("test/data/sub.png"));
+}

diff --git a/test/wscript b/test/wscript
index f10bb8247017f36a7754654eee7dc23a047de645..e1078d25b4cfa1d8299a918eca9b327b420a3560 100644 (file)
--- a/test/wscript
+++ b/test/wscript
@@ -91,7 +91,6 @@ def build(bld):
                  raw_convert_test.cc
                  read_dcp_test.cc
                  read_interop_subtitle_test.cc
-                 read_smpte_subtitle_test.cc
                  reel_asset_test.cc
                  recovery_test.cc
                  rgb_xyz_test.cc