Request redraw when stage visual properties change

Author: adazem009

src/scratch/target.cpp

@@ -4,6 +4,7 @@
 #include <scratchcpp/variable.h>
 #include <scratchcpp/list.h>
 #include <scratchcpp/block.h>
+#include <scratchcpp/iengine.h>
 
 #include "target_p.h"
 
@@ -209,6 +210,11 @@ void Target::setCostumeIndex(int newCostumeIndex)
 {
     if (newCostumeIndex >= 0 && newCostumeIndex < costumes().size())
         impl->costumeIndex = newCostumeIndex;
+
+    if (isStage()) {
+        if (impl->engine)
+            impl->engine->breakFrame();
+    }
 }
 
 /*! Returns the currently set costume. */

test/scratch_classes/CMakeLists.txt

@@ -51,7 +51,9 @@ add_executable(
 target_link_libraries(
   stage_test
   GTest::gtest_main
+  GTest::gmock_main
   scratchcpp
+  scratchcpp_mocks
 )
 
 gtest_discover_tests(stage_test)

test/scratch_classes/stage_test.cpp

@@ -1,4 +1,6 @@
 #include <scratchcpp/stage.h>
+#include <scratchcpp/costume.h>
+#include <enginemock.h>
 
 #include "../common.h"
 
@@ -10,6 +12,25 @@ TEST(StageTest, IsStage)
     ASSERT_TRUE(stage.isStage());
 }
 
+TEST(SpriteTest, CostumeIndex)
+{
+    Stage stage;
+    EngineMock engine;
+    stage.setEngine(&engine);
+    EXPECT_CALL(engine, breakFrame()).Times(2);
+
+    auto c1 = std::make_shared<Costume>("", "", "");
+    auto c2 = std::make_shared<Costume>("", "", "");
+    stage.addCostume(c1);
+    stage.addCostume(c2);
+
+    stage.setCostumeIndex(0);
+    ASSERT_EQ(stage.costumeIndex(), 0);
+
+    stage.setCostumeIndex(1);
+    ASSERT_EQ(stage.costumeIndex(), 1);
+}
+
 TEST(StageTest, Tempo)
 {
     Stage stage;