From c930dabc333d264b94e3b2586611a78955922b19 Mon Sep 17 00:00:00 2001
From: neeraj31285 <neeraj.singh31285@gmail.com>
Date: Sat, 24 Jan 2026 14:58:14 +0530
Subject: [PATCH 1/3] rtl::view, read-only for unique_ptr as well. no move out.

---
 .../RObjectReflecting_stdUniquePtr.cpp        | 316 ++++++++++--------
 .../rtl/detail/inc/RObjectUPtr.h              |   9 +-
 ReflectionTemplateLib/rtl/inc/view.hpp        |  13 +-
 3 files changed, 181 insertions(+), 157 deletions(-)

diff --git a/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp b/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp
index db416bf5..870eb981 100644
--- a/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp
+++ b/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp
@@ -53,19 +53,20 @@ namespace rtl::unit_test
             auto view = robj.view<std::unique_ptr<int>>();
             ASSERT_TRUE(view);
 
-            // get() - moves out the 'unique_ptr<int>' from 'robj'.
-            // 'robj' still remains alive and type-consistent, but now holds an empty unique_ptr
-            std::unique_ptr<int> uptr = view->get();
+            // get() - UPDATE: MOVING DISALLOWED NOW.
+            const std::unique_ptr<int>& uptr = view->get();
             ASSERT_TRUE(uptr);
+
             // RTL gave up the ownership after move-op.
-            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
+            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
+
             // Access the moved-out value
             EXPECT_EQ(*uptr, NUM);
 
-            int* ptr = uptr.release();
-            // Addresses must be same.
-            EXPECT_EQ(numPtr, ptr);
-            delete ptr;     //RTL must not delete again, once 'robj' out of scope.
+            //int* ptr = uptr.release();
+            //// Addresses must be same.
+            //EXPECT_EQ(numPtr, ptr);
+            //delete ptr;     //RTL must not delete again, once 'robj' out of scope.
         }
         // there must not be any crash.
         ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
@@ -89,19 +90,13 @@ namespace rtl::unit_test
             auto view = robj.view<std::unique_ptr<Node>>();
             ASSERT_TRUE(view);
 
-            // get() - moves out the 'unique_ptr<int>' from 'robj'.
-            // 'robj' still remains alive and type-consistent, but now holds an empty unique_ptr
-            std::unique_ptr<Node> uptr = view->get();
+            // get() - UPDATE: MOVING DISALLOWED NOW.
+            const std::unique_ptr<Node>& uptr = view->get();
             ASSERT_TRUE(uptr);
-            // RTL gave up the ownership after move-op.
-            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
-            // Access the moved-out value
+            // RTL still owns it.
+            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
+            // Access the value
             EXPECT_EQ(uptr->data(), NUM);
-
-            Node* ptr = uptr.release();
-            // Addresses must be same.
-            EXPECT_TRUE(ptr->data() == NUM);
-            delete ptr;     //RTL must not delete again, once 'robj' out of scope.
         }
         // there must not be any crash.
         ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
@@ -144,9 +139,8 @@ namespace rtl::unit_test
         // unique_ptr, ownership managed by RTL.
         ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
 
-        // A move from any view transfers ownership of the underlying object
-        // RObject remains alive and type-consistent, but now holds an empty unique_ptr
-        std::unique_ptr<int> uptr0 = view0->get();
+        // get() - UPDATE: MOVING DISALLOWED NOW.
+        const std::unique_ptr<int>& uptr0 = view0->get();
         ASSERT_TRUE(uptr0);
 
         // Access the moved-out value
@@ -155,191 +149,216 @@ namespace rtl::unit_test
         // Verify the original pointer address matches
         EXPECT_EQ(uptr0.get(), &valueNum);
 
-        // RTL gave up the ownership after move-op.
-        ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
+        // RTL still owns it
+        ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
 
         // RObject still exists with correct metadata, but the stored unique_ptr is now empty
         ASSERT_FALSE(robj.isEmpty());
 
+        // UPDATE: MOVING DISALLOWED NOW.
         // Any subsequent view will still be obtainable
         auto view3 = robj.view<std::unique_ptr<int>>();
         ASSERT_TRUE(view3);
 
