style: apply ruff-format and fix stray test assertions

Run ruff-format on changed files. Remove DuckDB attach assertions
accidentally included in test_config_ownership_defaults_to_empty —
those are already covered by test_load_duckdb_attach_config.

Signed-off-by: Gabe Pesco <gabe.pesco@milliman.com>
Signed-off-by: Gabe Pesco <PescoG@medinsight.milliman.com>

Author: Gabe Pesco

sqlmesh/core/config/ownership.py

@@ -11,9 +11,7 @@
 if t.TYPE_CHECKING:
     OwnershipMapping = t.Dict[re.Pattern, str]
 else:
-    OwnershipMapping = t.Annotated[
-        t.Dict[re.Pattern, str], BeforeValidator(compile_regex_mapping)
-    ]
+    OwnershipMapping = t.Annotated[t.Dict[re.Pattern, str], BeforeValidator(compile_regex_mapping)]
 
 
 class OwnershipConfig(BaseConfig):

tests/core/engine_adapter/test_spark.py

@@ -1095,9 +1095,7 @@ def test_table_format(adapter: SparkEngineAdapter, mocker: MockerFixture):
 def test_alter_schema_owner(make_mocked_engine_adapter: t.Callable):
     adapter = make_mocked_engine_adapter(SparkEngineAdapter)
     adapter.alter_schema_owner("catalog.my_schema", "svc_prod_spn")
-    assert to_sql_calls(adapter) == [
-        "ALTER SCHEMA `catalog`.`my_schema` OWNER TO `svc_prod_spn`"
-    ]
+    assert to_sql_calls(adapter) == ["ALTER SCHEMA `catalog`.`my_schema` OWNER TO `svc_prod_spn`"]
 
 
 def test_alter_schema_owner_three_part_name(make_mocked_engine_adapter: t.Callable):

tests/core/test_config.py

@@ -698,20 +698,6 @@ def test_config_ownership_defaults_to_empty():
     assert config.ownership.environment_owner_mapping == {}
     assert config.ownership.resolve_owner("prod") is None
 
-    attach_config_1 = config.gateways["another_gateway"].connection.catalogs.get("sqlite")
-
-    assert isinstance(attach_config_1, DuckDBAttachOptions)
-    assert attach_config_1.type == "sqlite"
-    assert attach_config_1.path == "test.db"
-    assert attach_config_1.read_only is False
-
-    attach_config_2 = config.gateways["another_gateway"].connection.catalogs.get("postgres")
-
-    assert isinstance(attach_config_2, DuckDBAttachOptions)
-    assert attach_config_2.type == "postgres"
-    assert attach_config_2.path == "dbname=postgres user=postgres host=127.0.0.1"
-    assert attach_config_2.read_only is True
-
 
 def test_ownership_config_physical_owner():
     # physical_owner is a simple optional string — no pattern matching.

tests/core/test_snapshot_evaluator.py

@@ -495,9 +495,7 @@ def test_promote_owner_applied_per_view(mocker: MockerFixture, adapter_mock, mak
         snapshot.categorize_as(SnapshotChangeCategory.BREAKING)
         snapshots.append(snapshot)
 
-    evaluator.promote(
-        snapshots, EnvironmentNamingInfo(name="test_env"), owner="svc_prod_spn"
-    )
+    evaluator.promote(snapshots, EnvironmentNamingInfo(name="test_env"), owner="svc_prod_spn")
 
     assert adapter_mock.alter_view_owner.call_count == 3
     called_owners = {c.args[1] for c in adapter_mock.alter_view_owner.call_args_list}
@@ -546,9 +544,7 @@ def test_create_without_physical_owner_skips_alter(
     adapter_mock.alter_table_owner.assert_not_called()
 
 
-def test_create_view_kind_skips_physical_owner(
-    mocker: MockerFixture, adapter_mock, make_snapshot
-):
+def test_create_view_kind_skips_physical_owner(mocker: MockerFixture, adapter_mock, make_snapshot):
     """ViewKind snapshots skip alter_table_owner even when physical_owner is set."""
     adapter_mock.get_data_objects.return_value = []
     evaluator = SnapshotEvaluator(adapter_mock)