From b1d8ad5a6f8fc6538bc9d405d12ec49eaf49a61f Mon Sep 17 00:00:00 2001 From: "Stephen McQuay (smcquay)" Date: Fri, 17 Jun 2016 09:15:09 -0700 Subject: [PATCH] fix for bad header value Go tool brought in changes that properly implement: http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2 merged in here: https://go-review.googlesource.com/#/c/23229/ This surfaced a bad http header value in our tests. Change-Id: I2c507bb4c92b060c78a7909600649dae2d22515d --- api_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/api_test.go b/api_test.go index 61dd5a7..acc995c 100644 --- a/api_test.go +++ b/api_test.go @@ -650,7 +650,7 @@ func TestRoundTrip(t *testing.T) { if err != nil { t.Fatalf("Failed to parse response body: %v", err) } - tok := strings.Trim(string(bs), "new token: ") + tok := strings.TrimSpace(strings.Trim(string(bs), "new token: ")) u = fmt.Sprintf("%s/foo", ts.URL) body := strings.NewReader(`{"repo": "https://s.mcquay.me/sm/vain"}`) @@ -742,7 +742,7 @@ func TestForgot(t *testing.T) { if err != nil { t.Fatalf("Failed to parse response body: %v", err) } - recTok := strings.Trim(string(ft), "new token: ") + recTok := strings.TrimSpace(strings.Trim(string(ft), "new token: ")) if iniTok == recTok { t.Fatalf("tokens should not be the same; old token %s, new token %s", iniTok, recTok)