Merge branch 'master' into format-fix
[dcpomatic.git] / test / test.cc
index 386aead2eca463f2b9671fcd112f6291acd5223e..4c468e22d86331a55d834f1ad8f322121835a3d5 100644 (file)
@@ -39,7 +39,7 @@
 #include "subtitle.h"
 #include "scaler.h"
 #include "ffmpeg_decoder.h"
-#include "external_audio_decoder.h"
+#include "sndfile_decoder.h"
 #define BOOST_TEST_DYN_LINK
 #define BOOST_TEST_MODULE dvdomatic_test
 #include <boost/test/unit_test.hpp>
@@ -94,9 +94,19 @@ BOOST_AUTO_TEST_CASE (make_black_test)
        libdcp::Size in_size (512, 512);
        libdcp::Size out_size (1024, 1024);
 
-       {
-               /* Plain RGB input */
-               boost::shared_ptr<Image> foo (new SimpleImage (AV_PIX_FMT_RGB24, in_size, true));
+       list<AVPixelFormat> pix_fmts;
+       pix_fmts.push_back (AV_PIX_FMT_RGB24);
+       pix_fmts.push_back (AV_PIX_FMT_YUV420P);
+       pix_fmts.push_back (AV_PIX_FMT_YUV422P10LE);
+       pix_fmts.push_back (AV_PIX_FMT_YUV444P9LE);
+       pix_fmts.push_back (AV_PIX_FMT_YUV444P9BE);
+       pix_fmts.push_back (AV_PIX_FMT_YUV444P10LE);
+       pix_fmts.push_back (AV_PIX_FMT_YUV444P10BE);
+       pix_fmts.push_back (AV_PIX_FMT_UYVY422);
+
+       int N = 0;
+       for (list<AVPixelFormat>::const_iterator i = pix_fmts.begin(); i != pix_fmts.end(); ++i) {
+               boost::shared_ptr<Image> foo (new SimpleImage (*i, in_size, true));
                foo->make_black ();
                boost::shared_ptr<Image> bar = foo->scale_and_convert_to_rgb (out_size, 0, Scaler::from_id ("bicubic"), true);
                
@@ -108,38 +118,8 @@ BOOST_AUTO_TEST_CASE (make_black_test)
                        }
                        p += bar->stride()[0];
                }
-       }
 
-       {
-               /* YUV420P input */
-               boost::shared_ptr<Image> foo (new SimpleImage (AV_PIX_FMT_YUV420P, in_size, true));
-               foo->make_black ();
-               boost::shared_ptr<Image> bar = foo->scale_and_convert_to_rgb (out_size, 0, Scaler::from_id ("bicubic"), true);
-               
-               uint8_t* p = bar->data()[0];
-               for (int y = 0; y < bar->size().height; ++y) {
-                       uint8_t* q = p;
-                       for (int x = 0; x < bar->line_size()[0]; ++x) {
-                               BOOST_CHECK_EQUAL (*q++, 0);
-                       }
-                       p += bar->stride()[0];
-               }
-       }
-
-       {
-               /* YUV422P10LE input */
-               boost::shared_ptr<Image> foo (new SimpleImage (AV_PIX_FMT_YUV422P10LE, in_size, true));
-               foo->make_black ();
-               boost::shared_ptr<Image> bar = foo->scale_and_convert_to_rgb (out_size, 0, Scaler::from_id ("bicubic"), true);
-               
-               uint8_t* p = bar->data()[0];
-               for (int y = 0; y < bar->size().height; ++y) {
-                       uint8_t* q = p;
-                       for (int x = 0; x < bar->line_size()[0]; ++x) {
-                               BOOST_CHECK_EQUAL (*q++, 0);
-                       }
-                       p += bar->stride()[0];
-               }
+               ++N;
        }
 }
 
@@ -212,7 +192,7 @@ BOOST_AUTO_TEST_CASE (stream_test)
        BOOST_CHECK_EQUAL (a.name(), "hello there world");
        BOOST_CHECK_EQUAL (a.to_string(), "ffmpeg 4 44100 1 hello there world");
 
-       ExternalAudioStream e ("external 44100 1", boost::optional<int> (1));
+       SndfileStream e ("external 44100 1", boost::optional<int> (1));
        BOOST_CHECK_EQUAL (e.sample_rate(), 44100);
        BOOST_CHECK_EQUAL (e.channel_layout(), 1);
        BOOST_CHECK_EQUAL (e.to_string(), "external 44100 1");
@@ -242,11 +222,39 @@ BOOST_AUTO_TEST_CASE (format_test)
        
        Format const * f = Format::from_nickname ("Flat");
        BOOST_CHECK (f);
