Tests: PityTest - PityUnit change constructor semantics (parent)

src/PityUnit.hh

@@ -28,7 +28,13 @@ namespace pEp {
             // Constructors are private
             PityUnit() = delete;
             explicit PityUnit<TestContext>(
-                AbstractPityUnit* const parent,
+                const std::string& name,
+                TestFunction test_func = nullptr,
+                TestContext* perspective = nullptr,
+                ExecutionMode exec_mode = ExecutionMode::FUNCTION);
+
+            explicit PityUnit<TestContext>(
+                AbstractPityUnit& parent,
                 const std::string& name,
                 TestFunction test_func = nullptr,
                 TestContext* perspective = nullptr,

src/PityUnit.hxx

@@ -20,14 +20,25 @@
 
 namespace pEp {
     namespace PityTest11 {
-            // CONSTRUCTOR
-            template<class TestContext>
-            PityUnit<TestContext>::PityUnit(
-                AbstractPityUnit *const parent,
-                const std::string &name,
-                TestFunction test_func,
-                TestContext *perspective,
-                ExecutionMode exec_mode) :
+
+        template<class TestContext>
+        PityUnit<TestContext>::PityUnit(
+            const std::string &name,
+            TestFunction test_func,
+            TestContext *perspective,
+            ExecutionMode exec_mode) :
+            AbstractPityUnit(name, exec_mode),
+            _perspective{ perspective }, _test_func{ test_func }
+        {
+        }
+
+        template<class TestContext>
+        PityUnit<TestContext>::PityUnit(
+            AbstractPityUnit &parent,
+            const std::string &name,
+            TestFunction test_func,
+            TestContext *perspective,
+            ExecutionMode exec_mode) :
             AbstractPityUnit(parent, name, exec_mode),
             _perspective{ perspective }, _test_func{ test_func }
         {
@@ -59,7 +70,7 @@ namespace pEp {
             if (_perspective != nullptr) {
                 ret = _perspective;
             } else {
-                if (!_isRootUnit()) {
+                if (!isRoot()) {
                     ret = (dynamic_cast<PityUnit<TestContext> *>(getParent()))->getPerspective();
                 }
             }