fix #479: Add bubble text length limit

Author: Jessie-JY2728

src/scratch/target.cpp

@@ -466,7 +466,8 @@ const std::string &Target::bubbleText() const
  */
 void Target::setBubbleText(const std::string &text)
 {
-    impl->bubbleText = text;
+    size_t limit = 330;
+    impl->bubbleText = text.substr(0, limit);
 }
 
 /*! Returns the engine. */

test/scratch_classes/target_test.cpp

@@ -603,6 +603,25 @@ TEST(TargetTest, BubbleText)
 
     target.setBubbleText("world");
     ASSERT_EQ(target.bubbleText(), "world");
+
+    // longstr.length = 384, should be limited to 330 in bubble text
+    std::string longstr =
+        "EY8OUNzAqwgh7NRGk5TzCP3dkAhJy9TX"
+        "Y9mqKElPjdQpKddYqjyCwUk2hx6YgVZV"
+        "6BOdmZGxDMs8Hjv8W9G6j4gTxAWdOkzs"
+        "8Ih80xzEDbvLilWsDwoB6FxH2kVVI4xs"
+        "IXOETNQ6QMsCKLWc5XjHk2BS9nYvDGpJ"
+        "uEmp9zIzFGT1kRSrOlU3ZwnN1YtvqFx"
+        "3hkWVNtJ71dQ0PJHhOVQPUy19V01SPu3"
+        "KIIS2wdSUVAc4RYMzepSveghzWbdcizy"
+        "Tm1KKAj4svu9YoL8b9vsolG8gKunvKO7"
+        "MurRKSeUbECELnJEKV6683xCq7RvmjAu"
+        "2djZ54apiQc1lTixWns5GoG0SVNuFzHl"
+        "q97qUiqiMecjVFM51YVif7c1Stip52Hl";
+
+    target.setBubbleText(longstr);
+    ASSERT_EQ(target.bubbleText().length(), 330);
+    ASSERT_EQ(target.bubbleText(), longstr.substr(0, 330));
 }
 
 TEST(TargetTest, Engine)