-       BOOST_CHECK_EQUAL (f->ratio_as_integer(shared_ptr<const Film> ()), 185);
+       BOOST_CHECK_EQUAL (f->dcp_size().width, 1998);
+       BOOST_CHECK_EQUAL (f->dcp_size().height, 1080);
        
        f = Format::from_nickname ("Scope");
        BOOST_CHECK (f);
-       BOOST_CHECK_EQUAL (f->ratio_as_integer(shared_ptr<const Film> ()), 239);
+       BOOST_CHECK_EQUAL (f->dcp_size().width, 2048);
+       BOOST_CHECK_EQUAL (f->dcp_size().height, 858);
+}
+
+/* Test VariableFormat-based scaling of content */
+BOOST_AUTO_TEST_CASE (scaling_test)
+{
+       shared_ptr<Film> film (new Film (test_film_dir ("scaling_test").string(), false));
+
+       /* 4:3 ratio */
+       film->set_size (libdcp::Size (320, 240));
+
+       /* This format should preserve aspect ratio of the source */
+       Format const * format = Format::from_id ("var-185");
+
+       /* We should have enough padding that the result is 4:3,
+          which would be 1440 pixels.
+       */
+       BOOST_CHECK_EQUAL (format->dcp_padding (film), (1998 - 1440) / 2);
+       
+       /* This crops it to 1.291666667 */
+       film->set_left_crop (5);
+       film->set_right_crop (5);
+
+       /* We should now have enough padding that the result is 1.29166667,
+          which would be 1395 pixels.
+       */
+       BOOST_CHECK_EQUAL (format->dcp_padding (film), rint ((1998 - 1395) / 2.0));
 }
 
 BOOST_AUTO_TEST_CASE (util_test)
@@ -275,9 +283,9 @@ public:
 void
 do_positive_delay_line_test (int delay_length, int data_length)
 {
-       NullLog log;
+       shared_ptr<NullLog> log (new NullLog);
        
-       DelayLine d (&log, 6, delay_length);
+       DelayLine d (log, 6, delay_length);
        shared_ptr<AudioBuffers> data (new AudioBuffers (6, data_length));
 
        int in = 0;
@@ -318,9 +326,9 @@ do_positive_delay_line_test (int delay_length, int data_length)
 void
 do_negative_delay_line_test (int delay_length, int data_length)
 {
-       NullLog log;
+       shared_ptr<NullLog> log (new NullLog);
 
-       DelayLine d (&log, 6, delay_length);
+       DelayLine d (log, 6, delay_length);
        shared_ptr<AudioBuffers> data (new AudioBuffers (6, data_length));
 
        int in = 0;
@@ -427,7 +435,7 @@ BOOST_AUTO_TEST_CASE (client_server_test)
 
        shared_ptr<Subtitle> subtitle (new Subtitle (Position (50, 60), sub_image));
 
-       FileLog log ("build/test/client_server_test.log");
+       shared_ptr<FileLog> log (new FileLog ("build/test/client_server_test.log"));
 
        shared_ptr<DCPVideoFrame> frame (
                new DCPVideoFrame (
@@ -443,14 +451,14 @@ BOOST_AUTO_TEST_CASE (client_server_test)
                        "",
                        0,
                        200000000,
-                       &log
+                       log
                        )
                );
 
        shared_ptr<EncodedData> locally_encoded = frame->encode_locally ();
        BOOST_ASSERT (locally_encoded);
        
-       Server* server = new Server (&log);
+       Server* server = new Server (log);
 
        new thread (boost::bind (&Server::run, server, 2));
 
@@ -498,7 +506,7 @@ BOOST_AUTO_TEST_CASE (have_dcp_test)
        BOOST_CHECK (f.have_dcp());
 
        p /= f.dcp_name();
-       p /= "video.mxf";
+       p /= f.dcp_video_mxf_filename();
        boost::filesystem::remove (p);
        BOOST_CHECK (!f.have_dcp ());
 }
@@ -521,8 +529,8 @@ BOOST_AUTO_TEST_CASE (make_dcp_with_range_test)
        BOOST_CHECK_EQUAL (JobManager::instance()->errors(), false);
 }
 
