updated error handling to return non-zero exit code for failed http call

Author: benPearce1

pkg/cmd/api/api.go

@@ -3,6 +3,7 @@ package api
 import (
 	"bytes"
 	"encoding/json"
+	"errors"
 	"io"
 	"net/http"
 	"os"
@@ -62,6 +63,10 @@ func apiRun(cmd *cobra.Command, f factory.Factory, path string) error {
 		return err
 	}
 
+	if resp.StatusCode < 200 || resp.StatusCode >= 300 {
+		return errors.New(string(body))
+	}
+
 	// Pretty-print if valid JSON, otherwise output raw
 	var prettyJSON bytes.Buffer
 	if err := json.Indent(&prettyJSON, body, "", "  "); err == nil {
@@ -70,9 +75,5 @@ func apiRun(cmd *cobra.Command, f factory.Factory, path string) error {
 		cmd.Print(string(body))
 	}
 
-	if resp.StatusCode < 200 || resp.StatusCode >= 300 {
-		OsExit(resp.StatusCode)
-	}
-
 	return nil
 }