make TestRename fail, make tests less verbose when they don't fail

TestRename fails now because MemMapFs.Rename does not change the filename in the parent's memDir
This commit is contained in:
Martin Bertschler 2015-11-22 14:26:00 +01:00
parent 3f6f746a75
commit 3fc0d6fb2b
1 changed files with 22 additions and 3 deletions

View File

@ -143,6 +143,22 @@ func TestRename(t *testing.T) {
if err != nil { if err != nil {
t.Fatalf("rename %q, %q failed: %v", to, from, err) t.Fatalf("rename %q, %q failed: %v", to, from, err)
} }
names, err := readDirNames(testDir, fs)
if err != nil {
t.Fatalf("readDirNames error: %v", err)
}
found := false
for _, e := range names {
if e == "renamefrom" {
t.Error("File is still called renamefrom")
}
if e == "renameto" {
found = true
}
}
if !found {
t.Error("File was not renamed to renameto")
}
defer fs.Remove(to) defer fs.Remove(to)
_, err = fs.Stat(to) _, err = fs.Stat(to)
if err != nil { if err != nil {
@ -512,9 +528,6 @@ func TestReaddirAll(t *testing.T) {
} }
func findNames(t *testing.T, root, sub []string, fs Fs) { func findNames(t *testing.T, root, sub []string, fs Fs) {
t.Logf("Names root: %v", root)
t.Logf("Names sub: %v", sub)
var foundRoot bool var foundRoot bool
for _, e := range root { for _, e := range root {
_, err := fs.Open(path.Join(testDir, e)) _, err := fs.Open(path.Join(testDir, e))
@ -526,6 +539,8 @@ func findNames(t *testing.T, root, sub []string, fs Fs) {
} }
} }
if !foundRoot { if !foundRoot {
t.Logf("Names root: %v", root)
t.Logf("Names sub: %v", sub)
t.Error("Didn't find subdirectory we") t.Error("Didn't find subdirectory we")
} }
@ -544,9 +559,13 @@ func findNames(t *testing.T, root, sub []string, fs Fs) {
} }
if !found1 { if !found1 {
t.Logf("Names root: %v", root)
t.Logf("Names sub: %v", sub)
t.Error("Didn't find testfile1") t.Error("Didn't find testfile1")
} }
if !found2 { if !found2 {
t.Logf("Names root: %v", root)
t.Logf("Names sub: %v", sub)
t.Error("Didn't find testfile2") t.Error("Didn't find testfile2")
} }
} }