Do not sync empty contacts not having URNs set

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

Hide file tree

Showing 2 changed files with 23 additions and 4 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
         )
@@ -271,10 +274,7 @@ def local_kwargs(self, org, remote):
             else:
                 gender = ""
 
-        scheme = ""
-        if remote.urns:
-            primary_urn = remote.urns[0]
-            scheme, path = primary_urn.split(":", 1)
+        scheme, path = remote.urns[0].split(":", 1)
 
         return {
             "backend": self.backend,

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",