From 5a1875120cceeaa90f065a4fa7dccffb8c535a71 Mon Sep 17 00:00:00 2001 From: Miek Gieben Date: Thu, 17 Aug 2017 18:42:14 +0100 Subject: [PATCH] mw/kubernetes: add reverse test case (#932) Add a non-arpa testcase to the reverse test. --- middleware/kubernetes/reverse_test.go | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/middleware/kubernetes/reverse_test.go b/middleware/kubernetes/reverse_test.go index 7cc51da17..759e615bd 100644 --- a/middleware/kubernetes/reverse_test.go +++ b/middleware/kubernetes/reverse_test.go @@ -96,6 +96,13 @@ func TestReverse(t *testing.T) { test.SOA("0.10.in-addr.arpa. 300 IN SOA ns.dns.0.10.in-addr.arpa. hostmaster.0.10.in-addr.arpa. 1502782828 7200 1800 86400 60"), }, }, + { + Qname: "example.org.cluster.local.", Qtype: dns.TypePTR, + Rcode: dns.RcodeSuccess, + Ns: []dns.RR{ + test.SOA("cluster.local. 300 IN SOA ns.dns.cluster.local. hostmaster.cluster.local. 1502989566 7200 1800 86400 60"), + }, + }, } ctx := context.TODO() @@ -109,9 +116,6 @@ func TestReverse(t *testing.T) { t.Errorf("Test %d: expected no error, got %v", i, err) return } - if tc.Error != nil { - continue - } resp := w.Msg if resp == nil {