tests: update coverage

Author: raisedadead

internal/config/config_test.go

@@ -711,3 +711,155 @@ func TestRemoveFromSlice(t *testing.T) {
 		})
 	}
 }
+
+func TestFormatStringSlice(t *testing.T) {
+	tests := []struct {
+		name     string
+		input    []string
+		expected string
+	}{
+		{"empty", nil, "[]"},
+		{"empty slice", []string{}, "[]"},
+		{"single", []string{"zoxide"}, `["zoxide"]`},
+		{"multiple", []string{"zoxide", "direnv"}, `["zoxide", "direnv"]`},
+		{"three", []string{"a", "b", "c"}, `["a", "b", "c"]`},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := formatStringSlice(tt.input)
+			if result != tt.expected {
+				t.Errorf("formatStringSlice(%v) = %q, want %q", tt.input, result, tt.expected)
+			}
+		})
+	}
+}
+
+func TestJoinStrings(t *testing.T) {
+	tests := []struct {
+		name     string
+		input    []string
+		sep      string
+		expected string
+	}{
+		{"empty", nil, ", ", ""},
+		{"single", []string{"a"}, ", ", "a"},
+		{"two", []string{"a", "b"}, ", ", "a, b"},
+		{"custom sep", []string{"x", "y", "z"}, "-", "x-y-z"},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := joinStrings(tt.input, tt.sep)
+			if result != tt.expected {
+				t.Errorf("joinStrings(%v, %q) = %q, want %q", tt.input, tt.sep, result, tt.expected)
+			}
+		})
+	}
+}
+
+func TestGenerateHooksSection(t *testing.T) {
+	tests := []struct {
+		name       string
+		selections IntegrationSelections
+		contains   []string
+		absent     []string
+	}{
+		{
+			"no integrations",
+			IntegrationSelections{},
+			[]string{"# [hooks]", "# post_clone = []"},
+			[]string{"\n[hooks]\n"},
+		},
+		{
+			"zoxide only",
+			IntegrationSelections{Zoxide: true},
+			[]string{"[hooks]\n", `post_clone = ["zoxide"]`, `post_add = ["zoxide"]`},
+			[]string{"# [hooks]"},
+		},
+		{
+			"github only",
+			IntegrationSelections{GitHub: true},
+			[]string{"[hooks]\n", `post_clone = ["gh-default"]`, "post_add = []"},
+			nil,
+		},
+		{
+			"direnv only",
+			IntegrationSelections{Direnv: true},
+			[]string{"[hooks]\n", "post_clone = []", `post_add = ["direnv"]`},
+			nil,
+		},
+		{
+			"all integrations",
+			IntegrationSelections{Zoxide: true, GitHub: true, Direnv: true},
+			[]string{`post_clone = ["zoxide", "gh-default"]`, `post_add = ["zoxide", "direnv"]`},
+			nil,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := generateHooksSection(tt.selections)
+			for _, s := range tt.contains {
+				if !strings.Contains(result, s) {
+					t.Errorf("expected %q to contain %q", result, s)
+				}
+			}
+			for _, s := range tt.absent {
+				if strings.Contains(result, s) {
+					t.Errorf("expected %q to NOT contain %q", result, s)
+				}
+			}
+		})
+	}
+}
+
+func TestGenerateWorkflowsSection(t *testing.T) {
+	tests := []struct {
+		name       string
+		selections IntegrationSelections
+		contains   []string
+	}{
+		{
+			"no extras",
+			IntegrationSelections{},
+			[]string{"[workflows.feature]", "[workflows.bugfix]", "github-issue", "post_add = []"},
+		},
+		{
+			"with direnv and zoxide",
+			IntegrationSelections{Direnv: true, Zoxide: true},
+			[]string{`post_add = ["direnv", "zoxide"]`, "[workflows.feature]", "[workflows.bugfix]"},
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := generateWorkflowsSection(tt.selections)
+			for _, s := range tt.contains {
+				if !strings.Contains(result, s) {
+					t.Errorf("expected %q to contain %q", result, s)
+				}
+			}
+		})
+	}
+}
+
+func TestGenerateConfigWithIntegrations(t *testing.T) {
+	// No integrations - should have commented hooks, no workflows
+	plain := GenerateConfigWithIntegrations(IntegrationSelections{})
+	if !strings.Contains(plain, "# [hooks]") {
+		t.Error("expected commented hooks section")
+	}
+	if strings.Contains(plain, "[workflows.") {
+		t.Error("expected no workflows section without GitHub")
+	}
+
+	// With GitHub - should have workflows section
+	withGH := GenerateConfigWithIntegrations(IntegrationSelections{GitHub: true})
+	if !strings.Contains(withGH, "[workflows.feature]") {
+		t.Error("expected workflows section with GitHub enabled")
+	}
+	if !strings.Contains(withGH, "[workflows.bugfix]") {
+		t.Error("expected bugfix workflow with GitHub enabled")
+	}
+}

internal/tui/panels/detail.go

@@ -141,19 +141,19 @@ func (dp *DetailPanel) renderInfo() string {
 
 	var b strings.Builder
 
-	b.WriteString(fmt.Sprintf("  %s  %s\n",
+	fmt.Fprintf(&b, "  %s  %s\n",
 		dp.tabStyles.label.Render("Branch:"),
-		dp.tabStyles.value.Render(dp.Branch)))
-	b.WriteString(fmt.Sprintf("  %s    %s\n",
+		dp.tabStyles.value.Render(dp.Branch))
+	fmt.Fprintf(&b, "  %s    %s\n",
 		dp.tabStyles.label.Render("Path:"),
-		dp.tabStyles.value.Render(dp.Path)))
-	b.WriteString(fmt.Sprintf("  %s  %s\n",
+		dp.tabStyles.value.Render(dp.Path))
+	fmt.Fprintf(&b, "  %s  %s\n",
 		dp.tabStyles.label.Render("Status:"),
-		dp.tabStyles.value.Render(dp.Status)))
+		dp.tabStyles.value.Render(dp.Status))
 
 	if dp.Commits != "" {
 		b.WriteString("\n")
-		b.WriteString(fmt.Sprintf("  %s\n", dp.tabStyles.section.Render("── Recent commits ──────────────")))
+		fmt.Fprintf(&b, "  %s\n", dp.tabStyles.section.Render("── Recent commits ──────────────"))
 		for _, line := range strings.Split(dp.Commits, "\n") {
 			if line != "" {
 				b.WriteString("  " + line + "\n")
@@ -163,15 +163,15 @@ func (dp *DetailPanel) renderInfo() string {
 
 	if dp.Files != "" {
 		b.WriteString("\n")
-		b.WriteString(fmt.Sprintf("  %s\n", dp.tabStyles.section.Render("── Modified files ──────────────")))
+		fmt.Fprintf(&b, "  %s\n", dp.tabStyles.section.Render("── Modified files ──────────────"))
 		for _, line := range strings.Split(dp.Files, "\n") {
 			if line != "" {
 				b.WriteString("  " + line + "\n")
 			}
 		}
 	} else if dp.Status == "clean" {
 		b.WriteString("\n")
-		b.WriteString(fmt.Sprintf("  %s\n", dp.tabStyles.section.Render("── Modified files ──────────────")))
+		fmt.Fprintf(&b, "  %s\n", dp.tabStyles.section.Render("── Modified files ──────────────"))
 		b.WriteString("  (no uncommitted changes)\n")
 	}