TestFileHoleCopy: improve error msg

Relates-to: #812
rfjakob · Dec 8, 2023 · 930dcfb · 930dcfb
1 parent f77477c
commit 930dcfb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/internal/fusefrontend_reverse/node_helpers.go b/internal/fusefrontend_reverse/node_helpers.go
@@ -115,6 +115,7 @@ func (n *Node) isRoot() bool {
 	return &rn.Node == n
 }
 
+// Handle LOOKUP gocryptfs.longname.*.name
 func (n *Node) lookupLongnameName(ctx context.Context, nameFile string, out *fuse.EntryOut) (ch *fs.Inode, errno syscall.Errno) {
 	d, errno := n.prepareAtSyscall("")
 	if errno != 0 {

diff --git a/tests/plaintextnames/file_holes_test.go b/tests/plaintextnames/file_holes_test.go
@@ -92,7 +92,7 @@ func doTestFileHoleCopy(t *testing.T, name string, writeOffsets []int64) {
 		// (looking at "filefrag -v", it seems like ext4 needs 4kB extra once
 		//  you have >=4 extents)
 		if st.Blocks != st0.Blocks && st.Blocks != st0.Blocks-8 && st.Blocks != st0.Blocks+8 {
-			t.Errorf("size changed: st0.Blocks=%d st%d.Blocks=%d", st0.Blocks, i, st.Blocks)
+			t.Errorf("error: size changed: st0.Blocks=%d st%d.Blocks=%d", st0.Blocks, i, st.Blocks)
 		}
 	}
 
@@ -123,7 +123,7 @@ func doTestFileHoleCopy(t *testing.T, name string, writeOffsets []int64) {
 	}
 }
 
-// TestFileHoleCopy creates a sparse times, copies it a few times, and check if
+// TestFileHoleCopy creates a sparse file, copies it a few times, and checks if
 // the copies are the same (including the location of holes and data sections).
 //
 // The test runs with -plaintextnames because that makes it easier to manipulate