Do not sync empty contacts not having URNs set

rapidpro · Sep 9, 2024 · b3319a7 · b3319a7
1 parent 8eea669
commit b3319a7
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/ureport/backend/rapidpro.py b/ureport/backend/rapidpro.py
@@ -174,6 +174,9 @@ def local_kwargs(self, org, remote):
         if reporter_group.lower() not in contact_groups_names:
             return None
 
+        if not remote.urns:
+            return None
+
         org_state_boundaries_data, org_district_boundaries_data, org_ward_boundaries_data = self.get_boundaries_data(
             org
         )

diff --git a/ureport/backend/tests/test_rapidpro.py b/ureport/backend/tests/test_rapidpro.py
@@ -344,6 +344,25 @@ def test_local_kwargs(self):
 
         self.assertIsNone(self.syncer.local_kwargs(self.nigeria, temba_contact))
 
+        temba_contact = TembaContact.create(
+            uuid="C-006",
+            name="Jan",
+            urns=[],
+            groups=[ObjectRef.create(uuid="G-001", name="ureporters"), ObjectRef.create(uuid="G-007", name="Actors")],
+            fields={
+                "registration_date": None,
+                "state": None,
+                "lga": None,
+                "occupation": None,
+                "born": None,
+                "gender": None,
+            },
+            language="eng",
+            created_on=json_date_to_datetime("2013-01-02T03:04:05.000"),
+        )
+
+        self.assertIsNone(self.syncer.local_kwargs(self.nigeria, temba_contact))
+
         temba_contact = TembaContact.create(
             uuid="C-006",
             name="Jan",