Rename DEVELOPMENT -> DEVELOP.md and add some stuff about player stress testing.

[dcpomatic.git] / test / empty_test.cc

diff --git a/test/empty_test.cc b/test/empty_test.cc
index 71397273942bbbe8312ec90bc1b4a42ffd0a8ff8..f0fa7138f46408a456e7db54df24fdbfdb29c1c8 100644 (file)
--- a/test/empty_test.cc
+++ b/test/empty_test.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2017 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2017-2020 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -24,10 +24,20 @@
 #include "lib/video_content.h"
 #include "lib/image_content.h"
 #include "lib/empty.h"
+#include "lib/player.h"
+#include "lib/decoder.h"
 #include "test.h"
 #include <boost/test/unit_test.hpp>
 
+using std::list;
 using boost::shared_ptr;
+using namespace dcpomatic;
+
+bool
+has_video (shared_ptr<Piece> piece)
+{
+        return piece->decoder && piece->decoder->video;
+}
 
 BOOST_AUTO_TEST_CASE (empty_test1)
 {
@@ -36,28 +46,34 @@ BOOST_AUTO_TEST_CASE (empty_test1)
        film->set_name ("empty_test1");
        film->set_container (Ratio::from_id ("185"));
        film->set_sequence (false);
-       shared_ptr<ImageContent> contentA (new ImageContent (film, "test/data/simple_testcard_640x480.png"));
-       shared_ptr<ImageContent> contentB (new ImageContent (film, "test/data/simple_testcard_640x480.png"));
+       shared_ptr<ImageContent> contentA (new ImageContent("test/data/simple_testcard_640x480.png"));
+       shared_ptr<ImageContent> contentB (new ImageContent("test/data/simple_testcard_640x480.png"));
 
        film->examine_and_add_content (contentA);
        film->examine_and_add_content (contentB);
-       wait_for_jobs ();
+       BOOST_REQUIRE (!wait_for_jobs());
 
        int const vfr = film->video_frame_rate ();
 
        contentA->video->set_scale (VideoContentScale (Ratio::from_id ("185")));
        contentA->video->set_length (3);
-       contentA->set_position (DCPTime::from_frames (2, vfr));
+       contentA->set_position (film, DCPTime::from_frames (2, vfr));
        contentB->video->set_scale (VideoContentScale (Ratio::from_id ("185")));
        contentB->video->set_length (1);
-       contentB->set_position (DCPTime::from_frames (7, vfr));
-
-       Empty black (film->content(), film->length(), bind(&Content::video, _1));
-       BOOST_REQUIRE_EQUAL (black._periods.size(), 2);
-       BOOST_CHECK (black._periods.front().from == DCPTime());
-       BOOST_CHECK (black._periods.front().to == DCPTime::from_frames(2, vfr));
-       BOOST_CHECK (black._periods.back().from == DCPTime::from_frames(5, vfr));
-       BOOST_CHECK (black._periods.back().to == DCPTime::from_frames(7, vfr));
+       contentB->set_position (film, DCPTime::from_frames (7, vfr));
+
+       shared_ptr<Player> player (new Player(film, film->playlist()));
+       Empty black (film, player->_pieces, bind(&has_video, _1));
+       BOOST_REQUIRE_EQUAL (black._periods.size(), 3);
+       list<dcpomatic::DCPTimePeriod>::const_iterator i = black._periods.begin();
+       BOOST_CHECK (i->from == DCPTime());
+       BOOST_CHECK (i->to == DCPTime::from_frames(2, vfr));
+       ++i;
+       BOOST_CHECK (i->from == DCPTime::from_frames(5, vfr));
+       BOOST_CHECK (i->to == DCPTime::from_frames(7, vfr));
+       ++i;
+       BOOST_CHECK (i->from == DCPTime::from_frames(8, vfr));
+       BOOST_CHECK (i->to == DCPTime::from_frames(24, vfr));
 }
 
 /** Some tests where the first empty period is not at time 0 */
@@ -68,24 +84,25 @@ BOOST_AUTO_TEST_CASE (empty_test2)
        film->set_name ("empty_test1");
        film->set_container (Ratio::from_id ("185"));
        film->set_sequence (false);
-       shared_ptr<ImageContent> contentA (new ImageContent (film, "test/data/simple_testcard_640x480.png"));
-       shared_ptr<ImageContent> contentB (new ImageContent (film, "test/data/simple_testcard_640x480.png"));
+       shared_ptr<ImageContent> contentA (new ImageContent("test/data/simple_testcard_640x480.png"));
+       shared_ptr<ImageContent> contentB (new ImageContent("test/data/simple_testcard_640x480.png"));
 
        film->examine_and_add_content (contentA);
        film->examine_and_add_content (contentB);
-       wait_for_jobs ();
+       BOOST_REQUIRE (!wait_for_jobs());
 
        int const vfr = film->video_frame_rate ();
 
        contentA->video->set_scale (VideoContentScale (Ratio::from_id ("185")));
        contentA->video->set_length (3);
-       contentA->set_position (DCPTime(0));
+       contentA->set_position (film, DCPTime(0));
        contentB->video->set_scale (VideoContentScale (Ratio::from_id ("185")));
        contentB->video->set_length (1);
-       contentB->set_position (DCPTime::from_frames (7, vfr));
+       contentB->set_position (film, DCPTime::from_frames(7, vfr));
 
-       Empty black (film->content(), film->length(), bind(&Content::video, _1));
-       BOOST_REQUIRE_EQUAL (black._periods.size(), 1);
+       shared_ptr<Player> player (new Player(film, film->playlist()));
+       Empty black (film, player->_pieces, bind(&has_video, _1));
+       BOOST_REQUIRE_EQUAL (black._periods.size(), 2);
        BOOST_CHECK (black._periods.front().from == DCPTime::from_frames(3, vfr));
        BOOST_CHECK (black._periods.front().to == DCPTime::from_frames(7, vfr));
 
@@ -97,5 +114,7 @@ BOOST_AUTO_TEST_CASE (empty_test2)
        black.set_position (DCPTime::from_frames (4, vfr));
        BOOST_CHECK (!black.done ());
        black.set_position (DCPTime::from_frames (7, vfr));
+       BOOST_CHECK (!black.done ());
+       black.set_position (DCPTime::from_frames (24, vfr));
        BOOST_CHECK (black.done ());
 }