Add setPosition() method to Sprite

Author: adazem009

include/scratchcpp/sprite.h

@@ -47,6 +47,8 @@ class LIBSCRATCHCPP_EXPORT Sprite
         double y() const;
         void setY(double newY);
 
+        void setPosition(double x, double y);
+
         double size() const;
         void setSize(double newSize);
 

src/scratch/sprite.cpp

@@ -184,6 +184,17 @@ void Sprite::setY(double newY)
         impl->iface->onYChanged(impl->y);
 }
 
+/* Sets the position of the sprite. */
+void Sprite::setPosition(double x, double y)
+{
+    setXY(x, y);
+
+    if (impl->iface) {
+        impl->iface->onXChanged(impl->x);
+        impl->iface->onYChanged(impl->y);
+    }
+}
+
 /*! Returns the size. */
 double Sprite::size() const
 {

test/scratch_classes/sprite_test.cpp

@@ -233,7 +233,7 @@ TEST(SpriteTest, XY)
 
     EngineMock engine;
     sprite.setEngine(&engine);
-    EXPECT_CALL(engine, requestRedraw()).Times(17);
+    EXPECT_CALL(engine, requestRedraw()).Times(18);
     EXPECT_CALL(engine, spriteFencingEnabled()).Times(4).WillRepeatedly(Return(false));
 
     sprite.setX(-53.25);
@@ -340,6 +340,13 @@ TEST(SpriteTest, XY)
     EXPECT_CALL(handler, onYChanged);
     sprite.setY(-340);
     ASSERT_EQ(sprite.y(), -340);
+
+    EXPECT_CALL(engine, spriteFencingEnabled()).WillOnce(Return(false));
+    EXPECT_CALL(handler, onXChanged);
+    EXPECT_CALL(handler, onYChanged);
+    sprite.setPosition(56, -23);
+    ASSERT_EQ(sprite.x(), 56);
+    ASSERT_EQ(sprite.y(), -23);
 }
 
 TEST(SpriteTest, Size)

test/target_interfaces/ispritehandler_test.cpp

@@ -93,6 +93,15 @@ TEST_F(ISpriteHandlerTest, Y)
     m_sprite.setY(207.08);
 }
 
+TEST_F(ISpriteHandlerTest, Position)
+{
+    EXPECT_CALL(m_handler, onXChanged(284.61));
+    EXPECT_CALL(m_handler, onYChanged(207.08));
+    EXPECT_CALL(m_engine, spriteFencingEnabled()).WillOnce(Return(false));
+    EXPECT_CALL(m_engine, requestRedraw());
+    m_sprite.setPosition(284.61, 207.08);
+}
+
 TEST_F(ISpriteHandlerTest, Size)
 {
     EXPECT_CALL(m_handler, onSizeChanged(53.8)).Times(1);