rpc_next_message off-by-one, fixes #9

Author: gryphius

python/makerbotapi/makerbotapi.py

@@ -475,7 +475,7 @@ def _rpc_get_next_message(self, buffer):
                 parenindex -= 1
                 if parenindex == 0:
                     message = buffer[0:pos]
-                    return message, buffer[pos + 1:]
+                    return message, buffer[pos:]
         return None, buffer
 
     def _handle_response(self, response):

python/makerbotapi/makerbotapi_test.py

@@ -164,6 +164,16 @@ def test__get_raw_camera_image_data(self):
         tpl = self.makerbot._get_raw_camera_image_data()
         self.assertEquals(tpl[:4], (153616, 320, 240, 1))
 
+    def test__rpc_get_next_message(self):
+        self.assertEqual(
+            self.makerbot._rpc_get_next_message("{foo}"), ("{foo}", ""),)
+        self.assertEqual(
+            self.makerbot._rpc_get_next_message("{foo}{bar}"), ("{foo}", "{bar}"),)
+        self.assertEqual(self.makerbot._rpc_get_next_message(
+            "{foo}{bar{baz}}"), ("{foo}", "{bar{baz}}"),)
+        self.assertEqual(self.makerbot._rpc_get_next_message(
+            "{foo{baz}}{bar{baz}}"), ("{foo{baz}}", "{bar{baz}}"),)
+
 
 class ModuleTest(unittest.TestCase):