Fix adding labels/annotations to resources or data to config maps/secrets when those were empty

[pdecat]

No description provided.

[pdecat]

Acceptance test failure

# make testacc TEST=./kubernetes TESTARGS='-run=TestAccKubernetesNamespace_basic'                                                 
==> Checking that code complies with gofmt requirements...        
TF_ACC=1 go test ./kubernetes -v -run=TestAccKubernetesNamespace_basic -timeout 120m                                                
=== RUN   TestAccKubernetesNamespace_basic                        
--- FAIL: TestAccKubernetesNamespace_basic (6.34s)
        testing.go:449: Step 1 error: Error applying: 1 error(s) occurred:

                * kubernetes_namespace.test: 1 error(s) occurred:

                * kubernetes_namespace.test: jsonpatch add operation does not apply: doc is missing path: /metadata/labels/TestLabelTwo
FAIL
exit status 1
FAIL    github.com/terraform-providers/terraform-provider-kubernetes/kubernetes 6.424s
GNUmakefile:15: recipe for target 'testacc' failed
make: *** [testacc] Error 1

[pdecat]

Unit tests result

# make test TEST=./kubernetes
==> Checking that code complies with gofmt requirements...
go test -i ./kubernetes || exit 1
echo ./kubernetes | \
        xargs -t -n4 go test  -timeout=30s -parallel=4
go test -timeout=30s -parallel=4 ./kubernetes 
ok      github.com/terraform-providers/terraform-provider-kubernetes/kubernetes 0.089s

Acceptance tests result for namespace

# make testacc TEST=./kubernetes TESTARGS='-run=TestAccKubernetesNamespace_.*'                                                    
==> Checking that code complies with gofmt requirements...        
TF_ACC=1 go test ./kubernetes -v -run=TestAccKubernetesNamespace_.* -timeout 120m                                                   
=== RUN   TestAccKubernetesNamespace_basic                        
--- PASS: TestAccKubernetesNamespace_basic (6.59s)                
=== RUN   TestAccKubernetesNamespace_importBasic                  
--- PASS: TestAccKubernetesNamespace_importBasic (6.35s)          
=== RUN   TestAccKubernetesNamespace_generatedName                                                                                  
--- PASS: TestAccKubernetesNamespace_generatedName (6.33s)        
=== RUN   TestAccKubernetesNamespace_withSpecialCharacters                                                                          
--- PASS: TestAccKubernetesNamespace_withSpecialCharacters (6.33s)                                                                                                                                                                                                       
=== RUN   TestAccKubernetesNamespace_importGeneratedName                                                                                                                                                                                                                 --- PASS: TestAccKubernetesNamespace_importGeneratedName (6.37s)                                                                                                                                                                                                         
PASS                                                                                                                                
ok      github.com/terraform-providers/terraform-provider-kubernetes/kubernetes 32.054s                                             

[pdecat]

Acceptance tests result for config map:

# make testacc TEST=./kubernetes TESTARGS='-run=TestAccKubernetesConfigMap_.*' 
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./kubernetes -v -run=TestAccKubernetesConfigMap_.* -timeout 120m
=== RUN   TestAccKubernetesConfigMap_basic
--- PASS: TestAccKubernetesConfigMap_basic (0.35s)
=== RUN   TestAccKubernetesConfigMap_importBasic
--- PASS: TestAccKubernetesConfigMap_importBasic (0.13s)
=== RUN   TestAccKubernetesConfigMap_generatedName
--- PASS: TestAccKubernetesConfigMap_generatedName (0.10s)
=== RUN   TestAccKubernetesConfigMap_importGeneratedName
--- PASS: TestAccKubernetesConfigMap_importGeneratedName (0.11s)
PASS
ok      github.com/terraform-providers/terraform-provider-kubernetes/kubernetes 0.759s

[pdecat]

Acceptance tests result for secret:

#  make testacc TEST=./kubernetes TESTARGS='-run=TestAccKubernetesSecret_.*'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./kubernetes -v -run=TestAccKubernetesSecret_.* -timeout 120m
=== RUN   TestAccKubernetesSecret_basic
--- PASS: TestAccKubernetesSecret_basic (0.38s)
=== RUN   TestAccKubernetesSecret_importBasic
--- PASS: TestAccKubernetesSecret_importBasic (0.10s)
=== RUN   TestAccKubernetesSecret_generatedName
--- PASS: TestAccKubernetesSecret_generatedName (0.08s)
=== RUN   TestAccKubernetesSecret_importGeneratedName
--- PASS: TestAccKubernetesSecret_importGeneratedName (0.10s)
PASS
ok      github.com/terraform-providers/terraform-provider-kubernetes/kubernetes 0.757s

[radeksimko]

Thanks for the fix and all attached tests @pdecat
This looks great.

Shipping... 🚢

[pdecat]

Thanks!