middleware/kubernetes: autopath in sub package (#848)

Put the autopath stuff in a separate sub package. Tests are still
included in the main kubernetes directory.

Next steps (after this is merged), is pulling the autopath handling
into the subpackage and fixing the tests.

1 files changed, 40 insertions, 40 deletions

diff --git a/middleware/kubernetes/setup_test.go b/middleware/kubernetes/setup_test.go
index 9512c8fbf..3cfc94bf1 100644
--- a/middleware/kubernetes/setup_test.go
+++ b/middleware/kubernetes/setup_test.go
@@ -8,6 +8,7 @@ import (
 	"testing"
 	"time"
 
+	"github.com/coredns/coredns/middleware/kubernetes/autopath"
 	"github.com/coredns/coredns/middleware/test"
 
 	"github.com/mholt/caddy"
@@ -42,7 +43,7 @@ func TestKubernetesParse(t *testing.T) {
 		expectedFallthrough   bool
 		expectedUpstreams     []string
 		expectedFederations   []Federation
-		expectedAutoPath      AutoPath
+		expectedAutoPath      *autopath.AutoPath
 	}{
 		// positive
 		{
@@ -59,7 +60,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"kubernetes keyword with multiple zones",
@@ -75,7 +76,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"kubernetes keyword with zone and empty braces",
@@ -92,7 +93,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"endpoint keyword with url",
@@ -110,7 +111,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"namespaces keyword with one namespace",
@@ -128,7 +129,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			nil,
-			AutoPath{},
+			nil,
 		},
 		{
 			"namespaces keyword with multiple namespaces",
@@ -146,7 +147,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"resync period in seconds",
@@ -164,7 +165,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"resync period in minutes",
@@ -182,7 +183,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"basic label selector",
@@ -200,7 +201,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"multi-label selector",
@@ -218,7 +219,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"fully specified valid config",
@@ -240,7 +241,7 @@ func TestKubernetesParse(t *testing.T) {
 			true,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// negative
 		{
@@ -257,7 +258,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"kubernetes keyword without a zone",
@@ -273,7 +274,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"endpoint keyword without an endpoint value",
@@ -291,7 +292,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"namespace keyword without a namespace value",
@@ -309,7 +310,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"resyncperiod keyword without a duration value",
@@ -327,7 +328,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"resync period no units",
@@ -345,7 +346,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"resync period invalid",
@@ -363,7 +364,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"labels with no selector value",
@@ -381,7 +382,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		{
 			"labels with invalid selector value",
@@ -399,7 +400,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// pods disabled
 		{
@@ -418,7 +419,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// pods insecure
 		{
@@ -437,7 +438,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// pods verified
 		{
@@ -456,7 +457,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// pods invalid
 		{
@@ -475,7 +476,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// cidrs ok
 		{
@@ -494,7 +495,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// cidrs ok
 		{
@@ -513,7 +514,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// fallthrough invalid
 		{
@@ -532,7 +533,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// Valid upstream
 		{
@@ -551,7 +552,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			[]string{"13.14.15.16:53"},
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// Invalid upstream
 		{
@@ -570,7 +571,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// Valid federations
 		{
@@ -593,7 +594,7 @@ func TestKubernetesParse(t *testing.T) {
 				{name: "foo", zone: "bar.crawl.com"},
 				{name: "fed", zone: "era.tion.com"},
 			},
-			AutoPath{},
+			nil,
 		},
 		// Invalid federations
 		{
@@ -612,7 +613,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			[]Federation{},
-			AutoPath{},
+			nil,
 		},
 		// autopath
 		{
@@ -631,8 +632,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			nil,
-			AutoPath{
-				Enabled:        true,
+			&autopath.AutoPath{
 				NDots:          1,
 				HostSearchPath: []string{"bar.com.", "baz.com."},
 				ResolvConfFile: autoPathResolvConfFile,
@@ -655,7 +655,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			nil,
-			AutoPath{},
+			nil,
 		},
 		{
 			"invalid autopath NDOTS",
@@ -673,7 +673,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			nil,
-			AutoPath{},
+			nil,
 		},
 		{
 			"invalid autopath RESOLV-CONF",
@@ -691,7 +691,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			nil,
-			AutoPath{},
+			nil,
 		},
 		{
 			"invalid autopath invalid option",
@@ -709,7 +709,7 @@ func TestKubernetesParse(t *testing.T) {
 			false,
 			nil,
 			nil,
-			AutoPath{},
+			nil,
 		},
 	}
 
@@ -811,8 +811,8 @@ func TestKubernetesParse(t *testing.T) {
 			}
 		}
 		// autopath
-		if !reflect.DeepEqual(test.expectedAutoPath, k8sController.AutoPath) {
-			t.Errorf("Test %d: Expected kubernetes controller to be initialized with autopath '%v'. Instead found autopath '%v' for input '%s'", i, test.expectedAutoPath, k8sController.AutoPath, test.input)
+		if !reflect.DeepEqual(test.expectedAutoPath, k8sController.autoPath) {
+			t.Errorf("Test %d: Expected kubernetes controller to be initialized with autopath '%v'. Instead found autopath '%v' for input '%s'", i, test.expectedAutoPath, k8sController.autoPath, test.input)
 		}
 	}
 }