Add cloneCount method to IEngine

Author: adazem009

include/scratchcpp/iengine.h

@@ -144,6 +144,9 @@ class LIBSCRATCHCPP_EXPORT IEngine
         /*! Sets the maximum number of clones (use -1 or any negative number to disable the limit). */
         virtual void setCloneLimit(int limit) = 0;
 
+        /*! Returns the current number of clones. */
+        virtual int cloneCount() const = 0;
+
         /*! Returns true if sprite fencing is enabled. */
         virtual bool spriteFencingEnabled() const = 0;
 

src/engine/internal/engine.cpp

@@ -592,6 +592,11 @@ void Engine::setCloneLimit(int limit)
     m_cloneLimit = limit < 0 ? -1 : limit;
 }
 
+int Engine::cloneCount() const
+{
+    return m_clones.size();
+}
+
 bool Engine::spriteFencingEnabled() const
 {
     return m_spriteFencingEnabled;

src/engine/internal/engine.h

@@ -67,6 +67,8 @@ class Engine : public IEngine
         int cloneLimit() const override;
         void setCloneLimit(int limit) override;
 
+        int cloneCount() const override;
+
         bool spriteFencingEnabled() const override;
         void setSpriteFencingEnabled(bool enable) override;
 

test/engine/engine_test.cpp

@@ -796,6 +796,7 @@ TEST(EngineTest, CloneLimit)
     ASSERT_TRUE(p.load());
     auto engine = p.engine();
     ASSERT_EQ(engine->cloneLimit(), 300);
+    ASSERT_EQ(engine->cloneCount(), 0);
 
     // TODO: Set "infinite" FPS and remove this (#254)
     engine->setFps(100000);
@@ -808,6 +809,7 @@ TEST(EngineTest, CloneLimit)
     ASSERT_EQ(GET_VAR(stage, "count")->value().toInt(), 300);
     ASSERT_VAR(stage, "delete_passed");
     ASSERT_TRUE(GET_VAR(stage, "delete_passed")->value().toBool());
+    ASSERT_EQ(engine->cloneCount(), 300);
 
     engine->setCloneLimit(475);
     ASSERT_EQ(engine->cloneLimit(), 475);
@@ -816,6 +818,7 @@ TEST(EngineTest, CloneLimit)
     ASSERT_EQ(GET_VAR(stage, "count")->value().toInt(), 475);
     ASSERT_VAR(stage, "delete_passed");
     ASSERT_TRUE(GET_VAR(stage, "delete_passed")->value().toBool());
+    ASSERT_EQ(engine->cloneCount(), 475);
 
     engine->setCloneLimit(0);
     ASSERT_EQ(engine->cloneLimit(), 0);
@@ -824,6 +827,7 @@ TEST(EngineTest, CloneLimit)
     ASSERT_EQ(GET_VAR(stage, "count")->value().toInt(), 0);
     ASSERT_VAR(stage, "delete_passed");
     ASSERT_TRUE(GET_VAR(stage, "delete_passed")->value().toBool());
+    ASSERT_EQ(engine->cloneCount(), 0);
 
     engine->setCloneLimit(-1);
     ASSERT_EQ(engine->cloneLimit(), -1);
@@ -832,6 +836,7 @@ TEST(EngineTest, CloneLimit)
     ASSERT_GT(GET_VAR(stage, "count")->value().toInt(), 500);
     ASSERT_VAR(stage, "delete_passed");
     ASSERT_TRUE(GET_VAR(stage, "delete_passed")->value().toBool());
+    ASSERT_GT(engine->cloneCount(), 500);
 
     engine->setCloneLimit(-5);
     ASSERT_EQ(engine->cloneLimit(), -1);
@@ -840,6 +845,10 @@ TEST(EngineTest, CloneLimit)
     ASSERT_GT(GET_VAR(stage, "count")->value().toInt(), 500);
     ASSERT_VAR(stage, "delete_passed");
     ASSERT_TRUE(GET_VAR(stage, "delete_passed")->value().toBool());
+    ASSERT_GT(engine->cloneCount(), 500);
+
+    engine->stop();
+    ASSERT_EQ(engine->cloneCount(), 0);
 }
 
 // TODO: Uncomment this after fixing #256 and #257

test/mocks/enginemock.h

@@ -53,6 +53,8 @@ class EngineMock : public IEngine
         MOCK_METHOD(int, cloneLimit, (), (const, override));
         MOCK_METHOD(void, setCloneLimit, (int), (override));
 
+        MOCK_METHOD(int, cloneCount, (), (const, override));
+
         MOCK_METHOD(bool, spriteFencingEnabled, (), (const, override));
         MOCK_METHOD(void, setSpriteFencingEnabled, (bool), (override));