diff --git a/internal/email/factory_test.go b/internal/email/factory_test.go index 1c104d5b..569d115f 100644 --- a/internal/email/factory_test.go +++ b/internal/email/factory_test.go @@ -255,14 +255,6 @@ func TestNewSenderFromEnvironment_EmailEnabled(t *testing.T) { } // Unset / empty → factory takes the default-enabled path. - t.Run("unset_falls_through", func(t *testing.T) { - orig, hadOrig := os.LookupEnv("EMAIL_ENABLED") - t.Cleanup(func() { - if hadOrig { - os.Setenv("EMAIL_ENABLED", orig) - } else { - os.Unsetenv("EMAIL_ENABLED") - } t.Run("unset_falls_through", func(t *testing.T) { prev, hadPrev := os.LookupEnv("EMAIL_ENABLED") os.Unsetenv("EMAIL_ENABLED") @@ -276,8 +268,6 @@ func TestNewSenderFromEnvironment_EmailEnabled(t *testing.T) { aws(t) ctx := context.Background() sender, err := NewSenderFromEnvironment(ctx) - ctx := context.Background() - sender, err := NewSenderFromEnvironment(ctx) require.NoError(t, err) _, ok := sender.(*Sender) assert.True(t, ok, "Expected AWS Sender when EMAIL_ENABLED is unset")