Made tests pickier about ReadDir.

geesefs-0-30-9
Aaron Jacobs 2015-04-01 09:05:26 +11:00
parent 304ce5a08e
commit 40fdee8c78
3 changed files with 17 additions and 16 deletions

View File

@ -22,6 +22,7 @@ import (
"syscall" "syscall"
"testing" "testing"
"github.com/jacobsa/fuse/fusetesting"
"github.com/jacobsa/fuse/samples" "github.com/jacobsa/fuse/samples"
"github.com/jacobsa/fuse/samples/hellofs" "github.com/jacobsa/fuse/samples/hellofs"
. "github.com/jacobsa/oglematchers" . "github.com/jacobsa/oglematchers"
@ -54,7 +55,7 @@ func (t *HelloFSTest) SetUp(ti *TestInfo) {
//////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////
func (t *HelloFSTest) ReadDir_Root() { func (t *HelloFSTest) ReadDir_Root() {
entries, err := ioutil.ReadDir(t.Dir) entries, err := fusetesting.ReadDirPicky(t.Dir)
AssertEq(nil, err) AssertEq(nil, err)
AssertEq(2, len(entries)) AssertEq(2, len(entries))
@ -78,7 +79,7 @@ func (t *HelloFSTest) ReadDir_Root() {
} }
func (t *HelloFSTest) ReadDir_Dir() { func (t *HelloFSTest) ReadDir_Dir() {
entries, err := ioutil.ReadDir(path.Join(t.Dir, "dir")) entries, err := fusetesting.ReadDirPicky(path.Join(t.Dir, "dir"))
AssertEq(nil, err) AssertEq(nil, err)
AssertEq(1, len(entries)) AssertEq(1, len(entries))
@ -94,7 +95,7 @@ func (t *HelloFSTest) ReadDir_Dir() {
} }
func (t *HelloFSTest) ReadDir_NonExistent() { func (t *HelloFSTest) ReadDir_NonExistent() {
_, err := ioutil.ReadDir(path.Join(t.Dir, "foobar")) _, err := fusetesting.ReadDirPicky(path.Join(t.Dir, "foobar"))
AssertNe(nil, err) AssertNe(nil, err)
ExpectThat(err, Error(HasSubstr("no such file"))) ExpectThat(err, Error(HasSubstr("no such file")))

View File

@ -98,7 +98,7 @@ func (t *MemFSTest) SetUp(ti *TestInfo) {
//////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////
func (t *MemFSTest) ContentsOfEmptyFileSystem() { func (t *MemFSTest) ContentsOfEmptyFileSystem() {
entries, err := ioutil.ReadDir(t.Dir) entries, err := fusetesting.ReadDirPicky(t.Dir)
AssertEq(nil, err) AssertEq(nil, err)
ExpectThat(entries, ElementsAre()) ExpectThat(entries, ElementsAre())
@ -148,13 +148,13 @@ func (t *MemFSTest) Mkdir_OneLevel() {
ExpectEq(0, fi.ModTime().Sub(createTime)) ExpectEq(0, fi.ModTime().Sub(createTime))
// Read the directory. // Read the directory.
entries, err = ioutil.ReadDir(dirName) entries, err = fusetesting.ReadDirPicky(dirName)
AssertEq(nil, err) AssertEq(nil, err)
ExpectThat(entries, ElementsAre()) ExpectThat(entries, ElementsAre())
// Read the root. // Read the root.
entries, err = ioutil.ReadDir(t.Dir) entries, err = fusetesting.ReadDirPicky(t.Dir)
AssertEq(nil, err) AssertEq(nil, err)
AssertEq(1, len(entries)) AssertEq(1, len(entries))
@ -209,13 +209,13 @@ func (t *MemFSTest) Mkdir_TwoLevels() {
ExpectEq(0, fi.ModTime().Sub(createTime)) ExpectEq(0, fi.ModTime().Sub(createTime))
// Read the directory. // Read the directory.
entries, err = ioutil.ReadDir(path.Join(t.Dir, "parent/dir")) entries, err = fusetesting.ReadDirPicky(path.Join(t.Dir, "parent/dir"))
AssertEq(nil, err) AssertEq(nil, err)
ExpectThat(entries, ElementsAre()) ExpectThat(entries, ElementsAre())
// Read the parent. // Read the parent.
entries, err = ioutil.ReadDir(path.Join(t.Dir, "parent")) entries, err = fusetesting.ReadDirPicky(path.Join(t.Dir, "parent"))
AssertEq(nil, err) AssertEq(nil, err)
AssertEq(1, len(entries)) AssertEq(1, len(entries))
@ -496,7 +496,7 @@ func (t *MemFSTest) UnlinkFile_Exists() {
ExpectThat(err, Error(HasSubstr("no such file"))) ExpectThat(err, Error(HasSubstr("no such file")))
// Nothing should be in the directory. // Nothing should be in the directory.
entries, err := ioutil.ReadDir(t.Dir) entries, err := fusetesting.ReadDirPicky(t.Dir)
AssertEq(nil, err) AssertEq(nil, err)
ExpectThat(entries, ElementsAre()) ExpectThat(entries, ElementsAre())
} }
@ -526,7 +526,7 @@ func (t *MemFSTest) UnlinkFile_StillOpen() {
AssertEq(nil, err) AssertEq(nil, err)
// The directory should no longer contain it. // The directory should no longer contain it.
entries, err := ioutil.ReadDir(t.Dir) entries, err := fusetesting.ReadDirPicky(t.Dir)
AssertEq(nil, err) AssertEq(nil, err)
ExpectThat(entries, ElementsAre()) ExpectThat(entries, ElementsAre())
@ -586,7 +586,7 @@ func (t *MemFSTest) Rmdir_Empty() {
t.Clock.AdvanceTime(time.Second) t.Clock.AdvanceTime(time.Second)
// There should be nothing left in the parent. // There should be nothing left in the parent.
entries, err = ioutil.ReadDir(path.Join(t.Dir, "foo")) entries, err = fusetesting.ReadDirPicky(path.Join(t.Dir, "foo"))
AssertEq(nil, err) AssertEq(nil, err)
ExpectThat(entries, ElementsAre()) ExpectThat(entries, ElementsAre())
@ -601,7 +601,7 @@ func (t *MemFSTest) Rmdir_Empty() {
AssertEq(nil, err) AssertEq(nil, err)
// Now the root directory should be empty, too. // Now the root directory should be empty, too.
entries, err = ioutil.ReadDir(t.Dir) entries, err = fusetesting.ReadDirPicky(t.Dir)
AssertEq(nil, err) AssertEq(nil, err)
ExpectThat(entries, ElementsAre()) ExpectThat(entries, ElementsAre())
@ -661,12 +661,12 @@ func (t *MemFSTest) Rmdir_OpenedForReading() {
// Attempt to read from the directory. This shouldn't see any junk from the // Attempt to read from the directory. This shouldn't see any junk from the
// new directory. It should either succeed with an empty result or should // new directory. It should either succeed with an empty result or should
// return ENOENT. // return ENOENT.
entries, err := f.Readdir(0) names, err := f.Readdirnames(0)
if err != nil { if err != nil {
ExpectThat(err, Error(HasSubstr("no such file"))) ExpectThat(err, Error(HasSubstr("no such file")))
} else { } else {
ExpectThat(entries, ElementsAre()) ExpectThat(names, ElementsAre())
} }
} }

View File

@ -410,11 +410,11 @@ func (t *PosixTest) RmdirWhileOpenedForReading() {
// Attempt to read from the directory. This shouldn't see any junk from the // Attempt to read from the directory. This shouldn't see any junk from the
// new directory. It should either succeed with an empty result or should // new directory. It should either succeed with an empty result or should
// return ENOENT. // return ENOENT.
entries, err := f.Readdir(0) names, err := f.Readdirnames(0)
if err != nil { if err != nil {
ExpectThat(err, Error(HasSubstr("no such file"))) ExpectThat(err, Error(HasSubstr("no such file")))
} else { } else {
ExpectThat(entries, ElementsAre()) ExpectThat(names, ElementsAre())
} }
} }