From 7c73ec405fdb55bd78d82d764999b5af6d81e973 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Sun, 13 Oct 2019 23:47:17 +0200 Subject: Fix failure to load OV after adding a VF to a project. This has the same cause as 19f51503621a57794bd79bac053c9e6549a69f46 i.e. the DCPDecoder re-use optimisation. This commit tries to re-fix 19f515 in a more general way which also takes into account the OV/VF bug. It also adds a unit test. --- test/dcp_decoder_test.cc | 140 +++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 140 insertions(+) create mode 100644 test/dcp_decoder_test.cc (limited to 'test/dcp_decoder_test.cc') diff --git a/test/dcp_decoder_test.cc b/test/dcp_decoder_test.cc new file mode 100644 index 000000000..64d35a2eb --- /dev/null +++ b/test/dcp_decoder_test.cc @@ -0,0 +1,140 @@ +/* + Copyright (C) 2019 Carl Hetherington + + This file is part of DCP-o-matic. + + DCP-o-matic 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. + + DCP-o-matic 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 DCP-o-matic. If not, see . + +*/ + +/** @file test/dcp_decoder_test.cc + * @brief Test DCPDecoder class. + * @ingroup selfcontained + */ + +#include "lib/film.h" +#include "lib/dcp_content.h" +#include "lib/dcp_decoder.h" +#include "lib/content_factory.h" +#include "lib/player.h" +#include "lib/examine_content_job.h" +#include "lib/job_manager.h" +#include "lib/config.h" +#include "test.h" +#include +#include +#include + +using std::list; +using std::string; +using std::vector; +using boost::shared_ptr; + +/* Check that DCPDecoder reuses old data when it should */ +BOOST_AUTO_TEST_CASE (check_reuse_old_data_test) +{ + /* Make some DCPs */ + + shared_ptr ov = new_test_film2 ("check_reuse_old_data_ov"); + ov->examine_and_add_content (content_factory("test/data/flat_red.png").front()); + BOOST_REQUIRE (!wait_for_jobs()); + ov->make_dcp (); + BOOST_REQUIRE (!wait_for_jobs()); + + shared_ptr vf = new_test_film2 ("check_reuse_old_data_vf"); + shared_ptr ov_content(new DCPContent(ov->dir(ov->dcp_name(false)))); + vf->examine_and_add_content (ov_content); + vf->examine_and_add_content (content_factory("test/data/L.wav").front()); + BOOST_REQUIRE (!wait_for_jobs()); + ov_content->set_reference_video (true); + vf->make_dcp (); + BOOST_REQUIRE (!wait_for_jobs()); + + shared_ptr encrypted = new_test_film2 ("check_reuse_old_data_decrypted"); + encrypted->examine_and_add_content (content_factory("test/data/flat_red.png").front()); + BOOST_REQUIRE (!wait_for_jobs()); + encrypted->set_encrypted (true); + encrypted->make_dcp (); + BOOST_REQUIRE (!wait_for_jobs()); + + dcp::DCP encrypted_dcp (encrypted->dir(encrypted->dcp_name())); + encrypted_dcp.read (); + + dcp::EncryptedKDM kdm = encrypted->make_kdm ( + Config::instance()->decryption_chain()->leaf(), + vector(), + encrypted_dcp.cpls().front()->file().get(), + dcp::LocalTime ("2014-07-21T00:00:00+00:00"), + dcp::LocalTime ("2024-07-21T00:00:00+00:00"), + dcp::MODIFIED_TRANSITIONAL_1, + true, 0 + ); + + + /* Add just the OV to a new project, move it around a bit and check that + the _reels get reused. + */ + shared_ptr test = new_test_film2 ("check_reuse_old_data_test1"); + ov_content.reset (new DCPContent(ov->dir(ov->dcp_name(false)))); + test->examine_and_add_content (ov_content); + BOOST_REQUIRE (!wait_for_jobs()); + shared_ptr player (new Player(test, test->playlist())); + + shared_ptr decoder = boost::dynamic_pointer_cast(player->_pieces.front()->decoder); + BOOST_REQUIRE (decoder); + list > reels = decoder->reels(); + + ov_content->set_position (test, dcpomatic::DCPTime(96000)); + decoder = boost::dynamic_pointer_cast(player->_pieces.front()->decoder); + BOOST_REQUIRE (decoder); + BOOST_REQUIRE (reels == decoder->reels()); + + /* Add the VF to a new project, then add the OV and check that the + _reels did not get reused. + */ + test = new_test_film2 ("check_reuse_old_data_test2"); + shared_ptr vf_content (new DCPContent(vf->dir(vf->dcp_name(false)))); + test->examine_and_add_content (vf_content); + BOOST_REQUIRE (!wait_for_jobs()); + player.reset (new Player(test, test->playlist())); + + decoder = boost::dynamic_pointer_cast(player->_pieces.front()->decoder); + BOOST_REQUIRE (decoder); + reels = decoder->reels(); + + vf_content->add_ov (ov->dir(ov->dcp_name(false))); + JobManager::instance()->add (shared_ptr(new ExamineContentJob(test, vf_content))); + BOOST_REQUIRE (!wait_for_jobs()); + decoder = boost::dynamic_pointer_cast(player->_pieces.front()->decoder); + BOOST_REQUIRE (decoder); + BOOST_REQUIRE (reels != decoder->reels()); + + /* Add a KDM to an encrypted DCP and check that the _reels did not get reused */ + test = new_test_film2 ("check_reuse_old_data_test3"); + shared_ptr encrypted_content (new DCPContent(encrypted->dir(encrypted->dcp_name(false)))); + test->examine_and_add_content (encrypted_content); + BOOST_REQUIRE (!wait_for_jobs()); + player.reset (new Player(test, test->playlist())); + + decoder = boost::dynamic_pointer_cast(player->_pieces.front()->decoder); + BOOST_REQUIRE (decoder); + reels = decoder->reels(); + + encrypted_content->add_kdm (kdm); + JobManager::instance()->add (shared_ptr(new ExamineContentJob(test, encrypted_content))); + BOOST_REQUIRE (!wait_for_jobs()); + decoder = boost::dynamic_pointer_cast(player->_pieces.front()->decoder); + BOOST_REQUIRE (decoder); + BOOST_REQUIRE (reels != decoder->reels()); +} -- cgit v1.2.3