Move get_parent_handles/get_child_handles to DbReadBase; fix proxy traversal.

Move the Python base implementations of get_parent_handles and
get_child_handles from DbGeneric up to DbReadBase so that proxy
databases (PrivateProxyDb, LivingProxyDb, etc.) inherit them
automatically. Previously, calling these methods through a proxy
would fall through __getattr__ to the SQL implementation, bypassing
proxy filtering. Now proxies use the base Python implementation which
calls get_raw_person_data/get_raw_family_data through the proxy's own
filtering layer.

Also rename test_graph.py to graph_test.py so CI discovers it via
the *_test.py pattern, and add proxy tests for get_parent_handles
and get_child_handles to cover this path.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: dsblank

gramps/gen/db/base.py

@@ -967,13 +967,40 @@ def get_parent_handles(self, person_handle, first_only=True):
         """
         Return handles of all parents of a person across their parent families.
         """
-        raise NotImplementedError
+        person = self.get_raw_person_data(person_handle)
+        if person is None:
+            return []
+        families = person.parent_family_list
+        if first_only and families:
+            families = [families[0]]
+        result = []
+        for fam_handle in families:
+            family = self.get_raw_family_data(fam_handle)
+            if family is None:
+                continue
+            if family.father_handle:
+                result.append(family.father_handle)
+            if family.mother_handle:
+                result.append(family.mother_handle)
+        return result
 
     def get_child_handles(self, person_handle, first_only=True):
         """
         Return handles of all children of a person across their families.
         """
-        raise NotImplementedError
+        person = self.get_raw_person_data(person_handle)
+        if person is None:
+            return []
+        families = person.family_list
+        if first_only and families:
+            families = [families[0]]
+        result = []
+        for fam_handle in families:
+            family = self.get_raw_family_data(fam_handle)
+            if family is None:
+                continue
+            result.extend(cr.ref for cr in family.child_ref_list if cr.ref)
+        return result
 
     def get_raw_place_data(self, handle):
         """

gramps/gen/db/generic.py

@@ -1867,45 +1867,6 @@ def get_raw_person_data(self, handle):
     def get_raw_family_data(self, handle):
         return self._get_raw_data(FAMILY_KEY, handle)
 
-    def get_parent_handles(self, person_handle, first_only=True):
-        """
-        Return handles of all parents of a person across their parent families.
-        """
-        person = self.get_raw_person_data(person_handle)
-        if person is None:
-            return []
-        families = person.parent_family_list
-        if first_only and families:
-            families = [families[0]]
-        result = []
-        for fam_handle in families:
-            family = self.get_raw_family_data(fam_handle)
-            if family is None:
-                continue
-            if family.father_handle:
-                result.append(family.father_handle)
-            if family.mother_handle:
-                result.append(family.mother_handle)
-        return result
-
-    def get_child_handles(self, person_handle, first_only=True):
-        """
-        Return handles of all children of a person across their families.
-        """
-        person = self.get_raw_person_data(person_handle)
-        if person is None:
-            return []
-        families = person.family_list
-        if first_only and families:
-            families = [families[0]]
-        result = []
-        for fam_handle in families:
-            family = self.get_raw_family_data(fam_handle)
-            if family is None:
-                continue
-            result.extend(cr.ref for cr in family.child_ref_list if cr.ref)
-        return result
-
     def get_raw_source_data(self, handle):
         return self._get_raw_data(SOURCE_KEY, handle)
 

gramps/gen/proxy/test/proxies_test.py

@@ -112,6 +112,32 @@ def test_person_with_private_data(self):
         person = self.db.basedb.get_person_from_handle(handle)
         self.assertTrue(len(person.attribute_list) == 3)
 
+    def test_get_parent_handles_private_person(self):
+        # A private person's parents should not be returned through the proxy.
+        handle = "0GDKQC54XKSWZKEBWW"
+        parents = self.db.get_parent_handles(handle)
+        self.assertEqual(parents, [])
+
+    def test_get_parent_handles_non_private_person(self):
+        # A non-private person's parents should match the base db.
+        handle = "0FWJQCLYEP736P3YZK"
+        parents = self.db.get_parent_handles(handle)
+        parents_base = self.db.basedb.get_parent_handles(handle)
+        self.assertEqual(sorted(parents), sorted(parents_base))
+
+    def test_get_child_handles_private_person(self):
+        # A private person's children should not be returned through the proxy.
+        handle = "0GDKQC54XKSWZKEBWW"
+        children = self.db.get_child_handles(handle)
+        self.assertEqual(children, [])
+
+    def test_get_child_handles_non_private_person(self):
+        # A non-private person's children should match the base db.
+        handle = "0FWJQCLYEP736P3YZK"
+        children = self.db.get_child_handles(handle)
+        children_base = self.db.basedb.get_child_handles(handle)
+        self.assertEqual(sorted(children), sorted(children_base))
+
 
 class LivingProxyTest(unittest.TestCase):
     """
@@ -155,6 +181,32 @@ def test_dead_person(self):
         person = self.db.get_person_from_handle(handle)
         self.assertIs(person, None)
 
+    def test_get_parent_handles_dead_person(self):
+        # A living-excluded person's parents should not be returned.
+        handle = "66TJQC6CC7ZWL9YZ64"
+        parents = self.db.get_parent_handles(handle)
+        self.assertEqual(parents, [])
+
+    def test_get_parent_handles_live_person(self):
+        # A visible person's parents should match the base db.
+        handle = "004KQCGYT27EEPQHK"
+        parents = self.db.get_parent_handles(handle)
+        parents_base = self.db.basedb.get_parent_handles(handle)
+        self.assertEqual(sorted(parents), sorted(parents_base))
+
+    def test_get_child_handles_dead_person(self):
+        # A living-excluded person's children should not be returned.
+        handle = "66TJQC6CC7ZWL9YZ64"
+        children = self.db.get_child_handles(handle)
+        self.assertEqual(children, [])
+
+    def test_get_child_handles_live_person(self):
+        # A visible person's children should match the base db.
+        handle = "004KQCGYT27EEPQHK"
+        children = self.db.get_child_handles(handle)
+        children_base = self.db.basedb.get_child_handles(handle)
+        self.assertEqual(sorted(children), sorted(children_base))
+
 
 class LivingPrivateProxyTest(unittest.TestCase):
     """