diff --git a/plugin/cancel/cancel_test.go b/plugin/cancel/cancel_test.go index ceba9f5d2..f77551809 100644 --- a/plugin/cancel/cancel_test.go +++ b/plugin/cancel/cancel_test.go @@ -34,7 +34,6 @@ func (s sleepPlugin) ServeDNS(ctx context.Context, w dns.ResponseWriter, r *dns. return 0, nil } } - return 0, nil } func TestCancel(t *testing.T) { diff --git a/plugin/done.go b/plugin/done.go index 3f53273da..c6ff8633b 100644 --- a/plugin/done.go +++ b/plugin/done.go @@ -10,5 +10,4 @@ func Done(ctx context.Context) bool { default: return false } - return false } diff --git a/plugin/etcd/msg/service_test.go b/plugin/etcd/msg/service_test.go index a5038ac2c..f334aa5ce 100644 --- a/plugin/etcd/msg/service_test.go +++ b/plugin/etcd/msg/service_test.go @@ -123,12 +123,3 @@ func TestGroup(t *testing.T) { t.Fatalf("Failure to group seventh set: %v", sx) } } - -func BenchmarkNewSRV(b *testing.B) { - s := &Service{Host: "www,example.org", Port: 8080} - for n := 0; n < b.N; n++ { - srv := s.NewSRV("www.example.org.", 16) - // this assignment makes sure s.NewSRV doesn't get optimized out - srv = srv - } -} diff --git a/plugin/kubernetes/controller_test.go b/plugin/kubernetes/controller_test.go index 37d4705e4..f945684cb 100644 --- a/plugin/kubernetes/controller_test.go +++ b/plugin/kubernetes/controller_test.go @@ -78,7 +78,7 @@ func generateEndpoints(cidr string, client kubernetes.Interface) { }, } ep.ObjectMeta.Name = "svc" + strconv.Itoa(count) - _, err = client.CoreV1().Endpoints("testns").Create(ep) + client.CoreV1().Endpoints("testns").Create(ep) count++ } } diff --git a/test/secondary_test.go b/test/secondary_test.go index fd9d4d562..1aef05d29 100644 --- a/test/secondary_test.go +++ b/test/secondary_test.go @@ -73,7 +73,7 @@ func TestSecondaryZoneTransfer(t *testing.T) { var r *dns.Msg // This is now async; we need to wait for it to be transferred. for i := 0; i < 10; i++ { - r, err = dns.Exchange(m, udp) + r, _ = dns.Exchange(m, udp) if len(r.Answer) != 0 { break }