From 727c848d0a3eb43e33593d2ac25a5ddae8714db8 Mon Sep 17 00:00:00 2001
From: chilianyi <stark@yunify.com>
Date: Wed, 24 May 2023 18:29:01 +0800
Subject: [PATCH] Fix the issue of removing argocd resources without cascade
 parameter (#949)

---
 pkg/kapis/gitops/v1alpha1/argocd/route_test.go | 8 ++++++++
 pkg/kapis/gitops/v1alpha1/gitops/handler.go    | 5 +----
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/pkg/kapis/gitops/v1alpha1/argocd/route_test.go b/pkg/kapis/gitops/v1alpha1/argocd/route_test.go
index e3620e90..c4ecb6d8 100644
--- a/pkg/kapis/gitops/v1alpha1/argocd/route_test.go
+++ b/pkg/kapis/gitops/v1alpha1/argocd/route_test.go
@@ -236,6 +236,14 @@ func TestPublicAPIs(t *testing.T) {
 		},
 		Spec: v1alpha1.ApplicationSpec{
 			Kind: v1alpha1.ArgoCD,
+			ArgoApp: &v1alpha1.ArgoApplication{
+				Operation: nil,
+				Spec: v1alpha1.ArgoApplicationSpec{
+					SyncPolicy: &v1alpha1.SyncPolicy{
+						Automated: nil,
+					},
+				},
+			},
 		},
 	}
 
diff --git a/pkg/kapis/gitops/v1alpha1/gitops/handler.go b/pkg/kapis/gitops/v1alpha1/gitops/handler.go
index ba1b2f8b..86f7f642 100644
--- a/pkg/kapis/gitops/v1alpha1/gitops/handler.go
+++ b/pkg/kapis/gitops/v1alpha1/gitops/handler.go
@@ -86,9 +86,7 @@ func (h *Handler) DelApplication(req *restful.Request, res *restful.Response) {
 	}
 	err := h.Get(ctx, objectKey, application)
 	if err == nil {
-		switch application.Spec.Kind {
-		case v1alpha1.ArgoCD:
-			// add the Argo CD resources finalizer if cascade is true
+		if argo := application.Spec.ArgoApp; argo != nil {
 			if cascade == "true" {
 				if k8sutil.AddFinalizer(&application.ObjectMeta, v1alpha1.ArgoCDResourcesFinalizer) {
 					if err = h.Update(ctx, application); err != nil {
@@ -102,7 +100,6 @@ func (h *Handler) DelApplication(req *restful.Request, res *restful.Response) {
 					}
 				}
 			}
-		case v1alpha1.FluxCD:
 		}
 		err = h.Delete(ctx, application)
 	}