-/* Test the constructor of DCPFrameRate */
-BOOST_AUTO_TEST_CASE (dcp_frame_rate_test)
+/* Test best_dcp_frame_rate and FrameRateConversion */
+BOOST_AUTO_TEST_CASE (best_dcp_frame_rate_test)
 {
        /* Run some tests with a limited range of allowed rates */
        
@@ -532,71 +540,82 @@ BOOST_AUTO_TEST_CASE (dcp_frame_rate_test)
        afr.push_back (30);
        Config::instance()->set_allowed_dcp_frame_rates (afr);
 
-       DCPFrameRate dfr = DCPFrameRate (60);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 30);
-       BOOST_CHECK_EQUAL (dfr.skip, true);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
+       int best = best_dcp_frame_rate (60);
+       FrameRateConversion frc = FrameRateConversion (60, best);
+       BOOST_CHECK_EQUAL (best, 30);
+       BOOST_CHECK_EQUAL (frc.skip, true);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
        
-       dfr = DCPFrameRate (50);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 25);
-       BOOST_CHECK_EQUAL (dfr.skip, true);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
-
-       dfr = DCPFrameRate (48);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 24);
-       BOOST_CHECK_EQUAL (dfr.skip, true);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
+       best = best_dcp_frame_rate (50);
+       frc = FrameRateConversion (50, best);
+       BOOST_CHECK_EQUAL (best, 25);
+       BOOST_CHECK_EQUAL (frc.skip, true);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
+
+       best = best_dcp_frame_rate (48);
+       frc = FrameRateConversion (48, best);
+       BOOST_CHECK_EQUAL (best, 24);
+       BOOST_CHECK_EQUAL (frc.skip, true);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
        
-       dfr = DCPFrameRate (30);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 30);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
-
-       dfr = DCPFrameRate (29.97);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 30);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, true);
+       best = best_dcp_frame_rate (30);
+       frc = FrameRateConversion (30, best);
+       BOOST_CHECK_EQUAL (best, 30);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
+
+       best = best_dcp_frame_rate (29.97);
+       frc = FrameRateConversion (29.97, best);
+       BOOST_CHECK_EQUAL (best, 30);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, true);
        
