transport: send correct time signature info to juce playhead

Author: mfisher31

src/engine/transport.cpp

@@ -122,7 +122,7 @@ void Transport::postProcess (int nframes)
     if (ts.beatDivisor() != nextBeatDivisor.get())
     {
         ts.setBeatType ((unsigned short) nextBeatDivisor.get());
-        ts.setBeatDivisor (1 << ts.beatType());
+        ts.setBeatDivisor ((unsigned short) nextBeatDivisor.get());
         monitor->beatDivisor.set (nextBeatDivisor.get());
         updateTimeScale = true;
     }

test/CMakeLists.txt

@@ -49,6 +49,7 @@ add_test(NAME "ScriptManagerTest" COMMAND test_element --run_test=ScriptManagerT
 add_test(NAME "ScriptPlayground" COMMAND test_element --run_test=ScriptPlayground)
 add_test(NAME "SessionChangedTest" COMMAND test_element --run_test=SessionChangedTest)
 add_test(NAME "ShuttleTests" COMMAND test_element --run_test=ShuttleTests)
+add_test(NAME "TransportTests" COMMAND test_element --run_test=TransportTests)
 add_test(NAME "ToggleGridTest" COMMAND test_element --run_test=ToggleGridTest)
 add_test(NAME "UpdateTests" COMMAND test_element --run_test=UpdateTests)
 add_test(NAME "VelocityCurveTest" COMMAND test_element --run_test=VelocityCurveTest)

test/transporttests.cpp

@@ -0,0 +1,32 @@
+// SPDX-FileCopyrightText: Copyright (C) Kushview, LLC.
+// SPDX-License-Identifier: GPL-3.0-or-later
+
+#include <boost/test/unit_test.hpp>
+#include <element/transport.hpp>
+#include "tempo.hpp"
+
+using element::Transport;
+
+BOOST_AUTO_TEST_SUITE (TransportTests)
+
+BOOST_AUTO_TEST_CASE (publishesCorrectTimeSignatureDenominator)
+{
+    Transport transport;
+
+    auto pos = transport.getPosition();
+    BOOST_REQUIRE (pos.hasValue());
+    BOOST_REQUIRE (pos->getTimeSignature().hasValue());
+    BOOST_REQUIRE_EQUAL (pos->getTimeSignature()->numerator, 4);
+    BOOST_REQUIRE_EQUAL (pos->getTimeSignature()->denominator, 4);
+
+    transport.requestMeter (7, element::BeatType::EighthNote);
+    transport.postProcess (0);
+
+    pos = transport.getPosition();
+    BOOST_REQUIRE (pos.hasValue());
+    BOOST_REQUIRE (pos->getTimeSignature().hasValue());
+    BOOST_REQUIRE_EQUAL (pos->getTimeSignature()->numerator, 7);
+    BOOST_REQUIRE_EQUAL (pos->getTimeSignature()->denominator, 8);
+}
+
+BOOST_AUTO_TEST_SUITE_END()