diff options
| author | Brad Fitzpatrick <bradfitz@golang.org> | 2015-07-28 07:53:37 +0200 |
|---|---|---|
| committer | Brad Fitzpatrick <bradfitz@golang.org> | 2015-07-28 06:23:38 +0000 |
| commit | d0729a6edebcaae1ab6554244fc6ef2ff36609bf (patch) | |
| tree | d3eb5241b9371df5bbe7c2c0aa9abc106e6d7e92 /src | |
| parent | deaf0333df5209985fc508c0e83cac789d780309 (diff) | |
| download | go-d0729a6edebcaae1ab6554244fc6ef2ff36609bf.tar.xz | |
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>
Diffstat (limited to 'src')
| -rw-r--r-- | src/encoding/json/stream_test.go | 29 |
1 files changed, 11 insertions, 18 deletions
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) } } |
