G2P-2555: Test cases resolved

Author: Dibik

g2p_registry_base/tests/test_reg_id.py

@@ -16,7 +16,13 @@ def test_01_display_name(self):
         id_type = self.id_type_model.create({"name": "Test ID Type", "id_validation": "[0-9]+"})
 
         reg_id = self.reg_id_model.create(
-            {"partner_id": partner.id, "id_type": id_type.id, "value": "123456" ,"status":"valid", "description":"Due to API"}
+            {
+                "partner_id": partner.id,
+                "id_type": id_type.id,
+                "value": "123456",
+                "status": "valid",
+                "description": "Due to API",
+            }
         )
 
         # Call the _compute_display_name method
@@ -35,7 +41,13 @@ def test_02_create_registrant_id(self):
         id_type = self.id_type_model.create({"name": "Test ID Type", "id_validation": "[0-9]+"})
 
         reg_id = self.reg_id_model.create(
-            {"partner_id": partner.id, "id_type": id_type.id, "value": "123456", "status":"valid", "description":"Due to API"}
+            {
+                "partner_id": partner.id,
+                "id_type": id_type.id,
+                "value": "123456",
+                "status": "valid",
+                "description": "Due to API",
+            }
         )
         self.assertEqual(reg_id.value, "123456", "Registrant ID value is not as expected.")
 
@@ -44,15 +56,29 @@ def test_03_invalid_id_value(self):
         id_type = self.id_type_model.create({"name": "Test ID Type", "id_validation": "[0-9]+"})
 
         with self.assertRaises(ValidationError) as context:
-            self.reg_id_model.create({"partner_id": partner.id, "id_type": id_type.id, "value": "abc", "status":"valid", "description":"Due to API"})
+            self.reg_id_model.create(
+                {
+                    "partner_id": partner.id,
+                    "id_type": id_type.id,
+                    "value": "abc",
+                    "status": "valid",
+                    "description": "Due to API",
+                }
+            )
 
         self.assertIn("The provided Test ID Type ID 'abc' is invalid.", str(context.exception))
 
     def test_04_name_search(self):
         partner = self.partner_model.create({"name": "Test Partner", "is_registrant": True})
         id_type = self.id_type_model.create({"name": "Test ID Type"})
         reg_id = self.reg_id_model.create(
-            {"partner_id": partner.id, "id_type": id_type.id, "value": "Test Value","status":"valid", "description":"Due to API"}
+            {
+                "partner_id": partner.id,
+                "id_type": id_type.id,
+                "value": "Test Value",
+                "status": "valid",
+                "description": "Due to API",
+            }
         )
         search_result = self.reg_id_model._name_search("Test Partner")
         self.assertIn(reg_id.id, search_result, "Expected record not found in search result")

g2p_registry_group/tests/test_groups.py

@@ -115,7 +115,12 @@ def test_02_add_id(self):
                 "name": "Testing ID Type",
             }
         )
-        vals = {"id_type": id_type.id, "value": "112233445566778899"}
+        vals = {
+            "id_type": id_type.id,
+            "value": "112233445566778899",
+            "status": "valid",
+            "description": "Due to API",
+        }
 
         self.group_1.write({"reg_ids": [(0, 0, vals)]})
         expected_value = "112233445566778899"

g2p_registry_individual/tests/test_individuals.py

@@ -139,7 +139,12 @@ def test_04_add_id(self):
                 "name": "Testing ID Type",
             }
         )
-        vals = {"id_type": id_type.id, "value": "112233445566778899"}
+        vals = {
+            "id_type": id_type.id,
+            "value": "112233445566778899",
+            "status": "valid",
+            "description": "Due to API",
+        }
 
         self.registrant_1.write({"reg_ids": [(0, 0, vals)]})
         expected_value = "112233445566778899"