From f58eeef49d3a59f314bf69d5eeb176695f82ae5b Mon Sep 17 00:00:00 2001 From: Andrew Savinykh <658865+AndrewSav@users.noreply.github.com> Date: Sun, 15 Feb 2026 21:28:23 +1300 Subject: [PATCH] check return value from `unmarshalRoot` in test and fail if false Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> --- service/internal/config/config_reloader_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/service/internal/config/config_reloader_test.go b/service/internal/config/config_reloader_test.go index f96544e..39dad61 100644 --- a/service/internal/config/config_reloader_test.go +++ b/service/internal/config/config_reloader_test.go @@ -99,7 +99,10 @@ func TestEnvInConfig(t *testing.T) { t.Errorf("Error loading YAML: %v", err) continue } - unmarshalRoot(k, cfg) + if !unmarshalRoot(k, cfg) { + t.Errorf("Error unmarshalling config for env=%q", tt.input) + continue + } field := tt.selector(cfg) assert.Equal(t, tt.output, field, "Unmarshaled config field doesn't match expected value: env=%q", tt.input)