Skip to content

Commit cefb301

Browse files
intel352claude
andcommitted
fix: update TriggerTypes test to match disabled trigger implementation
TriggerTypes() intentionally returns empty because the host adapter does not yet support the CreateTrigger RPC. The test was still expecting ["bento"] and panicking on index out of range. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 7c88804 commit cefb301

1 file changed

Lines changed: 5 additions & 7 deletions

File tree

internal/plugin_test.go

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -206,13 +206,11 @@ func TestBentoPlugin_TriggerTypes(t *testing.T) {
206206

207207
types := triggerProvider.TriggerTypes()
208208

209-
expected := []string{"bento"}
210-
if len(types) != len(expected) {
211-
t.Errorf("expected %d trigger types, got %d", len(expected), len(types))
212-
}
213-
214-
if types[0] != expected[0] {
215-
t.Errorf("expected trigger type %q, got %q", expected[0], types[0])
209+
// TriggerTypes intentionally returns empty because the host adapter
210+
// does not yet support the CreateTrigger RPC. Advertising trigger types
211+
// would cause a registration failure that prevents step.bento from loading.
212+
if len(types) != 0 {
213+
t.Errorf("expected 0 trigger types (triggers disabled), got %d: %v", len(types), types)
216214
}
217215
}
218216

0 commit comments

Comments
 (0)