Merge pull request #101 from bmelville/98

Adds support for "dm deployed-instances <type>:<version>"

dm/dm.go

@@ -111,10 +111,7 @@ func main() {
 		fmt.Printf("Templates:\n")
 		for _, t := range templates {
 			fmt.Printf("%s:%s\n", t.Name, t.Version)
-			downloadURL, err := git.GetURL(t)
+			downloadURL := getDownloadUrl(t)
-			if err != nil {
-				log.Printf("Failed to get download URL for template %s:%s", t.Name, t.Version)
-			}
 
 			fmt.Printf("\tdownload URL: %s\n", downloadURL)
 		}
@@ -160,8 +157,9 @@ func main() {
 			usage()
 		}
 
-		path := fmt.Sprintf("types/%s/instances", url.QueryEscape(args[1]))
+		tUrl := getTypeUrl(args[1])
-		action := fmt.Sprintf("list deployed instances of type %s", args[1])
+		path := fmt.Sprintf("types/%s/instances", url.QueryEscape(tUrl))
+		action := fmt.Sprintf("list deployed instances of type %s", tUrl)
 		callService(path, "GET", action, nil)
 	default:
 		usage()
@@ -211,28 +209,38 @@ func describeType(args []string) {
 		usage()
 	}
 
-	var tUrl string
+	tUrl := getTypeUrl(args[1])
+	schemaUrl := tUrl + ".schema"
+	fmt.Println(callHttp(schemaUrl, "GET", "get schema for type ("+tUrl+")", nil))
+}
 
-	if strings.HasPrefix(args[1], "http://") || strings.HasPrefix(args[1], "https://") {
+func getTypeUrl(tName string) string {
+	if isHttp(tName) {
 		// User can pass raw URL to template.
-		tUrl = args[1]
+		return tName
-	} else {
+	}
+
 	// User can pass registry type.
-		t := getRegistryType(args[1])
+	t := getRegistryType(tName)
 	if t == nil {
-			log.Fatalf("Invalid type name, must be in the form \"<type-name>:<version>\": %s", args[1])
+		log.Fatalf("Invalid type name, must be in the form \"<type-name>:<version>\": %s", tName)
 	}
 
+	return getDownloadUrl(*t)
+}
+
+func getDownloadUrl(t registry.Type) string {
 	git := getGitRegistry()
-		url, err := git.GetURL(*t)
+	url, err := git.GetURL(t)
 	if err != nil {
-			log.Fatalf("Failed to fetch type information for %s: %s", args[1], err)
+		log.Fatalf("Failed to fetch type information for \"%s:%s\": %s", t.Name, t.Version, err)
-		}
-		tUrl = url
 	}
 
-	schemaUrl := tUrl + ".schema"
+	return url
-	fmt.Println(callHttp(schemaUrl, "GET", "get schema for type ("+tUrl+")", nil))
+}
+
+func isHttp(t string) bool {
+	return strings.HasPrefix(t, "http://") || strings.HasPrefix(t, "https://")
 }
 
 func loadTemplate(args []string) *expander.Template {
@@ -279,11 +287,7 @@ func getRegistryType(fullType string) *registry.Type {
 }
 
 func buildTemplateFromType(t registry.Type) *expander.Template {
-	git := getGitRegistry()
+	downloadURL := getDownloadUrl(t)
-	downloadURL, err := git.GetURL(t)
-	if err != nil {
-		log.Fatalf("Failed to get download URL for type %s:%s\n%s\n", t.Name, t.Version, err)
-	}
 
 	props := make(map[string]interface{})
 	if *properties != "" {