X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=cmd%2Fsafcm%2Fconfig%2Ffiles_test.go;h=46da59055ee602646257b77ebea609ab84e07081;hb=583a2695a3ddc9c98e0a03d9f1bad8df30afe887;hp=9f226cd7b502e17e28f8db54527e693241753923;hpb=304a68c565d58ae52dd39cf6e2d6ef52cada7d19;p=safcm%2Fsafcm.git diff --git a/cmd/safcm/config/files_test.go b/cmd/safcm/config/files_test.go index 9f226cd..46da590 100644 --- a/cmd/safcm/config/files_test.go +++ b/cmd/safcm/config/files_test.go @@ -19,6 +19,7 @@ import ( "fmt" "io/fs" "os" + "runtime" "syscall" "testing" @@ -45,6 +46,9 @@ func TestLoadFiles(t *testing.T) { t.Fatal(err) } + // Regular users cannot create sticky files + skipInvalidSticky := runtime.GOOS == "freebsd" + chmod("files-invalid-perm-dir/files", 0500) defer chmod("files-invalid-perm-dir/files", 0700) chmod("files-invalid-perm-dir/files/etc/", 0755) @@ -58,9 +62,11 @@ func TestLoadFiles(t *testing.T) { chmod("files-invalid-perm-file-executable/files", 0755) chmod("files-invalid-perm-file-executable/files/etc", 0755) chmod("files-invalid-perm-file-executable/files/etc/rc.local", 0750) - chmod("files-invalid-perm-file-sticky/files", 0755) - chmod("files-invalid-perm-file-sticky/files/etc", 0755) - chmod("files-invalid-perm-file-sticky/files/etc/resolv.conf", 01644) + if !skipInvalidSticky { + chmod("files-invalid-perm-file-sticky/files", 0755) + chmod("files-invalid-perm-file-sticky/files/etc", 0755) + chmod("files-invalid-perm-file-sticky/files/etc/resolv.conf", 01644) + } err = syscall.Mkfifo("files-invalid-type/files/invalid", 0644) if err != nil { @@ -79,18 +85,21 @@ via "safcm fixperms". tests := []struct { group string + skip bool exp map[string]*safcm.File expErr error }{ { "empty", + false, nil, nil, }, { "group", + false, map[string]*safcm.File{ "/": { Path: "/", @@ -169,31 +178,37 @@ host3.example.net { "files-invalid-type", + false, nil, fmt.Errorf("files-invalid-type: \"files-invalid-type/files/invalid\": file type not supported"), }, { "files-invalid-perm-dir", + false, nil, fmt.Errorf("files-invalid-perm-dir: \"files-invalid-perm-dir/files\": invalid permissions 0500" + errMsg), }, { "files-invalid-perm-dir-setgid", + false, nil, fmt.Errorf("files-invalid-perm-dir-setgid: \"files-invalid-perm-dir-setgid/files/etc\": invalid permissions 02755" + errMsg), }, { "files-invalid-perm-file", + false, nil, fmt.Errorf("files-invalid-perm-file: \"files-invalid-perm-file/files/etc/resolv.conf\": invalid permissions 0600" + errMsg), }, { "files-invalid-perm-file-executable", + false, nil, fmt.Errorf("files-invalid-perm-file-executable: \"files-invalid-perm-file-executable/files/etc/rc.local\": invalid permissions 0750" + errMsg), }, { "files-invalid-perm-file-sticky", + skipInvalidSticky, nil, fmt.Errorf("files-invalid-perm-file-sticky: \"files-invalid-perm-file-sticky/files/etc/resolv.conf\": invalid permissions 01644" + errMsg), }, @@ -201,6 +216,10 @@ host3.example.net for _, tc := range tests { t.Run(tc.group, func(t *testing.T) { + if tc.skip { + t.SkipNow() + } + res, err := LoadFiles(tc.group) testutil.AssertEqual(t, "res", res, tc.exp) testutil.AssertErrorEqual(t, "err", err, tc.expErr)