Skip to content

Commit

Permalink
fix: [MSSQL] entries without prohibit_update also failed (#1065)
Browse files Browse the repository at this point in the history
[#185149499]

We were passing default and required fields to the update
operation and some of these fields have prohibit update so
every entry was failing but not necessarily for the right
reason.
  • Loading branch information
fnaranjo-vmw authored Aug 9, 2023
1 parent f5af30a commit f2e681e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion integration-tests/mssql_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ var _ = Describe("MSSQL", Label("MSSQL"), func() {

DescribeTable("should prevent updating properties flagged as `prohibit_update` because it can result in the recreation of the service instance",
func(prop string, value any) {
err := broker.Update(instanceID, msSQLServiceName, customMSSQLPlan["name"].(string), buildProperties(defaultProperties(), requiredProperties(), map[string]any{prop: value}))
err := broker.Update(instanceID, msSQLServiceName, customMSSQLPlan["name"].(string), map[string]any{prop: value})

Expect(err).To(MatchError(
ContainSubstring(
Expand Down

0 comments on commit f2e681e

Please sign in to comment.