Merge pull request #322 from Smeb/fixup-mutate-annotations

bugfix MutateAnnotations

klient/decoder/decoder.go

@@ -254,7 +254,7 @@ func MutateAnnotations(overrides map[string]string) DecodeOption {
 		annotations := obj.GetAnnotations()
 		if annotations == nil {
 			annotations = make(map[string]string)
-			obj.SetLabels(annotations)
+			obj.SetAnnotations(annotations)
 		}
 		for key, value := range overrides {
 			annotations[key] = value

klient/decoder/decoder_test.go

@@ -32,6 +32,7 @@ import (
 )
 
 const (
+	testAnnotation       = "annotationvalue"
 	testLabel            = "labelvalue"
 	serviceAccountPrefix = "example-sa*"
 )
@@ -255,6 +256,26 @@ func TestDecodersWithMutateFunc(t *testing.T) {
 	})
 }
 
+func TestMutateAnnotations(t *testing.T) {
+	testObj := &v1.Pod{
+		ObjectMeta: metav1.ObjectMeta{
+			Annotations: map[string]string{"injected": "original"},
+		},
+	}
+	options := &decoder.Options{}
+	opt := decoder.MutateAnnotations(map[string]string{"injected": testAnnotation})
+	opt(options)
+	for _, fn := range options.MutateFuncs {
+		if err := fn(testObj); err != nil {
+			t.Fatal(err)
+		}
+	}
+
+	if testObj.Annotations["injected"] != testAnnotation {
+		t.Fatal("expected object annotation to be overwritten")
+	}
+}
+
 func TestHandlerFuncs(t *testing.T) {
 	handlerNS := &v1.Namespace{ObjectMeta: metav1.ObjectMeta{Name: "handler-test"}}
 	res, err := resources.New(cfg)