Merge pull request #536 from Liujingfang1/mergenil

Update TransformerConfig.Merge function to handle nil
This commit is contained in:
k8s-ci-robot
2018-11-07 19:19:33 -08:00
committed by GitHub
2 changed files with 8 additions and 0 deletions

View File

@@ -64,6 +64,9 @@ func (t *TransformerConfig) AddNamereferenceFieldSpec(nbrs NameBackReferences) {
// Merge merges two TransformerConfigs objects into a new TransformerConfig object
func (t *TransformerConfig) Merge(input *TransformerConfig) *TransformerConfig {
if input == nil {
return t
}
merged := &TransformerConfig{}
merged.NamePrefix = append(t.NamePrefix, input.NamePrefix...)
merged.NameSpace = append(t.NameSpace, input.NameSpace...)

View File

@@ -141,4 +141,9 @@ func TestMerge(t *testing.T) {
if !reflect.DeepEqual(actual, expected) {
t.Fatalf("expected: %v\n but got: %v\n", expected, actual)
}
actual = cfga.Merge(nil)
if !reflect.DeepEqual(actual, cfga) {
t.Fatalf("expected: %v\n but got: %v\n", cfga, actual)
}
}