From 94e8fc12ea5756a130e7ac9ed67ddd519b5f3a22 Mon Sep 17 00:00:00 2001 From: Jakob Unterwurzacher Date: Sat, 15 Aug 2020 17:31:25 +0200 Subject: v2api/reverse: finish -exclude Tests pass now. --- tests/reverse/correctness_test.go | 2 +- tests/reverse/exclude_test.go | 2 ++ tests/reverse/inomap_test.go | 4 ++-- 3 files changed, 5 insertions(+), 3 deletions(-) (limited to 'tests') diff --git a/tests/reverse/correctness_test.go b/tests/reverse/correctness_test.go index 47b8b73..6b7ed5f 100644 --- a/tests/reverse/correctness_test.go +++ b/tests/reverse/correctness_test.go @@ -87,7 +87,7 @@ func TestSymlinkDentrySize(t *testing.T) { fi, err := os.Lstat(mnt + "/" + symlinkResponse.Result) if err != nil { - t.Errorf("Lstat: %v", err) + t.Fatalf("Lstat: %v", err) } target, err := os.Readlink(mnt + "/" + symlinkResponse.Result) diff --git a/tests/reverse/exclude_test.go b/tests/reverse/exclude_test.go index b5d0f5b..c493d95 100644 --- a/tests/reverse/exclude_test.go +++ b/tests/reverse/exclude_test.go @@ -118,6 +118,7 @@ func testExclude(t *testing.T, flag string) { cExclude := encryptExcludeTestPaths(t, sock, pExclude) // Check that "excluded" paths are not there and "ok" paths are there for _, v := range cExclude { + t.Logf("File %q should be invisible", v) if test_helpers.VerifyExistence(t, mnt+"/"+v) { t.Errorf("File %q is visible, but should be excluded", v) } @@ -126,6 +127,7 @@ func testExclude(t *testing.T, flag string) { } } for _, v := range cOk { + t.Logf("File %q should be visible", v) if !test_helpers.VerifyExistence(t, mnt+"/"+v) { t.Errorf("File %q is hidden, but should be visible", v) } diff --git a/tests/reverse/inomap_test.go b/tests/reverse/inomap_test.go index 5f7fb45..e6fc525 100644 --- a/tests/reverse/inomap_test.go +++ b/tests/reverse/inomap_test.go @@ -45,7 +45,7 @@ func findIno(dir string, ino uint64) string { // ├── gocryptfs.longname.e31v1ax4h_F0l4jhlN8kCjaWWMq8rO9VVBZ15IYsV50 <---- child // └── gocryptfs.longname.e31v1ax4h_F0l4jhlN8kCjaWWMq8rO9VVBZ15IYsV50.name <---- name // -// And verifies that the inode numbers match what we expect. +// It verifies that the inode numbers match what we expect. func TestVirtualFileIno(t *testing.T) { if plaintextnames { t.Skip("plaintextnames mode does not have virtual files") @@ -111,7 +111,7 @@ func TestVirtualFileIno(t *testing.T) { var st2 syscall.Stat_t err = syscall.Lstat(encryptedParent+"/"+entry, &st2) if err != nil { - t.Logf("stat %q: %v", entry, err) + t.Errorf("stat %q: %v", entry, err) continue } if entry == "gocryptfs.diriv" { -- cgit v1.2.3