diff options
author | 2019-01-19 11:23:13 +0000 | |
---|---|---|
committer | 2019-01-19 11:23:13 +0000 | |
commit | 725becd134752caead1aa2b2cf3a72d01c13b599 (patch) | |
tree | 300dbfeac60f6cffd86c3aeb0b68d772415e6521 /plugin/kubernetes/handler_test.go | |
parent | 278303ca0df6e6167788fa3435f6ea7c6253157b (diff) | |
download | coredns-725becd134752caead1aa2b2cf3a72d01c13b599.tar.gz coredns-725becd134752caead1aa2b2cf3a72d01c13b599.tar.zst coredns-725becd134752caead1aa2b2cf3a72d01c13b599.zip |
Stop importing testing in the main binary (#2479)
* Stop importing testing in the main binary
Stop importing "testing" into the main binary:
* test/helpers.go imported it; remote that and change function signature
* update all tests that use this
Signed-off-by: Miek Gieben <miek@miek.nl>
* Drop import testing from metrics plugin
Signed-off-by: Miek Gieben <miek@miek.nl>
* more fiddling
Signed-off-by: Miek Gieben <miek@miek.nl>
Diffstat (limited to 'plugin/kubernetes/handler_test.go')
-rw-r--r-- | plugin/kubernetes/handler_test.go | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/plugin/kubernetes/handler_test.go b/plugin/kubernetes/handler_test.go index 21a2ddbbf..6b0724045 100644 --- a/plugin/kubernetes/handler_test.go +++ b/plugin/kubernetes/handler_test.go @@ -371,9 +371,13 @@ func TestServeDNS(t *testing.T) { } // Before sorting, make sure that CNAMES do not appear after their target records - test.CNAMEOrder(t, resp) + if err := test.CNAMEOrder(resp); err != nil { + t.Error(err) + } - test.SortAndCheck(t, resp, tc) + if err := test.SortAndCheck(resp, tc); err != nil { + t.Error(err) + } } } @@ -417,10 +421,13 @@ func TestNotSyncedServeDNS(t *testing.T) { t.Fatalf("Test %d, got nil message and no error for %q", i, r.Question[0].Name) } - // Before sorting, make sure that CNAMES do not appear after their target records - test.CNAMEOrder(t, resp) + if err := test.CNAMEOrder(resp); err != nil { + t.Error(err) + } - test.SortAndCheck(t, resp, tc) + if err := test.SortAndCheck(resp, tc); err != nil { + t.Error(err) + } } } |