From 1da125e46f0dce4de43b81786bb867567db4fec7 Mon Sep 17 00:00:00 2001 From: Som Samantray Date: Thu, 30 Jul 2026 22:22:48 +0530 Subject: [PATCH] test: clarify empty-env-var semantics in default test --- ui/config_test.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/ui/config_test.go b/ui/config_test.go index 575c0a3..41ee1e7 100644 --- a/ui/config_test.go +++ b/ui/config_test.go @@ -7,6 +7,10 @@ import ( ) func TestConfig_HighPerformancePagerDefault(t *testing.T) { + // caarlos0/env treats a set-but-empty var the same as unset when a + // default exists (env.getOr: `exists && value == "" && defExists`), + // so setting "" here exercises the same envDefault path as a truly + // unset var. t.Setenv("GLOW_HIGH_PERFORMANCE_PAGER", "") cfg, err := env.ParseAs[Config]()