From 044a205454606dc5afd9eb0421c9f4c3aad3d3f8 Mon Sep 17 00:00:00 2001 From: John Murret Date: Wed, 17 Jan 2024 07:25:35 -0700 Subject: [PATCH] check error in TestDNSCycleRecursorCheckAllFail before asserting response to stop panic in CI. (#20231) --- agent/dns_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/agent/dns_test.go b/agent/dns_test.go index 399899527218e..8480beb62dec5 100644 --- a/agent/dns_test.go +++ b/agent/dns_test.go @@ -513,7 +513,8 @@ func TestDNSCycleRecursorCheckAllFail(t *testing.T) { m.SetQuestion("google.com.", dns.TypeA) // Agent request client := new(dns.Client) - in, _, _ := client.Exchange(m, agent.DNSAddr()) + in, _, err := client.Exchange(m, agent.DNSAddr()) + require.NoError(t, err) // Verify if we hit SERVFAIL from Consul require.Equal(t, dns.RcodeServerFailure, in.Rcode) }