From af8759a9c1d965a3220024d1857aea219d62d898 Mon Sep 17 00:00:00 2001
From: Marcos Magueta <maguetamarcos@gmail.com>
Date: Mon, 18 May 2026 12:50:25 -0300
Subject: Branching and object registry rework (#11)

---
 test/ObjectManager.cpp | 75 ++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 75 insertions(+)
 create mode 100644 test/ObjectManager.cpp

(limited to 'test/ObjectManager.cpp')

diff --git a/test/ObjectManager.cpp b/test/ObjectManager.cpp
new file mode 100644
index 0000000..e97bcb2
--- /dev/null
+++ b/test/ObjectManager.cpp
@@ -0,0 +1,75 @@
+#include <catch2/catch_test_macros.hpp>
+
+#include "ObjectManager.h"
+
+#include <memory>
+
+// ---------------------------------------------------------------------------
+// Step 5 — Unregister
+//
+// Sessions are disposable and must be removable from the registry on close.
+// Unregister is the splice-out primitive that rnt_session_close, the
+// LifecycleManager TryCollect path, and any future explicit-delete API all
+// share.
+// ---------------------------------------------------------------------------
+
+namespace
+{
+    std::unique_ptr<nt::ObjectManager::object_type> session_type()
+    {
+        auto t      = std::make_unique<nt::ObjectManager::object_type>();
+        t->label    = SESSION;
+        t->disposable = true;
+        t->methods  = { OPEN, CLOSE };
+        return t;
+    }
+}
+
+TEST_CASE("Unregister removes the entry and Find returns nullptr", "[step5][object-manager]")
+{
+    nt::ObjectManager om;
+    om.Register({"system", "sessions", "alpha"},
+                std::make_unique<nt::ObjectManager::Session>(),
+                session_type());
+    REQUIRE(om.Find({"system", "sessions", "alpha"}) != nullptr);
+
+    REQUIRE(om.Unregister({"system", "sessions", "alpha"}) == true);
+    REQUIRE(om.Find({"system", "sessions", "alpha"}) == nullptr);
+}
+
+TEST_CASE("Unregister returns false for missing paths", "[step5][object-manager]")
+{
+    nt::ObjectManager om;
+    REQUIRE(om.Unregister({"never", "registered"}) == false);
+}
+
+TEST_CASE("Unregister splices a middle entry without disturbing neighbours",
+          "[step5][object-manager]")
+{
+    nt::ObjectManager om;
+    om.Register({"a"}, std::make_unique<nt::ObjectManager::Session>(), session_type());
+    om.Register({"b"}, std::make_unique<nt::ObjectManager::Session>(), session_type());
+    om.Register({"c"}, std::make_unique<nt::ObjectManager::Session>(), session_type());
+
+    REQUIRE(om.Unregister({"b"}) == true);
+
+    REQUIRE(om.Find({"a"}) != nullptr);
+    REQUIRE(om.Find({"b"}) == nullptr);
+    REQUIRE(om.Find({"c"}) != nullptr);
+}
+
+TEST_CASE("Unregister at the head of the list works", "[step5][object-manager]")
+{
+    nt::ObjectManager om;
+    om.Register({"first"},  std::make_unique<nt::ObjectManager::Session>(), session_type());
+    om.Register({"second"}, std::make_unique<nt::ObjectManager::Session>(), session_type());
+
+    // The most recently registered entry is the list head (LIFO).
+    REQUIRE(om.Unregister({"second"}) == true);
+    REQUIRE(om.Find({"second"}) == nullptr);
+    REQUIRE(om.Find({"first"})  != nullptr);
+
+    // And the new head is removable too.
+    REQUIRE(om.Unregister({"first"}) == true);
+    REQUIRE(om.Find({"first"}) == nullptr);
+}
-- 
cgit v1.2.3