From 87131088210bd887da4fc72ef913a2e975247510 Mon Sep 17 00:00:00 2001 From: Caleb Doxsey Date: Tue, 30 Aug 2022 10:02:15 -0600 Subject: [PATCH] autocert: fix flaky test (#3591) --- internal/autocert/manager_test.go | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/internal/autocert/manager_test.go b/internal/autocert/manager_test.go index fadf5a7752e..6e9598681d4 100644 --- a/internal/autocert/manager_test.go +++ b/internal/autocert/manager_test.go @@ -262,12 +262,15 @@ func TestConfig(t *testing.T) { var initialOCSPStaple []byte var certValidTime *time.Time mgr.OnConfigChange(ctx, func(ctx context.Context, cfg *config.Config) { - log.Info(ctx).Msg("OnConfigChange") + if len(cfg.AutoCertificates) == 0 { + return + } + cert := cfg.AutoCertificates[0] if initialOCSPStaple == nil { initialOCSPStaple = cert.OCSPStaple } else { - if bytes.Compare(initialOCSPStaple, cert.OCSPStaple) != 0 { + if !bytes.Equal(initialOCSPStaple, cert.OCSPStaple) { log.Info(ctx).Msg("OCSP updated") ocspUpdated <- true }