use http.StatusOK instead of 200

This commit is contained in:
tanshanshan 2018-09-27 08:33:27 +08:00
parent 516876b232
commit acdc45628f
2 changed files with 4 additions and 3 deletions

View File

@ -19,6 +19,7 @@ package master
import ( import (
"encoding/json" "encoding/json"
"fmt" "fmt"
"net/http"
"reflect" "reflect"
"strings" "strings"
"testing" "testing"
@ -98,8 +99,8 @@ func TestOpenAPIDelegationChainPlumbing(t *testing.T) {
result := kubeclient.RESTClient().Get().AbsPath("/swagger.json").Do() result := kubeclient.RESTClient().Get().AbsPath("/swagger.json").Do()
status := 0 status := 0
result.StatusCode(&status) result.StatusCode(&status)
if status != 200 { if status != http.StatusOK {
t.Fatalf("GET /swagger.json failed: expected status=%d, got=%d", 200, status) t.Fatalf("GET /swagger.json failed: expected status=%d, got=%d", http.StatusOK, status)
} }
raw, err := result.Raw() raw, err := result.Raw()

View File

@ -54,7 +54,7 @@ func scrapeMetrics(s *httptest.Server) ([]*prometheuspb.MetricFamily, error) {
return nil, fmt.Errorf("Unable to contact metrics endpoint of master: %v", err) return nil, fmt.Errorf("Unable to contact metrics endpoint of master: %v", err)
} }
defer resp.Body.Close() defer resp.Body.Close()
if resp.StatusCode != 200 { if resp.StatusCode != http.StatusOK {
return nil, fmt.Errorf("Non-200 response trying to scrape metrics from master: %v", resp) return nil, fmt.Errorf("Non-200 response trying to scrape metrics from master: %v", resp)
} }