BUGFIX: Ignores non-configurable properties when associating existing collection item GUIDs

CHANGELOG.md

@@ -47,6 +47,12 @@ can be found in [Pivotal Documentation](docs.pivotal.io/platform-automation).
   This ensures that commands are not kept in `bash` history.
   The environment variable `OM_PASSWORD` will overwrite the password value in `env.yml`.
 
+## 6.1.3
+
+### Bug Fixes
+- `configure-product` will no longer assign a new guid for unnamed collections
+   - that haven't changed but contain non-configurable properties
+
 ## 6.1.2
 
 ### Bug Fixes

api/staged_products_property_collection_guid_assigner.go

@@ -115,6 +115,10 @@ func (item responsePropertyCollectionItem) getFieldValuesExceptGUID() map[interf
 		if key == "guid" {
 			continue
 		}
+		isNotConfigurable := !valueObj.(map[interface{}]interface{})["configurable"].(bool)
+		if isNotConfigurable {
+			continue
+		}
 		extractedValues[key] = valueObj.(map[interface{}]interface{})["value"]
 	}
 

api/staged_products_property_collection_guid_assigner_test.go.go

@@ -302,5 +302,43 @@ var _ = Describe("ResponsePropertyCollection", func() {
 			Expect(ok).To(BeTrue())
 			Expect(guid).To(Equal("28bab1d3-4a4b-48d5-8dac-two"))
 		})
+		It("ignores non-configurable properties when finding items that are equivalent", func() {
+			existingCollection, err := parseResponsePropertyCollection(unmarshalJSONLikeApiGetStagedProductProperties(`[
+				{
+					"guid": {
+						"type": "uuid",
+						"configurable": false,
+						"credential": false,
+						"value": "28bab1d3-4a4b-with-non-configurable-properties",
+						"optional": false
+					},
+					"non_configurable_property": {
+						"type": "string",
+						"configurable": false,
+						"credential": false,
+						"value": "this property can't be configured",
+						"optional": false
+					},
+					"configurable_property": {
+						"type": "string",
+						"configurable": true,
+						"credential": false,
+						"value": "this property can be configured",
+						"optional": false
+					}
+				}
+			]`))
+			Expect(err).To(BeNil())
+			updatedCollection, err := parseUpdatedPropertyCollection(unmarshalJSON(`{ "value":[
+				{
+					"configurable_property": "this property can be configured"
+				}
+			]}`))
+			Expect(err).To(BeNil())
+
+			guid, ok := existingCollection.findGUIDForEquivalentlItem(updatedCollection[0])
+			Expect(ok).To(BeTrue())
+			Expect(guid).To(Equal("28bab1d3-4a4b-with-non-configurable-properties"))
+		})
 	})
 })