test: update workflow tests for batch 2.5a (version 2, 15 steps)

Author: papadopouloskyriakos

internal/flowengine/workflows/workflows_test.go

@@ -63,20 +63,21 @@ func TestAppStoreInstallWorkflowSteps(t *testing.T) {
 	if w.Type() != "appstore_install" {
 		t.Errorf("expected type 'appstore_install', got %q", w.Type())
 	}
-	if w.Version() != 1 {
-		t.Errorf("expected version 1, got %d", w.Version())
+	if w.Version() != 2 {
+		t.Errorf("expected version 2, got %d", w.Version())
 	}
 
 	steps := w.Steps()
-	if len(steps) != 13 {
-		t.Fatalf("expected 13 steps, got %d", len(steps))
+	if len(steps) != 15 {
+		t.Fatalf("expected 15 steps, got %d", len(steps))
 	}
 
 	// Verify step order
 	expectedNames := []string{
 		"validate", "read_manifest", "allocate_port", "process_manifest",
 		"create_dirs", "remap_volumes", "write_compose", "deploy_stack",
-		"wait_convergence", "add_dns", "create_proxy", "insert_db", "health_check",
+		"wait_convergence", "add_dns", "create_proxy", "insert_db",
+		"store_volumes", "detect_webui", "health_check",
 	}
 	for i, name := range expectedNames {
 		if steps[i].Name != name {
@@ -110,7 +111,8 @@ func TestAppStoreInstallWorkflowSteps(t *testing.T) {
 	// Verify read-only steps have no compensation
 	readOnlySteps := map[string]bool{
 		"validate": true, "read_manifest": true, "process_manifest": true,
-		"remap_volumes": true, "wait_convergence": true, "health_check": true,
+		"remap_volumes": true, "wait_convergence": true, "store_volumes": true,
+		"detect_webui": true, "health_check": true,
 	}
 	for _, step := range steps {
 		if readOnlySteps[step.Name] && step.Compensate != "" {
@@ -182,7 +184,7 @@ func TestAllWorkflowsHaveNonEmptyActions(t *testing.T) {
 		steps int
 	}{
 		{"app_install", 9},
-		{"appstore_install", 13},
+		{"appstore_install", 15},
 		{"appstore_remove", 6},
 	} {
 		var stepCount int