Backport of check error in TestDNSCycleRecursorCheckAllFail before as…

…serting response to stop panic in CI. into release/1.17.x (#20235)

* no-op commit due to failed cherry-picking

* check error in TestDNSCycleRecursorCheckAllFail before asserting response to stop panic in CI. (#20231)

---------

Co-authored-by: temp <temp@hashicorp.com>
Co-authored-by: John Murret <john.murret@hashicorp.com>

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)
 }