Lock school while creating domains and syncing with Profile

Author: PetarSimonovic

lib/concepts/school_email_domain/operations/create.rb

@@ -7,7 +7,7 @@ def call(school:, domain:, token:)
         response = OperationResponse.new
         response[:school_email_domain] = nil
         response[:school_email_domain] = build_domain(school, domain)
-        SchoolEmailDomain.transaction do
+        school.with_lock do
           response[:school_email_domain].save!
           update_profile(school, token)
         end

spec/concepts/school_email_domain/create_spec.rb

@@ -43,6 +43,14 @@
       )
     end
 
+    it 'locks the school while creating and syncing to Profile' do
+      allow(school).to receive(:with_lock).and_call_original
+
+      described_class.call(school:, domain:, token:)
+
+      expect(school).to have_received(:with_lock)
+    end
+
     context 'when multiple domains already exist' do
       before do
         create(:school_email_domain, school:, domain: 'first.edu')