Skip to content

Commit f7284fd

Browse files
committed
fix: drop integration-test references to retired Secrets/Refresh/ConnectionTypes APIs
The regeneration removes hotdata.api.secrets_api, hotdata.api.refresh_api, hotdata.api.connection_types_api, ConnectionsApi.check_connection_health, and their models, but the hand-written integration suite still imported them, so every test failed at collection. - drop the secrets_api / refresh_api / connection_types_api fixtures and imports from tests/integration/conftest.py - delete test_secrets_crud.py and test_connection_types_read.py; drop the check_connection_health assertions from test_connections_read.py - document the breaking removals in CHANGELOG.md and update README
1 parent 8c5129a commit f7284fd

6 files changed

Lines changed: 13 additions & 119 deletions

File tree

‎CHANGELOG.md‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
77

88
## [Unreleased]
99

10+
### Removed
11+
12+
- **Breaking:** the Secrets, Refresh, and Connection Types endpoints were retired
13+
from the HotData API and are gone from the SDK: `SecretsApi`, `RefreshApi`,
14+
`ConnectionTypesApi`, and their request/response models
15+
(`CreateSecretRequest`, `UpdateSecretRequest`, `RefreshRequest`, etc.).
16+
- **Breaking:** `ConnectionsApi.check_connection_health` and
17+
`ConnectionHealthResponse` are removed.
18+
- **Breaking:** the `data_refresh_table` / `data_refresh_connection` `JobType`
19+
values and the corresponding `JobResult` variants
20+
(`TableRefreshResult` / `ConnectionRefreshResult`) are removed.
21+
1022
### Changed
1123

1224
- chore(databases): clarify fork naming behavior

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# hotdata
22

3-
Official Python client for the [Hotdata](https://www.hotdata.dev) HTTP API: workspaces, connections, datasets, SQL queries, results, secrets, uploads, indexes, jobs, embedding providers, and workspace context.
3+
Official Python client for the [Hotdata](https://www.hotdata.dev) HTTP API: workspaces, connections, datasets, SQL queries, results, uploads, indexes, jobs, embedding providers, and workspace context.
44

55
## Requirements
66

‎tests/integration/conftest.py‎

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -14,17 +14,14 @@
1414
import pytest
1515

1616
from hotdata import ApiClient, Configuration
17-
from hotdata.api.connection_types_api import ConnectionTypesApi
1817
from hotdata.api.connections_api import ConnectionsApi
1918
from hotdata.api.database_context_api import DatabaseContextApi
2019
from hotdata.api.databases_api import DatabasesApi
2120
from hotdata.api.embedding_providers_api import EmbeddingProvidersApi
2221
from hotdata.api.indexes_api import IndexesApi
2322
from hotdata.api.information_schema_api import InformationSchemaApi
2423
from hotdata.api.jobs_api import JobsApi
25-
from hotdata.api.refresh_api import RefreshApi
2624
from hotdata.api.saved_queries_api import SavedQueriesApi
27-
from hotdata.api.secrets_api import SecretsApi
2825
from hotdata.api.workspaces_api import WorkspacesApi
2926
from hotdata.exceptions import ApiException
3027
from hotdata.models.create_database_request import CreateDatabaseRequest
@@ -151,11 +148,6 @@ def saved_queries_api(api_client: ApiClient) -> SavedQueriesApi:
151148
return SavedQueriesApi(api_client)
152149

153150

154-
@pytest.fixture
155-
def secrets_api(api_client: ApiClient) -> SecretsApi:
156-
return SecretsApi(api_client)
157-
158-
159151
@pytest.fixture
160152
def database_context_api(api_client: ApiClient) -> DatabaseContextApi:
161153
return DatabaseContextApi(api_client)
@@ -166,11 +158,6 @@ def embedding_providers_api(api_client: ApiClient) -> EmbeddingProvidersApi:
166158
return EmbeddingProvidersApi(api_client)
167159

168160

169-
@pytest.fixture
170-
def connection_types_api(api_client: ApiClient) -> ConnectionTypesApi:
171-
return ConnectionTypesApi(api_client)
172-
173-
174161
@pytest.fixture
175162
def jobs_api(api_client: ApiClient) -> JobsApi:
176163
return JobsApi(api_client)
@@ -186,11 +173,6 @@ def uploads_api(api_client: ApiClient) -> UploadsApi:
186173
return UploadsApi(api_client)
187174

188175

189-
@pytest.fixture
190-
def refresh_api(api_client: ApiClient) -> RefreshApi:
191-
return RefreshApi(api_client)
192-
193-
194176
@pytest.fixture
195177
def scratch_database(databases_api: DatabasesApi, sdkci_name) -> Iterator[str]:
196178
"""Yields the id of a fresh, isolated database, deleting it on teardown.

‎tests/integration/test_connection_types_read.py‎

Lines changed: 0 additions & 27 deletions
This file was deleted.

‎tests/integration/test_connections_read.py‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,5 @@ def test_connections_read(connections_api: ConnectionsApi, connection_id: str) -
2828
f"seeded connection {connection_id} not in list_connections"
2929
)
3030

31-
health = connections_api.check_connection_health(connection_id)
32-
assert health.connection_id == connection_id
33-
assert health.healthy, f"seeded connection unhealthy: {health.error}"
34-
3531
# purge_connection_cache returns None on success.
3632
connections_api.purge_connection_cache(connection_id)

‎tests/integration/test_secrets_crud.py‎

Lines changed: 0 additions & 69 deletions
This file was deleted.

0 commit comments

Comments
 (0)