mock the systray for tests

Author: tstromberg

cmd/goose/main.go

@@ -87,6 +87,7 @@ type App struct {
 	enableAutoBrowser   bool
 	hideStaleIncoming   bool
 	noCache             bool
+	systrayInterface    SystrayInterface // For mocking systray operations in tests
 	hiddenOrgs          map[string]bool
 	seenOrgs            map[string]bool
 
@@ -215,6 +216,7 @@ func main() {
 		enableAutoBrowser:  false, // Default to false for safety
 		browserRateLimiter: NewBrowserRateLimiter(browserOpenDelay, maxBrowserOpensMinute, maxBrowserOpensDay),
 		startTime:          startTime,
+		systrayInterface:   &RealSystray{}, // Use real systray implementation
 		seenOrgs:           make(map[string]bool),
 		hiddenOrgs:         make(map[string]bool),
 		// Deprecated fields for test compatibility

cmd/goose/main_test.go

@@ -299,7 +299,7 @@ func TestSoundPlaybackDuringTransitions(t *testing.T) {
 		enableAudioCues:     true,
 		initialLoadComplete: true, // Set to true to allow sound playback
 		menuInitialized:     true,
-		lastMenuTitles:      []string{"placeholder"}, // Initialize to prevent menu rebuilds during testing
+		systrayInterface:    &MockSystray{}, // Use mock systray to avoid Windows-specific panics
 	}
 
 	tests := []struct {

cmd/goose/ui.go

@@ -388,7 +388,7 @@ func (app *App) rebuildMenu(ctx context.Context) {
 	// Rebuild entire menu
 
 	// Clear all existing menu items
-	systray.ResetMenu()
+	app.systrayInterface.ResetMenu()
 
 	// Check for auth error first
 	if app.authError != "" {