+        // UPDATE: MOVING DISALLOWED NOW.
         // But the unique_ptr inside is now empty due to the earlier move
-        std::unique_ptr<int> uptr3 = view3->get();
-        ASSERT_TRUE(uptr3 == nullptr);
+        const std::unique_ptr<int>& uptr3 = view3->get();
+        //ASSERT_TRUE(uptr3 == nullptr);
 
+        // UPDATE: MOVING DISALLOWED NOW.
         // All earlier views now yield empty unique_ptrs as well- no dangling pointers, no UB
-        std::unique_ptr<int> uptr2 = view2->get();
-        ASSERT_TRUE(uptr3 == nullptr);
+        const std::unique_ptr<int>& uptr2 = view2->get();
+        //ASSERT_TRUE(uptr3 == nullptr);
 
-        std::unique_ptr<int> uptr1 = view1->get();
-        ASSERT_TRUE(uptr3 == nullptr);
+        // UPDATE: MOVING DISALLOWED NOW.
+        const std::unique_ptr<int>& uptr1 = view1->get();
+        //ASSERT_TRUE(uptr3 == nullptr);
 
+        // UPDATE: MOVING DISALLOWED NOW.
         // Even reusing the moved-from view0 is safe- just returns empty
-        std::unique_ptr<int> uptr00 = view0->get();
-        ASSERT_TRUE(uptr00 == nullptr);
+        const std::unique_ptr<int>& uptr00 = view0->get();
+        //ASSERT_TRUE(uptr00 == nullptr);
     }
 
 
     TEST(RObject_reflecting_unique_ptr, init_with_lvalue)
     {
-        constexpr const int NUM = 16238;
-        std::unique_ptr<Node> uptr = std::make_unique<Node>(NUM);
-        ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
-        // Reflect a move-only type directly into RObject
-        RObject robj = reflect(std::move(uptr));
-        ASSERT_FALSE(robj.isEmpty());
+        {
+            constexpr const int NUM = 16238;
+            std::unique_ptr<Node> uptr = std::make_unique<Node>(NUM);
+            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
+            // Reflect a move-only type directly into RObject
+            RObject robj = reflect(std::move(uptr));
+            ASSERT_FALSE(robj.isEmpty());
 
-        // RObject can transparently expose the pointee type
-        EXPECT_TRUE(robj.canViewAs<Node>());
+            // RObject can transparently expose the pointee type
+            EXPECT_TRUE(robj.canViewAs<Node>());
 
-        auto nodeView = robj.view<Node>();
-        ASSERT_TRUE(nodeView);
+            auto nodeView = robj.view<Node>();
+            ASSERT_TRUE(nodeView);
 
-        const Node& node = nodeView->get();
-        EXPECT_EQ(node.data(), NUM);
+            const Node& node = nodeView->get();
+            EXPECT_EQ(node.data(), NUM);
 
-        // RObject can also reflect as the original move-only type
-        EXPECT_TRUE(robj.canViewAs<std::unique_ptr<Node>>());
-        {
-            // Multiple independent views to the same stored object- all valid before a move
-            auto view0 = robj.view<std::unique_ptr<Node>>();
-            ASSERT_TRUE(view0);
+            // RObject can also reflect as the original move-only type
+            EXPECT_TRUE(robj.canViewAs<std::unique_ptr<Node>>());
+            {
+                // Multiple independent views to the same stored object- all valid before a move
+                auto view0 = robj.view<std::unique_ptr<Node>>();
+                ASSERT_TRUE(view0);
 
-            auto view1 = robj.view<std::unique_ptr<Node>>();
-            ASSERT_TRUE(view1);
+                auto view1 = robj.view<std::unique_ptr<Node>>();
+                ASSERT_TRUE(view1);
 
-            auto view2 = robj.view<std::unique_ptr<Node>>();
-            ASSERT_TRUE(view2);
+                auto view2 = robj.view<std::unique_ptr<Node>>();
+                ASSERT_TRUE(view2);
 
-            // unique_ptr, ownership managed by RTL.
-            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
+                // unique_ptr, ownership managed by RTL.
+                ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
 
-            // A move from any view transfers ownership of the underlying object
-            // RObject remains alive and type-consistent, but now holds an empty unique_ptr
-            std::unique_ptr<Node> uptr0 = view0->get();
-            ASSERT_TRUE(uptr0);
+                // UPDATE: MOVING DISALLOWED NOW.
+                // A move from any view transfers ownership of the underlying object
+                // RObject remains alive and type-consistent, but now holds an empty unique_ptr
+                const std::unique_ptr<Node>& uptr0 = view0->get();
+                ASSERT_TRUE(uptr0);
 
-            // Access the moved-out value
-            EXPECT_EQ(uptr0->data(), NUM);
+                // Access the moved-out value
+                EXPECT_EQ(uptr0->data(), NUM);
 
-            // Verify the original pointer address matches
-            EXPECT_EQ(uptr0.get(), &node);
+                // Verify the original pointer address matches
+                EXPECT_EQ(uptr0.get(), &node);
 
-            // RObject still exists with correct metadata, but the stored unique_ptr is now empty
-            ASSERT_FALSE(robj.isEmpty());
+                // RObject still exists with correct metadata, but the stored unique_ptr is now empty
+                ASSERT_FALSE(robj.isEmpty());
 
-            // RTL gave up the ownership after move-op.
-            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
-            // Node still exists.
-            EXPECT_TRUE(Node::instanceCount() == 1);
+                // RTL still owns it
+                ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
+                // Node still exists.
+                EXPECT_TRUE(Node::instanceCount() == 1);
 
-            // Any subsequent view will still be obtainable
-            auto view3 = robj.view<std::unique_ptr<Node>>();
-            ASSERT_TRUE(view3);
+                // Any subsequent view will still be obtainable
+                auto view3 = robj.view<std::unique_ptr<Node>>();
+                ASSERT_TRUE(view3);
 
-            // But the unique_ptr inside is now empty due to the earlier move
-            std::unique_ptr<Node> uptr3 = view3->get();
-            ASSERT_TRUE(uptr3 == nullptr);
+                // UPDATE: MOVING DISALLOWED NOW.
+                // But the unique_ptr inside is now empty due to the earlier move
+                const std::unique_ptr<Node>& uptr3 = view3->get();
+                //ASSERT_TRUE(uptr3 == nullptr);
 
-            // All earlier views now yield empty unique_ptrs as well- no dangling pointers, no UB
-            std::unique_ptr<Node> uptr2 = view2->get();
-            ASSERT_TRUE(uptr3 == nullptr);
+                // UPDATE: MOVING DISALLOWED NOW.
+                // All earlier views now yield empty unique_ptrs as well- no dangling pointers, no UB
+                const std::unique_ptr<Node>& uptr2 = view2->get();
+                //ASSERT_TRUE(uptr3 == nullptr);
 
-            std::unique_ptr<Node> uptr1 = view1->get();
-            ASSERT_TRUE(uptr3 == nullptr);
+                // UPDATE: MOVING DISALLOWED NOW.
+                const std::unique_ptr<Node>& uptr1 = view1->get();
+                //ASSERT_TRUE(uptr3 == nullptr);
 
-            // Even reusing the moved-from view0 is safe- just returns empty
-            std::unique_ptr<Node> uptr00 = view0->get();
-            ASSERT_TRUE(uptr00 == nullptr);
+                // UPDATE: MOVING DISALLOWED NOW.
+                // Even reusing the moved-from view0 is safe- just returns empty
+                const std::unique_ptr<Node>& uptr00 = view0->get();
+                //ASSERT_TRUE(uptr00 == nullptr);
+            }
+            EXPECT_TRUE(Node::instanceCount() == 1);
+            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
         }
         EXPECT_TRUE(Node::instanceCount() == 0);
         EXPECT_TRUE(Node::assertResourcesReleased());
+        ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
     }
 
 
 
     TEST(RObject_reflecting_unique_ptr, destructor_call__reflectecting_lvalue)
     {
-        const int NUM = 24028;
-        std::unique_ptr<Node> nodePtr = std::make_unique<Node>(NUM);
-        RObject robj = reflect(nodePtr);
-        ASSERT_FALSE(robj.isEmpty());
-
-        // Check if RObject can reflect as `Node`
-        EXPECT_TRUE(robj.canViewAs<Node>());
         {
-            auto view = robj.view<Node>();
-            ASSERT_TRUE(view);
+            const int NUM = 24028;
+            std::unique_ptr<Node> nodePtr = std::make_unique<Node>(NUM);
+            RObject robj = reflect(nodePtr);
+            ASSERT_FALSE(robj.isEmpty());
 
-            const Node& node = view->get();
-            EXPECT_EQ(node.data(), NUM);
-            // Ensure no copy is made for viewing.
-            EXPECT_TRUE(Node::instanceCount() == 1);
-        }
-        // Check if RObject can reflect as `shared_ptr<Node>`
-        EXPECT_FALSE(robj.canViewAs<std::shared_ptr<Node>>());
-        {
-            // Get a view of the view as `shared_ptr<int>`
-            auto view = robj.view<std::shared_ptr<Node>>();
-            ASSERT_FALSE(view);
-        }
-        // Check if RObject can reflect as `unique_ptr<Node>`
-        EXPECT_TRUE(robj.canViewAs<std::unique_ptr<Node>>());
-        {
-            // Get a view of the view as `shared_ptr<int>`
-            auto view = robj.view<std::unique_ptr<Node>>();
-            EXPECT_TRUE(view);
-            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() != 0);
+            // Check if RObject can reflect as `Node`
+            EXPECT_TRUE(robj.canViewAs<Node>());
             {
-                std::unique_ptr<Node> movedOutPtr = view->get();
+                auto view = robj.view<Node>();
+                ASSERT_TRUE(view);
+
+                const Node& node = view->get();
+                EXPECT_EQ(node.data(), NUM);
+                // Ensure no copy is made for viewing.
+                EXPECT_TRUE(Node::instanceCount() == 1);
+            }
+            // Check if RObject can reflect as `shared_ptr<Node>`
+            EXPECT_FALSE(robj.canViewAs<std::shared_ptr<Node>>());
+            {
+                // Get a view of the view as `shared_ptr<int>`
+                auto view = robj.view<std::shared_ptr<Node>>();
+                ASSERT_FALSE(view);
+            }
+            // Check if RObject can reflect as `unique_ptr<Node>`
+            EXPECT_TRUE(robj.canViewAs<std::unique_ptr<Node>>());
+            {
+                // Get a view of the view as `shared_ptr<int>`
+                auto view = robj.view<std::unique_ptr<Node>>();
+                EXPECT_TRUE(view);
+                ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() != 0);
+                {
+                    // UPDATE: MOVING DISALLOWED NOW.
+                    const std::unique_ptr<Node>& movedOutPtr = view->get();
+                }
+                EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
+                EXPECT_TRUE(Node::instanceCount() == 1);
             }
-            EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
-            EXPECT_TRUE(Node::instanceCount() == 0);
-            EXPECT_TRUE(Node::assertResourcesReleased());
         }
