aboutsummaryrefslogtreecommitdiff
path: root/plugin/grpc/setup_test.go
diff options
context:
space:
mode:
authorGravatar IƱigo <inigohu@gmail.com> 2019-03-14 08:12:28 +0100
committerGravatar Miek Gieben <miek@miek.nl> 2019-03-14 07:12:28 +0000
commit7b6cb76237d151ffa056c742ec281a17548fa089 (patch)
tree2100dd26d9e41f0be6fb365de9aeb334d7a94075 /plugin/grpc/setup_test.go
parent0d8e1cf8b4ac157ef08837d0ff1e83597b8d1211 (diff)
downloadcoredns-7b6cb76237d151ffa056c742ec281a17548fa089.tar.gz
coredns-7b6cb76237d151ffa056c742ec281a17548fa089.tar.zst
coredns-7b6cb76237d151ffa056c742ec281a17548fa089.zip
plugin/grpc: New gRPC plugin (#2667)
* plugin/grpc: New gRPC plugin * some changes after the first review: - remove healthcheck. gRPC already has this implicitly implemented - some naming and stetic changes - fix some comments - other minor fixes * plugin/grpc: New gRPC plugin * some changes after the first review: - remove healthcheck. gRPC already has this implicitly implemented - some naming and stetic changes - fix some comments - other minor fixes * add OWNERS file and change plugin order * remove Rcode checker
Diffstat (limited to 'plugin/grpc/setup_test.go')
-rw-r--r--plugin/grpc/setup_test.go156
1 files changed, 156 insertions, 0 deletions
diff --git a/plugin/grpc/setup_test.go b/plugin/grpc/setup_test.go
new file mode 100644
index 000000000..fb470a541
--- /dev/null
+++ b/plugin/grpc/setup_test.go
@@ -0,0 +1,156 @@
+package grpc
+
+import (
+ "io/ioutil"
+ "os"
+ "reflect"
+ "strings"
+ "testing"
+
+ "github.com/mholt/caddy"
+)
+
+func TestSetup(t *testing.T) {
+ tests := []struct {
+ input string
+ shouldErr bool
+ expectedFrom string
+ expectedIgnored []string
+ expectedErr string
+ }{
+ // positive
+ {"grpc . 127.0.0.1", false, ".", nil, ""},
+ {"grpc . 127.0.0.1 {\nexcept miek.nl\n}\n", false, ".", nil, ""},
+ {"grpc . 127.0.0.1", false, ".", nil, ""},
+ {"grpc . 127.0.0.1:53", false, ".", nil, ""},
+ {"grpc . 127.0.0.1:8080", false, ".", nil, ""},
+ {"grpc . [::1]:53", false, ".", nil, ""},
+ {"grpc . [2003::1]:53", false, ".", nil, ""},
+ // negative
+ {"grpc . a27.0.0.1", true, "", nil, "not an IP"},
+ {"grpc . 127.0.0.1 {\nblaatl\n}\n", true, "", nil, "unknown property"},
+ {`grpc . ::1
+ grpc com ::2`, true, "", nil, "plugin"},
+ }
+
+ for i, test := range tests {
+ c := caddy.NewTestController("grpc", test.input)
+ g, err := parseGRPC(c)
+
+ if test.shouldErr && err == nil {
+ t.Errorf("Test %d: expected error but found %s for input %s", i, err, test.input)
+ }
+
+ if err != nil {
+ if !test.shouldErr {
+ t.Errorf("Test %d: expected no error but found one for input %s, got: %v", i, test.input, err)
+ }
+
+ if !strings.Contains(err.Error(), test.expectedErr) {
+ t.Errorf("Test %d: expected error to contain: %v, found error: %v, input: %s", i, test.expectedErr, err, test.input)
+ }
+ }
+
+ if !test.shouldErr && g.from != test.expectedFrom {
+ t.Errorf("Test %d: expected: %s, got: %s", i, test.expectedFrom, g.from)
+ }
+ if !test.shouldErr && test.expectedIgnored != nil {
+ if !reflect.DeepEqual(g.ignored, test.expectedIgnored) {
+ t.Errorf("Test %d: expected: %q, actual: %q", i, test.expectedIgnored, g.ignored)
+ }
+ }
+ }
+}
+
+func TestSetupTLS(t *testing.T) {
+ tests := []struct {
+ input string
+ shouldErr bool
+ expectedServerName string
+ expectedErr string
+ }{
+ // positive
+ {`grpc . 127.0.0.1 {
+tls_servername dns
+}`, false, "dns", ""},
+ {`grpc . 127.0.0.1 {
+tls_servername dns
+}`, false, "", ""},
+ {`grpc . 127.0.0.1 {
+tls
+}`, false, "", ""},
+ {`grpc . 127.0.0.1`, false, "", ""},
+ }
+
+ for i, test := range tests {
+ c := caddy.NewTestController("dns", test.input)
+ g, err := parseGRPC(c)
+
+ if test.shouldErr && err == nil {
+ t.Errorf("Test %d: expected error but found %s for input %s", i, err, test.input)
+ }
+
+ if err != nil {
+ if !test.shouldErr {
+ t.Errorf("Test %d: expected no error but found one for input %s, got: %v", i, test.input, err)
+ }
+
+ if !strings.Contains(err.Error(), test.expectedErr) {
+ t.Errorf("Test %d: expected error to contain: %v, found error: %v, input: %s", i, test.expectedErr, err, test.input)
+ }
+ }
+
+ if !test.shouldErr && test.expectedServerName != "" && g.tlsConfig != nil && test.expectedServerName != g.tlsConfig.ServerName {
+ t.Errorf("Test %d: expected: %q, actual: %q", i, test.expectedServerName, g.tlsConfig.ServerName)
+ }
+ }
+}
+
+func TestSetupResolvconf(t *testing.T) {
+ const resolv = "resolv.conf"
+ if err := ioutil.WriteFile(resolv,
+ []byte(`nameserver 10.10.255.252
+nameserver 10.10.255.253`), 0666); err != nil {
+ t.Fatalf("Failed to write resolv.conf file: %s", err)
+ }
+ defer os.Remove(resolv)
+
+ tests := []struct {
+ input string
+ shouldErr bool
+ expectedErr string
+ expectedNames []string
+ }{
+ // pass
+ {`grpc . ` + resolv, false, "", []string{"10.10.255.252:53", "10.10.255.253:53"}},
+ }
+
+ for i, test := range tests {
+ c := caddy.NewTestController("grpc", test.input)
+ f, err := parseGRPC(c)
+
+ if test.shouldErr && err == nil {
+ t.Errorf("Test %d: expected error but found %s for input %s", i, err, test.input)
+ continue
+ }
+
+ if err != nil {
+ if !test.shouldErr {
+ t.Errorf("Test %d: expected no error but found one for input %s, got: %v", i, test.input, err)
+ }
+
+ if !strings.Contains(err.Error(), test.expectedErr) {
+ t.Errorf("Test %d: expected error to contain: %v, found error: %v, input: %s", i, test.expectedErr, err, test.input)
+ }
+ }
+
+ if !test.shouldErr {
+ for j, n := range test.expectedNames {
+ addr := f.proxies[j].addr
+ if n != addr {
+ t.Errorf("Test %d, expected %q, got %q", j, n, addr)
+ }
+ }
+ }
+ }
+}