Fix some DCP generation bugs and update some tests.

6 files changed, 43 insertions, 63 deletions

diff --git a/test/dcp_test.cc b/test/dcp_test.cc
index 5a698684b..795b4dfe3 100644
--- a/test/dcp_test.cc
+++ b/test/dcp_test.cc
@@ -21,7 +21,14 @@ BOOST_AUTO_TEST_CASE (make_dcp_test)
 {
 	shared_ptr<Film> film = new_test_film ("make_dcp_test");
 	film->set_name ("test_film2");
-	film->add_content (shared_ptr<FFmpegContent> (new FFmpegContent (film, "../../../test/test.mp4")));
+	system ("pwd");
+	film->examine_and_add_content (shared_ptr<FFmpegContent> (new FFmpegContent (film, "test/test.mp4")));
+
+	/* Wait for the examine to finish */
+	while (JobManager::instance()->work_to_do ()) {
+		dcpomatic_sleep (1);
+	}
+	
 	film->set_container (Container::from_id ("185"));
 	film->set_dcp_content_type (DCPContentType::from_pretty_name ("Test"));
 	film->make_dcp ();
@@ -46,21 +53,3 @@ BOOST_AUTO_TEST_CASE (have_dcp_test)
 	boost::filesystem::remove (p);
 	BOOST_CHECK (!f.have_dcp ());
 }
-
-BOOST_AUTO_TEST_CASE (make_dcp_with_range_test)
-{
-	shared_ptr<Film> film = new_test_film ("make_dcp_with_range_test");
-	film->set_name ("test_film3");
-	film->add_content (shared_ptr<Content> (new FFmpegContent (film, "../../../test/test.mp4")));
-//	film->examine_content ();
-	film->set_container (Container::from_id ("185"));
-	film->set_dcp_content_type (DCPContentType::from_pretty_name ("Test"));
-	film->make_dcp ();
-
-	while (JobManager::instance()->work_to_do() && !JobManager::instance()->errors()) {
-		dcpomatic_sleep (1);
-	}
-
-	BOOST_CHECK_EQUAL (JobManager::instance()->errors(), false);
-}
-
diff --git a/test/film_metadata_test.cc b/test/film_metadata_test.cc
index 8309de7f1..315461b22 100644
--- a/test/film_metadata_test.cc
+++ b/test/film_metadata_test.cc
@@ -31,14 +31,13 @@ BOOST_AUTO_TEST_CASE (film_metadata_test)
 	BOOST_CHECK (f->dcp_content_type() == 0);
 
 	f->set_name ("fred");
-//	BOOST_CHECK_THROW (f->add_content ("jim"), OpenFileError);
 	f->set_dcp_content_type (DCPContentType::from_pretty_name ("Short"));
 	f->set_container (Container::from_id ("185"));
 	f->set_ab (true);
 	f->write_metadata ();
 
 	stringstream s;
-	s << "diff -u test/metadata.ref " << test_film << "/metadata";
+	s << "diff -u test/metadata.xml.ref " << test_film << "/metadata.xml";
 	BOOST_CHECK_EQUAL (::system (s.str().c_str ()), 0);
 
 	shared_ptr<Film> g (new Film (test_film));
diff --git a/test/metadata.ref b/test/metadata.ref
deleted file mode 100644
index b7a031883..000000000
--- a/test/metadata.ref
+++ /dev/null
@@ -1,42 +0,0 @@
-version 4
-name fred
-use_dci_name 1
-content 
-trust_content_header 1
-dcp_content_type SHR
-format 185
-left_crop 1
-right_crop 2
-top_crop 3
-bottom_crop 4
-filter pphb
-filter unsharp
-scaler bicubic
-trim_start 42
-trim_end 99
-trim_type cpl
-dcp_ab 1
-use_content_audio 1
-audio_gain 0
-audio_delay 0
-still_duration 10
-with_subtitles 0
-subtitle_offset 0
-subtitle_scale 1
-colour_lut 0
-j2k_bandwidth 200000000
-audio_language 
-subtitle_language 
-territory 
-rating 
-studio 
-facility 
-package_type 
-dci_date 20130211
-dcp_frame_rate 0
-width 0
-height 0
-length 0
-content_digest 
-external_audio_stream external 0 0
-source_frame_rate 0
diff --git a/test/metadata.xml.ref b/test/metadata.xml.ref
new file mode 100644
index 000000000..be791c741
--- /dev/null
+++ b/test/metadata.xml.ref
@@ -0,0 +1,31 @@
+<?xml version="1.0"?>
+<Metadata>
+  <Version>4</Version>
+  <Name>fred</Name>
+  <UseDCIName>1</UseDCIName>
+  <DCPContentType>SHR</DCPContentType>
+  <Container>185</Container>
+  <Scaler>bicubic</Scaler>
+  <AB>1</AB>
+  <WithSubtitles>0</WithSubtitles>
+  <SubtitleOffset>0</SubtitleOffset>
+  <SubtitleScale>1</SubtitleScale>
+  <ColourLUT>0</ColourLUT>
+  <J2KBandwidth>200000000</J2KBandwidth>
+  <DCIMetadata>
+    <AudioLanguage></AudioLanguage>
+    <SubtitleLanguage></SubtitleLanguage>
+    <Territory></Territory>
+    <Rating></Rating>
+    <Studio></Studio>
+    <Facility></Facility>
+    <PackageType></PackageType>
+  </DCIMetadata>
+  <DCPVideoFrameRate>0</DCPVideoFrameRate>
+  <DCIDate>20130211</DCIDate>
+  <DCPAudioChannels>6</DCPAudioChannels>
+  <Playlist>
+    <Loop>1</Loop>
+    <SequenceVideo>1</SequenceVideo>
+  </Playlist>
+</Metadata>
diff --git a/test/pixel_formats_test.cc b/test/pixel_formats_test.cc
index 08c9f2d2d..fb2278fdb 100644
--- a/test/pixel_formats_test.cc
+++ b/test/pixel_formats_test.cc
@@ -62,6 +62,7 @@ BOOST_AUTO_TEST_CASE (pixel_formats_test)
 		f->width = 640;
 		f->height = 480;
 		f->format = static_cast<int> (i->format);
+		av_frame_get_buffer (f, true);
 		SimpleImage t (f);
 		BOOST_CHECK_EQUAL(t.components(), i->components);
 		BOOST_CHECK_EQUAL(t.lines(0), i->lines[0]);
diff --git a/test/test.cc b/test/test.cc
index cbbd0443d..7e2715a02 100644
--- a/test/test.cc
+++ b/test/test.cc
@@ -63,6 +63,8 @@ struct TestConfig
 		Config::instance()->set_servers (vector<ServerDescription*> ());
 		Config::instance()->set_server_port (61920);
 		Config::instance()->set_default_dci_metadata (DCIMetadata ());
+		Config::instance()->set_default_container (0);
+		Config::instance()->set_default_dcp_content_type (0);
 	}
 };