From: Brad Fitzpatrick <bradfitz@golang.org>
Date: Tue, 28 Jul 2015 05:53:37 +0000 (+0200)
Subject: encoding/json: test style tweaks
X-Git-Tag: go1.5beta3~32
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=d0729a6edebcaae1ab6554244fc6ef2ff36609bf;p=gostls13.git

encoding/json: test style tweaks

Rename test name from Http to HTTP, and fix some style nits.

Change-Id: I00fe1cecd69ca2f50be86a76ec90031c2f921707
Reviewed-on: https://go-review.googlesource.com/12760
Reviewed-by: Andrew Gerrand <adg@golang.org>
---

diff --git a/src/encoding/json/stream_test.go b/src/encoding/json/stream_test.go
index 1e9d3d0dce..c2e30408cd 100644
--- a/src/encoding/json/stream_test.go
+++ b/src/encoding/json/stream_test.go
@@ -319,43 +319,36 @@ func TestDecodeInStream(t *testing.T) {
 
 }
 
-const raw = `{ "foo": "bar" }`
+// Test from golang.org/issue/11893
+func TestHTTPDecoding(t *testing.T) {
+	const raw = `{ "foo": "bar" }`
 
-func makeHTTP() io.ReadCloser {
-	mux := http.NewServeMux()
-	mux.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
+	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		w.Write([]byte(raw))
-	})
-	ts := httptest.NewServer(mux)
+	}))
 	defer ts.Close()
 	res, err := http.Get(ts.URL)
 	if err != nil {
 		log.Fatalf("GET failed: %v", err)
 	}
-	return res.Body
-}
-
-func TestHttpDecoding(t *testing.T) {
+	defer res.Body.Close()
 
 	foo := struct {
 		Foo string
 	}{}
 
-	rc := makeHTTP()
-	defer rc.Close()
-
-	d := NewDecoder(rc)
-	err := d.Decode(&foo)
+	d := NewDecoder(res.Body)
+	err = d.Decode(&foo)
 	if err != nil {
-		t.Errorf("Unexpected error %v", err)
+		t.Fatalf("Decode: %v", err)
 	}
 	if foo.Foo != "bar" {
-		t.Errorf("Expected \"bar\", was %v", foo.Foo)
+		t.Errorf("decoded %q; want \"bar\"", foo.Foo)
 	}
 
 	// make sure we get the EOF the second time
 	err = d.Decode(&foo)
 	if err != io.EOF {
-		t.Errorf("Expected io.EOF, was %v", err)
+		t.Errorf("err = %v; want io.EOF", err)
 	}
 }