remove unnecessary nil checks; default case

add tests for TLSConfig object

nomad/rpc.go

@@ -87,6 +87,7 @@ func (s *Server) listen(ctx context.Context) {
 			select {
 			case <-ctx.Done():
 				return
+			default:
 			}
 
 			s.logger.Printf("[ERR] nomad.rpc: failed to accept RPC conn: %v", err)

nomad/structs/config/tls.go

@@ -191,14 +191,6 @@ func (t *TLSConfig) Merge(b *TLSConfig) *TLSConfig {
 // NewConfig should never be nil- calling code is responsible for walways
 // passing a valid TLSConfig object
 func (t *TLSConfig) Equals(newConfig *TLSConfig) bool {
-	if t == nil {
-		return false
-	}
-
-	if t == nil && newConfig != nil {
-		return false
-	}
-
 	return t.EnableRPC == newConfig.EnableRPC &&
 		t.CAFile == newConfig.CAFile &&
 		t.CertFile == newConfig.CertFile &&

nomad/structs/config/tls_test.go

@@ -25,3 +25,39 @@ func TestTLSConfig_Merge(t *testing.T) {
 	new := a.Merge(b)
 	assert.Equal(b, new)
 }
+
+func TestTLS_Equals_TrueWhenEmpty(t *testing.T) {
+	assert := assert.New(t)
+	a := &TLSConfig{}
+	b := &TLSConfig{}
+	assert.True(a.Equals(b))
+}
+
+func TestTLS_Equals_FalseWhenUnequal(t *testing.T) {
+	assert := assert.New(t)
+	a := &TLSConfig{CAFile: "abc", CertFile: "def", KeyFile: "ghi"}
+	b := &TLSConfig{CAFile: "jkl", CertFile: "def", KeyFile: "ghi"}
+	assert.False(a.Equals(b))
+}
+
+func TestTLS_Equals_TrueWhenEqual(t *testing.T) {
+	assert := assert.New(t)
+	a := &TLSConfig{CAFile: "abc", CertFile: "def", KeyFile: "ghi"}
+	b := &TLSConfig{CAFile: "abc", CertFile: "def", KeyFile: "ghi"}
+	assert.True(a.Equals(b))
+}
+
+func TestTLS_Copy(t *testing.T) {
+	assert := assert.New(t)
+	a := &TLSConfig{CAFile: "abc", CertFile: "def", KeyFile: "ghi"}
+	aCopy := a.Copy()
+	assert.True(a.Equals(aCopy))
+}
+
+// GetKeyLoader should always return an initialized KeyLoader for a TLSConfig
+// object
+func TestTLS_GetKeyloader(t *testing.T) {
+	assert := assert.New(t)
+	a := &TLSConfig{}
+	assert.NotNil(a.GetKeyLoader())
+}