Don't use subtests.

This commit is contained in:
Nuno Cruces 2023-04-12 00:22:04 +01:00
parent 2ab3f8ff53
commit d9bcc1c0bf
1 changed files with 43 additions and 47 deletions

View File

@ -186,7 +186,7 @@ func TestOffset(t *testing.T) {
const want = "hello, world\n" const want = "hello, world\n"
t.Run("Read", func(t *testing.T) { // Read the entire file.
got := make([]byte, len(want)) got := make([]byte, len(want))
n, err := file.Read(got) n, err := file.Read(got)
if err != nil { if err != nil {
@ -198,34 +198,31 @@ func TestOffset(t *testing.T) {
if string(got) != want { if string(got) != want {
t.Fatalf("Read: %q", got) t.Fatalf("Read: %q", got)
} }
})
t.Run("EOF", func(t *testing.T) { // Try to read one byte; confirm we're at the EOF.
var buf [1]byte var buf [1]byte
n, err := file.Read(buf[:]) n, err = file.Read(buf[:])
if err != io.EOF { if err != io.EOF {
t.Fatal("Read:", err) t.Fatal("Read:", err)
} }
if n != 0 { if n != 0 {
t.Fatal("Read:", n) t.Fatal("Read:", n)
} }
})
t.Run("Offset", func(t *testing.T) { // Use seek to get the offset at the EOF.
seeker := file.(io.Seeker) seeker := file.(io.Seeker)
n, err := seeker.Seek(0, io.SeekCurrent) off, err := seeker.Seek(0, io.SeekCurrent)
if err != nil { if err != nil {
t.Fatal("Read:", err) t.Fatal("Seek:", err)
} }
if n != int64(len(want)) { if off != int64(len(want)) {
t.Fatal("Read:", n) t.Fatal("Seek:", off)
} }
})
t.Run("ReadAt", func(t *testing.T) { // Use ReadAt to read the entire file, ignoring the offset.
at := file.(io.ReaderAt) at := file.(io.ReaderAt)
got := make([]byte, len(want)) got = make([]byte, len(want))
n, err := at.ReadAt(got, 0) n, err = at.ReadAt(got, 0)
if err != nil { if err != nil {
t.Fatal("ReadAt:", err) t.Fatal("ReadAt:", err)
} }
@ -235,5 +232,4 @@ func TestOffset(t *testing.T) {
if string(got) != want { if string(got) != want {
t.Fatalf("ReadAt: %q", got) t.Fatalf("ReadAt: %q", got)
} }
})
} }