From 9f53218f1629e5d0122ca0adda0620b972779042 Mon Sep 17 00:00:00 2001
From: jedarden <github@jedarden.com>
Date: Tue, 7 Apr 2026 10:21:02 -0400
Subject: [PATCH] fix: correct no-op trigger update test expectation

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>
---
 mothership/internal/api/triggers_test.go | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/mothership/internal/api/triggers_test.go b/mothership/internal/api/triggers_test.go
index 02e8171..0d02c7e 100644
--- a/mothership/internal/api/triggers_test.go
+++ b/mothership/internal/api/triggers_test.go
@@ -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,
 		},
 	}