From 99b2e7e4c3893265955a9cf708710858e77d0b10 Mon Sep 17 00:00:00 2001
From: Steve Yegge <steve.yegge@gmail.com>
Date: Sun, 7 Dec 2025 20:57:13 +1100
Subject: [PATCH] test(autoimport): add unit tests for getLocalSyncBranch
 (bd-6y5)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Tests cover:
- Missing config.yaml
- Config without sync-branch key
- Unquoted, double-quoted, and single-quoted values
- Environment variable precedence
- Empty files and whitespace-only content
- Comments (both full-line and inline)
- Special characters in branch names
- Nested sync-branch (should be ignored)
- Mixed config files

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 cmd/bd/autoimport_test.go | 118 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 118 insertions(+)

diff --git a/cmd/bd/autoimport_test.go b/cmd/bd/autoimport_test.go
index b7736374..31c1ec63 100644
--- a/cmd/bd/autoimport_test.go
+++ b/cmd/bd/autoimport_test.go
@@ -298,3 +298,121 @@ func TestBoolToFlag(t *testing.T) {
 		})
 	}
 }
+
+func TestGetLocalSyncBranch(t *testing.T) {
+	tests := []struct {
+		name        string
+		configYAML  string
+		envVar      string
+		want        string
+		createFile  bool
+	}{
+		{
+			name:       "no config.yaml exists",
+			createFile: false,
+			want:       "",
+		},
+		{
+			name:       "config.yaml has no sync-branch key",
+			configYAML: "issue-prefix: test\nauthor: testuser\n",
+			createFile: true,
+			want:       "",
+		},
+		{
+			name:       "sync-branch without quotes",
+			configYAML: "sync-branch: my-branch\n",
+			createFile: true,
+			want:       "my-branch",
+		},
+		{
+			name:       "sync-branch with double quotes",
+			configYAML: `sync-branch: "my-quoted-branch"` + "\n",
+			createFile: true,
+			want:       "my-quoted-branch",
+		},
+		{
+			name:       "sync-branch with single quotes",
+			configYAML: `sync-branch: 'single-quoted'` + "\n",
+			createFile: true,
+			want:       "single-quoted",
+		},
+		{
+			name:       "env var takes precedence",
+			configYAML: "sync-branch: config-branch\n",
+			createFile: true,
+			envVar:     "env-branch",
+			want:       "env-branch",
+		},
+		{
+			name:       "empty file",
+			configYAML: "",
+			createFile: true,
+			want:       "",
+		},
+		{
+			name:       "whitespace-only lines",
+			configYAML: "   \n\t\n  \n",
+			createFile: true,
+			want:       "",
+		},
+		{
+			name:       "sync-branch after comments",
+			configYAML: "# This is a comment\n# sync-branch: fake\nsync-branch: real-branch\n",
+			createFile: true,
+			want:       "real-branch",
+		},
+		{
+			name:       "sync-branch with trailing comment",
+			configYAML: "sync-branch: branch-name # inline comment not valid YAML but test it\n",
+			createFile: true,
+			want:       "branch-name",
+		},
+		{
+			name:       "sync-branch with special characters",
+			configYAML: "sync-branch: feature/my-branch_v2.0\n",
+			createFile: true,
+			want:       "feature/my-branch_v2.0",
+		},
+		{
+			name:       "sync-branch indented under section (not top-level)",
+			configYAML: "settings:\n  sync-branch: nested-branch\n",
+			createFile: true,
+			want:       "", // Only top-level sync-branch should be read
+		},
+		{
+			name:       "mixed config with sync-branch",
+			configYAML: "issue-prefix: bd\nauthor: steve\nsync-branch: beads-sync\npriority: P2\n",
+			createFile: true,
+			want:       "beads-sync",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			// Create temp beads directory
+			tmpDir := t.TempDir()
+			beadsDir := filepath.Join(tmpDir, ".beads")
+			if err := os.MkdirAll(beadsDir, 0755); err != nil {
+				t.Fatalf("Failed to create beads dir: %v", err)
+			}
+
+			// Create config.yaml if needed
+			if tt.createFile {
+				configPath := filepath.Join(beadsDir, "config.yaml")
+				if err := os.WriteFile(configPath, []byte(tt.configYAML), 0644); err != nil {
+					t.Fatalf("Failed to write config.yaml: %v", err)
+				}
+			}
+
+			// Set env var if specified
+			if tt.envVar != "" {
+				t.Setenv("BEADS_SYNC_BRANCH", tt.envVar)
+			}
+
+			got := getLocalSyncBranch(beadsDir)
+			if got != tt.want {
+				t.Errorf("getLocalSyncBranch() = %q, want %q", got, tt.want)
+			}
+		})
+	}
+}