Add ScratchConfiguration test

src/scratchconfiguration_p.cpp

@@ -1,13 +1,17 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include <scratchcpp/iextension.h>
+#include <algorithm>
 
 #include "scratchconfiguration_p.h"
 
 using namespace libscratchcpp;
 
 void ScratchConfigurationPrivate::registerExtension(std::shared_ptr<IExtension> extension)
 {
+    if (std::find(extensions.begin(), extensions.end(), extension) != extensions.cend())
+        return;
+
     extensions.push_back(extension);
 }
 

test/CMakeLists.txt

@@ -23,3 +23,4 @@ add_subdirectory(virtual_machine)
 add_subdirectory(scratch_classes)
 add_subdirectory(target_interfaces)
 add_subdirectory(blocks)
+add_subdirectory(scratchconfiguration)

test/scratchconfiguration/CMakeLists.txt

@@ -0,0 +1,20 @@
+add_executable(
+  scratchconfiguration_test
+  scratchconfiguration_test.cpp
+  extensionbase.cpp
+  extensionbase.h
+  extension1.cpp
+  extension1.h
+  extension2.cpp
+  extension2.h
+  extension3.cpp
+  extension3.h
+)
+
+target_link_libraries(
+  scratchconfiguration_test
+  GTest::gtest_main
+  scratchcpp
+)
+
+gtest_discover_tests(scratchconfiguration_test)

test/scratchconfiguration/extension1.cpp

@@ -0,0 +1,8 @@
+#include "extension1.h"
+
+using namespace libscratchcpp;
+
+std::string Extension1::name() const
+{
+    return "ext 1";
+}

test/scratchconfiguration/extension1.h

@@ -0,0 +1,14 @@
+#pragma once
+
+#include "extensionbase.h"
+
+namespace libscratchcpp
+{
+
+class Extension1 : public ExtensionBase
+{
+    public:
+        std::string name() const override;
+};
+
+} // namespace libscratchcpp

test/scratchconfiguration/extension2.cpp

@@ -0,0 +1,8 @@
+#include "extension2.h"
+
+using namespace libscratchcpp;
+
+std::string Extension2::name() const
+{
+    return "ext 2";
+}

test/scratchconfiguration/extension2.h

@@ -0,0 +1,14 @@
+#pragma once
+
+#include "extensionbase.h"
+
+namespace libscratchcpp
+{
+
+class Extension2 : public ExtensionBase
+{
+    public:
+        std::string name() const override;
+};
+
+} // namespace libscratchcpp

test/scratchconfiguration/extension3.cpp

@@ -0,0 +1,8 @@
+#include "extension3.h"
+
+using namespace libscratchcpp;
+
+std::string Extension3::name() const
+{
+    return "ext 3";
+}

test/scratchconfiguration/extension3.h

@@ -0,0 +1,14 @@
+#pragma once
+
+#include "extensionbase.h"
+
+namespace libscratchcpp
+{
+
+class Extension3 : public ExtensionBase
+{
+    public:
+        std::string name() const override;
+};
+
+} // namespace libscratchcpp

test/scratchconfiguration/extensionbase.cpp

@@ -0,0 +1,12 @@
+#include "extensionbase.h"
+
+using namespace libscratchcpp;
+
+std::string ExtensionBase::description() const
+{
+    return "";
+}
+
+void ExtensionBase::registerSections(IEngine *engine)
+{
+}

test/scratchconfiguration/extensionbase.h

@@ -0,0 +1,16 @@
+#pragma once
+
+#include <scratchcpp/iextension.h>
+
+namespace libscratchcpp
+{
+
+class ExtensionBase : public IExtension
+{
+    public:
+        std::string description() const override;
+
+        void registerSections(IEngine *engine) override;
+};
+
+} // namespace libscratchcpp

test/scratchconfiguration/scratchconfiguration_test.cpp

@@ -0,0 +1,37 @@
+#include <scratchcpp/scratchconfiguration.h>
+
+#include "../common.h"
+#include "extension1.h"
+#include "extension2.h"
+#include "extension3.h"
+
+using namespace libscratchcpp;
+
+class ScratchConfigurationTest : public testing::Test
+{
+    public:
+        void SetUp() override
+        {
+            m_extension1 = std::make_shared<Extension1>();
+            m_extension2 = std::make_shared<Extension2>();
+            m_extension3 = std::make_shared<Extension3>();
+        }
+
+        std::shared_ptr<IExtension> m_extension1;
+        std::shared_ptr<IExtension> m_extension2;
+        std::shared_ptr<IExtension> m_extension3;
+};
+
+TEST_F(ScratchConfigurationTest, Extensions)
+{
+    ScratchConfiguration::registerExtension(m_extension1);
+    ScratchConfiguration::registerExtension(m_extension2);
+
+    ASSERT_EQ(ScratchConfiguration::getExtension("ext 1"), m_extension1.get());
+    ASSERT_EQ(ScratchConfiguration::getExtension("ext 2"), m_extension2.get());
+    ASSERT_EQ(ScratchConfiguration::getExtension("ext 3"), nullptr);
+
+    ASSERT_EQ(ScratchConfiguration::getExtension<Extension1>(), m_extension1.get());
+    ASSERT_EQ(ScratchConfiguration::getExtension<Extension2>(), m_extension2.get());
+    ASSERT_EQ(ScratchConfiguration::getExtension<Extension3>(), nullptr);
+}