Scale windows VM bug: catch error for len(vms) < 1 (#2059)

Cecile Robert-Michon · jackfrancis · commit 1d33229664d0 · 2018-01-16T11:06:19.000-08:00
* add win config to tests * catch 0 vms error * Revert "add win config to tests" This reverts commit 3ffca42. * rm comment
diff --git a/cmd/scale.go b/cmd/scale.go
@@ -188,6 +188,8 @@ func (sc *scaleCmd) run(cmd *cobra.Command, args []string) error {
 		vms, err := sc.client.ListVirtualMachines(sc.resourceGroupName)
 		if err != nil {
 			log.Fatalln("failed to get vms in the resource group. Error: %s", err.Error())
+		} else if len(*vms.Value) < 1 {
+			log.Fatalln("The provided resource group does not contain any vms.")
 		}
 		for _, vm := range *vms.Value {
 

Original file line number	Diff line number	Diff line change
`@@ -188,6 +188,8 @@ func (sc scaleCmd) run(cmd cobra.Command, args []string) error {`
`188`	`188`	`vms, err := sc.client.ListVirtualMachines(sc.resourceGroupName)`
`189`	`189`	`if err != nil {`
`190`	`190`	`log.Fatalln("failed to get vms in the resource group. Error: %s", err.Error())`
	`191`	`+ } else if len(*vms.Value) < 1 {`
	`192`	`+ log.Fatalln("The provided resource group does not contain any vms.")`
`191`	`193`	`}`
`192`	`194`	`for _, vm := range *vms.Value {`
`193`	`195`