aboutsummaryrefslogtreecommitdiff
path: root/core/https/handler_test.go
diff options
context:
space:
mode:
authorGravatar Miek Gieben <miek@miek.nl> 2016-08-19 17:14:17 -0700
committerGravatar GitHub <noreply@github.com> 2016-08-19 17:14:17 -0700
commit9ac3cab1b7b1b1e78f86ce3c6a80fbee312162e6 (patch)
tree437e9755927c33af16276ad2602a6da115f948cb /core/https/handler_test.go
parenta1989c35231b0e5ea271b2f68d82c1a63e697cd0 (diff)
downloadcoredns-9ac3cab1b7b1b1e78f86ce3c6a80fbee312162e6.tar.gz
coredns-9ac3cab1b7b1b1e78f86ce3c6a80fbee312162e6.tar.zst
coredns-9ac3cab1b7b1b1e78f86ce3c6a80fbee312162e6.zip
Make CoreDNS a server type plugin for Caddy (#220)
* Make CoreDNS a server type plugin for Caddy Remove code we don't need and port all middleware over. Fix all tests and rework the documentation. Also make `go generate` build a caddy binary which we then copy into our directory. This means `go build`-builds remain working as-is. And new etc instances in each etcd test for better isolation. Fix more tests and rework test.Server with the newer support Caddy offers. Fix Makefile to support new mode of operation.
Diffstat (limited to 'core/https/handler_test.go')
-rw-r--r--core/https/handler_test.go63
1 files changed, 0 insertions, 63 deletions
diff --git a/core/https/handler_test.go b/core/https/handler_test.go
deleted file mode 100644
index 016799ffb..000000000
--- a/core/https/handler_test.go
+++ /dev/null
@@ -1,63 +0,0 @@
-package https
-
-import (
- "net"
- "net/http"
- "net/http/httptest"
- "testing"
-)
-
-func TestRequestCallbackNoOp(t *testing.T) {
- // try base paths that aren't handled by this handler
- for _, url := range []string{
- "http://localhost/",
- "http://localhost/foo.html",
- "http://localhost/.git",
- "http://localhost/.well-known/",
- "http://localhost/.well-known/acme-challenging",
- } {
- req, err := http.NewRequest("GET", url, nil)
- if err != nil {
- t.Fatalf("Could not craft request, got error: %v", err)
- }
- rw := httptest.NewRecorder()
- if RequestCallback(rw, req) {
- t.Errorf("Got true with this URL, but shouldn't have: %s", url)
- }
- }
-}
-
-func TestRequestCallbackSuccess(t *testing.T) {
- expectedPath := challengeBasePath + "/asdf"
-
- // Set up fake acme handler backend to make sure proxying succeeds
- var proxySuccess bool
- ts := httptest.NewUnstartedServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- proxySuccess = true
- if r.URL.Path != expectedPath {
- t.Errorf("Expected path '%s' but got '%s' instead", expectedPath, r.URL.Path)
- }
- }))
-
- // Custom listener that uses the port we expect
- ln, err := net.Listen("tcp", "127.0.0.1:"+AlternatePort)
- if err != nil {
- t.Fatalf("Unable to start test server listener: %v", err)
- }
- ts.Listener = ln
-
- // Start our engines and run the test
- ts.Start()
- defer ts.Close()
- req, err := http.NewRequest("GET", "http://127.0.0.1:"+AlternatePort+expectedPath, nil)
- if err != nil {
- t.Fatalf("Could not craft request, got error: %v", err)
- }
- rw := httptest.NewRecorder()
-
- RequestCallback(rw, req)
-
- if !proxySuccess {
- t.Fatal("Expected request to be proxied, but it wasn't")
- }
-}