adds some tests too

Entire-Checkpoint: 11c16f0c4d1f

Author: Soph

cmd/entire/cli/labs_test.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"strings"
 	"testing"
+	"unicode/utf8"
 )
 
 func TestLabsCmd_PrintsExperimentalCommandList(t *testing.T) {
@@ -97,6 +98,73 @@ func TestRootHelp_ShowsLabsButHidesReview(t *testing.T) {
 	}
 }
 
+// summaryColumns returns, for each non-empty rendered row, the rune offset at
+// which the summary begins (i.e. the column after the padded invocation).
+func summaryColumns(t *testing.T, commands []experimentalCommandInfo) []int {
+	t.Helper()
+	var cols []int
+	for _, line := range strings.Split(renderExperimentalCommands(commands), "\n") {
+		if line == "" {
+			continue
+		}
+		info := indexOfSummary(t, line, commands)
+		cols = append(cols, info)
+	}
+	return cols
+}
+
+// indexOfSummary finds the rune offset of a row's summary text within the line.
+func indexOfSummary(t *testing.T, line string, commands []experimentalCommandInfo) int {
+	t.Helper()
+	for _, info := range commands {
+		if idx := strings.Index(line, info.Summary); idx >= 0 {
+			return utf8.RuneCountInString(line[:idx])
+		}
+	}
+	t.Fatalf("no known summary found in rendered line %q", line)
+	return -1
+}
+
+func TestRenderExperimentalCommands_SummariesAlign(t *testing.T) {
+	t.Parallel()
+
+	cols := summaryColumns(t, experimentalCommands)
+	if len(cols) < 2 {
+		t.Fatalf("expected multiple experimental commands, got %d", len(cols))
+	}
+	for i, col := range cols {
+		if col != cols[0] {
+			t.Fatalf("summary column %d (%d) does not match first column (%d); descriptions are misaligned", i, col, cols[0])
+		}
+	}
+}
+
+func TestRenderExperimentalCommands_ColumnWidthAdjustsToLongest(t *testing.T) {
+	t.Parallel()
+
+	short := []experimentalCommandInfo{
+		{Name: "a", Invocation: "entire a", Summary: "first"},
+		{Name: "b", Invocation: "entire b", Summary: "second"},
+	}
+	long := []experimentalCommandInfo{
+		{Name: "a", Invocation: "entire a", Summary: "first"},
+		{Name: "verylongcommand", Invocation: "entire verylongcommand", Summary: "second"},
+	}
+
+	shortCol := summaryColumns(t, short)[0]
+	longCol := summaryColumns(t, long)[0]
+
+	if longCol <= shortCol {
+		t.Fatalf("column should widen for a longer invocation: short=%d long=%d", shortCol, longCol)
+	}
+	// All rows in the long set must still align despite differing invocation lengths.
+	for i, col := range summaryColumns(t, long) {
+		if col != longCol {
+			t.Fatalf("row %d column %d does not match %d", i, col, longCol)
+		}
+	}
+}
+
 func TestLabsRegistryCommandsExistAtCanonicalPaths(t *testing.T) {
 	t.Parallel()