Function TestHarnessForTask

internal/app/harness_test.go:39–69

TestHarnessForTask covers the column → adapter lookup. NULL and empty resolve to claude+nil error (back-compat). Unknown non-empty names resolve to nil+error so callers refuse rather than silently coerce.

(t *testing.T)

// non-empty names resolve to nil+error so callers refuse rather
// than silently coerce.
func TestHarnessForTask(t *testing.T) {
	cases := []struct {
		name    string
		harness sql.NullString
		want    harness.Name
		wantErr bool
	}{
		{"null column → claude", sql.NullString{}, harness.NameClaude, false},
		{"empty string → claude", sql.NullString{Valid: true, String: ""}, harness.NameClaude, false},
		{"claude pin", sql.NullString{Valid: true, String: "claude"}, harness.NameClaude, false},
		{"unknown name → error", sql.NullString{Valid: true, String: "future"}, "", true},
	}
	for _, tc := range cases {
		t.Run(tc.name, func(t *testing.T) {
			task := &flowdb.Task{Harness: tc.harness}
			h, err := harnessForTask(task)
			if tc.wantErr {
				if err == nil {
					t.Errorf("got nil error, want non-nil")
				}
				return
			}
			if err != nil {
				t.Fatalf("unexpected error: %v", err)
			}
			if got := h.Name(); got != tc.want {
				t.Errorf("got %v, want %v", got, tc.want)
			}
		})
	}
}

// TestCmdDoPersistsHarnessOnBootstrap pins the contract: the first
// `flow do` on a previously-unbound task writes the chosen harness