Add clone() method to Sound

Author: adazem009

include/scratchcpp/sound.h

@@ -32,6 +32,8 @@ class LIBSCRATCHCPP_EXPORT Sound : public Asset
 
         virtual bool isPlaying();
 
+        std::shared_ptr<Sound> clone() const;
+
     protected:
         void processData(unsigned int size, void *data) override;
 

src/scratch/sound.cpp

@@ -62,8 +62,24 @@ bool Sound::isPlaying()
     return impl->player->isPlaying();
 }
 
+/*! Returns an independent copy of the sound which is valid for as long as the original sound exists. */
+std::shared_ptr<Sound> Sound::clone() const
+{
+    auto sound = std::make_shared<Sound>(name(), id(), dataFormat());
+    sound->setRate(rate());
+    sound->setSampleCount(sampleCount());
+    sound->impl->player->loadCopy(impl->player.get());
+    sound->impl->player->setVolume(impl->player->volume());
+    sound->setData(dataSize(), const_cast<void *>(data()));
+
+    return sound;
+}
+
 void Sound::processData(unsigned int size, void *data)
 {
+    if (impl->player->isLoaded())
+        return;
+
     if (!impl->player->load(size, data, impl->rate))
         std::cerr << "Failed to load sound " << name() << std::endl;
 }

test/assets/sound_test.cpp

@@ -60,7 +60,12 @@ TEST_F(SoundTest, ProcessData)
     const char *data = "abc";
     void *dataPtr = const_cast<void *>(static_cast<const void *>(data));
 
-    EXPECT_CALL(*m_player, load(3, dataPtr, 44100));
+    EXPECT_CALL(*m_player, isLoaded()).WillOnce(Return(false));
+    EXPECT_CALL(*m_player, load(3, dataPtr, 44100)).WillOnce(Return(true));
+    sound.setData(3, dataPtr);
+
+    EXPECT_CALL(*m_player, isLoaded()).WillOnce(Return(true));
+    EXPECT_CALL(*m_player, load).Times(0);
     sound.setData(3, dataPtr);
 }
 
@@ -100,3 +105,31 @@ TEST_F(SoundTest, IsPlaying)
 
     SoundPrivate::audioOutput = nullptr;
 }
+
+TEST_F(SoundTest, Clone)
+{
+    Sound sound("sound1", "a", "wav");
+    sound.setRate(44100);
+    sound.setSampleCount(10000);
+
+    const char *data = "abc";
+    void *dataPtr = const_cast<void *>(static_cast<const void *>(data));
+
+    EXPECT_CALL(*m_player, isLoaded()).WillOnce(Return(false));
+    EXPECT_CALL(*m_player, load(3, dataPtr, 44100)).WillOnce(Return(true));
+    sound.setData(3, dataPtr);
+
+    auto clonePlayer = std::make_shared<AudioPlayerMock>();
+    EXPECT_CALL(m_playerFactory, createAudioPlayer()).WillOnce(Return(clonePlayer));
+    EXPECT_CALL(*clonePlayer, loadCopy(m_player.get())).WillOnce(Return(true));
+    EXPECT_CALL(*m_player, volume()).WillOnce(Return(0.45));
+    EXPECT_CALL(*clonePlayer, setVolume(0.45));
+    EXPECT_CALL(*clonePlayer, isLoaded()).WillOnce(Return(true));
+    auto clone = sound.clone();
+    ASSERT_TRUE(clone);
+    ASSERT_EQ(clone->name(), sound.name());
+    ASSERT_EQ(clone->id(), sound.id());
+    ASSERT_EQ(clone->dataFormat(), sound.dataFormat());
+    ASSERT_EQ(clone->rate(), sound.rate());
+    ASSERT_EQ(clone->sampleCount(), sound.sampleCount());
+}