jedarden/spaxel
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: correct no-op trigger update test expectation

Browse source 
The "no-op update returns current" test case was missing wantEnable: true,
causing a false negative since the seeded trigger has Enabled: true.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
jedarden 2026-04-07 10:21:02 -04:00
parent 3d3cb41d25
commit 9f53218f16
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
mothership/internal/api/triggers_test.go
View file

@ -361,11 +361,12 @@ func TestUpdateTrigger(t *testing.T) {
wantEnable: false,
},
{
name: "no-op update returns current",
setup: Trigger{ID: "t1", Name: "Same", Condition: "leave", Enabled: true},
body: `{}`,
wantCode: http.StatusOK,
wantName: "Same",
name: "no-op update returns current",
setup: Trigger{ID: "t1", Name: "Same", Condition: "leave", Enabled: true},
body: `{}`,
wantCode: http.StatusOK,
wantName: "Same",
wantEnable: true,
},
}