final touches

Titan-Systems · Dec 18, 2024 · 74d7518 · 74d7518
1 parent 7522a5f
commit 74d7518
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/tests/test_blueprint.py b/tests/test_blueprint.py
@@ -643,8 +643,7 @@ def test_blueprint_vars_in_owner(session_ctx):
         resources=[res.Schema(name="schema", owner="role_{{ var.role_name }}", database="STATIC_DATABASE")],
         vars={"role_name": "role123"},
     )
-    manifest = blueprint.generate_manifest(session_ctx)
-    # assert manifest.resources[1].data["owner"] == "role_{{ var.role_name }}"
+    assert blueprint.generate_manifest(session_ctx)
 
 
 def test_blueprint_allowlist(session_ctx, remote_state):

diff --git a/titan/resources/resource.py b/titan/resources/resource.py
@@ -93,7 +93,6 @@ def _coerce_resource_field(field_value, field_type):
             return VarString(field_value)
         elif isinstance(field_value, (Resource, VarString, str)):
             return convert_role_ref(field_value)
-            # return field_value
         else:
             raise TypeError
 
@@ -180,9 +179,6 @@ def to_dict(self, account_edition: AccountEdition):
 
         def _serialize_field(field, value):
             if field.name == "owner":
-                # if isinstance(value, str):
-                #     return value
-                # else:
                 return str(value.fqn)
             elif isinstance(value, ResourcePointer):
                 return str(value.fqn)