-       dfr = DCPFrameRate (25);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 25);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
-
-       dfr = DCPFrameRate (24);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 24);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
-
-       dfr = DCPFrameRate (14.5);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 30);
-       BOOST_CHECK_EQUAL (dfr.repeat, true);
-       BOOST_CHECK_EQUAL (dfr.change_speed, true);
-
-       dfr = DCPFrameRate (12.6);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 25);
-       BOOST_CHECK_EQUAL (dfr.repeat, true);
-       BOOST_CHECK_EQUAL (dfr.change_speed, true);
-
-       dfr = DCPFrameRate (12.4);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 25);
-       BOOST_CHECK_EQUAL (dfr.repeat, true);
-       BOOST_CHECK_EQUAL (dfr.change_speed, true);
-
-       dfr = DCPFrameRate (12);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 24);
-       BOOST_CHECK_EQUAL (dfr.repeat, true);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
+       best = best_dcp_frame_rate (25);
+       frc = FrameRateConversion (25, best);
+       BOOST_CHECK_EQUAL (best, 25);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
+
+       best = best_dcp_frame_rate (24);
+       frc = FrameRateConversion (24, best);
+       BOOST_CHECK_EQUAL (best, 24);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
+
+       best = best_dcp_frame_rate (14.5);
+       frc = FrameRateConversion (14.5, best);
+       BOOST_CHECK_EQUAL (best, 30);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, true);
+       BOOST_CHECK_EQUAL (frc.change_speed, true);
+
+       best = best_dcp_frame_rate (12.6);
+       frc = FrameRateConversion (12.6, best);
+       BOOST_CHECK_EQUAL (best, 25);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, true);
+       BOOST_CHECK_EQUAL (frc.change_speed, true);
+
+       best = best_dcp_frame_rate (12.4);
+       frc = FrameRateConversion (12.4, best);
+       BOOST_CHECK_EQUAL (best, 25);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, true);
+       BOOST_CHECK_EQUAL (frc.change_speed, true);
+
+       best = best_dcp_frame_rate (12);
+       frc = FrameRateConversion (12, best);
+       BOOST_CHECK_EQUAL (best, 24);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, true);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
 
        /* Now add some more rates and see if it will use them
           in preference to skip/repeat.
@@ -607,29 +626,59 @@ BOOST_AUTO_TEST_CASE (dcp_frame_rate_test)
        afr.push_back (60);
        Config::instance()->set_allowed_dcp_frame_rates (afr);
 
-       dfr = DCPFrameRate (60);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 60);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
+       best = best_dcp_frame_rate (60);
+       frc = FrameRateConversion (60, best);
+       BOOST_CHECK_EQUAL (best, 60);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
        
-       dfr = DCPFrameRate (50);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 50);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
-
-       dfr = DCPFrameRate (48);
-       BOOST_CHECK_EQUAL (dfr.frames_per_second, 48);
-       BOOST_CHECK_EQUAL (dfr.skip, false);
-       BOOST_CHECK_EQUAL (dfr.repeat, false);
-       BOOST_CHECK_EQUAL (dfr.change_speed, false);
+       best = best_dcp_frame_rate (50);
+       frc = FrameRateConversion (50, best);
+       BOOST_CHECK_EQUAL (best, 50);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
+
+       best = best_dcp_frame_rate (48);
+       frc = FrameRateConversion (48, best);
+       BOOST_CHECK_EQUAL (best, 48);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, false);
+
+       /* Check some out-there conversions (not the best) */
+       
+       frc = FrameRateConversion (14.99, 24);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, true);
+       BOOST_CHECK_EQUAL (frc.change_speed, true);
+
+       /* Check some conversions with limited DCP targets */
+
+       afr.clear ();
+       afr.push_back (24);
+       Config::instance()->set_allowed_dcp_frame_rates (afr);
+
+       best = best_dcp_frame_rate (25);
+       frc = FrameRateConversion (25, best);
+       BOOST_CHECK_EQUAL (best, 24);
+       BOOST_CHECK_EQUAL (frc.skip, false);
+       BOOST_CHECK_EQUAL (frc.repeat, false);
+       BOOST_CHECK_EQUAL (frc.change_speed, true);
 }
 
 BOOST_AUTO_TEST_CASE (audio_sampling_rate_test)
 {
+       std::list<int> afr;
+       afr.push_back (24);
+       afr.push_back (25);
+       afr.push_back (30);
+       Config::instance()->set_allowed_dcp_frame_rates (afr);
+
        shared_ptr<Film> f = new_test_film ("audio_sampling_rate_test");
-       f->set_frames_per_second (24);
+       f->set_source_frame_rate (24);
+       f->set_dcp_frame_rate (24);
 
        f->set_content_audio_stream (shared_ptr<AudioStream> (new FFmpegAudioStream ("a", 42, 48000, 0)));
        BOOST_CHECK_EQUAL (f->target_audio_sample_rate(), 48000);
@@ -640,13 +689,36 @@ BOOST_AUTO_TEST_CASE (audio_sampling_rate_test)
        f->set_content_audio_stream (shared_ptr<AudioStream> (new FFmpegAudioStream ("a", 42, 80000, 0)));
        BOOST_CHECK_EQUAL (f->target_audio_sample_rate(), 96000);
 
-       f->set_frames_per_second (23.976);
+       f->set_source_frame_rate (23.976);
+       f->set_dcp_frame_rate (best_dcp_frame_rate (23.976));
        f->set_content_audio_stream (shared_ptr<AudioStream> (new FFmpegAudioStream ("a", 42, 48000, 0)));
        BOOST_CHECK_EQUAL (f->target_audio_sample_rate(), 47952);
 
-       f->set_frames_per_second (29.97);
+       f->set_source_frame_rate (29.97);
+       f->set_dcp_frame_rate (best_dcp_frame_rate (29.97));
+       BOOST_CHECK_EQUAL (f->dcp_frame_rate (), 30);
        f->set_content_audio_stream (shared_ptr<AudioStream> (new FFmpegAudioStream ("a", 42, 48000, 0)));
        BOOST_CHECK_EQUAL (f->target_audio_sample_rate(), 47952);
+
+       f->set_source_frame_rate (25);
+       f->set_dcp_frame_rate (24);
+       f->set_content_audio_stream (shared_ptr<AudioStream> (new FFmpegAudioStream ("a", 42, 48000, 0)));
+       BOOST_CHECK_EQUAL (f->target_audio_sample_rate(), 50000);
+
+       f->set_source_frame_rate (25);
+       f->set_dcp_frame_rate (24);
+       f->set_content_audio_stream (shared_ptr<AudioStream> (new FFmpegAudioStream ("a", 42, 44100, 0)));
+       BOOST_CHECK_EQUAL (f->target_audio_sample_rate(), 50000);
+
+       /* Check some out-there conversions (not the best) */
+       
+       f->set_source_frame_rate (14.99);
+       f->set_dcp_frame_rate (25);
+       f->set_content_audio_stream (shared_ptr<AudioStream> (new FFmpegAudioStream ("a", 42, 16000, 0)));
+       /* The FrameRateConversion within target_audio_sample_rate should choose to double-up
+          the 14.99 fps video to 30 and then run it slow at 25.
+       */
+       BOOST_CHECK_EQUAL (f->target_audio_sample_rate(), rint (48000 * 2 * 14.99 / 25));
 }
 
 class TestJob : public Job