+        EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
+        EXPECT_TRUE(Node::instanceCount() == 0);
+        EXPECT_TRUE(Node::assertResourcesReleased());
     }
 
 
     TEST(RObject_reflecting_unique_ptr, destructor_call__reflectecting_rvalue)
     {
-        const int NUM = 28228;
-        RObject robj = reflect(std::make_unique<Node>(NUM));
-        ASSERT_FALSE(robj.isEmpty());
-
-        // Check if RObject can reflect as `Node`
-        EXPECT_TRUE(robj.canViewAs<Node>());
         {
-            auto view = robj.view<Node>();
-            ASSERT_TRUE(view);
+            const int NUM = 28228;
+            RObject robj = reflect(std::make_unique<Node>(NUM));
+            ASSERT_FALSE(robj.isEmpty());
 
-            const Node& node = view->get();
-            EXPECT_EQ(node.data(), NUM);
-            // Ensure no copy is made for viewing.
-            EXPECT_TRUE(Node::instanceCount() == 1);
-        }
-        // Check if RObject can reflect as `shared_ptr<Node>`
-        EXPECT_FALSE(robj.canViewAs<std::shared_ptr<Node>>());
-        {
-            // Get a view of the view as `shared_ptr<int>`
-            auto view = robj.view<std::shared_ptr<Node>>();
-            ASSERT_FALSE(view);
-        }
-        // Check if RObject can reflect as `unique_ptr<Node>`
-        EXPECT_TRUE(robj.canViewAs<std::unique_ptr<Node>>());
-        {
-            // Get a view of the view as `shared_ptr<int>`
-            auto view = robj.view<std::unique_ptr<Node>>();
-            EXPECT_TRUE(view);
-            ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() != 0);
+            // Check if RObject can reflect as `Node`
+            EXPECT_TRUE(robj.canViewAs<Node>());
+            {
+                auto view = robj.view<Node>();
+                ASSERT_TRUE(view);
+
+                const Node& node = view->get();
+                EXPECT_EQ(node.data(), NUM);
+                // Ensure no copy is made for viewing.
+                EXPECT_TRUE(Node::instanceCount() == 1);
+            }
+            // Check if RObject can reflect as `shared_ptr<Node>`
+            EXPECT_FALSE(robj.canViewAs<std::shared_ptr<Node>>());
             {
-                std::unique_ptr<Node> movedOutPtr = view->get();
+                // Get a view of the view as `shared_ptr<int>`
+                auto view = robj.view<std::shared_ptr<Node>>();
+                ASSERT_FALSE(view);
+            }
+            // Check if RObject can reflect as `unique_ptr<Node>`
+            EXPECT_TRUE(robj.canViewAs<std::unique_ptr<Node>>());
+            {
+                // Get a view of the view as `shared_ptr<int>`
+                auto view = robj.view<std::unique_ptr<Node>>();
+                EXPECT_TRUE(view);
+                ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() != 0);
+                {
+                    // UPDATE: MOVING DISALLOWED NOW.
+                    const std::unique_ptr<Node>& movedOutPtr = view->get();
+                }
+                EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
+                EXPECT_TRUE(Node::instanceCount() == 1);
             }
