Skip to content

Commit

Permalink
generic tests have been fixed (ydb-platform#4093)
Browse files Browse the repository at this point in the history
  • Loading branch information
dorooleg authored Apr 25, 2024
1 parent 25cd2e9 commit ed958e3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ydb/tests/fq/generic/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,10 @@
def settings() -> Settings:
return Settings.from_env()


@pytest.fixture
def mvp_external_ydb_endpoint(request) -> str:
return request.param["endpoint"] if request is not None else None
return request.param["endpoint"] if request is not None and hasattr(request, "param") else None


@pytest.fixture
Expand Down
1 change: 1 addition & 0 deletions ydb/tests/fq/generic/test_ydb.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
class TestYdb:
@yq_v2
@pytest.mark.parametrize("fq_client", [{"folder_id": "my_folder"}], indirect=True)
@pytest.mark.parametrize("mvp_external_ydb_endpoint", [{"endpoint": "tests-fq-generic-ydb:2136"}], indirect=True)
def test_simple(self, fq_client: FederatedQueryClient, settings: Settings):
table_name = 'simple_table'
conn_name = f'conn_{table_name}'
Expand Down

0 comments on commit ed958e3

Please sign in to comment.