From: Dmitri Shuralyov Date: Sat, 16 Nov 2019 23:19:33 +0000 (+0000) Subject: net/http: rename tests for Redirect and StripPrefix X-Git-Tag: go1.14beta1~114 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=6ba1bf393af0d4fb71e29a0e7f0f52502cff29eb;p=gostls13.git net/http: rename tests for Redirect and StripPrefix Before May 2018, I mistakenly thought the _suffix naming convention¹ used by examples also applied to tests. Thanks to a code review comment² from Ian Lance Taylor, I have since learned that is not true. This trivial change fixes some collateral damage from my earlier misunderstanding, resulting in improved test naming consistency. ¹ https://golang.org/pkg/testing/#hdr-Examples ² https://go-review.googlesource.com/c/go/+/112935/1/src/path/filepath/path_test.go#1075 Change-Id: I555f60719629eb64bf2f096aa3dd5e00851827cd Reviewed-on: https://go-review.googlesource.com/c/go/+/207446 Run-TryBot: Dmitri Shuralyov TryBot-Result: Gobot Gobot Reviewed-by: Emmanuel Odeke Reviewed-by: Brad Fitzpatrick --- diff --git a/src/net/http/serve_test.go b/src/net/http/serve_test.go index a2ab52b829..1a690efb49 100644 --- a/src/net/http/serve_test.go +++ b/src/net/http/serve_test.go @@ -2632,7 +2632,7 @@ func TestRedirect(t *testing.T) { // Test that Redirect sets Content-Type header for GET and HEAD requests // and writes a short HTML body, unless the request already has a Content-Type header. -func TestRedirect_contentTypeAndBody(t *testing.T) { +func TestRedirectContentTypeAndBody(t *testing.T) { type ctHeader struct { Values []string } @@ -2911,7 +2911,7 @@ func TestStripPrefix(t *testing.T) { } // https://golang.org/issue/18952. -func TestStripPrefix_notModifyRequest(t *testing.T) { +func TestStripPrefixNotModifyRequest(t *testing.T) { h := StripPrefix("/foo", NotFoundHandler()) req := httptest.NewRequest("GET", "/foo/bar", nil) h.ServeHTTP(httptest.NewRecorder(), req)