Merge pull request firmata#365 from firmata/string-parser-test

add tests for sending and receiving strings

Author: soundanalogous

test/firmata_test/firmata_test.ino

@@ -58,6 +58,12 @@ void setupDigitalPort()
   _digitalPortValue = 0;
 }
 
+char * _receivedString;
+void handleStringCallback(char *str)
+{
+  _receivedString = str;
+}
+
 test(processWriteDigital_0)
 {
   setupDigitalPort();
@@ -134,3 +140,33 @@ test(setFirmwareVersionDoesNotLeakMemory)
 
   assertEqual(0, initialMemory - freeMemory());
 }
+
+test(sendStringShouldEncode2BytesPerChar)
+{
+  FakeStream stream;
+  Firmata.begin(stream);
+  // reset the buffer because the firmware name string will be sent on Firmata.begin
+  stream.reset();
+
+  char testString[] = "hi!";
+  Firmata.sendString(testString);
+
+  byte expected[] = { START_SYSEX, STRING_DATA, 'h', 0, 'i', 0, '!', 0, END_SYSEX };
+
+  int len = stream.bytesWritten().length();
+  assertEqual(sizeof(expected), len);
+  for (byte i = 0; i < len; i++) {
+    assertEqual(expected[i], (byte)stream.bytesWritten().charAt(i));
+  }
+}
+
+test(receivedStringShouldDecodeFrom2BytesPerChar)
+{
+  Firmata.attach(STRING_DATA, handleStringCallback);
+
+  byte message[] = { START_SYSEX, STRING_DATA, 'b', 0, 'y', 0, 'e', 0, '!', 0, END_SYSEX };
+  processMessage(message, 11);
+
+  assertEqual("bye!", _receivedString);
+}
+