Cleanup: test tidying.
[dcpomatic.git] / test / shuffler_test.cc
index 879f2e079ade4145b35730b1e4f1820014c54df4..2099a0923dfdd052439900e07f333718d212f78a 100644 (file)
@@ -1,17 +1,44 @@
-#include "lib/shuffler.h"
-#include "lib/piece.h"
+/*
+    Copyright (C) 2020-2021 Carl Hetherington <cth@carlh.net>
+
+    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 <http://www.gnu.org/licenses/>.
+
+*/
+
+
 #include "lib/content_video.h"
+#include "lib/piece.h"
+#include "lib/shuffler.h"
 #include <boost/test/unit_test.hpp>
 
+
 using std::list;
-using boost::shared_ptr;
-using boost::weak_ptr;
+using std::make_shared;
+using std::shared_ptr;
+using std::weak_ptr;
 using boost::optional;
+#if BOOST_VERSION >= 106100
+using namespace boost::placeholders;
+#endif
+
 
 static void
 push (Shuffler& s, int frame, Eyes eyes)
 {
-       shared_ptr<Piece> piece (new Piece (shared_ptr<Content>(), shared_ptr<Decoder>(), FrameRateChange(24, 24)));
+       auto piece = make_shared<Piece>(shared_ptr<Content>(), shared_ptr<Decoder>(), FrameRateChange(24, 24));
        ContentVideo cv;
        cv.frame = frame;
        cv.eyes = eyes;
@@ -42,10 +69,10 @@ BOOST_AUTO_TEST_CASE (shuffler_test1)
        s.Video.connect (boost::bind (&receive, _1, _2));
 
        for (int i = 0; i < 10; ++i) {
-               push (s, i, EYES_LEFT);
-               push (s, i, EYES_RIGHT);
-               check (i, EYES_LEFT, __LINE__);
-               check (i, EYES_RIGHT, __LINE__);
+               push (s, i, Eyes::LEFT);
+               push (s, i, Eyes::RIGHT);
+               check (i, Eyes::LEFT, __LINE__);
+               check (i, Eyes::RIGHT, __LINE__);
        }
 }
 
@@ -56,14 +83,14 @@ BOOST_AUTO_TEST_CASE (shuffler_test2)
        s.Video.connect (boost::bind (&receive, _1, _2));
 
        for (int i = 0; i < 10; i += 2) {
-               push (s, i, EYES_LEFT);
-               push (s, i + 1, EYES_LEFT);
-               push (s, i, EYES_RIGHT);
-               push (s, i + 1, EYES_RIGHT);
-               check (i, EYES_LEFT, __LINE__);
-               check (i, EYES_RIGHT, __LINE__);
-               check (i + 1, EYES_LEFT, __LINE__);
-               check (i + 1, EYES_RIGHT, __LINE__);
+               push (s, i, Eyes::LEFT);
+               push (s, i + 1, Eyes::LEFT);
+               push (s, i, Eyes::RIGHT);
+               push (s, i + 1, Eyes::RIGHT);
+               check (i, Eyes::LEFT, __LINE__);
+               check (i, Eyes::RIGHT, __LINE__);
+               check (i + 1, Eyes::LEFT, __LINE__);
+               check (i + 1, Eyes::RIGHT, __LINE__);
        }
 }
 
@@ -73,25 +100,25 @@ BOOST_AUTO_TEST_CASE (shuffler_test3)
        Shuffler s;
        s.Video.connect (boost::bind (&receive, _1, _2));
 
-       push (s, 0, EYES_LEFT);
-       check (0, EYES_LEFT, __LINE__);
-       push (s, 0, EYES_RIGHT);
-       check (0, EYES_RIGHT, __LINE__);
-       push (s, 1, EYES_LEFT);
-       check (1, EYES_LEFT, __LINE__);
-       push (s, 1, EYES_RIGHT);
-       check (1, EYES_RIGHT, __LINE__);
-       push (s, 2, EYES_RIGHT);
-       push (s, 3, EYES_LEFT);
-       push (s, 3, EYES_RIGHT);
-       push (s, 4, EYES_LEFT);
-       push (s, 4, EYES_RIGHT);
+       push (s, 0, Eyes::LEFT);
+       check (0, Eyes::LEFT, __LINE__);
+       push (s, 0, Eyes::RIGHT);
+       check (0, Eyes::RIGHT, __LINE__);
+       push (s, 1, Eyes::LEFT);
+       check (1, Eyes::LEFT, __LINE__);
+       push (s, 1, Eyes::RIGHT);
+       check (1, Eyes::RIGHT, __LINE__);
+       push (s, 2, Eyes::RIGHT);
+       push (s, 3, Eyes::LEFT);
+       push (s, 3, Eyes::RIGHT);
+       push (s, 4, Eyes::LEFT);
+       push (s, 4, Eyes::RIGHT);
        s.flush ();
