Add tests for cluster_by parameter for DYNAMIC_TABLE

littleK0i · littleK0i · commit e0303f06287c · 2025-06-25T20:21:24.000+02:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## [0.54.2] - 2025-06-25
+
+- Added check for `cluster_by` parameter for existing dynamic tables (thanks to gcv-epalmer).
+
 ## [0.54.1] - 2025-06-24
 
 - Added `unique` parameter for `SEMANTIC_VIEW` table. Semantic views should now be fully supported.
diff --git a/snowddl/version.py b/snowddl/version.py
@@ -1 +1 @@
-__version__ = "0.54.1"
+__version__ = "0.54.2"
diff --git a/test/_config/step1/db1/sc1/dynamic_table/dt004_dt1.yaml b/test/_config/step1/db1/sc1/dynamic_table/dt004_dt1.yaml
@@ -0,0 +1,8 @@
+text: |-
+  SELECT id, name
+  FROM dt004_tb1
+
+warehouse: dt004_wh1
+target_lag: 1 hour
+
+cluster_by: [id]
diff --git a/test/_config/step1/db1/sc1/params.yaml b/test/_config/step1/db1/sc1/params.yaml
@@ -8,6 +8,7 @@ owner_warehouse_usage:
   - dt001_wh2
   - dt002_wh1
   - dt003_wh1
+  - dt004_wh1
   - mp001_wh1
   - pp001_wh1
   - rp001_wh1
diff --git a/test/_config/step1/db1/sc1/table/dt004_tb1.yaml b/test/_config/step1/db1/sc1/table/dt004_tb1.yaml
@@ -0,0 +1,5 @@
+columns:
+  id: NUMBER(10,0)
+  name: VARCHAR(255)
+
+change_tracking: true
diff --git a/test/_config/step1/warehouse.yaml b/test/_config/step1/warehouse.yaml
@@ -16,6 +16,9 @@ dt002_wh1:
 dt003_wh1:
   size: XSMALL
 
+dt004_wh1:
+  size: XSMALL
+
 mp001_wh1:
   size: XSMALL
 
diff --git a/test/_config/step2/db1/sc1/dynamic_table/dt004_dt1.yaml b/test/_config/step2/db1/sc1/dynamic_table/dt004_dt1.yaml
@@ -0,0 +1,8 @@
+text: |-
+  SELECT id, name
+  FROM dt004_tb1
+
+warehouse: dt004_wh1
+target_lag: 1 hour
+
+cluster_by: [id, name]
diff --git a/test/_config/step2/db1/sc1/params.yaml b/test/_config/step2/db1/sc1/params.yaml
@@ -7,6 +7,7 @@ owner_warehouse_usage:
   - dt001_wh2
   - dt002_wh1
   - dt003_wh1
+  - dt004_wh1
   - mp001_wh1
   - pp001_wh1
   - rp001_wh1
diff --git a/test/_config/step2/db1/sc1/table/dt004_tb1.yaml b/test/_config/step2/db1/sc1/table/dt004_tb1.yaml
@@ -0,0 +1,5 @@
+columns:
+  id: NUMBER(10,0)
+  name: VARCHAR(255)
+
+change_tracking: true
diff --git a/test/_config/step2/warehouse.yaml b/test/_config/step2/warehouse.yaml
@@ -19,6 +19,9 @@ dt002_wh1:
 dt003_wh1:
   size: XSMALL
 
+dt004_wh1:
+  size: XSMALL
+
 mp001_wh1:
   size: XSMALL
 
diff --git a/test/_config/step3/db1/sc1/dynamic_table/dt004_dt1.yaml b/test/_config/step3/db1/sc1/dynamic_table/dt004_dt1.yaml
@@ -0,0 +1,6 @@
+text: |-
+  SELECT id, name
+  FROM dt004_tb1
+
+warehouse: dt004_wh1
+target_lag: 1 hour
diff --git a/test/_config/step3/db1/sc1/params.yaml b/test/_config/step3/db1/sc1/params.yaml
@@ -0,0 +1,2 @@
+owner_warehouse_usage:
+  - dt004_wh1
diff --git a/test/_config/step3/db1/sc1/table/dt004_tb1.yaml b/test/_config/step3/db1/sc1/table/dt004_tb1.yaml
@@ -0,0 +1,5 @@
+columns:
+  id: NUMBER(10,0)
+  name: VARCHAR(255)
+
+change_tracking: true
diff --git a/test/_config/step3/warehouse.yaml b/test/_config/step3/warehouse.yaml
@@ -4,6 +4,9 @@ al001_wh2:
 dt002_wh1:
   size: XSMALL
 
+dt004_wh1:
+  size: XSMALL
+
 pm001_wh1:
   size: XSMALL
 
diff --git a/test/dynamic_table/dt004.py b/test/dynamic_table/dt004.py
@@ -0,0 +1,19 @@
+def test_step1(helper):
+    dynamic_table_show = helper.show_dynamic_table("db1", "sc1", "dt004_dt1")
+
+    assert dynamic_table_show["automatic_clustering"] == "ON"
+    assert dynamic_table_show["cluster_by"] == "LINEAR(id)"
+
+
+def test_step2(helper):
+    dynamic_table_show = helper.show_dynamic_table("db1", "sc1", "dt004_dt1")
+
+    assert dynamic_table_show["automatic_clustering"] == "ON"
+    assert dynamic_table_show["cluster_by"] == "LINEAR(id, name)"
+
+
+def test_step3(helper):
+    dynamic_table_show = helper.show_dynamic_table("db1", "sc1", "dt004_dt1")
+
+    assert dynamic_table_show["automatic_clustering"] == "OFF"
+    assert dynamic_table_show["cluster_by"] == ""

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-__version__ = "0.54.1"`
	`1`	`+__version__ = "0.54.2"`