]> Cypherpunks repositories - gostls13.git/commitdiff
io: simplify tests by removing redundant statements
authorOleksandr Redko <oleksandr.red+github@gmail.com>
Tue, 5 Nov 2024 11:09:37 +0000 (13:09 +0200)
committerGopher Robot <gobot@golang.org>
Tue, 5 Nov 2024 19:52:23 +0000 (19:52 +0000)
Change-Id: I4bcaa6b42571626c88e3374c328bbfe993476242
Reviewed-on: https://go-review.googlesource.com/c/go/+/625295
Auto-Submit: Ian Lance Taylor <iant@google.com>
Reviewed-by: Robert Griesemer <gri@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Commit-Queue: Ian Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>

src/io/io_test.go

index 9491ffae614c61603a1c914aa1ec15123a59b007..38bec8243e7baa5e76a6fb128ba204785b082cab 100644 (file)
@@ -505,7 +505,7 @@ func TestNopCloserWriterToForwarding(t *testing.T) {
 func TestOffsetWriter_Seek(t *testing.T) {
        tmpfilename := "TestOffsetWriter_Seek"
        tmpfile, err := os.CreateTemp(t.TempDir(), tmpfilename)
-       if err != nil || tmpfile == nil {
+       if err != nil {
                t.Fatalf("CreateTemp(%s) failed: %v", tmpfilename, err)
        }
        defer tmpfile.Close()
@@ -564,15 +564,12 @@ func TestOffsetWriter_Seek(t *testing.T) {
 func TestOffsetWriter_WriteAt(t *testing.T) {
        const content = "0123456789ABCDEF"
        contentSize := int64(len(content))
-       tmpdir, err := os.MkdirTemp(t.TempDir(), "TestOffsetWriter_WriteAt")
-       if err != nil {
-               t.Fatal(err)
-       }
+       tmpdir := t.TempDir()
 
        work := func(off, at int64) {
                position := fmt.Sprintf("off_%d_at_%d", off, at)
                tmpfile, err := os.CreateTemp(tmpdir, position)
-               if err != nil || tmpfile == nil {
+               if err != nil {
                        t.Fatalf("CreateTemp(%s) failed: %v", position, err)
                }
                defer tmpfile.Close()
@@ -642,7 +639,7 @@ func TestOffsetWriter_Write(t *testing.T) {
        makeOffsetWriter := func(name string) (*OffsetWriter, *os.File) {
                tmpfilename := "TestOffsetWriter_Write_" + name
                tmpfile, err := os.CreateTemp(tmpdir, tmpfilename)
-               if err != nil || tmpfile == nil {
+               if err != nil {
                        t.Fatalf("CreateTemp(%s) failed: %v", tmpfilename, err)
                }
                return NewOffsetWriter(tmpfile, 0), tmpfile