[tests] Prevent test leaks when loading plugins (sofa-framework#5344)

* [tests] Prevent test leaks when loading plugins

* missing plugin

* fix tests

* missing plugin

* missing plugins

* missing plugins

* missing plugins

Author: alxbilger

Sofa/Component/Collision/Detection/Algorithm/tests/CollisionPipeline_test.cpp

@@ -76,8 +76,12 @@ class TestCollisionPipeline : public BaseSimulationTest {
 
     void doSetUp() override
     {
-        sofa::simpleapi::importPlugin(Sofa.Component.StateContainer);
-        sofa::simpleapi::importPlugin(Sofa.Component.Collision);
+        this->loadPlugins({
+            Sofa.Component.StateContainer,
+            Sofa.Component.Collision.Detection.Algorithm,
+            Sofa.Component.Collision.Detection.Intersection,
+            Sofa.Component.Collision.Response.Contact
+        });
     }
 
     void doTearDown() override

Sofa/Component/Collision/Detection/Intersection/tests/LocalMinDistance_test.cpp

@@ -56,11 +56,7 @@ namespace
 struct TestLocalMinDistance : public BaseSimulationTest {
     void doSetUp() override
     {
-        sofa::simpleapi::importPlugin(Sofa.Component.StateContainer);
-        sofa::simpleapi::importPlugin(Sofa.Component.Collision.Detection.Intersection);
-    }
-    void doTearDown() override
-    {
+        loadPlugins({Sofa.Component.StateContainer, Sofa.Component.Collision.Detection.Intersection});
     }
 
     void checkAttributes();

Sofa/Component/Collision/Geometry/tests/Sphere_test.cpp

@@ -72,16 +72,12 @@ struct TestSphere : public BaseSimulationTest
 {
     void doSetUp() override
     {
-        sofa::simpleapi::importPlugin(Sofa.Component.StateContainer);
-        sofa::simpleapi::importPlugin(Sofa.Component.Collision.Geometry);
+        loadPlugins({Sofa.Component.StateContainer, Sofa.Component.Collision.Geometry});
 
         m_proxIntersection = sofa::core::objectmodel::New<MinProximityIntersection>();
         m_proxIntersection->setAlarmDistance(1.0);
         m_proxIntersection->setContactDistance(1.0);
     }
-    void doTearDown() override
-    {
-    }
 
     bool rigidRigid1();
     bool rigidRigid2();

Sofa/Component/Engine/Select/tests/MeshROI_test.cpp

@@ -63,9 +63,11 @@ struct MeshROI_test : public BaseSimulationTest,
 
     void doSetUp() override
     {
-        simpleapi::importPlugin(Sofa.Component.Engine.Select);
-        simpleapi::importPlugin(Sofa.Component.Topology.Container.Constant);
-        simpleapi::importPlugin(Sofa.Component.IO.Mesh);
+        loadPlugins({
+            Sofa.Component.Engine.Select,
+            Sofa.Component.Topology.Container.Constant,
+            Sofa.Component.IO.Mesh
+        });
 
         // SetUp3
         const string scene2 =

Sofa/Component/Engine/Select/tests/MeshSubsetEngine_test.cpp

@@ -33,7 +33,7 @@ struct MeshSubsetEngine_test : public testing::BaseSimulationTest
 
     void doSetUp() override
     {
-        simpleapi::importPlugin(Sofa.Component.Engine.Select);
+        loadPlugins({Sofa.Component.Engine.Select});
 
         m_root = simulation::getSimulation()->createNewNode("root");
         ASSERT_NE(nullptr, m_root);

Sofa/Component/Engine/Select/tests/PlaneROI_test.cpp

@@ -71,6 +71,8 @@ struct PlaneROI_test : public BaseSimulationTest,
         m_node1 = m_simu->createNewGraph("root");
         m_node1->addObject(m_thisObject);
 
+        this->loadPlugins({Sofa.Component.Engine.Select});
+
 
         // SetUp2
         const string scene1 =

Sofa/Component/IO/Mesh/tests/MeshExporter_test.cpp

@@ -79,8 +79,10 @@ class MeshExporter_test
 
     void doSetUp() override
     {
-        sofa::simpleapi::importPlugin(Sofa.Component.StateContainer);
-        sofa::simpleapi::importPlugin(Sofa.Component.Topology.Container.Grid);
+        loadPlugins({
+            Sofa.Component.StateContainer,
+            Sofa.Component.Topology.Container.Grid
+        });
     }
 
     void doTearDown() override

Sofa/Component/IO/Mesh/tests/STLExporter_test.cpp

@@ -67,9 +67,11 @@ class STLExporter_test : public BaseSimulationTest {
 
     void doSetUp() override
     {
-        sofa::simpleapi::importPlugin(Sofa.Component.StateContainer);
-        sofa::simpleapi::importPlugin(Sofa.Component.Visual);
-        sofa::simpleapi::importPlugin(Sofa.Component.IO.Mesh);
+        loadPlugins({
+            Sofa.Component.StateContainer,
+            Sofa.Component.Visual,
+            Sofa.Component.IO.Mesh
+        });
     }
 
     void doTearDown() override

Sofa/Component/LinearSystem/tests/MappingGraph_test.cpp

@@ -176,7 +176,7 @@ TEST(MappingGraph, diamondMapping)
     sofa::simulation::Node::SPtr root = simulation->createNewGraph("root");
     EXPECT_EQ(root->getName(), "root");
 
-    sofa::simpleapi::importPlugin(Sofa.Component.Mapping.Linear);
+    const auto plugins = sofa::testing::makeScopedPlugin({Sofa.Component.Mapping.Linear});
 
     const auto top = sofa::core::objectmodel::New<sofa::component::statecontainer::MechanicalObject<sofa::defaulttype::Vec3Types> >();
     root->addObject(top);

Sofa/Component/Mass/tests/DiagonalMass_test.cpp

@@ -98,10 +98,12 @@ class DiagonalMass_test : public BaseTest
 
     void doSetUp() override
     {
-        sofa::simpleapi::importPlugin(Sofa.Component.Topology.Container.Dynamic);
-        sofa::simpleapi::importPlugin(Sofa.Component.Topology.Container.Grid);
-        sofa::simpleapi::importPlugin(Sofa.Component.StateContainer);
-        sofa::simpleapi::importPlugin(Sofa.Component.Mass);
+        this->loadPlugins({
+            Sofa.Component.Topology.Container.Dynamic,
+            Sofa.Component.Topology.Container.Grid,
+            Sofa.Component.StateContainer,
+            Sofa.Component.Mass
+        });
 
         simulation = simulation::getSimulation();
         root = simulation::getSimulation()->createNewGraph("root");