Add onCloned method to sprite interface

Author: adazem009

include/scratchcpp/ispritehandler.h

@@ -16,6 +16,8 @@ class LIBSCRATCHCPP_EXPORT ISpriteHandler
 
         virtual void onSpriteChanged(Sprite *sprite) = 0;
 
+        virtual void onCloned(Sprite *clone) = 0;
+
         virtual void onCostumeChanged(const char *data) = 0;
 
         virtual void onVisibleChanged(bool visible) = 0;

src/scratch/sprite.cpp

@@ -85,6 +85,9 @@ std::shared_ptr<Sprite> Sprite::clone()
         // Call "when I start as clone" scripts
         eng->initClone(clone.get());
 
+        if (impl->iface)
+            impl->iface->onCloned(clone.get());
+
         return clone;
     }
 

test/mocks/spritehandlermock.h

@@ -10,6 +10,8 @@ class SpriteHandlerMock : public ISpriteHandler
     public:
         MOCK_METHOD(void, onSpriteChanged, (Sprite *), (override));
 
+        MOCK_METHOD(void, onCloned, (Sprite *), (override));
+
         MOCK_METHOD(void, onCostumeChanged, (const char *), (override));
 
         MOCK_METHOD(void, onVisibleChanged, (bool), (override));

test/target_interfaces/ispritehandler_test.cpp

@@ -1,22 +1,41 @@
 #include <spritehandlermock.h>
+#include <enginemock.h>
 
 #include "../common.h"
 
 using namespace libscratchcpp;
 
+using ::testing::Return;
+using ::testing::_;
+using ::testing::SaveArg;
+
 class ISpriteHandlerTest : public testing::Test
 {
     public:
         void SetUp() override
         {
             EXPECT_CALL(m_handler, onSpriteChanged(&m_sprite)).Times(1);
             m_sprite.setInterface(&m_handler);
+            m_sprite.setEngine(&m_engine);
         }
 
         Sprite m_sprite;
         SpriteHandlerMock m_handler;
+        EngineMock m_engine;
 };
 
+TEST_F(ISpriteHandlerTest, Clone)
+{
+    Sprite *clone;
+    Sprite *cloneArg;
+    EXPECT_CALL(m_engine, initClone(_)).WillOnce(SaveArg<0>(&clone));
+    EXPECT_CALL(m_handler, onCloned(_)).WillOnce(SaveArg<0>(&cloneArg));
+
+    m_sprite.clone();
+    ASSERT_TRUE(clone);
+    ASSERT_EQ(cloneArg, clone);
+}
+
 TEST_F(ISpriteHandlerTest, Visible)
 {
     EXPECT_CALL(m_handler, onVisibleChanged(false)).Times(1);