From a44c38fc63b13b26544b64a396dc0fe076ae0fbf Mon Sep 17 00:00:00 2001
From: Jane Liu L <jane.l.liu@ericsson.com>
Date: Tue, 27 Jul 2021 16:37:13 +0800
Subject: [PATCH] Improve UT: add cross namespace helmchart test

Signed-off-by: Jane Liu L <jane.l.liu@ericsson.com>
---
 controllers/helmchart_controller_test.go | 54 ++++++++++++++++++++++++
 1 file changed, 54 insertions(+)

diff --git a/controllers/helmchart_controller_test.go b/controllers/helmchart_controller_test.go
index c88fa3395..dbe354b9b 100644
--- a/controllers/helmchart_controller_test.go
+++ b/controllers/helmchart_controller_test.go
@@ -407,6 +407,60 @@ var _ = Describe("HelmChartReconciler", func() {
 			Expect(chart.Status.Artifact.Revision).Should(Equal("0.1.1"))
 		})
 
+		It("Cross namespace", func() {
+			Expect(helmServer.PackageChart(path.Join("testdata/charts/helmchart"))).Should(Succeed())
+			Expect(helmServer.GenerateIndex()).Should(Succeed())
+
+			repositoryKey := types.NamespacedName{
+				Name:      "helmrepository-sample-" + randStringRunes(5),
+				Namespace: namespace.Name,
+			}
+			Expect(k8sClient.Create(context.Background(), &sourcev1.HelmRepository{
+				ObjectMeta: metav1.ObjectMeta{
+					Name:      repositoryKey.Name,
+					Namespace: repositoryKey.Namespace,
+				},
+				Spec: sourcev1.HelmRepositorySpec{
+					URL:      helmServer.URL(),
+					Interval: metav1.Duration{Duration: indexInterval},
+				},
+			})).Should(Succeed())
+
+			key := types.NamespacedName{
+				Name:      "helmchart-sample-" + randStringRunes(5),
+				Namespace: "default",
+			}
+			created := &sourcev1.HelmChart{
+				ObjectMeta: metav1.ObjectMeta{
+					Name:      key.Name,
+					Namespace: key.Namespace,
+				},
+				Spec: sourcev1.HelmChartSpec{
+					Chart:   "helmchart",
+					Version: "",
+					SourceRef: sourcev1.LocalHelmChartSourceReference{
+						Kind: sourcev1.HelmRepositoryKind,
+						Name: repositoryKey.Name,
+					},
+					Interval: metav1.Duration{Duration: pullInterval},
+				},
+			}
+			Expect(k8sClient.Create(context.Background(), created)).Should(Succeed())
+
+			got := &sourcev1.HelmChart{}
+			Eventually(func() bool {
+				_ = k8sClient.Get(context.Background(), key, got)
+				for _, c := range got.Status.Conditions {
+					if strings.Contains(c.Message, "failed to retrieve source") {
+						return true
+					}
+				}
+				return false
+			}, timeout, interval).Should(BeTrue())
+
+			Expect(k8sClient.Delete(context.Background(), created)).Should(Succeed())
+		})
+
 		It("Authenticates when credentials are provided", func() {
 			helmServer.Stop()
 			var username, password = "john", "doe"