Skip to content

Commit fc3ff2f

Browse files
committed
test(clickhouse): add missing task_kind to taskRuns test fixtures
Migration 029 added `task_kind` to `task_runs_v2`, and TASK_RUN_COLUMNS was updated, but the four test-data arrays in src/taskRuns.test.ts were not. ClickHouse rejects the inserts with "Cannot parse input: expected ',' before: ']'" because the array length is one short of the column count. All 7 internal/clickhouse unit-test shards on PR #3173 fail on this. Pre-existing bug (predates my Sessions work) but blocking CI; verified the fix locally — `vitest run src/taskRuns.test.ts` now passes 4/4.
1 parent c9b5236 commit fc3ff2f

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

internal-packages/clickhouse/src/taskRuns.test.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ describe("Task Runs V2", () => {
8484
null, // max_duration_in_seconds
8585
"", // trigger_source
8686
"", // root_trigger_source
87+
"", // task_kind
8788
null, // is_warm_start
8889
];
8990

@@ -215,6 +216,7 @@ describe("Task Runs V2", () => {
215216
null, // max_duration_in_seconds
216217
"", // trigger_source
217218
"", // root_trigger_source
219+
"", // task_kind
218220
null, // is_warm_start
219221
];
220222

@@ -269,6 +271,7 @@ describe("Task Runs V2", () => {
269271
null, // max_duration_in_seconds
270272
"", // trigger_source
271273
"", // root_trigger_source
274+
"", // task_kind
272275
null, // is_warm_start
273276
];
274277

@@ -370,6 +373,7 @@ describe("Task Runs V2", () => {
370373
null, // max_duration_in_seconds
371374
"", // trigger_source
372375
"", // root_trigger_source
376+
"", // task_kind
373377
null, // is_warm_start
374378
];
375379

0 commit comments

Comments
 (0)