fix setting of level

Author: aphralG

internal/config/config.go

@@ -794,11 +794,11 @@ func resolveLog() *Log {
 
 	if !slices.Contains(validLevels, logLevel) {
 		slog.Warn("Invalid log level set, defaulting to 'info'", "log_level", logLevel)
-		logLevel = "info"
+		viperInstance.Set(LogLevelKey, "info")
 	}
 
 	return &Log{
-		Level: logLevel,
+		Level: viperInstance.GetString(LogLevelKey),
 		Path:  viperInstance.GetString(LogPathKey),
 	}
 }
@@ -1176,16 +1176,14 @@ func isHealthExtensionSet() bool {
 }
 
 func resolveCollectorLog() *Log {
-	log := &Log{
-		Level: viperInstance.GetString(CollectorLogLevelKey),
-		Path:  viperInstance.GetString(CollectorLogPathKey),
+	if !viperInstance.IsSet(CollectorLogLevelKey) {
+		viperInstance.Set(CollectorLogLevelKey, strings.ToUpper(viperInstance.GetString(LogLevelKey)))
 	}
 
-	if !viperInstance.IsSet(CollectorLogLevelKey) && strings.ToLower(viperInstance.GetString(LogLevelKey)) != "warn" {
-		log.Level = viperInstance.GetString(LogLevelKey)
+	return &Log{
+		Level: viperInstance.GetString(CollectorLogLevelKey),
+		Path:  viperInstance.GetString(CollectorLogPathKey),
 	}
-
-	return log
 }
 
 func resolveCommand() *Command {

internal/config/config_test.go

@@ -277,7 +277,7 @@ func TestResolveLog(t *testing.T) {
 			logLevel:         "DEBUG",
 			logPath:          "./logs/nginx.log",
 			expectedLogPath:  "./logs/nginx.log",
-			expectedLogLevel: "debug",
+			expectedLogLevel: "DEBUG",
 		},
 	}
 
@@ -341,8 +341,6 @@ func TestResolveCollector(t *testing.T) {
 }
 
 func TestResolveCollectorLog(t *testing.T) {
-	viperInstance = viper.NewWithOptions(viper.KeyDelimiter(KeyDelimiter))
-	viperInstance.SetDefault(CollectorLogLevelKey, DefCollectorLogLevel)
 	tests := []struct {
 		name             string
 		logLevel         string
@@ -357,15 +355,15 @@ func TestResolveCollectorLog(t *testing.T) {
 			logPath:          "/tmp/collector.log",
 			agentLogLevel:    "debug",
 			expectedLogPath:  "/tmp/collector.log",
-			expectedLogLevel: "debug",
+			expectedLogLevel: "DEBUG",
 		},
 		{
 			name:             "Test 2: Agent Log Level is Warn",
 			logLevel:         "",
 			logPath:          "/tmp/collector.log",
 			agentLogLevel:    "warn",
 			expectedLogPath:  "/tmp/collector.log",
-			expectedLogLevel: "INFO",
+			expectedLogLevel: "WARN",
 		},
 		{
 			name:             "Test 3: OTel Log Level Set In Config",
@@ -379,6 +377,7 @@ func TestResolveCollectorLog(t *testing.T) {
 
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
+			viperInstance = viper.NewWithOptions(viper.KeyDelimiter(KeyDelimiter))
 			viperInstance.Set(CollectorLogPathKey, test.logPath)
 			viperInstance.Set(LogLevelKey, test.agentLogLevel)