-            EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
-            EXPECT_TRUE(Node::instanceCount() == 0);
-            EXPECT_TRUE(Node::assertResourcesReleased());
         }
+        EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
+        EXPECT_TRUE(Node::instanceCount() == 0);
+        EXPECT_TRUE(Node::assertResourcesReleased());
     }
 
 
@@ -366,7 +385,8 @@ namespace rtl::unit_test
                 ASSERT_TRUE(view);
                 ASSERT_FALSE(robj.isEmpty());
 
-                std::unique_ptr<Node> uptrNode = std::move(view->get());
+                // UPDATE: MOVING DISALLOWED NOW.
+                const std::unique_ptr<Node>& uptrNode = std::move(view->get());
                 EXPECT_EQ(uptrNode->data(), NUM);
             }
         }
diff --git a/ReflectionTemplateLib/rtl/detail/inc/RObjectUPtr.h b/ReflectionTemplateLib/rtl/detail/inc/RObjectUPtr.h
index 4e9f72c6..98af8df2 100644
--- a/ReflectionTemplateLib/rtl/detail/inc/RObjectUPtr.h
+++ b/ReflectionTemplateLib/rtl/detail/inc/RObjectUPtr.h
@@ -82,13 +82,8 @@ namespace rtl::detail
             return m_uniquePtr.get();
         }
 
