Misc ioutil deprecation adjustments

To make the tests pass.

* Replace io => os.ReadFile in magefile.go
* Adjust failing image test vs fs.DirEntry
* Adjust poller test

See #10732
This commit is contained in:
Bjørn Erik Pedersen 2023-03-01 09:34:13 +01:00
parent d453c12742
commit a669467d98
3 changed files with 15 additions and 3 deletions

View file

@ -7,7 +7,6 @@ import (
"bytes" "bytes"
"errors" "errors"
"fmt" "fmt"
"io"
"os" "os"
"path" "path"
"path/filepath" "path/filepath"
@ -298,7 +297,7 @@ func TestCoverHTML() error {
if err := sh.Run(goexe, "test", "-coverprofile="+cover, "-covermode=count", pkg); err != nil { if err := sh.Run(goexe, "test", "-coverprofile="+cover, "-covermode=count", pkg); err != nil {
return err return err
} }
b, err := io.ReadFile(cover) b, err := os.ReadFile(cover)
if err != nil { if err != nil {
if os.IsNotExist(err) { if os.IsNotExist(err) {
continue continue

View file

@ -18,6 +18,7 @@ import (
"fmt" "fmt"
"image" "image"
"image/gif" "image/gif"
"io/fs"
"math/big" "math/big"
"math/rand" "math/rand"
"os" "os"
@ -56,6 +57,14 @@ var eq = qt.CmpEquals(
cmp.Comparer(func(p1, p2 os.FileInfo) bool { cmp.Comparer(func(p1, p2 os.FileInfo) bool {
return p1.Name() == p2.Name() && p1.Size() == p2.Size() && p1.IsDir() == p2.IsDir() return p1.Name() == p2.Name() && p1.Size() == p2.Size() && p1.IsDir() == p2.IsDir()
}), }),
cmp.Comparer(func(d1, d2 fs.DirEntry) bool {
p1, err1 := d1.Info()
p2, err2 := d2.Info()
if err1 != nil || err2 != nil {
return false
}
return p1.Name() == p2.Name() && p1.Size() == p2.Size() && p1.IsDir() == p2.IsDir()
}),
cmp.Comparer(func(p1, p2 *genericResource) bool { return p1 == p2 }), cmp.Comparer(func(p1, p2 *genericResource) bool { return p1 == p2 }),
cmp.Comparer(func(m1, m2 media.Type) bool { cmp.Comparer(func(m1, m2 media.Type) bool {
return m1.Type() == m2.Type() return m1.Type() == m2.Type()

View file

@ -34,10 +34,14 @@ func TestPollerAddRemove(t *testing.T) {
c.Assert(w.Add("foo"), qt.Not(qt.IsNil)) c.Assert(w.Add("foo"), qt.Not(qt.IsNil))
c.Assert(w.Remove("foo"), qt.Not(qt.IsNil)) c.Assert(w.Remove("foo"), qt.Not(qt.IsNil))
f, err := os.CreateTemp(t.TempDir(), "asdf") f, err := os.CreateTemp("", "asdf")
if err != nil { if err != nil {
t.Fatal(err) t.Fatal(err)
} }
c.Cleanup(func() {
c.Assert(w.Close(), qt.IsNil)
os.Remove(f.Name())
})
c.Assert(w.Add(f.Name()), qt.IsNil) c.Assert(w.Add(f.Name()), qt.IsNil)
c.Assert(w.Remove(f.Name()), qt.IsNil) c.Assert(w.Remove(f.Name()), qt.IsNil)