clean up the code

Signed-off-by: Stephanie <yangcao@redhat.com>

Author: yangcao77

go.mod

@@ -15,7 +15,7 @@ require (
 	github.com/openshift/api v0.0.0-20200930075302-db52bc4ef99f
 	github.com/pkg/errors v0.9.1
 	github.com/spf13/afero v1.2.2
-	github.com/stretchr/testify v1.6.1 // indirect
+	github.com/stretchr/testify v1.6.1
 	github.com/xeipuuv/gojsonschema v1.2.0
 	k8s.io/api v0.19.0
 	k8s.io/apimachinery v0.19.0

go.sum

@@ -494,6 +494,7 @@ gopkg.in/yaml.v2 v2.3.0 h1:clyUAQHOM3G0M3f5vQj7LuJrETvjVot3Z5el9nffUtU=
 gopkg.in/yaml.v2 v2.3.0/go.mod h1:hI93XBmqTisBFMUTm0b8Fm+jr3Dg1NNxqwp+5A1VGuI=
 gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c h1:dUUwHk2QECo/6vqA44rthZ8ie2QXMNeKRTHCNY2nXvo=
 gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
+gotest.tools v2.2.0+incompatible h1:VsBPFP1AI068pPrMxtb/S8Zkgf9xEmTLJjfM+P5UIEo=
 gotest.tools v2.2.0+incompatible/go.mod h1:DsYFclhRJ6vuDpmuTbkuFWG+y2sxOXAzmJt81HFBacw=
 honnef.co/go/tools v0.0.0-20190102054323-c2f93a96b099/go.mod h1:rf3lG4BRIbNafJWhAfAdb/ePZxsR/4RtNHQocxwk9r4=
 honnef.co/go/tools v0.0.0-20190106161140-3f1c8253044a/go.mod h1:rf3lG4BRIbNafJWhAfAdb/ePZxsR/4RtNHQocxwk9r4=

pkg/devfile/parser/context/context.go

@@ -72,7 +72,7 @@ func (d *DevfileCtx) Populate() (err error) {
 	}
 	klog.V(4).Infof("absolute devfile path: '%s'", d.absPath)
 	if d.uriMap == nil {
-		d.uriMap= make(map[string]bool)
+		d.uriMap = make(map[string]bool)
 	}
 	if d.uriMap[d.absPath] {
 		return fmt.Errorf("URI %v is recursively referenced", d.absPath)
@@ -93,11 +93,7 @@ func (d *DevfileCtx) PopulateFromURL() (err error) {
 		return err
 	}
 	if d.uriMap == nil {
-		d.uriMap= make(map[string]bool)
-	}
-	fmt.Printf("uriMap length is: %v", len(d.uriMap))
-	for k,_ := range d.uriMap {
-		fmt.Printf("uriMap now contains: %s", k)
+		d.uriMap = make(map[string]bool)
 	}
 	if d.uriMap[d.url] {
 		return fmt.Errorf("URI %v is recursively referenced", d.url)
@@ -150,8 +146,6 @@ func (d *DevfileCtx) GetURIMap() map[string]bool {
 }
 
 // GetURIMap func returns current devfile uri map
-func (d *DevfileCtx) SetURIMap(uriMap map[string]bool){
+func (d *DevfileCtx) SetURIMap(uriMap map[string]bool) {
 	d.uriMap = uriMap
 }
-
-

pkg/devfile/parser/parse.go

@@ -185,51 +185,38 @@ func parseFromURI(uri string, curDevfileCtx devfileCtx.DevfileCtx) (DevfileObj,
 	}
 	// NewDevfileCtx
 	var d DevfileObj
-	// absolute URL address
-	if strings.HasPrefix(uri, "http://") || strings.HasPrefix(uri, "https://") {
-		// return ParseFromURL(uri)
-		d.Ctx = devfileCtx.NewURLDevfileCtx(uri)
-		d.Ctx.SetURIMap(curDevfileCtx.GetURIMap())
-		// Fill the fields of DevfileCtx struct
-		err = d.Ctx.PopulateFromURL()
-		if err != nil {
-			return DevfileObj{}, err
-		}
-		return parseDevfile(d, true)
-	}
+	absoluteURL := strings.HasPrefix(uri, "http://") || strings.HasPrefix(uri, "https://")
 
 	// relative path on disk
-	if curDevfileCtx.GetAbsPath() != "" {
+	if !absoluteURL && curDevfileCtx.GetAbsPath() != "" {
 		d.Ctx = devfileCtx.NewDevfileCtx(path.Join(path.Dir(curDevfileCtx.GetAbsPath()), uri))
 		d.Ctx.SetURIMap(curDevfileCtx.GetURIMap())
 
 		// Fill the fields of DevfileCtx struct
 		err = d.Ctx.Populate()
-		if err!= nil {
+		if err != nil {
 			return DevfileObj{}, err
 		}
-		// return Parse(path.Join(path.Dir(curDevfileCtx.GetAbsPath()), uri))
 		return parseDevfile(d, true)
 	}
 
-	if curDevfileCtx.GetURL() != "" {
+	// absolute URL address
+	if absoluteURL {
+		d.Ctx = devfileCtx.NewURLDevfileCtx(uri)
+	} else if curDevfileCtx.GetURL() != "" {
 		u, err := url.Parse(curDevfileCtx.GetURL())
 		if err != nil {
 			return DevfileObj{}, err
 		}
-
 		u.Path = path.Join(path.Dir(u.Path), uri)
 		d.Ctx = devfileCtx.NewURLDevfileCtx(u.String())
-		d.Ctx.SetURIMap(curDevfileCtx.GetURIMap())
-		// Fill the fields of DevfileCtx struct
-		err = d.Ctx.PopulateFromURL()
-		if err != nil {
-			return DevfileObj{}, err
-		}
-		return parseDevfile(d, true)
-		// u.String() is the joint absolute URL path
-		// return ParseFromURL(u.String())
 	}
+	d.Ctx.SetURIMap(curDevfileCtx.GetURIMap())
+	// Fill the fields of DevfileCtx struct
+	err = d.Ctx.PopulateFromURL()
+	if err != nil {
+		return DevfileObj{}, err
+	}
+	return parseDevfile(d, true)
 
-	return DevfileObj{}, fmt.Errorf("fail to parse from uri: %s", uri)
 }