-        std::unique_ptr<T> release() const 
-        {
-            if (m_uniquePtr) {
-                RObject::getInstanceCounter()--;//.fetch_sub(1, std::memory_order_relaxed);
-                return std::move(m_uniquePtr);
-            }
-            return nullptr;
+        const std::unique_ptr<T>& cref() const {
+            return m_uniquePtr;
         }
 
     private:
diff --git a/ReflectionTemplateLib/rtl/inc/view.hpp b/ReflectionTemplateLib/rtl/inc/view.hpp
index 552750af..2155f6fc 100644
--- a/ReflectionTemplateLib/rtl/inc/view.hpp
+++ b/ReflectionTemplateLib/rtl/inc/view.hpp
@@ -44,6 +44,12 @@ namespace rtl
 
         const detail::RObjectUPtr<T>& m_uptrRef;
 
+        //_asType move() const {
+        //    return std::move(m_uptrRef.release());
+        //}
+
+        //friend rtl::RObject;
+
     public:
 
         //  Construct from reference (no copy, no default init)
@@ -55,8 +61,11 @@ namespace rtl
         view& operator=(view&&) = delete;
         view& operator=(const view&) = delete;
 
-        _asType get() const {
-            return std::move(m_uptrRef.release());
+        // UPDATE: MOVING DISALLOWED NOW.
+	    // cannot move a std::unique_ptr out once its inside rtl::RObject. it can only be viewed.
+	    // done for keeping the rtl::view strictly Read-only and a consistent interface.
+        const _asType& get() const {
+            return m_uptrRef.cref();
         }
     };
 }
