diff options
author | Ian Davis <nospam@iandavis.com> | 2018-09-10 09:27:47 +0100 |
---|---|---|
committer | Daniel Martí <mvdan@mvdan.cc> | 2018-09-16 23:31:17 +0000 |
commit | 930ce09ca3a1ab5f886c2327fc143b9ae075807b (patch) | |
tree | e34b4dc4c0964c2783911410889a70e9b9a22a01 /src/image | |
parent | adcecbe05ef812bc8ff477dec47720a2cfc273e3 (diff) | |
download | go-930ce09ca3a1ab5f886c2327fc143b9ae075807b.tar.gz go-930ce09ca3a1ab5f886c2327fc143b9ae075807b.zip |
image/png: minor cleanup of a few tests
Removes a redundant err check and replaces some returns in a testing
loop with continue to prevent skipping unrelated test cases when
a failure is encountered.
Change-Id: Ic1a560751b95bb0ef8dfa957e057e0fa0c2b281d
Reviewed-on: https://go-review.googlesource.com/134236
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>
Diffstat (limited to 'src/image')
-rw-r--r-- | src/image/png/reader_test.go | 4 | ||||
-rw-r--r-- | src/image/png/writer_test.go | 4 |
2 files changed, 2 insertions, 6 deletions
diff --git a/src/image/png/reader_test.go b/src/image/png/reader_test.go index 66bcfcb437..33dcd3debc 100644 --- a/src/image/png/reader_test.go +++ b/src/image/png/reader_test.go @@ -364,10 +364,6 @@ func TestReader(t *testing.T) { } defer sf.Close() sb := bufio.NewScanner(sf) - if err != nil { - t.Error(fn, err) - continue - } // Compare the two, in SNG format, line by line. for { diff --git a/src/image/png/writer_test.go b/src/image/png/writer_test.go index 1107ea0e7f..6c5e942310 100644 --- a/src/image/png/writer_test.go +++ b/src/image/png/writer_test.go @@ -61,12 +61,12 @@ func TestWriter(t *testing.T) { m1, err := readPNG(qfn) if err != nil { t.Error(fn, err) - return + continue } m2, err := encodeDecode(m1) if err != nil { t.Error(fn, err) - return + continue } // Compare the two. err = diff(m0, m2) |