Errors handling

Signed-off-by: Maayan Hadasi <mguetta@redhat.com>

config/config.json

@@ -1,3 +1,4 @@
 {
-	"RETRY_NUM": "10"
+	"RETRY_NUM": "10",
+	"KEEP": "0"
 }

config/configuration.go

@@ -5,4 +5,5 @@ type Configuration struct {
 	JIRA_PASSWORD string `env:"JIRA_PASSWORD"`
 	JIRA_URL      string `env:"JIRA_URL"`
 	RETRY_NUM     string `env:"RETRY_NUM"`
+	KEEP          string `env:"KEEP"`
 }

e2e/jiraintegration/jira_connection.go

@@ -25,11 +25,13 @@ const (
 
 var _ = Describe("Jira Connection", Ordered, func() {
 	AfterAll(func() {
+		// Resources cleanup
 		// Delete tracker instance and the associated identity after
-
-		//TODO - error handling
-		Tracker.Delete(jiraInstance.ID)
-		Identity.Delete(jiraBasicAuth.ID)
+		if keep, _ := strconv.ParseBool(Config.KEEP); keep {
+			return
+		}
+		Expect(Tracker.Delete(jiraInstance.ID)).To(Succeed())
+		Expect(Identity.Delete(jiraBasicAuth.ID)).To(Succeed())
 	})
 
 	Context("Create new Jira instance", func() {

e2e/metrics/application_metrics.go

@@ -1,9 +1,12 @@
 package metrics
 
 import (
+	"strconv"
+
 	. "github.com/onsi/ginkgo/v2"
 	. "github.com/onsi/gomega"
 
+	. "github.com/konveyor/go-konveyor-tests/config"
 	"github.com/konveyor/go-konveyor-tests/utils"
 	"github.com/konveyor/tackle2-hub/api"
 )
@@ -23,6 +26,9 @@ var _ = Describe("Application Metrics", Ordered, func() {
 
 	AfterAll(func() {
 		// Resources cleanup
+		if keep, _ := strconv.ParseBool(Config.KEEP); keep {
+			return
+		}
 		utils.DeleteApplicationsBySlice(apps)
 	})
 
@@ -39,7 +45,7 @@ var _ = Describe("Application Metrics", Ordered, func() {
 		It("Should decrease the applications gauge", func() {
 			// Delete the last application item
 			lastApplicationItem := apps[len(apps)-1]
-			Application.Delete(lastApplicationItem.ID)
+			utils.DeleteApplicationsBySlice([]api.Application{lastApplicationItem})
 			metricValue--
 			Expect(GetMetricValue(metricName)).To(Equal(metricValue))
 		})

utils/application_utils.go

@@ -6,6 +6,7 @@ import (
 	"github.com/konveyor/go-konveyor-tests/data"
 	"github.com/konveyor/go-konveyor-tests/hack/uniq"
 	"github.com/konveyor/tackle2-hub/api"
+	. "github.com/onsi/gomega"
 )
 
 func CreateMultipleApplications(numberOfApplications int) []api.Application {
@@ -17,15 +18,15 @@ func CreateMultipleApplications(numberOfApplications int) []api.Application {
 		//// Create a local copy
 		application := data.ApplicationSamples[randomIndex]
 		uniq.ApplicationName(&application)
-		//// TODO: dealing with create returns error
-		Application.Create(&application)
+		Expect(Application.Create(&application)).To(Succeed())
 		appSlice = append(appSlice, application)
 	}
 	return appSlice
 }
 
 func DeleteApplicationsBySlice(apps []api.Application) {
 	for i := 0; i < len(apps); i++ {
-		Application.Delete(apps[i].ID)
+		Expect(Application.Delete(apps[i].ID)).To(Succeed())
+		return
 	}
 }