\ No newline at end of file

From 8d8a3a7726b986b77a8bd60c9f33c0396097e2b9 Mon Sep 17 00:00:00 2001
From: neeraj31285 <neeraj.singh31285@gmail.com>
Date: Sat, 24 Jan 2026 18:42:12 +0530
Subject: [PATCH 2/3] cleanup.

---
 .../RObjectReflecting_stdUniquePtr.cpp        | 70 ++-----------------
 1 file changed, 5 insertions(+), 65 deletions(-)

diff --git a/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp b/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp
index 870eb981..ca949a9f 100644
--- a/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp
+++ b/RTLTestRunApp/src/RObjectTests/RObjectReflecting_stdUniquePtr.cpp
@@ -8,6 +8,8 @@
 using namespace test_utils;
 using namespace rtl;
 
+// TODO: Refine these, remove moot cases after this-> UPDATE: MOVING DISALLOWED NOW.
+
 namespace rtl::unit_test
 {
     TEST(RObject_reflecting_unique_ptr, clone_on__heap_stack)
@@ -63,10 +65,8 @@ namespace rtl::unit_test
             // Access the moved-out value
             EXPECT_EQ(*uptr, NUM);
 
+            // this is compile error now.
             //int* ptr = uptr.release();
-            //// Addresses must be same.
-            //EXPECT_EQ(numPtr, ptr);
-            //delete ptr;     //RTL must not delete again, once 'robj' out of scope.
         }
         // there must not be any crash.
         ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 0);
@@ -151,33 +151,6 @@ namespace rtl::unit_test
 
         // RTL still owns it
         ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
-
-        // RObject still exists with correct metadata, but the stored unique_ptr is now empty
-        ASSERT_FALSE(robj.isEmpty());
-
-        // UPDATE: MOVING DISALLOWED NOW.
-        // Any subsequent view will still be obtainable
-        auto view3 = robj.view<std::unique_ptr<int>>();
-        ASSERT_TRUE(view3);
-
-        // UPDATE: MOVING DISALLOWED NOW.
-        // But the unique_ptr inside is now empty due to the earlier move
-        const std::unique_ptr<int>& uptr3 = view3->get();
-        //ASSERT_TRUE(uptr3 == nullptr);
-
-        // UPDATE: MOVING DISALLOWED NOW.
-        // All earlier views now yield empty unique_ptrs as well- no dangling pointers, no UB
-        const std::unique_ptr<int>& uptr2 = view2->get();
-        //ASSERT_TRUE(uptr3 == nullptr);
-
-        // UPDATE: MOVING DISALLOWED NOW.
-        const std::unique_ptr<int>& uptr1 = view1->get();
-        //ASSERT_TRUE(uptr3 == nullptr);
-
-        // UPDATE: MOVING DISALLOWED NOW.
-        // Even reusing the moved-from view0 is safe- just returns empty
-        const std::unique_ptr<int>& uptr00 = view0->get();
-        //ASSERT_TRUE(uptr00 == nullptr);
     }
 
 
@@ -235,29 +208,6 @@ namespace rtl::unit_test
                 ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
                 // Node still exists.
                 EXPECT_TRUE(Node::instanceCount() == 1);
