vrr/Pulse
2
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-04-28 03:20:11 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

test: update config and monitoring tests after proxy removal

Browse source 
Remove references to sensor proxy config fields in test cases.
This commit is contained in:
rcourtman 2026-01-21 12:03:30 +00:00
parent 7599915b8f
commit 925815c3e7
3 changed files with 14 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

17
internal/config/config_env_overrides_test.go
View file

@ -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)
}

2
internal/config/config_load_extra_coverage_test.go
View file

@ -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)

15
internal/monitoring/monitor_extra_coverage_test.go
View file

@ -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(),