From 63eb0a3a1fa96d1ce24621b3aab16f7285751e7e Mon Sep 17 00:00:00 2001 From: Jordan Liggitt Date: Tue, 23 Oct 2018 13:48:05 -0400 Subject: [PATCH] Add system root unit test Kubernetes-commit: fbd5597e9914fc24ece3b21e6015748ed865de7b --- transport/transport_test.go | 38 ++++++++++++++++++++++++++++++------- 1 file changed, 31 insertions(+), 7 deletions(-) diff --git a/transport/transport_test.go b/transport/transport_test.go index 2e9896a0..eead38aa 100644 --- a/transport/transport_test.go +++ b/transport/transport_test.go @@ -93,20 +93,32 @@ stR0Yiw0buV6DL/moUO0HIM9Bjh96HJp+LxiIS6UCdIhMPp5HoQa func TestNew(t *testing.T) { testCases := map[string]struct { - Config *Config - Err bool - TLS bool - TLSCert bool - TLSErr bool - Default bool + Config *Config + Err bool + TLS bool + TLSCert bool + TLSErr bool + Default bool + Insecure bool + DefaultRoots bool }{ "default transport": { Default: true, Config: &Config{}, }, + "insecure": { + TLS: true, + Insecure: true, + DefaultRoots: true, + Config: &Config{TLS: TLSConfig{ + Insecure: true, + }}, + }, + "server name": { - TLS: true, + TLS: true, + DefaultRoots: true, Config: &Config{TLS: TLSConfig{ ServerName: "foo", }}, @@ -266,6 +278,18 @@ func TestNew(t *testing.T) { return } + switch { + case testCase.DefaultRoots && transport.TLSClientConfig.RootCAs != nil: + t.Fatalf("got %#v, expected nil root CAs", transport.TLSClientConfig.RootCAs) + case !testCase.DefaultRoots && transport.TLSClientConfig.RootCAs == nil: + t.Fatalf("got %#v, expected non-nil root CAs", transport.TLSClientConfig.RootCAs) + } + + switch { + case testCase.Insecure != transport.TLSClientConfig.InsecureSkipVerify: + t.Fatalf("got %#v, expected %#v", transport.TLSClientConfig.InsecureSkipVerify, testCase.Insecure) + } + switch { case testCase.TLSCert && transport.TLSClientConfig.GetClientCertificate == nil: t.Fatalf("got %#v, expected TLSClientConfig.GetClientCertificate", transport.TLSClientConfig)