rename tests

Author: sean-breen

internal/config/config_test.go

@@ -170,47 +170,47 @@ func TestResolveAllowedDirectories(t *testing.T) {
 			expected:       []string{"/etc/nginx-agent"},
 		},
 		{
-			name:           "Absolute path",
+			name:           "Test 1: Absolute path",
 			configuredDirs: []string{"/etc/agent/"},
 			expected:       []string{"/etc/nginx-agent", "/etc/agent"},
 		},
 		{
-			name:           "Absolute paths",
+			name:           "Test 2: Absolute paths",
 			configuredDirs: []string{"/etc/nginx/"},
 			expected:       []string{"/etc/nginx-agent", "/etc/nginx"},
 		},
 		{
-			name:           "Absolute path with multiple slashes",
+			name:           "Test 3: Absolute path with multiple slashes",
 			configuredDirs: []string{"/etc///////////nginx-agent/"},
 			expected:       []string{"/etc/nginx-agent"},
 		},
 		{
-			name:           "Absolute path with directory traversal",
+			name:           "Test 4: Absolute path with directory traversal",
 			configuredDirs: []string{"/etc/nginx/../nginx-agent"},
 			expected:       []string{"/etc/nginx-agent"},
 		},
 		{
-			name:           "Absolute path with repeat directory traversal",
+			name:           "Test 5: Absolute path with repeat directory traversal",
 			configuredDirs: []string{"/etc/nginx-agent/../../../../../nginx-agent"},
 			expected:       []string{"/etc/nginx-agent"},
 		},
 		{
-			name:           "Absolute path with control characters",
+			name:           "Test 6: Absolute path with control characters",
 			configuredDirs: []string{"/etc/nginx-agent/\\x08../tmp/"},
 			expected:       []string{"/etc/nginx-agent"},
 		},
 		{
-			name:           "Absolute path with invisible characters",
+			name:           "Test 7: Absolute path with invisible characters",
 			configuredDirs: []string{"/etc/nginx-agent/ㅤㅤㅤ/tmp/"},
 			expected:       []string{"/etc/nginx-agent"},
 		},
 		{
-			name:           "Absolute path with escaped invisible characters",
+			name:           "Test 8: Absolute path with escaped invisible characters",
 			configuredDirs: []string{"/etc/nginx-agent/\\\\ㅤ/tmp/"},
 			expected:       []string{"/etc/nginx-agent"},
 		},
 		{
-			name: "Mixed paths",
+			name: "Test 9: Mixed paths",
 			configuredDirs: []string{
 				"nginx-agent",
 				"",
@@ -223,7 +223,7 @@ func TestResolveAllowedDirectories(t *testing.T) {
 			expected: []string{"/etc/nginx-agent", "/etc/nginx"},
 		},
 		{
-			name:           "Relative path",
+			name:           "Test 10: Relative path",
 			configuredDirs: []string{"nginx-agent"},
 			expected:       []string{"/etc/nginx-agent"},
 		},

internal/config/types_test.go

@@ -22,47 +22,47 @@ func TestTypes_isAllowedDir(t *testing.T) {
 		allowed     bool
 	}{
 		{
-			name:    "File is in allowed directory",
+			name:    "Test 1:File is in allowed directory",
 			allowed: true,
 			allowedDirs: []string{
 				"/etc/nginx",
 			},
 			filePath: "/etc/nginx/nginx.conf",
 		},
 		{
-			name:    "File is in allowed directory with hyphen",
+			name:    "Test 2: File is in allowed directory with hyphen",
 			allowed: true,
 			allowedDirs: []string{
 				"/etc/nginx-agent",
 			},
 			filePath: "/etc/nginx-agent/nginx.conf",
 		},
 		{
-			name:    "File exists and is in a subdirectory of allowed directory",
+			name:    "Test 3: File exists and is in a subdirectory of allowed directory",
 			allowed: true,
 			allowedDirs: []string{
 				"/etc/nginx",
 			},
 			filePath: "/etc/nginx/conf.d/nginx.conf",
 		},
 		{
-			name:    "File exists and is outside allowed directory",
+			name:    "Test 4: File exists and is outside allowed directory",
 			allowed: false,
 			allowedDirs: []string{
 				"/etc/nginx",
 			},
 			filePath: "/etc/test/nginx.conf",
 		},
 		{
-			name:    "File does not exist but is in allowed directory",
+			name:    "Test 5: File does not exist but is in allowed directory",
 			allowed: true,
 			allowedDirs: []string{
 				"/etc/nginx",
 			},
 			filePath: "/etc/nginx/idontexist.conf",
 		},
 		{
-			name:    "File does not exist and is outside allowed directory",
+			name:    "Test 6: Test File does not exist and is outside allowed directory",
 			allowed: false,
 			allowedDirs: []string{
 				"/etc/nginx",
@@ -79,7 +79,7 @@ func TestTypes_isAllowedDir(t *testing.T) {
 		})
 	}
 
-	t.Run("Symlink in allowed directory", func(t *testing.T) {
+	t.Run("Test 7: Symlink in allowed directory", func(t *testing.T) {
 		allowedDirs := []string{"/etc/nginx"}
 		filePath := "file.conf"
 		symlinkPath := "file_link"