Remove duplicated TempFile testsuit (coredns#2508)

diogenxs · Jan 28, 2019 · ffe79b0 · ffe79b0
1 parent f300fa9
commit ffe79b0
Show file tree

Hide file tree

Showing 8 changed files with 25 additions and 31 deletions.
diff --git a/test/ds_file_test.go b/test/ds_file_test.go
@@ -28,7 +28,7 @@ var dsTestCases = []mtest.Case{
 
 func TestLookupDS(t *testing.T) {
 	t.Parallel()
-	name, rm, err := TempFile(".", miekNL)
+	name, rm, err := mtest.TempFile(".", miekNL)
 	if err != nil {
 		t.Fatalf("Failed to create zone: %s", err)
 	}

diff --git a/test/file.go b/test/file.go
diff --git a/test/file_cname_proxy_test.go b/test/file_cname_proxy_test.go
@@ -3,13 +3,15 @@ package test
 import (
 	"testing"
 
+	"github.com/coredns/coredns/plugin/test"
+
 	"github.com/miekg/dns"
 )
 
 func TestZoneExternalCNAMELookupWithoutProxy(t *testing.T) {
 	t.Parallel()
 
-	name, rm, err := TempFile(".", exampleOrg)
+	name, rm, err := test.TempFile(".", exampleOrg)
 	if err != nil {
 		t.Fatalf("Failed to create zone: %s", err)
 	}
@@ -41,7 +43,7 @@ func TestZoneExternalCNAMELookupWithoutProxy(t *testing.T) {
 func TestZoneExternalCNAMELookupWithProxy(t *testing.T) {
 	t.Parallel()
 
-	name, rm, err := TempFile(".", exampleOrg)
+	name, rm, err := test.TempFile(".", exampleOrg)
 	if err != nil {
 		t.Fatalf("Failed to create zone: %s", err)
 	}

diff --git a/test/file_reload_test.go b/test/file_reload_test.go
@@ -5,6 +5,7 @@ import (
 	"testing"
 	"time"
 
+	"github.com/coredns/coredns/plugin/test"
 	"github.com/coredns/coredns/plugin/file"
 
 	"github.com/miekg/dns"
@@ -13,7 +14,7 @@ import (
 func TestZoneReload(t *testing.T) {
 	file.TickTime = 1 * time.Second
 
-	name, rm, err := TempFile(".", exampleOrg)
+	name, rm, err := test.TempFile(".", exampleOrg)
 	if err != nil {
 		t.Fatalf("Failed to create zone: %s", err)
 	}

diff --git a/test/file_serve_test.go b/test/file_serve_test.go
@@ -3,13 +3,15 @@ package test
 import (
 	"testing"
 
+	"github.com/coredns/coredns/plugin/test"
+
 	"github.com/miekg/dns"
 )
 
 func TestZoneEDNS0Lookup(t *testing.T) {
 	t.Parallel()
 
-	name, rm, err := TempFile(".", `$ORIGIN example.org.
+	name, rm, err := test.TempFile(".", `$ORIGIN example.org.
 @	3600 IN	SOA sns.dns.icann.org. noc.dns.icann.org. (
 		2017042745 ; serial
 		7200       ; refresh (2 hours)
@@ -56,7 +58,7 @@ www     IN AAAA ::1
 func TestZoneNoNS(t *testing.T) {
 	t.Parallel()
 
-	name, rm, err := TempFile(".", `$ORIGIN example.org.
+	name, rm, err := test.TempFile(".", `$ORIGIN example.org.
 @	3600 IN	SOA sns.dns.icann.org. noc.dns.icann.org. (
 		2017042745 ; serial
 		7200       ; refresh (2 hours)

diff --git a/test/file_srv_additional_test.go b/test/file_srv_additional_test.go
@@ -3,13 +3,15 @@ package test
 import (
 	"testing"
 
+	"github.com/coredns/coredns/plugin/test"
+
 	"github.com/miekg/dns"
 )
 
 func TestZoneSRVAdditional(t *testing.T) {
 	t.Parallel()
 
-	name, rm, err := TempFile(".", exampleOrg)
+	name, rm, err := test.TempFile(".", exampleOrg)
 	if err != nil {
 		t.Fatalf("Failed to create zone: %s", err)
 	}

diff --git a/test/file_test.go b/test/file_test.go
@@ -1,10 +1,14 @@
 package test
 
-import "testing"
+import (
+	"testing"
+
+	"github.com/coredns/coredns/plugin/test"
+)
 
 func TestTempFile(t *testing.T) {
 	t.Parallel()
-	_, f, e := TempFile(".", "test")
+	_, f, e := test.TempFile(".", "test")
 	if e != nil {
 		t.Fatalf("Failed to create temp file: %s", e)
 	}

diff --git a/test/file_upstream_test.go b/test/file_upstream_test.go
@@ -3,11 +3,13 @@ package test
 import (
 	"testing"
 
+	"github.com/coredns/coredns/plugin/test"
+
 	"github.com/miekg/dns"
 )
 
 func TestFileUpstream(t *testing.T) {
-	name, rm, err := TempFile(".", `$ORIGIN example.org.
+	name, rm, err := test.TempFile(".", `$ORIGIN example.org.
 @	3600 IN	SOA sns.dns.icann.org. noc.dns.icann.org. (
 		2017042745 ; serial
 		7200       ; refresh (2 hours)
@@ -61,7 +63,7 @@ www 3600 IN CNAME   www.example.net.
 // TestFileUpstreamAdditional runs two CoreDNS servers that serve example.org and foo.example.org.
 // example.org contains a cname to foo.example.org; this should be resolved via upstream.Self.
 func TestFileUpstreamAdditional(t *testing.T) {
-	name, rm, err := TempFile(".", `$ORIGIN example.org.
+	name, rm, err := test.TempFile(".", `$ORIGIN example.org.
 @	3600 IN	SOA sns.dns.icann.org. noc.dns.icann.org. 2017042745 7200 3600 1209600 3600
 
 	3600 IN NS b.iana-servers.net.
@@ -73,7 +75,7 @@ www 3600 IN CNAME   www.foo
 	}
 	defer rm()
 
-	name2, rm2, err2 := TempFile(".", `$ORIGIN foo.example.org.
+	name2, rm2, err2 := test.TempFile(".", `$ORIGIN foo.example.org.
 @	3600 IN	SOA sns.dns.icann.org. noc.dns.icann.org. 2017042745 7200 3600 1209600 3600
 
 	3600 IN NS b.iana-servers.net.