-
-                // Any subsequent view will still be obtainable
-                auto view3 = robj.view<std::unique_ptr<Node>>();
-                ASSERT_TRUE(view3);
-
-                // UPDATE: MOVING DISALLOWED NOW.
-                // But the unique_ptr inside is now empty due to the earlier move
-                const std::unique_ptr<Node>& uptr3 = view3->get();
-                //ASSERT_TRUE(uptr3 == nullptr);
-
-                // UPDATE: MOVING DISALLOWED NOW.
-                // All earlier views now yield empty unique_ptrs as well- no dangling pointers, no UB
-                const std::unique_ptr<Node>& uptr2 = view2->get();
-                //ASSERT_TRUE(uptr3 == nullptr);
-
-                // UPDATE: MOVING DISALLOWED NOW.
-                const std::unique_ptr<Node>& uptr1 = view1->get();
-                //ASSERT_TRUE(uptr3 == nullptr);
-
-                // UPDATE: MOVING DISALLOWED NOW.
-                // Even reusing the moved-from view0 is safe- just returns empty
-                const std::unique_ptr<Node>& uptr00 = view0->get();
-                //ASSERT_TRUE(uptr00 == nullptr);
             }
             EXPECT_TRUE(Node::instanceCount() == 1);
             ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
@@ -293,7 +243,7 @@ namespace rtl::unit_test
             {
                 // Get a view of the view as `shared_ptr<int>`
                 auto view = robj.view<std::shared_ptr<Node>>();
-                ASSERT_FALSE(view);
+                EXPECT_FALSE(view);
             }
             // Check if RObject can reflect as `unique_ptr<Node>`
             EXPECT_TRUE(robj.canViewAs<std::unique_ptr<Node>>());
@@ -301,11 +251,6 @@ namespace rtl::unit_test
                 // Get a view of the view as `shared_ptr<int>`
                 auto view = robj.view<std::unique_ptr<Node>>();
                 EXPECT_TRUE(view);
-                ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() != 0);
-                {
-                    // UPDATE: MOVING DISALLOWED NOW.
-                    const std::unique_ptr<Node>& movedOutPtr = view->get();
-                }
                 EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
                 EXPECT_TRUE(Node::instanceCount() == 1);
             }
@@ -347,11 +292,6 @@ namespace rtl::unit_test
                 // Get a view of the view as `shared_ptr<int>`
                 auto view = robj.view<std::unique_ptr<Node>>();
                 EXPECT_TRUE(view);
-                ASSERT_TRUE(rtl::getRtlManagedHeapInstanceCount() != 0);
-                {
-                    // UPDATE: MOVING DISALLOWED NOW.
-                    const std::unique_ptr<Node>& movedOutPtr = view->get();
-                }
                 EXPECT_TRUE(rtl::getRtlManagedHeapInstanceCount() == 1);
                 EXPECT_TRUE(Node::instanceCount() == 1);
             }
@@ -386,7 +326,7 @@ namespace rtl::unit_test
                 ASSERT_FALSE(robj.isEmpty());
 
                 // UPDATE: MOVING DISALLOWED NOW.
-                const std::unique_ptr<Node>& uptrNode = std::move(view->get());
+                const std::unique_ptr<Node>& uptrNode = view->get();
                 EXPECT_EQ(uptrNode->data(), NUM);
             }
         }

From 634a6244dffb5d37fc9dc1a04dd45c5adffe6cfb Mon Sep 17 00:00:00 2001
From: neeraj31285 <neeraj.singh31285@gmail.com>
Date: Sun, 25 Jan 2026 00:17:31 +0530
Subject: [PATCH 3/3] ignore test-project for codecov.

---
 .codecov.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.codecov.yml b/.codecov.yml
index bd061d04..1d20178a 100644
--- a/.codecov.yml
+++ b/.codecov.yml
@@ -14,5 +14,6 @@ comment:
 ignore:
   - "CxxTestProps/**"
   - "CxxTestUtils/**"
+  - "RTLTestRunApp/**"
   - "RTLBenchmarkApp/**"
   - "ReflectionTemplateLib/rtl/detail/src/RObjectConverters_string.cpp"
\ No newline at end of file