[test] Cover presentation helper output

[ibobgunardi]

Closes #273.

Adds direct output tests for:

• printWorktreeResult copied, missing, and symlink-skipped rows
• printSyncDryRun merge dry-run output

Checks:

• gofmt -w cmd/add_test.go cmd/sync_test.go
• git diff --check

Note: I could not run go test locally because this host has Go 1.25.1 while go.mod requires Go 1.26.4, and automatic toolchain switching hung in the sandbox.

[lugassawan]

Medium — Test name TestPrintWorktreeResultCopySkipBranches is misleading.

The suffix Branches does not describe what is tested. The test covers Copied, Skipped, and SkippedSymlinks output lines — no branch-plural concept is involved. A reader scanning test names builds a wrong mental model.

Suggested fix: Rename to TestPrintWorktreeResultCopyAndSkipped or TestPrintWorktreeResultAllFields.

[lugassawan]

Medium — The push=false branch of printSyncDryRun is not tested.

The guard if push { ... } (in cmd/sync.go) is the only conditional branch in the function, yet this test covers only push=true. A regression that unconditionally prints the push line would not be caught.

Suggested fix: Add a second test case, e.g.:

func TestPrintSyncDryRunRebaseNoPush(t *testing.T) {
    cmd, buf := newTestCmd()
    printSyncDryRun(cmd, "feature/login", "main", false, false)
    got := buf.String()
    if !strings.Contains(got, "would rebase") {
        t.Fatalf("expected rebase verb, got: %q", got)
    }
    if strings.Contains(got, "would push") {
        t.Fatalf("push line must be absent, got: %q", got)
    }
}

[lugassawan]

Low — The empty-fields path of printWorktreeResult is not tested.

The three len > 0 guards (for Copied, Skipped, SkippedSymlinks) are untested with the current test. A complementary test would verify those lines are correctly suppressed when the slices are nil.

Suggested fix: Add TestPrintWorktreeResultMinimal with an AddResult where all slice fields are nil, and assert the output contains only the header, Branch, and Path lines (and does NOT contain "Copied:", "Skipped", or "symlinks").

Changing to comment — findings need addressing before approval.

[lugassawan]

Review Decision: COMMENT

---

Reviewed by reviewer (swe-workbench). Posted 3 inline comments, deduped 0.