Add clone method to List

Author: adazem009

include/scratchcpp/list.h

@@ -39,6 +39,8 @@ class LIBSCRATCHCPP_EXPORT List
 
         std::string toString() const;
 
+        std::shared_ptr<List> clone();
+
     private:
         spimpl::unique_impl_ptr<ListPrivate> impl;
 };

src/scratch/list.cpp

@@ -76,3 +76,14 @@ std::string List::toString() const
 
     return ret;
 }
+
+/*! Creates a copy of the list. */
+std::shared_ptr<List> List::clone()
+{
+    auto copy = std::make_shared<List>(id(), impl->name);
+
+    for (const Value &item : *this)
+        copy->push_back(item);
+
+    return copy;
+}

src/scratch/sprite.cpp

@@ -61,13 +61,8 @@ std::shared_ptr<Sprite> Sprite::clone()
 
         const auto &l = lists();
 
-        for (auto list : l) {
-            auto newList = std::make_shared<List>(list->id(), list->name());
-            clone->addList(newList);
-
-            for (const Value &item : *list)
-                newList->push_back(item);
-        }
+        for (auto list : l)
+            clone->addList(list->clone());
 
         clone->setCurrentCostume(currentCostume());
         clone->setLayerOrder(layerOrder());

test/scratch_classes/list_test.cpp

@@ -170,3 +170,22 @@ TEST(ListTest, ToString)
     list.push_back(8);
     ASSERT_EQ(list.toString(), "098");
 }
+
+TEST(ListTest, Clone)
+{
+    List list("abc", "test list");
+    list.push_back("Lorem");
+    list.push_back("ipsum");
+    list.push_back("dolor");
+    list.push_back("sit");
+    list.push_back("amet");
+
+    std::shared_ptr<List> clone = list.clone();
+    ASSERT_TRUE(clone);
+    ASSERT_EQ(clone->id(), list.id());
+    ASSERT_EQ(clone->name(), list.name());
+    ASSERT_EQ(clone->size(), list.size());
+
+    for (std::size_t i = 0; i < list.size(); i++)
+        ASSERT_EQ(list[i], (*clone)[i]);
+}