test: fix tests after changes in MacOS packaging (#1233)

Adjusts tests after changes in - SumoLogic/sumologic-otel-collector-packaging#28 - SumoLogic/sumologic-otel-collector-packaging#30
SumoLogic · Aug 29, 2023 · af84a3b · af84a3b
1 parent 35ff990
commit af84a3b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 7 deletions.
diff --git a/pkg/scripts_test/check.go b/pkg/scripts_test/check.go
@@ -55,14 +55,8 @@ func checkRun(c check) {
 	require.Equal(c.test, c.expectedInstallCode, c.code, "unexpected installation script error code")
 }
 
-func checkConfigDirectoryOwnershipAndPermissions(c check) {
-	PathHasOwner(c.test, etcPath, rootUser, rootGroup)
-	PathHasPermissions(c.test, etcPath, etcPathPermissions)
-}
-
 func checkConfigCreated(c check) {
 	require.FileExists(c.test, configPath, "configuration has not been created properly")
-	checkConfigDirectoryOwnershipAndPermissions(c)
 }
 
 func checkConfigNotCreated(c check) {

diff --git a/pkg/scripts_test/check_darwin.go b/pkg/scripts_test/check_darwin.go
@@ -12,6 +12,9 @@ import (
 
 func checkConfigFilesOwnershipAndPermissions(ownerName string, ownerGroup string) func(c check) {
 	return func(c check) {
+		PathHasPermissions(c.test, etcPath, etcPathPermissions)
+		PathHasOwner(c.test, etcPath, ownerName, ownerGroup)
+
 		etcPathGlob := filepath.Join(etcPath, "*")
 		etcPathNestedGlob := filepath.Join(etcPath, "*", "*")
 

diff --git a/pkg/scripts_test/consts_darwin.go b/pkg/scripts_test/consts_darwin.go
@@ -13,7 +13,7 @@ const (
 	configPathDirPermissions        uint32 = 0770
 	configPathFilePermissions       uint32 = 0440
 	confDPathFilePermissions        uint32 = 0644
-	etcPathPermissions              uint32 = 0755
+	etcPathPermissions              uint32 = 0751
 
 	rootGroup   string = "wheel"
 	rootUser    string = "root"