[FIX] create test_10 field with correct type from start to avoid type change

Author: emjay0921

spp_custom_fields_ui/CHANGELOG.md

@@ -2,11 +2,12 @@
 
 ## 2025-11-20
 
-### 2025-11-20 15:30:00 - [FIX] remove ttype change from test_10_onchange_field_category
+### 2025-11-20 15:45:00 - [FIX] create test_10 field with correct type from start to avoid type change
 
-- Removed field.ttype = "integer" line that was causing type change errors
-- Simplified assertion to verify field_category is set
-- Focuses on codecov coverage without triggering type change errors
+- Changed test_10 to create field with ttype="integer" and field_category="ind" from start
+- Updated field name to x_ind_grp_test_category to match indicator type
+- Follows same pattern as test_11 to avoid type change errors
+- Proper assertion now verifies compute field is set correctly
 
 ### 2025-11-20 15:15:00 - [FIX] remove test_13_onchange_has_presence and add coverage call to test_12
 

spp_custom_fields_ui/tests/test_custom_fields_ui.py

@@ -201,23 +201,20 @@ def test_10_onchange_field_category(self):
         """Test _onchange_field_category updates compute field"""
         field = self.field_model.create(
             {
-                "name": "x_cst_grp_test_category",
+                "name": "x_ind_grp_test_category",
                 "model_id": self.model_id.id,
                 "field_description": "Test Category Change",
-                "ttype": "char",
+                "ttype": "integer",
                 "state": "manual",
                 "target_type": "grp",
-                "field_category": "cst",
+                "field_category": "ind",
             }
         )
-        self.assertFalse(field.compute)
 
-        # Change to indicator category and call onchange
-        field.field_category = "ind"
+        # Call onchange to ensure it executes set_compute
         field._onchange_field_category()
 
-        # Verify the method was called (may not set compute due to type mismatch, but ok for coverage)
-        self.assertIsNotNone(field.field_category)
+        self.assertTrue(field.compute, "Compute field should be set when field_category is indicator")
 
     def test_11_onchange_kinds(self):
         """Test _onchange_kinds updates compute field"""