Function TestRenderFailIfMultiNodes_UsesCommandName

pkg/engine/render_test.go:4093–4135

TestRenderFailIfMultiNodes_UsesCommandName guards the contract for Options.CommandName: the multi-node rejection error must reference the calling subcommand the caller passed in, never the historical hardcoded literal that pre-dated this option. An empty value falls back to the neutral "talm".

(t *testing.T)

// hardcoded literal that pre-dated this option. An empty value falls back
// to the neutral "talm".
func TestRenderFailIfMultiNodes_UsesCommandName(t *testing.T) {
	tests := []struct {
		name        string
		commandName string
		wantInError string
	}{
		{"talm apply", "talm apply", "talm apply"},
		{"talm template", "talm template", "talm template"},
		{"empty falls back to talm", "", "talm"},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			ctx := client.WithNodes(context.Background(), "10.0.0.1", "10.0.0.2")
			opts := Options{
				Offline:     false,
				CommandName: tt.commandName,
			}
			_, err := Render(ctx, nil, opts)
			if err == nil {
				t.Fatalf("Render expected an error, got nil")
			}
			if !strings.Contains(err.Error(), tt.wantInError) {
				t.Errorf("error = %q, expected to contain %q", err.Error(), tt.wantInError)
			}
		})
	}

	t.Run("non-CommandName subcommand names must not leak into the error", func(t *testing.T) {
		// If a caller passes "talm apply", the error must not carry any
		// other subcommand name — historically the call site here emitted
		// "talm template" unconditionally.
		ctx := client.WithNodes(context.Background(), "10.0.0.1", "10.0.0.2")
		opts := Options{Offline: false, CommandName: "talm apply"}
		_, err := Render(ctx, nil, opts)
		if err == nil {
			t.Fatal("Render expected an error, got nil")
		}
		if strings.Contains(err.Error(), "talm template") {
			t.Errorf("error must not mention 'talm template' when CommandName is 'talm apply'; got %q", err.Error())
		}
	})
}

// TestRenderInvalidTalosVersion verifies that malformed TalosVersion values
// surface a user-friendly error before template rendering, instead of the