diff options
| author | Brad Fitzpatrick <bradfitz@golang.org> | 2016-05-03 19:54:49 +0000 |
|---|---|---|
| committer | Brad Fitzpatrick <bradfitz@golang.org> | 2016-05-03 21:01:13 +0000 |
| commit | 01182425f847e4c98a53c60d0994175e21fd06dd (patch) | |
| tree | f2fc61c4c570ef8e310b6282db2f2e1630532649 /src/strings/reader_test.go | |
| parent | 15f7a66f3686d24fd5ad233c6c6b1ff22daa42ae (diff) | |
| download | go-01182425f847e4c98a53c60d0994175e21fd06dd.tar.xz | |
strings, bytes: fix Reader 0 byte read at EOF
0 byte reads at EOF weren't returning EOF.
Change-Id: I19b5fd5a72e83d49566a230ce4067be03f00d14b
Reviewed-on: https://go-review.googlesource.com/22740
Reviewed-by: Bryan Mills <bcmills@google.com>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/strings/reader_test.go')
| -rw-r--r-- | src/strings/reader_test.go | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/strings/reader_test.go b/src/strings/reader_test.go index 6e9d904b9d..bf40eb1a31 100644 --- a/src/strings/reader_test.go +++ b/src/strings/reader_test.go @@ -22,14 +22,15 @@ func TestReader(t *testing.T) { n int want string wantpos int64 + readerr error seekerr string }{ {seek: io.SeekStart, off: 0, n: 20, want: "0123456789"}, {seek: io.SeekStart, off: 1, n: 1, want: "1"}, {seek: io.SeekCurrent, off: 1, wantpos: 3, n: 2, want: "34"}, {seek: io.SeekStart, off: -1, seekerr: "strings.Reader.Seek: negative position"}, - {seek: io.SeekStart, off: 1 << 33, wantpos: 1 << 33}, - {seek: io.SeekCurrent, off: 1, wantpos: 1<<33 + 1}, + {seek: io.SeekStart, off: 1 << 33, wantpos: 1 << 33, readerr: io.EOF}, + {seek: io.SeekCurrent, off: 1, wantpos: 1<<33 + 1, readerr: io.EOF}, {seek: io.SeekStart, n: 5, want: "01234"}, {seek: io.SeekCurrent, n: 5, want: "56789"}, {seek: io.SeekEnd, off: -1, n: 1, wantpos: 9, want: "9"}, @@ -50,8 +51,8 @@ func TestReader(t *testing.T) { } buf := make([]byte, tt.n) n, err := r.Read(buf) - if err != nil { - t.Errorf("%d. read = %v", i, err) + if err != tt.readerr { + t.Errorf("%d. read = %v; want %v", i, err, tt.readerr) continue } got := string(buf[:n]) |