Add cron install regression coverage for CLI and TUI

Close the remaining cron-install test gaps after aligning CLI and TUI scheduling behavior. Add a TUI wizard regression test that proves blank cron input resolves to the installer default (02:00) even when CRON_SCHEDULE is set in the environment, and add a CLI wizard regression test that aborting exactly at the cron prompt propagates the interactive abort and leaves backup.env unwritten. Introduce minimal test seams for the install wizard runner and cron prompt boundary to exercise the real command/wizard paths without changing production semantics.

Author: tis24dev

cmd/proxsave/install.go

@@ -25,6 +25,7 @@ var (
 	newInstallConfirmTUI             = wizard.ConfirmNewInstall
 	newInstallRunInstall             = runInstall
 	newInstallRunInstallTUI          = runInstallTUI
+	configureCronTimeFunc            = configureCronTime
 )
 
 type installConfigResult struct {
@@ -484,7 +485,7 @@ func runConfigWizardCLI(ctx context.Context, reader *bufio.Reader, configPath, t
 	}
 
 	logging.DebugStepBootstrap(bootstrap, "install config wizard (cli)", "configuring cron time")
-	cronTime, err := configureCronTime(ctx, reader, cronutil.DefaultTime)
+	cronTime, err := configureCronTimeFunc(ctx, reader, cronutil.DefaultTime)
 	if err != nil {
 		return installConfigResult{}, wrapInstallError(err)
 	}

cmd/proxsave/install_test.go

@@ -629,6 +629,31 @@ func TestRunConfigWizardCLISkipLeavesCronScheduleEmpty(t *testing.T) {
 	}
 }
 
+func TestRunConfigWizardCLIAbortAtCronPromptDoesNotWriteConfig(t *testing.T) {
+	configPath := filepath.Join(t.TempDir(), "env", "backup.env")
+	tmpConfigPath := configPath + ".tmp"
+
+	originalConfigureCronTime := configureCronTimeFunc
+	t.Cleanup(func() { configureCronTimeFunc = originalConfigureCronTime })
+
+	configureCronTimeFunc = func(ctx context.Context, reader *bufio.Reader, defaultCron string) (string, error) {
+		return "", errInteractiveAborted
+	}
+
+	reader := bufio.NewReader(strings.NewReader("n\nn\nn\nn\nn\nn\n"))
+
+	_, err := runConfigWizardCLI(context.Background(), reader, configPath, tmpConfigPath, "/opt/proxsave", nil)
+	if !errors.Is(err, errInteractiveAborted) {
+		t.Fatalf("expected errInteractiveAborted, got %v", err)
+	}
+	if _, statErr := os.Stat(configPath); !os.IsNotExist(statErr) {
+		t.Fatalf("expected config file not to exist, got err=%v", statErr)
+	}
+	if _, statErr := os.Stat(tmpConfigPath); !os.IsNotExist(statErr) {
+		t.Fatalf("expected temp config file not to exist, got err=%v", statErr)
+	}
+}
+
 func createTempFile(t *testing.T, content string) string {
 	t.Helper()
 	f, err := os.CreateTemp(t.TempDir(), "config-*.env")

internal/tui/wizard/install.go

@@ -59,7 +59,10 @@ const (
 
 var (
 	// ErrInstallCancelled is returned when the user aborts the install wizard.
-	ErrInstallCancelled       = errors.New("installation aborted by user")
+	ErrInstallCancelled    = errors.New("installation aborted by user")
+	runInstallWizardRunner = func(app *tui.App, root, focus tview.Primitive) error {
+		return app.SetRoot(root, true).SetFocus(focus).Run()
+	}
 	checkExistingConfigRunner = func(app *tui.App, root, focus tview.Primitive) error {
 		return app.SetRoot(root, true).SetFocus(focus).Run()
 	}
@@ -451,8 +454,9 @@ func RunInstallWizard(ctx context.Context, configPath string, baseDir string, bu
 		SetBorderColor(tui.ProxmoxOrange).
 		SetBackgroundColor(tcell.ColorBlack)
 
-	// Run the app - ignore errors from normal app termination
-	_ = app.SetRoot(flex, true).SetFocus(form.Form).Run()
+	if err := runInstallWizardRunner(app, flex, form.Form); err != nil {
+		return nil, err
+	}
 
 	if data == nil {
 		return nil, ErrInstallCancelled

internal/tui/wizard/install_test.go

@@ -7,8 +7,10 @@ import (
 	"strings"
 	"testing"
 
+	"github.com/gdamore/tcell/v2"
 	"github.com/rivo/tview"
 
+	cronutil "github.com/tis24dev/proxsave/internal/cron"
 	"github.com/tis24dev/proxsave/internal/tui"
 )
 
@@ -223,6 +225,37 @@ func TestApplyInstallDataCronAndNotifications(t *testing.T) {
 	assertContains("ENCRYPT_ARCHIVE", "false")
 }
 
+func TestRunInstallWizardBlankCronIgnoresEnvOverride(t *testing.T) {
+	t.Setenv("CRON_SCHEDULE", "5 1 * * *")
+
+	originalRunner := runInstallWizardRunner
+	t.Cleanup(func() { runInstallWizardRunner = originalRunner })
+
+	runInstallWizardRunner = func(app *tui.App, root, focus tview.Primitive) error {
+		form, ok := focus.(*tview.Form)
+		if !ok {
+			t.Fatalf("focus primitive = %T, want *tview.Form", focus)
+		}
+		button := form.GetButton(0)
+		if button == nil {
+			t.Fatal("expected install button")
+		}
+		button.InputHandler()(tcell.NewEventKey(tcell.KeyEnter, 0, tcell.ModNone), nil)
+		return nil
+	}
+
+	data, err := RunInstallWizard(t.Context(), "/tmp/proxsave/backup.env", "/opt/proxsave", "sig", "")
+	if err != nil {
+		t.Fatalf("RunInstallWizard returned error: %v", err)
+	}
+	if data == nil {
+		t.Fatal("expected wizard data")
+	}
+	if data.CronTime != cronutil.DefaultTime {
+		t.Fatalf("CronTime = %q, want %q", data.CronTime, cronutil.DefaultTime)
+	}
+}
+
 func TestCheckExistingConfigActions(t *testing.T) {
 	tmp := t.TempDir()
 	configPath := filepath.Join(tmp, "prox.env")