Add clone method to Variable

Author: adazem009

include/scratchcpp/variable.h

@@ -29,6 +29,8 @@ class LIBSCRATCHCPP_EXPORT Variable : public Entity
         bool isCloudVariable() const;
         void setIsCloudVariable(bool isCloudVariable);
 
+        std::shared_ptr<Variable> clone();
+
     private:
         spimpl::unique_impl_ptr<VariablePrivate> impl;
 };

src/scratch/sprite.cpp

@@ -57,7 +57,7 @@ std::shared_ptr<Sprite> Sprite::clone()
         const auto &vars = variables();
 
         for (auto var : vars)
-            clone->addVariable(std::make_shared<Variable>(var->id(), var->name(), var->value()));
+            clone->addVariable(var->clone());
 
         const auto &l = lists();
 

src/scratch/variable.cpp

@@ -48,3 +48,9 @@ void Variable::setIsCloudVariable(bool isCloudVariable)
 {
     impl->isCloudVariable = isCloudVariable;
 }
+
+/*! Creates a copy of the variable. */
+std::shared_ptr<Variable> Variable::clone()
+{
+    return std::make_shared<Variable>(id(), impl->name, impl->value, impl->isCloudVariable);
+}

test/scratch_classes/variable_test.cpp

@@ -56,3 +56,23 @@ TEST(VariableTest, IsCloudVariable)
     var.setIsCloudVariable(true);
     ASSERT_TRUE(var.isCloudVariable());
 }
+
+TEST(VariableTest, Clone)
+{
+    std::shared_ptr<Variable> clone;
+    std::vector<std::shared_ptr<Variable>> vars;
+
+    vars.push_back(std::make_shared<Variable>("abc", "var1"));
+    vars.push_back(std::make_shared<Variable>("abc", "var2", "test"));
+    vars.push_back(std::make_shared<Variable>("abc", "var3", "test", true));
+    vars.push_back(std::make_shared<Variable>("abc", "var4", "test", false));
+
+    for (auto var : vars) {
+        clone = var->clone();
+        ASSERT_TRUE(clone);
+        ASSERT_EQ(clone->id(), var->id());
+        ASSERT_EQ(clone->name(), var->name());
+        ASSERT_EQ(clone->value(), var->value());
+        ASSERT_EQ(clone->isCloudVariable(), var->isCloudVariable());
+    }
+}