Changed return management for download, install and upgrade command

Author: Rocketct

commands/core/download.go

@@ -50,7 +50,6 @@ func PlatformDownload(ctx context.Context, req *rpc.PlatformDownloadReq, progres
 	}
 	err = downloadPlatform(pm, platform, progressCallback)
 	if err != nil {
-		formatter.PrintError(err, "Error Installing "+platform.String())
 		return nil, err
 	}
 	for _, tool := range tools {

commands/core/install.go

@@ -34,7 +34,6 @@ func PlatformInstall(ctx context.Context, req *rpc.PlatformInstallReq) (*rpc.Pla
 
 	err = installPlatform(pm, platform, tools)
 	if err != nil {
-		formatter.PrintError(err, "Error Installing "+platform.String())
 		return nil, err
 	}
 

commands/core/upgrade.go

@@ -44,7 +44,10 @@ func PlatformUpgrade(ctx context.Context, req *rpc.PlatformUpgradeReq) (*rpc.Pla
 		PlatformVersion:      version}
 	pm := commands.GetPackageManager(req)
 
-	UpgradePlatform(pm, ref)
+	err := UpgradePlatform(pm, ref)
+	if err != nil {
+		return nil, err
+	}
 	return &rpc.PlatformUpgradeResp{}, nil
 }
 
@@ -81,7 +84,10 @@ func UpgradePlatform(pm *packagemanager.PackageManager, platformRef *packagemana
 		if err != nil {
 			return fmt.Errorf("Platform " + platformRef.String() + " is not installed")
 		}
-		installPlatform(pm, platform, tools)
+		err = installPlatform(pm, platform, tools)
+		if err != nil {
+			return err
+		}
 	}
 	return nil
 }