Safe repr for Group (#1890)

Author: nguyenv

HISTORY.md

@@ -1,5 +1,9 @@
 # Next
 
+## Bug Fixes
+
+* Add safe `Group.__repr__` [#1890](https://github.com/TileDB-Inc/TileDB-Py/pull/1890)
+
 ## Improvements
 
 * Warn when `os.fork()` is used in the presence of a Tiledb context [#1876](https://github.com/TileDB-Inc/TileDB-Py/pull/1876/files).

tiledb/group.py

@@ -389,6 +389,10 @@ def __contains__(self, member: str) -> bool:
         return self._has_member(member)
 
     def __repr__(self):
+        # use safe repr if pybind11 constructor failed
+        if self._ctx is None:
+            return object.__repr__(self)
+
         return self._dump(True)
 
     def __enter__(self):

tiledb/tests/test_group.py

@@ -342,6 +342,10 @@ def test_set_config(self):
             with tiledb.Group(group_uri, config=cfg) as G:
                 assert len(G) == sz
 
+    def test_group_does_not_exist(self):
+        with self.assertRaises(tiledb.TileDBError):
+            tiledb.Group("does-not-exist")
+
 
 class GroupMetadataTest(GroupTestCase):
     @pytest.mark.parametrize(