TextBubble: Move to front when text is set

Author: adazem009

src/scratch/textbubble.cpp

@@ -2,6 +2,7 @@
 
 #include <scratchcpp/textbubble.h>
 #include <scratchcpp/value.h>
+#include <scratchcpp/iengine.h>
 
 #include "textbubble_p.h"
 
@@ -70,6 +71,13 @@ void TextBubble::setText(const std::string &text)
     impl->text = converted.substr(0, limit);
 
     impl->textChanged(impl->text);
+
+    if (!impl->text.empty()) {
+        IEngine *eng = engine();
+
+        if (eng)
+            eng->moveDrawableToFront(this);
+    }
 }
 
 /*! Emits when the text changes. */

test/scratch_classes/CMakeLists.txt

@@ -98,7 +98,9 @@ add_executable(
 
 target_link_libraries(
   textbubble_test
-  GTest::gtest_main
+  GTest::gmock_main
+  scratchcpp
+  scratchcpp_mocks
   scratchcpp
 )
 

test/scratch_classes/textbubble_test.cpp

@@ -1,4 +1,5 @@
 #include <scratchcpp/textbubble.h>
+#include <enginemock.h>
 
 #include "../common.h"
 
@@ -36,8 +37,12 @@ TEST(TextBubbleTest, BubbleText)
     bubble.setText("hello");
     ASSERT_EQ(bubble.text(), "hello");
 
+    EngineMock engine;
+    bubble.setEngine(&engine);
+    EXPECT_CALL(engine, moveDrawableToFront(&bubble));
     bubble.setText("world");
     ASSERT_EQ(bubble.text(), "world");
+    bubble.setEngine(nullptr);
 
     // longstr.length = 384, should be limited to 330 in bubble text
     std::string longstr =