Skip to content

Commit ae31c4f

Browse files
committed
Remove postfix
1 parent 44964be commit ae31c4f

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

tests/catalog/test_sql.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def fixture_another_random_identifier(warehouse: Path, database_name: str, table
6767
def catalog_memory(warehouse: Path) -> Generator[SqlCatalog, None, None]:
6868
props = {
6969
"uri": "sqlite+pysqlite:///:memory:",
70-
"warehouse": f"file://{warehouse}/memory",
70+
"warehouse": f"file://{warehouse}",
7171
}
7272
catalog = SqlCatalog("test_sql_catalog", **props)
7373
catalog.create_tables()
@@ -79,7 +79,7 @@ def catalog_memory(warehouse: Path) -> Generator[SqlCatalog, None, None]:
7979
def catalog_sqlite(warehouse: Path) -> Generator[SqlCatalog, None, None]:
8080
props = {
8181
"uri": "sqlite:////tmp/sql-catalog.db",
82-
"warehouse": f"file://{warehouse}/sqlite",
82+
"warehouse": f"file://{warehouse}",
8383
}
8484
catalog = SqlCatalog("test_sql_catalog", **props)
8585
catalog.create_tables()
@@ -397,7 +397,13 @@ def test_rename_table_from_self_identifier(
397397
with pytest.raises(NoSuchTableError):
398398
catalog.load_table(random_identifier)
399399

400-
400+
@pytest.mark.parametrize(
401+
'catalog',
402+
[
403+
lazy_fixture('catalog_memory'),
404+
lazy_fixture('catalog_sqlite'),
405+
],
406+
)
401407
def test_rename_table_to_existing_one(
402408
catalog: SqlCatalog, table_schema_nested: Schema, random_identifier: Identifier, another_random_identifier: Identifier
403409
) -> None:

0 commit comments

Comments
 (0)