Skip to content

Commit 92c8ac9

Browse files
[FIX] spp_registry_name_suffix: add placeholder name to satisfy check constraint
Tests were failing because res.partner has a check constraint (res_partner_check_name) requiring a non-empty name at INSERT time. Since name_change() is an onchange method that doesn't run during programmatic create, we provide a temporary placeholder name that gets overwritten when name_change() is called.
1 parent 238b623 commit 92c8ac9

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

spp_registry_name_suffix/tests/test_name_suffix.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ def test_03_name_with_suffix(self):
4040
"""Test that suffix is appended to the computed name."""
4141
individual = self.env["res.partner"].create(
4242
{
43+
"name": "Temp", # Required by res_partner_check_name constraint
4344
"family_name": "Doe",
4445
"given_name": "John",
4546
"suffix_id": self.suffix_jr.id,
@@ -59,6 +60,7 @@ def test_04_name_without_suffix(self):
5960
"""Test that name is computed correctly without suffix."""
6061
individual = self.env["res.partner"].create(
6162
{
63+
"name": "Temp", # Required by res_partner_check_name constraint
6264
"family_name": "Doe",
6365
"given_name": "Jane",
6466
"is_registrant": True,
@@ -76,6 +78,7 @@ def test_05_name_with_all_fields(self):
7678
"""Test name with all fields including addl_name and suffix."""
7779
individual = self.env["res.partner"].create(
7880
{
81+
"name": "Temp", # Required by res_partner_check_name constraint
7982
"family_name": "Smith",
8083
"given_name": "Robert",
8184
"addl_name": "James",
@@ -113,6 +116,7 @@ def test_07_suffix_update_triggers_name_change(self):
113116
"""Test that updating suffix and calling name_change updates name."""
114117
individual = self.env["res.partner"].create(
115118
{
119+
"name": "Temp", # Required by res_partner_check_name constraint
116120
"family_name": "Johnson",
117121
"given_name": "Michael",
118122
"is_registrant": True,
@@ -136,6 +140,7 @@ def test_08_suffix_removal(self):
136140
"""Test that removing suffix updates the name correctly."""
137141
individual = self.env["res.partner"].create(
138142
{
143+
"name": "Temp", # Required by res_partner_check_name constraint
139144
"family_name": "Williams",
140145
"given_name": "Sarah",
141146
"suffix_id": self.suffix_jr.id,

0 commit comments

Comments
 (0)