fixes replacement of /releases/name=os-conf? where last token was marked optional

Author: cppforlife

patch/replace_op.go

@@ -72,9 +72,13 @@ func (op ReplaceOp) Apply(doc interface{}) (interface{}, error) {
 			}
 
 			if typedToken.Optional && len(idxs) == 0 {
-				obj = map[interface{}]interface{}{typedToken.Key: typedToken.Value}
-				prevUpdate(append(typedObj, obj))
-				// no need to change prevUpdate since matching item can only be a map
+				if isLast {
+					prevUpdate(append(typedObj, op.Value))
+				} else {
+					obj = map[interface{}]interface{}{typedToken.Key: typedToken.Value}
+					prevUpdate(append(typedObj, obj))
+					// no need to change prevUpdate since matching item can only be a map
+				}
 			} else {
 				if len(idxs) != 1 {
 					return nil, opMultipleMatchingIndexErr{NewPointer(tokens[:i+2]), idxs}

patch/replace_op_test.go

@@ -233,9 +233,7 @@ var _ = Describe("ReplaceOp.Apply", func() {
 				map[interface{}]interface{}{"xyz": "xyz"},
 				map[interface{}]interface{}{
 					"name": "val",
-					"efg": []interface{}{
-						map[interface{}]interface{}{"name": "val"},
-					},
+					"efg": []interface{}{1},
 				},
 			}))
 		})