Add sync before unmount on snapshotter layer test

dmcgowan · dmcgowan · commit 1be571b14f96 · 2023-05-17T14:25:27.000-07:00
Signed-off-by: Derek McGowan &lt;derek@mcg.dev&gt;
diff --git a/snapshots/testsuite/testsuite.go b/snapshots/testsuite/testsuite.go
@@ -959,6 +959,7 @@ func check128LayersMount(name string) func(ctx context.Context, t *testing.T, sn
 				t.Fatalf("[layer %d] preparing doesn't equal to flat after apply: %+v", i, err)
 			}
 
+			sync()
 			testutil.Unmount(t, preparing)
 
 			parent = filepath.Join(work, fmt.Sprintf("committed-%d", i))
diff --git a/snapshots/testsuite/testsuite_unix.go b/snapshots/testsuite/testsuite_unix.go
@@ -18,11 +18,19 @@
 
 package testsuite
 
-import "syscall"
+import (
+	"syscall"
+
+	"golang.org/x/sys/unix"
+)
 
 func clearMask() func() {
 	oldumask := syscall.Umask(0)
 	return func() {
 		syscall.Umask(oldumask)
 	}
 }
+
+func sync() {
+	unix.Sync()
+}
diff --git a/snapshots/testsuite/testsuite_windows.go b/snapshots/testsuite/testsuite_windows.go
@@ -19,3 +19,5 @@ package testsuite
 func clearMask() func() {
 	return func() {}
 }
+
+func sync() {}

Original file line number	Diff line number	Diff line change
`@@ -959,6 +959,7 @@ func check128LayersMount(name string) func(ctx context.Context, t *testing.T, sn`
`959`	`959`	`t.Fatalf("[layer %d] preparing doesn't equal to flat after apply: %+v", i, err)`
`960`	`960`	`}`
`961`	`961`
	`962`	`+ sync()`
`962`	`963`	`testutil.Unmount(t, preparing)`
`963`	`964`
`964`	`965`	`parent = filepath.Join(work, fmt.Sprintf("committed-%d", i))`