X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=capifcore%2Finternal%2Fhelmmanagement%2Fhelm_test.go;h=5eb83f37b2509acbe9b7f84cf3f8f35fc22a56b0;hb=ae8001da50fc2f29c76888d194f9ab2a24f573f7;hp=6f89c539a7c2eab30991db082eddf5867b713e7a;hpb=2ba4580c67fbe7994141e4cd2701f7bd22b69ebf;p=nonrtric%2Fplt%2Fsme.git

diff --git a/capifcore/internal/helmmanagement/helm_test.go b/capifcore/internal/helmmanagement/helm_test.go
index 6f89c53..5eb83f3 100644
--- a/capifcore/internal/helmmanagement/helm_test.go
+++ b/capifcore/internal/helmmanagement/helm_test.go
@@ -38,19 +38,40 @@ import (
 	"oransc.org/nonrtric/capifcore/internal/helmmanagement/mocks"
 )
 
-func TestSetUpRepo_repoShouldBeAddedToReposFile(t *testing.T) {
+func TestNoChartURL_reoNotSetUp(t *testing.T) {
+	managerUnderTest := NewHelmManager(nil)
+
+	res := managerUnderTest.SetUpRepo("repoName", "")
+
+	assert.Nil(t, res)
+	assert.False(t, managerUnderTest.setUp)
+}
+
+// func TestSetUpRepo_repoShouldBeAddedToReposFile(t *testing.T) {
+// 	settings := createReposFile(t)
+
+// 	managerUnderTest := NewHelmManager(settings)
+
+// 	repoName := "repoName"
+// 	repoURL := "http://url"
+// 	managerUnderTest.repo = getChartRepo(settings)
+
+// 	res := managerUnderTest.SetUpRepo(repoName, repoURL)
+
+// 	assert.Nil(t, res)
+// 	assert.True(t, containsRepo(settings.RepositoryConfig, repoName))
+// 	assert.True(t, managerUnderTest.setUp)
+// }
+
+func TestSetUpRepoFail_shouldNotBeSetUp(t *testing.T) {
 	settings := createReposFile(t)
 
 	managerUnderTest := NewHelmManager(settings)
 
-	repoName := "repoName"
-	repoURL := "http://url"
-	managerUnderTest.repo = getChartRepo(settings)
-
-	res := managerUnderTest.SetUpRepo(repoName, repoURL)
+	res := managerUnderTest.SetUpRepo("repoName", "repoURL")
 
-	assert.Nil(t, res)
-	assert.True(t, containsRepo(settings.RepositoryConfig, repoName))
+	assert.NotNil(t, res)
+	assert.False(t, managerUnderTest.setUp)
 }
 
 func createReposFile(t *testing.T) *cli.EnvSettings {