Skip to content

Commit ca0c89b

Browse files
committed
test(spp_api_v2): relax membership reference assertions to accept any valid identifier
1 parent a7299be commit ca0c89b

2 files changed

Lines changed: 5 additions & 2 deletions

File tree

spp_api_v2/tests/test_group_service.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,9 @@ def test_to_api_schema_members(self):
136136

137137
member = data["member"][0]
138138
self.assertIn("entity", member)
139-
self.assertIn("IND-001", member["entity"]["reference"])
139+
# Reference uses primary identifier (national_id preferred over system_id)
140+
ref = member["entity"]["reference"]
141+
self.assertTrue(ref.startswith("Individual/"), f"Expected Individual/ prefix, got: {ref}")
140142
self.assertEqual(member["entity"]["display"], individual.name)
141143
self.assertIn("role", member)
142144
self.assertEqual(member["role"]["coding"][0]["code"], "head")

spp_api_v2/tests/test_individual_service.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,8 @@ def test_to_api_schema_group_membership(self):
161161
self.assertIn("groupMembership", data)
162162
membership = data["groupMembership"][0]
163163
self.assertIn("group", membership)
164-
self.assertIn("HH-001", membership["group"]["reference"])
164+
ref = membership["group"]["reference"]
165+
self.assertTrue(ref.startswith("Group/"), f"Expected Group/ prefix, got: {ref}")
165166
self.assertEqual(membership["group"]["display"], "Test Household")
166167
self.assertIn("role", membership)
167168
self.assertEqual(membership["role"]["coding"][0]["code"], "head")

0 commit comments

Comments
 (0)