Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

Commit b1fec6f

Browse files
committed
Tests: Fix clickhouse URI for CI
1 parent afb7dd5 commit b1fec6f

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,4 +47,5 @@ jobs:
4747
DATADIFF_SNOWFLAKE_URI: '${{ secrets.DATADIFF_SNOWFLAKE_URI }}'
4848
DATADIFF_PRESTO_URI: '${{ secrets.DATADIFF_PRESTO_URI }}'
4949
DATADIFF_TRINO_URI: '${{ secrets.DATADIFF_TRINO_URI }}'
50+
DATADIFF_CLICKHOUSE_URI: 'clickhouse://clickhouse:Password1@localhost:9000/clickhouse'
5051
run: poetry run unittest-parallel -j 16

tests/common.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@
2121
TEST_ORACLE_CONN_STRING: str = None
2222
TEST_DATABRICKS_CONN_STRING: str = os.environ.get("DATADIFF_DATABRICKS_URI")
2323
TEST_TRINO_CONN_STRING: str = os.environ.get("DATADIFF_TRINO_URI") or None
24-
TEST_CLICKHOUSE_CONN_STRING: str = None # "clickhouse://clickhouse:Password1@localhost:9000/clickhouse"
24+
# clickhouse uri for provided docker - "clickhouse://clickhouse:Password1@localhost:9000/clickhouse"
25+
TEST_CLICKHOUSE_CONN_STRING: str = os.environ.get("DATADIFF_CLICKHOUSE_URI") or None
2526

2627
DEFAULT_N_SAMPLES = 50
2728
N_SAMPLES = int(os.environ.get("N_SAMPLES", DEFAULT_N_SAMPLES))

0 commit comments

Comments
 (0)