Merge pull request #829 from atlanhq/revert-820-fix/dynamo-db-attribute-parent-qn-validation

Revert "fix(assets): validate parent_qualified_name in DynamoDBAttribute creator"

Author: Aryamanz29

pyatlan/model/assets/dynamo_d_b_attribute.py

@@ -1914,9 +1914,6 @@ def creator(
                 ["name", "parent_qualified_name", "order"],
                 [name, parent_qualified_name, order],
             )
-            fields = parent_qualified_name.split("/")
-            if len(fields) != 4:
-                raise ValueError("Invalid parent_qualified_name")
             if connection_qualified_name:
                 connector_name = AtlanConnectorType.get_connector_name(
                     connection_qualified_name
@@ -1928,6 +1925,7 @@ def creator(
             if order < 0:
                 raise ValueError("Order must be be a positive integer")
 
+            fields = parent_qualified_name.split("/")
             qualified_name = f"{parent_qualified_name}/{name}"
             connection_qualified_name = connection_qualified_name or connection_qn
             table_name = parent_name or fields[3]

tests/unit/model/dynamo_db_attribute_test.py

@@ -108,25 +108,6 @@ def test_creator_with_explicit_connection_qualified_name():
     assert sut.order == 0
 
 
-@pytest.mark.parametrize(
-    "parent_qualified_name",
-    [
-        DYNAMODB_CONNECTION_QUALIFIED_NAME,
-        DYNAMODB_ATTRIBUTE_QUALIFIED_NAME,
-    ],
-)
-def test_creator_with_connection_qn_and_invalid_parent_raises_value_error(
-    parent_qualified_name,
-):
-    with pytest.raises(ValueError, match="Invalid parent_qualified_name"):
-        DynamoDBAttribute.creator(
-            name=DYNAMODB_ATTRIBUTE_NAME,
-            parent_qualified_name=parent_qualified_name,
-            order=1,
-            connection_qualified_name=DYNAMODB_CONNECTION_QUALIFIED_NAME,
-        )
-
-
 @pytest.mark.parametrize(
     "qualified_name, name, message",
     [