From 037b9fb9f2eb52195722be3accb5a33acfdaac35 Mon Sep 17 00:00:00 2001 From: Francois Best Date: Fri, 28 Oct 2016 09:15:56 +0200 Subject: [PATCH] Added test for new sendAfterTouch signature. --- .../tests/unit-tests_MidiOutput.cpp | 21 ++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/test/unit-tests/tests/unit-tests_MidiOutput.cpp b/test/unit-tests/tests/unit-tests_MidiOutput.cpp index 3c2b43c..59cd993 100644 --- a/test/unit-tests/tests/unit-tests_MidiOutput.cpp +++ b/test/unit-tests/tests/unit-tests_MidiOutput.cpp @@ -252,6 +252,10 @@ TEST(MidiOutput, sendPitchBend) TEST(MidiOutput, sendPolyPressure) { + // Note: sendPolyPressure is deprecated in favor of sendAfterTouch, which + // now supports both mono and poly AfterTouch messages. + // This test is kept for coverage until removal of sendPolyPressure. + SerialMock serial; MidiInterface midi(serial); Buffer buffer; @@ -265,7 +269,7 @@ TEST(MidiOutput, sendPolyPressure) EXPECT_THAT(buffer, ElementsAreArray({0xab, 42, 12, 0xa3, 47, 12})); } -TEST(MidiOutput, sendAfterTouch) +TEST(MidiOutput, sendAfterTouchMono) { SerialMock serial; MidiInterface midi(serial); @@ -280,6 +284,21 @@ TEST(MidiOutput, sendAfterTouch) EXPECT_THAT(buffer, ElementsAreArray({0xdb, 42, 0xd3, 47})); } +TEST(MidiOutput, sendAfterTouchPoly) +{ + SerialMock serial; + MidiInterface midi(serial); + Buffer buffer; + buffer.resize(6); + + midi.begin(); + midi.sendAfterTouch(42, 12, 12); + midi.sendAfterTouch(47, 12, 4); + EXPECT_EQ(serial.mTxBuffer.getLength(), 6); + serial.mTxBuffer.read(&buffer[0], 6); + EXPECT_THAT(buffer, ElementsAreArray({0xab, 42, 12, 0xa3, 47, 12})); +} + TEST(MidiOutput, sendSysEx) { typedef test_mocks::SerialMock<1024> SerialMock;