fix: fix bug where pacticipant with blank name was being created inst…

…ead of updating existing record

lib/pact_broker/domain/pacticipant.rb

@@ -44,6 +44,7 @@ class Pacticipant < Sequel::Model
       using PactBroker::StringRefinements
 
       plugin :insert_ignore, identifying_columns: [:name]
+      plugin :upsert, identifying_columns: [:name]
       plugin :timestamps, update_on_create: true
 
       set_primary_key :id

lib/pact_broker/pacticipants/repository.rb

@@ -75,7 +75,7 @@ def replace(pacticipant_name, open_struct_pacticipant)
           repository_name: open_struct_pacticipant.repository_name,
           repository_namespace: open_struct_pacticipant.repository_namespace,
           main_branch: open_struct_pacticipant.main_branch
-        ).save
+        ).upsert
       end
 
       def delete(pacticipant)

spec/lib/pact_broker/pacticipants/repository_spec.rb

@@ -62,6 +62,10 @@ module Pacticipants
 
         subject { Repository.new.replace("Bar", OpenStruct.new(main_branch: "bar", repository_url: "new_url")) }
 
+        it "does not create a new record" do
+          expect { subject }.to_not change { PactBroker::Domain::Pacticipant.count }
+        end
+
         it "replaces the pacticipant" do
           expect(subject.name).to eq "Bar"
           expect(subject.main_branch).to eq "bar"