diff --git a/internal/config/config_env_overrides_test.go b/internal/config/config_env_overrides_test.go index 0f7d7aef3..d5b5a7403 100644 --- a/internal/config/config_env_overrides_test.go +++ b/internal/config/config_env_overrides_test.go @@ -16,7 +16,6 @@ func TestLoad_EnvOverrides_Comprehensive(t *testing.T) { "BACKUP_POLLING_INTERVAL", "PVE_POLLING_INTERVAL", "ENABLE_TEMPERATURE_MONITORING", - "PULSE_ENABLE_SENSOR_PROXY", "PULSE_AUTH_HIDE_LOCAL_LOGIN", "PULSE_DISABLE_DOCKER_UPDATE_ACTIONS", "ENABLE_BACKUP_POLLING", @@ -39,7 +38,6 @@ func TestLoad_EnvOverrides_Comprehensive(t *testing.T) { t.Setenv("BACKUP_POLLING_INTERVAL", "30s") t.Setenv("PVE_POLLING_INTERVAL", "15s") t.Setenv("ENABLE_TEMPERATURE_MONITORING", "false") - t.Setenv("PULSE_ENABLE_SENSOR_PROXY", "true") t.Setenv("PULSE_AUTH_HIDE_LOCAL_LOGIN", "true") t.Setenv("PULSE_DISABLE_DOCKER_UPDATE_ACTIONS", "true") t.Setenv("ENABLE_BACKUP_POLLING", "false") @@ -57,7 +55,6 @@ func TestLoad_EnvOverrides_Comprehensive(t *testing.T) { assert.Equal(t, 30*time.Second, cfg.BackupPollingInterval) assert.Equal(t, 15*time.Second, cfg.PVEPollingInterval) assert.False(t, cfg.TemperatureMonitoringEnabled) - assert.True(t, cfg.EnableSensorProxy) assert.True(t, cfg.HideLocalLogin) assert.True(t, cfg.DisableDockerUpdateActions) assert.False(t, cfg.EnableBackupPolling) @@ -127,3 +124,17 @@ func TestLoad_EnvOverrides_AdaptivePolling_Alternative(t *testing.T) { require.NoError(t, err) assert.False(t, cfg.AdaptivePollingEnabled) } + +func TestLoad_EnvOverrides_PBSAndPMG(t *testing.T) { + tempDir := t.TempDir() + t.Setenv("PULSE_DATA_DIR", tempDir) + + t.Setenv("PBS_POLLING_INTERVAL", "45s") + t.Setenv("PMG_POLLING_INTERVAL", "120") + + cfg, err := Load() + require.NoError(t, err) + + assert.Equal(t, 45*time.Second, cfg.PBSPollingInterval) + assert.Equal(t, 120*time.Second, cfg.PMGPollingInterval) +} diff --git a/internal/config/config_load_extra_coverage_test.go b/internal/config/config_load_extra_coverage_test.go index b08549a7f..6595e3f5c 100644 --- a/internal/config/config_load_extra_coverage_test.go +++ b/internal/config/config_load_extra_coverage_test.go @@ -65,7 +65,6 @@ func TestLoad_EnvOverrides_Seconds(t *testing.T) { func TestLoad_EnvOverrides_More(t *testing.T) { t.Setenv("PULSE_DATA_DIR", t.TempDir()) - t.Setenv("PULSE_ENABLE_SENSOR_PROXY", "true") t.Setenv("PULSE_AUTH_HIDE_LOCAL_LOGIN", "true") t.Setenv("PULSE_DISABLE_DOCKER_UPDATE_ACTIONS", "true") t.Setenv("ENABLE_BACKUP_POLLING", "0") @@ -79,7 +78,6 @@ func TestLoad_EnvOverrides_More(t *testing.T) { cfg, err := Load() assert.NoError(t, err) - assert.True(t, cfg.EnableSensorProxy) assert.True(t, cfg.HideLocalLogin) assert.True(t, cfg.DisableDockerUpdateActions) assert.False(t, cfg.EnableBackupPolling) diff --git a/internal/monitoring/monitor_extra_coverage_test.go b/internal/monitoring/monitor_extra_coverage_test.go index a3f79cb55..a17c49a48 100644 --- a/internal/monitoring/monitor_extra_coverage_test.go +++ b/internal/monitoring/monitor_extra_coverage_test.go @@ -433,21 +433,6 @@ func TestMonitor_EnrichContainerMetadata_Extra(t *testing.T) { } } -func TestMonitor_TemperatureDiagnostics_Extra(t *testing.T) { - m := &Monitor{} - // Should return false/nil when tempCollector is nil - if m.HasSocketTemperatureProxy() { - t.Error("Expected HasSocketTemperatureProxy to be false when collector is nil") - } - if diag := m.SocketProxyHostDiagnostics(); diag != nil { - t.Error("Expected SocketProxyHostDiagnostics to be nil when collector is nil") - } - - m.tempCollector = NewTemperatureCollectorWithPort("root", "", 22) - m.HasSocketTemperatureProxy() - m.SocketProxyHostDiagnostics() -} - func TestMonitor_TokenBindings_Extra(t *testing.T) { m := &Monitor{ state: models.NewState(),