-       check (2, EYES_RIGHT, __LINE__);
-       check (3, EYES_LEFT, __LINE__);
-       check (3, EYES_RIGHT, __LINE__);
-       check (4, EYES_LEFT, __LINE__);
-       check (4, EYES_RIGHT, __LINE__);
+       check (2, Eyes::RIGHT, __LINE__);
+       check (3, Eyes::LEFT, __LINE__);
+       check (3, Eyes::RIGHT, __LINE__);
+       check (4, Eyes::LEFT, __LINE__);
+       check (4, Eyes::RIGHT, __LINE__);
 }
 
 /** One missing right eye image */
@@ -100,23 +127,70 @@ BOOST_AUTO_TEST_CASE (shuffler_test4)
        Shuffler s;
        s.Video.connect (boost::bind (&receive, _1, _2));
 
-       push (s, 0, EYES_LEFT);
-       check (0, EYES_LEFT, __LINE__);
-       push (s, 0, EYES_RIGHT);
-       check (0, EYES_RIGHT, __LINE__);
-       push (s, 1, EYES_LEFT);
-       check (1, EYES_LEFT, __LINE__);
-       push (s, 1, EYES_RIGHT);
-       check (1, EYES_RIGHT, __LINE__);
-       push (s, 2, EYES_LEFT);
-       push (s, 3, EYES_LEFT);
-       push (s, 3, EYES_RIGHT);
-       push (s, 4, EYES_LEFT);
-       push (s, 4, EYES_RIGHT);
+       push (s, 0, Eyes::LEFT);
+       check (0, Eyes::LEFT, __LINE__);
+       push (s, 0, Eyes::RIGHT);
+       check (0, Eyes::RIGHT, __LINE__);
+       push (s, 1, Eyes::LEFT);
+       check (1, Eyes::LEFT, __LINE__);
+       push (s, 1, Eyes::RIGHT);
+       check (1, Eyes::RIGHT, __LINE__);
+       push (s, 2, Eyes::LEFT);
+       push (s, 3, Eyes::LEFT);
+       push (s, 3, Eyes::RIGHT);
+       push (s, 4, Eyes::LEFT);
+       push (s, 4, Eyes::RIGHT);
        s.flush ();
-       check (2, EYES_LEFT, __LINE__);
-       check (3, EYES_LEFT, __LINE__);
-       check (3, EYES_RIGHT, __LINE__);
-       check (4, EYES_LEFT, __LINE__);
-       check (4, EYES_RIGHT, __LINE__);
+       check (2, Eyes::LEFT, __LINE__);
+       check (3, Eyes::LEFT, __LINE__);
+       check (3, Eyes::RIGHT, __LINE__);
+       check (4, Eyes::LEFT, __LINE__);
+       check (4, Eyes::RIGHT, __LINE__);
+}
+
+/** Only one eye */
+BOOST_AUTO_TEST_CASE (shuffler_test5)
+{
+       Shuffler s;
+       s.Video.connect (boost::bind (&receive, _1, _2));
+
+       /* One left should come out straight away */
+       push (s, 0, Eyes::LEFT);
+       check (0, Eyes::LEFT, __LINE__);
+
+       /* More lefts should be kept in the shuffler in the hope that some rights arrive */
+       for (int i = 0; i < s._max_size; ++i) {
+               push (s, i + 1, Eyes::LEFT);
+       }
+       BOOST_CHECK (pending_cv.empty ());
+
+       /* If enough lefts come the shuffler should conclude that there's no rights and start
+          giving out the lefts.
+       */
+       push (s, s._max_size + 1, Eyes::LEFT);
+       check (1, Eyes::LEFT, __LINE__);
+}
+
+/** One complete frame (L+R) missing.
+    Shuffler should carry on, skipping this frame, as the player will cope with it.
+*/
+BOOST_AUTO_TEST_CASE (shuffler_test6)
+{
+       Shuffler s;
+       s.Video.connect (boost::bind (&receive, _1, _2));
+
+       push (s, 0, Eyes::LEFT);
+       check (0, Eyes::LEFT, __LINE__);
+       push (s, 0, Eyes::RIGHT);
+       check (0, Eyes::RIGHT, __LINE__);
+
+       push (s, 2, Eyes::LEFT);
+       push (s, 2, Eyes::RIGHT);
+       check (2, Eyes::LEFT, __LINE__);
+       check (2, Eyes::RIGHT, __LINE__);
+
+       push (s, 3, Eyes::LEFT);
+       check (3, Eyes::LEFT, __LINE__);
+       push (s, 3, Eyes::RIGHT);
+       check (3, Eyes::RIGHT, __LINE__);
 }