feat: Send webhook on downgrade

getlago · Oct 16, 2023 · 8826575 · 8826575
1 parent bf385e7
commit 8826575
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 17 deletions.
diff --git a/app/services/subscriptions/terminate_service.rb b/app/services/subscriptions/terminate_service.rb
@@ -60,23 +60,15 @@ def terminate_and_start_next(timestamp:)
       # NOTE: Create an invoice for the terminated subscription
       #       if it has not been billed yet
       #       or only for the charges if subscription was billed in advance
-      BillSubscriptionJob.perform_later(
-        [subscription],
-        timestamp,
-      )
+      BillSubscriptionJob.perform_later([subscription], timestamp)
 
-      SendWebhookJob.perform_later(
-        'subscription.terminated',
-        subscription,
-      )
+      SendWebhookJob.perform_later('subscription.terminated', subscription)
+      SendWebhookJob.perform_later('subscription.started', next_subscription)
 
       result.subscription = next_subscription
       return result unless next_subscription.plan.pay_in_advance?
 
-      BillSubscriptionJob.perform_later(
-        [next_subscription],
-        timestamp,
-      )
+      BillSubscriptionJob.perform_later([next_subscription], timestamp)
 
       result
     rescue ActiveRecord::RecordInvalid => e

diff --git a/spec/services/subscriptions/create_service_spec.rb b/spec/services/subscriptions/create_service_spec.rb
@@ -402,7 +402,7 @@
 
             it 'enqueues a job to bill the existing subscription' do
               create_service.call
-              expect(BillSubscriptionJob).to have_been_enqueued.at_least(2).times
+              expect(BillSubscriptionJob).to have_been_enqueued.at_least(1).times
             end
           end
 
@@ -459,7 +459,7 @@
 
             it 'enqueues a job to bill the existing subscription' do
               create_service.call
-              expect(BillSubscriptionJob).to have_been_enqueued.at_least(1).times
+              expect(BillSubscriptionJob).to have_been_enqueued.at_least(2).times
             end
           end
 

diff --git a/spec/services/subscriptions/terminate_service_spec.rb b/spec/services/subscriptions/terminate_service_spec.rb
@@ -158,9 +158,9 @@
     end
 
     it 'enqueues a SendWebhookJob' do
-      expect do
-        terminate_service.terminate_and_start_next(timestamp:)
-      end.to have_enqueued_job(SendWebhookJob)
+      terminate_service.terminate_and_start_next(timestamp:)
+      expect(SendWebhookJob).to have_been_enqueued.with('subscription.terminated', subscription)
+      expect(SendWebhookJob).to have_been_enqueued.with('subscription.started', next_subscription)
     end
 
     context 'when terminated subscription is payed in arrear' do