Merge pull request #681 from atlanhq/APP-7697

APP-7697 : custom metadata object created via Python

Author: Aryamanz29

pyatlan/model/typedef.py

@@ -504,7 +504,7 @@ class Options(AtlanObject):
 
         def __setattr__(self, name, value):
             super().__setattr__(name, value)
-            if self._attr_def and name == "multi_value_select":
+            if self._attr_def and name == "multi_value_select" and value is True:
                 self._attr_def.cardinality = Cardinality.SET
                 if self._attr_def.type_name and "array<" not in str(
                     self._attr_def.type_name

tests/unit/test_typedef_model.py

@@ -16,6 +16,7 @@
 from pyatlan.model.typedef import (
     AtlanTagDef,
     AttributeDef,
+    Cardinality,
     CustomMetadataDef,
     EntityDef,
     EnumDef,
@@ -443,3 +444,33 @@ def test_attribute_create_with_limited_applicability(self, client: AtlanClient):
             assert getattr(options, attribute) == json.dumps(
                 list(applicable_kwargs.get(attribute))  # type: ignore[arg-type]
             )
+
+    def test_multi_value_select_setter_condition(self, client: AtlanClient):
+        with patch("pyatlan.model.typedef._get_all_qualified_names") as mock_get_qa:
+            mock_get_qa.return_value = set()
+
+            attr_def = AttributeDef.create(
+                client=client,
+                display_name="Test Attribute",
+                attribute_type=AtlanCustomAttributePrimitiveType.STRING,
+            )
+
+            assert attr_def.options is not None
+            assert attr_def.options.multi_value_select is False
+            assert attr_def.cardinality == Cardinality.SINGLE
+            assert attr_def.type_name == AtlanCustomAttributePrimitiveType.STRING.value
+
+            # Test 1: Setting multi_value_select to False should NOT trigger __setattr__ logic
+            attr_def.options.multi_value_select = False
+
+            assert attr_def.cardinality == Cardinality.SINGLE
+            assert attr_def.type_name == AtlanCustomAttributePrimitiveType.STRING.value
+
+            # Test 2: Setting multi_value_select to True should trigger __setattr__ logic
+            attr_def.options.multi_value_select = True
+
+            assert attr_def.cardinality == Cardinality.SET
+            assert (
+                attr_def.type_name
+                == f"array<{AtlanCustomAttributePrimitiveType.STRING.value}>"
+            )