X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=cmd%2Fsafcm-remote%2Fsync%2Ffiles_test.go;h=2e3ed3eb2bca67a994232a3a2d9dc2502d77d982;hb=006a69a50239a27ba7929c513897ef19259e06af;hp=635501195860b35b79333213d96f5aa6510ecf77;hpb=d12e83d4d64b295b8da5c0506d44e94a330ab272;p=safcm%2Fsafcm.git

diff --git a/cmd/safcm-remote/sync/files_test.go b/cmd/safcm-remote/sync/files_test.go
index 6355011..2e3ed3e 100644
--- a/cmd/safcm-remote/sync/files_test.go
+++ b/cmd/safcm-remote/sync/files_test.go
@@ -58,15 +58,15 @@ func TestSyncFiles(t *testing.T) {
 	tmpTestFilePath := "/tmp/safcm-sync-files-test-file"
 
 	tests := []struct {
-		name     string
-		skip     bool
-		req      safcm.MsgSyncReq
-		prepare  func()
+		name        string
+		skip        bool
+		req         safcm.MsgSyncReq
+		prepare     func()
 		expTriggers []string
-		expFiles []ft.File
-		expResp  safcm.MsgSyncResp
-		expDbg   []string
-		expErr   error
+		expFiles    []ft.File
+		expResp     safcm.MsgSyncResp
+		expDbg      []string
+		expErr      error
 	}{
 
 		// NOTE: Also update MsgSyncResp in safcm test cases when
@@ -415,10 +415,7 @@ func TestSyncFiles(t *testing.T) {
 				},
 			},
 			func() {
-				err = os.Chmod(".", 0750)
-				if err != nil {
-					panic(err)
-				}
+				ft.CreateDirectoryExists(".", 0750)
 				ft.CreateDirectory("dir", 0755)
 				ft.CreateFile("dir/file", "content\n", 0644)
 			},
@@ -834,7 +831,7 @@ func TestSyncFiles(t *testing.T) {
 
 			// Create separate test directory for each test case
 			path := filepath.Join(cwd, "testdata", "files-"+tc.name)
-			err = os.Mkdir(path, 0700)
+			err := os.Mkdir(path, 0700)
 			if err != nil {
 				t.Fatal(err)
 			}
@@ -850,9 +847,12 @@ func TestSyncFiles(t *testing.T) {
 			s, res := prepareSync(tc.req, &testRunner{
 				t: t,
 			})
-			s.setDefaults()
+			err = s.setDefaults()
+			if err != nil {
+				t.Fatal(err)
+			}
 
-			err := s.syncFiles()
+			err = s.syncFiles()
 			testutil.AssertErrorEqual(t, "err", err, tc.expErr)
 			dbg := res.Wait()
 			// Remove random file names from result
@@ -2374,7 +2374,7 @@ file
 		t.Run(tc.name, func(t *testing.T) {
 			// Create separate test directory for each test case
 			path := filepath.Join(cwd, "testdata", "file-"+tc.name)
-			err = os.Mkdir(path, 0700)
+			err := os.Mkdir(path, 0700)
 			if err != nil {
 				t.Fatal(err)
 			}
@@ -2390,13 +2390,16 @@ file
 			s, res := prepareSync(tc.req, &testRunner{
 				t: t,
 			})
-			s.setDefaults()
+			err = s.setDefaults()
+			if err != nil {
+				t.Fatal(err)
+			}
 
 			// Deterministic temporary symlink names
 			rand.Seed(0)
 
 			var changed bool
-			err := s.syncFile(tc.file, &changed)
+			err = s.syncFile(tc.file, &changed)
 			testutil.AssertErrorEqual(t, "err", err, tc.expErr)
 			dbg := res.Wait()
 			// Remove random file names from result