Skip to content

Commit 2f5436a

Browse files
Add S3 creds to rest_catalog fixture in tests/integration/test_catalog.py
The fixture had no S3 credentials, which was fine when every test in the file was metadata-only. The new replace tests do .append() and fail with ACCESS_DENIED against minio. Mirrors the credentials the hive_catalog fixture in the same file already uses.
1 parent 24ffc24 commit 2f5436a

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

‎tests/integration/test_catalog.py‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,15 @@ def sqlite_catalog_file(warehouse: Path) -> Generator[Catalog, None, None]:
8686

8787
@pytest.fixture(scope="function")
8888
def rest_catalog() -> Generator[Catalog, None, None]:
89-
test_catalog = RestCatalog("rest", uri="http://localhost:8181")
89+
test_catalog = RestCatalog(
90+
"rest",
91+
**{
92+
"uri": "http://localhost:8181",
93+
"s3.endpoint": "http://localhost:9000",
94+
"s3.access-key-id": "admin",
95+
"s3.secret-access-key": "password",
96+
},
97+
)
9098

9199
yield test_catalog
92100

0 commit comments

Comments
 (0)