getter functions for MemorySlot properties

research_analytics_suite/data_engine/memory/MemoryManager.py

@@ -125,8 +125,8 @@ async def create_slot(self, name: str, data: any, db_path: str = None, file_path
         _id = uuid.uuid4().hex[:8]
         memory_slot = MemorySlot(memory_id=_id, name=name, data=data, db_path=db_path, file_path=file_path)
         await memory_slot.setup()
-        self._data_cache.set(key=memory_slot._memory_id, data=memory_slot)
-        return memory_slot._memory_id
+        self._data_cache.set(key=memory_slot.memory_id, data=memory_slot)
+        return memory_slot.memory_id
 
     async def update_slot(self, memory_id: str, data: any) -> str:
         """
@@ -159,8 +159,8 @@ async def delete_slot(self, memory_id: str) -> None:
         memory_slot = self._data_cache.get_key(key=memory_id)
         if memory_slot:
             memory_slot.close()
-            if memory_slot._file_path:
-                os.remove(memory_slot._file_path)  # Remove memory-mapped file if it exists
+            if memory_slot.file_path:
+                os.remove(memory_slot.file_path)  # Remove memory-mapped file if it exists
             self._data_cache.delete(key=memory_id)
 
     async def list_slots(self) -> dict:

research_analytics_suite/data_engine/memory/MemorySlot.py

@@ -115,6 +115,24 @@ def name(self, value: str) -> None:
         """
         self._name = value
 
+    @property
+    def memory_id(self) -> str:
+        """Gets the unique identifier for the memory slot.
+
+        Returns:
+            str: The unique identifier for the memory slot.
+        """
+        return self._memory_id or ""
+
+    @property
+    def file_path(self) -> str:
+        """Gets the file path for memory-mapped storage.
+
+        Returns:
+            str: The file path for memory-mapped storage.
+        """
+        return self._file_path or ""
+
     @property
     async def data(self) -> any:
         """Gets the data stored in the memory slot."""

tests/data_engine/memory/test_memory_manager.py

@@ -112,7 +112,7 @@ async def test_update_slot(self):
     async def test_delete_slot(self):
         memory_id = "12345678"
         mock_slot = MagicMock(spec=MemorySlot)
-        mock_slot._file_path = "test_mmap.dat"
+        mock_slot.file_path = "test_mmap.dat"
         self.memory_manager._data_cache.get_key.return_value = mock_slot
 
         with patch('os.remove', MagicMock()) as mock_remove: