Add graphics effect methods to ISpriteHandler

Author: adazem009

include/scratchcpp/ispritehandler.h

@@ -41,6 +41,15 @@ class LIBSCRATCHCPP_EXPORT ISpriteHandler
         /*! Called when the rotation style changes. */
         virtual void onRotationStyleChanged(Sprite::RotationStyle rotationStyle) = 0;
 
+        /*!
+         * Called when the value of the given graphics effect changes.
+         * \note This method isn't called when all effects are cleared, use onGraphicsEffectsCleared() for this.
+         */
+        virtual void onGraphicsEffectChanged(IGraphicsEffect *effect, double value) = 0;
+
+        /*! Called when all graphics effects are cleared. */
+        virtual void onGraphicsEffectsCleared() = 0;
+
         /*!
          * Used to get the bounding rectangle of the sprite.
          * \note The rectangle must be relative to the stage, so make sure to use the sprite's coordinates.

src/scratch/sprite.cpp

@@ -411,6 +411,9 @@ void Sprite::setGraphicsEffectValue(IGraphicsEffect *effect, double value)
         if (eng)
             eng->requestRedraw();
     }
+
+    if (impl->iface)
+        impl->iface->onGraphicsEffectChanged(effect, value);
 }
 
 /*! Overrides Target#clearGraphicsEffects(). */
@@ -424,6 +427,9 @@ void Sprite::clearGraphicsEffects()
         if (eng)
             eng->requestRedraw();
     }
+
+    if (impl->iface)
+        impl->iface->onGraphicsEffectsCleared();
 }
 
 Target *Sprite::dataSource() const

test/mocks/spritehandlermock.h

@@ -20,5 +20,8 @@ class SpriteHandlerMock : public ISpriteHandler
         MOCK_METHOD(void, onSizeChanged, (double), (override));
         MOCK_METHOD(void, onDirectionChanged, (double), (override));
         MOCK_METHOD(void, onRotationStyleChanged, (Sprite::RotationStyle), (override));
+        MOCK_METHOD(void, onGraphicsEffectChanged, (IGraphicsEffect *, double), (override));
+        MOCK_METHOD(void, onGraphicsEffectsCleared, (), (override));
+
         MOCK_METHOD(Rect, boundingRect, (), (const, override));
 };

test/target_interfaces/ispritehandler_test.cpp

@@ -2,6 +2,7 @@
 #include <scratchcpp/rect.h>
 #include <spritehandlermock.h>
 #include <enginemock.h>
+#include <graphicseffectmock.h>
 
 #include "../common.h"
 
@@ -130,6 +131,19 @@ TEST_F(ISpriteHandlerTest, Costume)
     m_sprite.setCostumeIndex(1);
 }
 
+TEST_F(ISpriteHandlerTest, GraphicsEffects)
+{
+    GraphicsEffectMock effect;
+
+    EXPECT_CALL(m_handler, onGraphicsEffectChanged(&effect, 16.7));
+    EXPECT_CALL(m_engine, requestRedraw());
+    m_sprite.setGraphicsEffectValue(&effect, 16.7);
+
+    EXPECT_CALL(m_handler, onGraphicsEffectsCleared());
+    EXPECT_CALL(m_engine, requestRedraw());
+    m_sprite.clearGraphicsEffects();
+}
+
 TEST_F(ISpriteHandlerTest, BoundingRect)
 {
     EXPECT_CALL(m_handler, boundingRect()).WillOnce(Return(Rect(-44.6, 89.1, 20.5, -0.48)));