From b7035de4002166a46b5d891bbfb466a56cd3b8f4 Mon Sep 17 00:00:00 2001
From: Tanmay Deep Sharma <tanmaydeepsharma21@gmail.com>
Date: Fri, 7 Nov 2025 00:05:05 +0530
Subject: [PATCH] fix webhook handler spec

---
 .../v2/webhook_handler_service_spec.rb        | 39 +++++++++++++++----
 1 file changed, 31 insertions(+), 8 deletions(-)

diff --git a/spec/enterprise/services/enterprise/billing/v2/webhook_handler_service_spec.rb b/spec/enterprise/services/enterprise/billing/v2/webhook_handler_service_spec.rb
index 8a339af23..20abeaf63 100644
--- a/spec/enterprise/services/enterprise/billing/v2/webhook_handler_service_spec.rb
+++ b/spec/enterprise/services/enterprise/billing/v2/webhook_handler_service_spec.rb
@@ -16,24 +16,47 @@ describe Enterprise::Billing::V2::WebhookHandlerService do
     end
     let(:cadence_response) { OpenStruct.new(payer: OpenStruct.new(customer: 'cus_123')) }
     let(:event) do
-      double('Stripe::Event', type: 'v2.billing.pricing_plan_subscription.servicing_activated', related_object: OpenStruct.new(id: 'bpps_sub_123')) # rubocop:disable RSpec/VerifiedDoubles
+      event_double = double
+      allow(event_double).to receive(:blank?).and_return(false)
+      allow(event_double).to receive(:type).and_return('v2.billing.pricing_plan_subscription.servicing_activated')
+      allow(event_double).to receive(:related_object).and_return(OpenStruct.new(id: 'bpps_sub_123'))
+      event_double
     end
 
     before do
       account # Ensure account is created
       create(:installation_config, name: 'STRIPE_BUSINESS_PLAN_ID', value: pricing_plan_id)
-      # Mock the Stripe API calls
-      allow(StripeV2Client).to receive(:request)
-        .with(:get, '/v2/billing/pricing_plan_subscriptions/bpps_sub_123', anything, anything)
-        .and_return(subscription_response)
-      allow(StripeV2Client).to receive(:request)
-        .with(:get, '/v2/billing/cadences/cad_123', anything, anything)
-        .and_return(cadence_response)
+
+      # Mock the Stripe gem client methods
+      stripe_client_double = double
+      v2_double = double
+      billing_double = double
+      subscriptions_double = double
+      cadences_double = double
+
+      allow(service).to receive(:stripe_client).and_return(stripe_client_double)
+      allow(stripe_client_double).to receive(:v2).and_return(v2_double)
+      allow(v2_double).to receive(:billing).and_return(billing_double)
+      allow(billing_double).to receive(:pricing_plan_subscriptions).and_return(subscriptions_double)
+      allow(billing_double).to receive(:cadences).and_return(cadences_double)
+
+      allow(subscriptions_double).to receive(:retrieve).with('bpps_sub_123').and_return(subscription_response)
+      allow(cadences_double).to receive(:retrieve).with('cad_123').and_return(cadence_response)
     end
 
     it 'handles subscription activation' do
+      # Mock the subscription provisioning service
+      provisioning_service_double = instance_double(
+        Enterprise::Billing::V2::SubscriptionProvisioningService,
+        provision: { pricing_plan_id: pricing_plan_id, quantity: 1 }
+      )
+      allow(Enterprise::Billing::V2::SubscriptionProvisioningService).to receive(:new)
+        .with(account: account)
+        .and_return(provisioning_service_double)
+
       result = service.perform(event: event)
 
+      expect(provisioning_service_double).to have_received(:provision).with(subscription_id: 'bpps_sub_123')
       expect(result[:pricing_plan_id]).to eq(pricing_plan_id)
     end
   end