update fixed status code issue

Author: KenWilliamson

coverage.out

@@ -1,11 +1,13 @@
 mode: set
-github.com/GolangToolKits/go-http-proxy/goProxy.go:15.62,20.16 5 1
-github.com/GolangToolKits/go-http-proxy/goProxy.go:20.16,24.3 3 1
-github.com/GolangToolKits/go-http-proxy/goProxy.go:24.8,28.19 4 1
-github.com/GolangToolKits/go-http-proxy/goProxy.go:28.19,31.4 2 1
-github.com/GolangToolKits/go-http-proxy/goProxy.go:31.9,34.4 2 1
-github.com/GolangToolKits/go-http-proxy/goProxy.go:36.2,36.24 1 1
-github.com/GolangToolKits/go-http-proxy/goProxy.go:40.31,42.2 1 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:15.62,20.52 5 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:20.52,23.18 3 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:23.18,25.4 1 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:25.9,27.4 1 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:28.8,32.19 4 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:32.19,35.4 2 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:35.9,38.4 2 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:40.2,40.24 1 1
+github.com/GolangToolKits/go-http-proxy/goProxy.go:44.31,46.2 1 1
 github.com/GolangToolKits/go-http-proxy/mockGoProxy.go:17.66,21.18 4 1
 github.com/GolangToolKits/go-http-proxy/mockGoProxy.go:21.18,25.3 3 1
 github.com/GolangToolKits/go-http-proxy/mockGoProxy.go:26.2,26.41 1 1

goProxy.go

@@ -17,10 +17,14 @@ func (p *GoProxy) Do(req *http.Request, obj any) (bool, int) {
 	var statusCode int
 	client := &http.Client{}
 	resp, err := client.Do(req)
-	if err != nil {
+	if err != nil || resp.StatusCode != http.StatusOK {
 		log.Println("go-http-proxy Do err: ", err)
 		log.Println("resp in fail: ", resp)
-		statusCode = http.StatusNotFound
+		if resp != nil {
+			statusCode = resp.StatusCode
+		} else {
+			statusCode = http.StatusNotFound
+		}
 	} else {
 		defer resp.Body.Close()
 		decoder := json.NewDecoder(resp.Body)

goProxy_test.go

@@ -47,6 +47,7 @@ func TestGoProxy_New(t *testing.T) {
 func TestGoProxy_Do(t *testing.T) {
 
 	var sURL = "http://localhost:3002/rs/product/get/list/59/0/100"
+	var sURL4 = "http://localhost:3002/rs/product2/get/list/59/0/100"
 
 	r1, rErr := http.NewRequest("GET", sURL, nil)
 	if rErr != nil {
@@ -58,6 +59,15 @@ func TestGoProxy_Do(t *testing.T) {
 
 	r2, _ := http.NewRequest("GET", "http:///", nil)
 
+
+	r4, rErr4 := http.NewRequest("GET", sURL4, nil)
+	if rErr4 != nil {
+		fmt.Println("request error: ", rErr4)
+	}
+	r4.Header.Set("Content-Type", "application/json")
+	r4.Header.Set("apiKey", "GDG651GFD66FD16151sss651f651ff65555ddfhjklyy5")
+	r4.Header.Set("storeName", "defaultLocalStore")
+
 	var uRes []prod
 
 	type args struct {
@@ -99,6 +109,15 @@ func TestGoProxy_Do(t *testing.T) {
 			want:  false,
 			want1: 0,
 		},
+		{
+			name: "test 4",
+			args: args{
+				req: r4,
+				obj: uRes, //non pointer fail
+			},
+			want:  false,
+			want1: 404,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {