Merge pull request #664 from rubyforgood/partner_creation_fix

We had the background job to create partners in the controller happen…

Author: seanmarcia

app/models/partner.rb

@@ -26,6 +26,8 @@ class Partner < ApplicationRecord
       .order(:name)
   }
 
+  after_create :register_on_partnerbase
+
   # better to extract this outside of the model
   def self.import_csv(data, organization_id)
     CSV.parse(data, headers: true) do |row|
@@ -43,4 +45,8 @@ def self.csv_export_headers
   def csv_export_attributes
     [name, email]
   end
+
+  def register_on_partnerbase
+    UpdateDiaperPartnerJob.perform_async(id)
+  end
 end

spec/controllers/partners_controller_spec.rb

@@ -49,11 +49,6 @@
         expect { subject }.to change(Partner, :count).by(1)
       end
 
-      it "enqueues a UpdateDiaperPartnerJob job" do
-        expect(UpdateDiaperPartnerJob).to receive(:perform_async)
-        subject
-      end
-
       it "redirects to #index" do
         expect(subject).to redirect_to(partners_path)
       end

spec/models/partner_spec.rb

@@ -51,6 +51,12 @@
   #     end
   #   end
   # end
+
+  it 'fires send_email method as after_create method callbacks' do
+    expect(subject).to receive(:register_on_partnerbase)
+    subject.run_callbacks(:create)
+  end
+
   describe "import_csv" do
     let(:organization) { create(:organization) }