Add unique index to svc_inst_op on svc_inst_id

* Add unique index on column service_instance_id to prevent double entries for the same
  service_instance
* Remove duplicate service_instance_id to prepare for adding a uniqueness constraint
* Drop old index before creating the unique one

Compared to service bindings, service keys and route bindings there is a UNIQUE index
defined for the "resource_id" field; for service instances the UNIQUE keyword is missing.

db/migrations/20220818142407_add_unique_index_to_service_instance_operations_service_instance_id.rb

@@ -0,0 +1,24 @@
+Sequel.migration do
+  up do
+    # Remove duplicate service_instance_operations.service_instance_id to prepare for adding a uniqueness constraint
+    # assumption is that the max id = newest entry for the same svc_inst_id
+    max_of_dup_groups = self[:service_instance_operations].
+                        select(Sequel.function(:max, :id)).
+                        group_by(:service_instance_id).
+                        having { count.function.* >= 1 }
+
+    self[:service_instance_operations].exclude(id: max_of_dup_groups).each do |row|
+      self[:service_instance_operations].where(id: row[:id]).delete
+    end
+    alter_table :service_instance_operations do
+      drop_index :service_instance_id, name: :svc_instance_id_index, if_exists: true
+      add_index :service_instance_id, name: :svc_inst_op_svc_instance_id_unique_index, unique: true
+    end
+  end
+  down do
+    alter_table :service_instance_operations do
+      drop_index :service_instance_id
+      add_index :service_instance_id, name: :svc_instance_id_index
+    end
+  end
+end

spec/unit/models/services/service_instance_operation_spec.rb

@@ -51,5 +51,37 @@ module VCAP::CloudController
         expect(operation.state).to eq 'finished'
       end
     end
+
+    describe 'when two are created with the same id' do
+      describe 'when a ServiceInstanceOperation exists' do
+        let(:service_instance_attrs) do
+          {
+            name: 'bommel_instance',
+            space: VCAP::CloudController::Space.make
+          }
+        end
+
+        let(:service_instance) { ServiceInstance.create(service_instance_attrs) }
+        let(:operation_attributes2) do
+          {
+            service_instance_id: service_instance.id,
+            state: 'in progress',
+            description: '50% all the time',
+            type: 'create',
+            proposed_changes: {
+              name: 'pizza',
+              service_plan_guid: '1800-pizza',
+            },
+          }
+        end
+        before { ServiceInstanceOperation.create(operation_attributes2) }
+
+        it 'raises an exception when creating another ServiceInstanceOperation' do
+          expect {
+            ServiceInstanceOperation.create(operation_attributes2)
+          }.to raise_error(Sequel::UniqueConstraintViolation)
+        end
+      end
+    end
   end
 end