Skip to content

Commit 2896f1d

Browse files
d-bytebaseclaude
andcommitted
test: add required id field to approval flow test cases
Update all approval flow test cases to include the new required 'id' field. Use custom flow IDs (e.g., 'test-flow', 'test-ddl-approval-flow') instead of built-in 'bb.*' prefixed IDs for test cases. Fixes test failures caused by adding the required id field to ApprovalTemplate. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 2e90f4e commit 2896f1d

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

provider/resource_setting_test.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -226,6 +226,7 @@ resource "bytebase_setting" "%s" {
226226
level = "LOW"
227227
}
228228
flow {
229+
id = "test-flow"
229230
title = "Test"
230231
description = "Test"
231232
roles = ["roles/test"]
@@ -261,6 +262,7 @@ resource "bytebase_setting" "%s" {
261262
level = "LOW"
262263
}
263264
flow {
265+
id = "test-flow"
264266
title = "Test"
265267
description = "Test"
266268
roles = ["invalid-role"]
@@ -333,6 +335,7 @@ resource "bytebase_setting" "%s" {
333335
level = "LOW"
334336
}
335337
flow {
338+
id = "test-ddl-approval-flow"
336339
title = "DDL Approval Flow"
337340
description = "Approval flow for DDL operations"
338341
roles = [
@@ -369,6 +372,7 @@ resource "bytebase_setting" "%s" {
369372
level = "MODERATE"
370373
}
371374
flow {
375+
id = "test-updated-approval-flow"
372376
title = "Updated Approval Flow"
373377
description = "Updated approval flow"
374378
roles = [bytebase_role.approval_role_%s.name]

0 commit comments

Comments
 (0)