f

prometheus · May 15, 2019 · f14e079 · f14e079
1 parent 5d404dc
commit f14e079
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 13 deletions.
diff --git a/api/client.go b/api/client.go
@@ -16,7 +16,6 @@ package api
 
 import (
 	"context"
-	"fmt"
 	"io/ioutil"
 	"net"
 	"net/http"
@@ -43,13 +42,6 @@ func (w *ErrorAPI) Err() error {
 }
 
 func (w *ErrorAPI) Error() string {
-	if w == nil {
-		fmt.Println("w nil")
-	} else {
-		if w.err == nil {
-			fmt.Println("w err nil")
-		}
-	}
 	return w.err.Error()
 }
 

diff --git a/api/client_test.go b/api/client_test.go
@@ -16,7 +16,6 @@ package api
 import (
 	"context"
 	"encoding/json"
-	"fmt"
 	"net/http"
 	"net/http/httptest"
 	"net/url"
@@ -155,7 +154,6 @@ func TestDoGetFallback(t *testing.T) {
 	// Do a post, and ensure that the post succeeds.
 	_, b, err := DoGetFallback(client, context.TODO(), u, v)
 	if err != nil {
-		fmt.Println(err)
 		t.Fatalf("Error doing local request: %v", err)
 	}
 	resp := &testResponse{}

diff --git a/api/prometheus/v1/api_test.go b/api/prometheus/v1/api_test.go
@@ -27,6 +27,7 @@ import (
 
 	"github.com/prometheus/client_golang/api"
 	"github.com/prometheus/common/model"
+	"github.com/prometheus/tsdb/testutil"
 )
 
 type apiTest struct {
@@ -883,9 +884,7 @@ func TestAPIClientDo(t *testing.T) {
 			}
 
 			if test.expectedWarnings != nil {
-				if !reflect.DeepEqual(test.expectedWarnings, err.Warnings()) {
-					t.Errorf("expected warnings %v, actual=%v", test.expectedWarnings, err.Warnings())
-				}
+				testutil.Equals(t, test.expectedWarnings, err.Warnings(), "expected warnings %v, actual=%v", test.expectedWarnings, err.Warnings())
 			} else {
 				if err.Warnings() != nil {
 					t.Errorf("Unexpected warnings: %v", err.Warnings())