X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=test%2Fplay_test.cc;h=bbf70781a87d4600526428055228fc54809c1c62;hb=989d8622ffa3287af860574e71bb033ad6215686;hp=ed3e497ed6e3c4fc6248537ad1e98f7d22ce1241;hpb=e6f28e7cda23c1ba3c49cc1bf2dc1491c2f87160;p=dcpomatic.git

diff --git a/test/play_test.cc b/test/play_test.cc
index ed3e497ed..bbf70781a 100644
--- a/test/play_test.cc
+++ b/test/play_test.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2014 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -21,6 +21,7 @@
 #include "lib/player.h"
 #include "lib/ratio.h"
 #include "lib/dcp_content_type.h"
+#include "lib/player_video_frame.h"
 #include "test.h"
 
 /* This test needs stuff in Player that is only included in debug mode */
@@ -34,7 +35,7 @@ struct Video
 {
 	boost::shared_ptr<Content> content;
 	boost::shared_ptr<const Image> image;
-	DCPTime time;
+	Time time;
 };
 
 class PlayerWrapper
@@ -43,14 +44,14 @@ public:
 	PlayerWrapper (shared_ptr<Player> p)
 		: _player (p)
 	{
-		_player->Video.connect (bind (&PlayerWrapper::process_video, this, _1, _2, _5));
+		_player->Video.connect (bind (&PlayerWrapper::process_video, this, _1, _3));
 	}
 
-	void process_video (shared_ptr<PlayerImage> i, bool, DCPTime t)
+	void process_video (shared_ptr<PlayerVideoFrame> i, Time t)
 	{
 		Video v;
 		v.content = _player->_last_video;
-		v.image = i->image (PIX_FMT_RGB24, false);
+		v.image = i->image (PIX_FMT_RGB24);
 		v.time = t;
 		_queue.push_front (v);
 	}
@@ -67,7 +68,7 @@ public:
 		return v;
 	}
 
-	void seek (DCPTime t, bool ac)
+	void seek (Time t, bool ac)
 	{
 		_player->seek (t, ac);
 		_queue.clear ();
@@ -81,7 +82,7 @@ private:
 BOOST_AUTO_TEST_CASE (play_test)
 {
 	shared_ptr<Film> film = new_test_film ("play_test");
-	film->set_dcp_content_type (DCPContentType::from_dci_name ("FTR"));
+	film->set_dcp_content_type (DCPContentType::from_isdcf_name ("FTR"));
 	film->set_container (Ratio::from_id ("185"));
 	film->set_name ("play_test");
 
@@ -89,23 +90,25 @@ BOOST_AUTO_TEST_CASE (play_test)
 	film->examine_and_add_content (A);
 	wait_for_jobs ();
 
-	BOOST_CHECK_EQUAL (A->video_length_after_3d_combine().frames (24), 16);
+	BOOST_CHECK_EQUAL (A->video_length_after_3d_combine(), 16);
 
 	shared_ptr<FFmpegContent> B (new FFmpegContent (film, "test/data/red_30.mp4"));
 	film->examine_and_add_content (B);
 	wait_for_jobs ();
 
-	BOOST_CHECK_EQUAL (B->video_length_after_3d_combine().frames (30), 16);
+	BOOST_CHECK_EQUAL (B->video_length_after_3d_combine(), 16);
 	
 	/* Film should have been set to 25fps */
 	BOOST_CHECK_EQUAL (film->video_frame_rate(), 25);
 
-	BOOST_CHECK_EQUAL (A->position(), DCPTime ());
+	BOOST_CHECK_EQUAL (A->position(), 0);
 	/* A is 16 frames long at 25 fps */
-	BOOST_CHECK_EQUAL (B->position(), DCPTime::from_frames (16, 25));
+	BOOST_CHECK_EQUAL (B->position(), 16 * TIME_HZ / 25);
 
 	shared_ptr<Player> player = film->make_player ();
 	PlayerWrapper wrap (player);
+	/* Seek and audio don't get on at the moment */
+	player->disable_audio ();
 
 	for (int i = 0; i < 32; ++i) {
 		optional<Video> v = wrap.get_video ();
@@ -117,10 +120,10 @@ BOOST_AUTO_TEST_CASE (play_test)
 		}
 	}
 
-	player->seek (DCPTime::from_frames (6, 25), true);
+	wrap.seek (10 * TIME_HZ / 25, true);
 	optional<Video> v = wrap.get_video ();
 	BOOST_CHECK (v);
-	BOOST_CHECK_EQUAL (v.get().time, DCPTime::from_frames (6, 25));
+	BOOST_CHECK_EQUAL (v.get().time, 10 * TIME_HZ / 25);
 }
 
 #endif