stripe_model_callbacks 0.1.1 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e8960c2c38ee84c3e3bca2584f510517c5b2e376b644ded3f32e78e9cb8a854e
-  data.tar.gz: caaffd9c4bf3d43a93b6a1f0c1fc849816da7533febd828db507809c8882efcb
+  metadata.gz: 42089c7d894f5caccf7893fe4cb02f580e8a4cfbb6a72ba465894c5ebfb090f4
+  data.tar.gz: 9be3e19bc9f6028312f9449704a79f232b86053961f61770ae36a679e93b183e
 SHA512:
-  metadata.gz: 77eb0b8e66fbeebb8b0dd1034cbd87423ab3632ac7be297704ad22ebc9fda8cdb460a53896d97e183d7c04ec48d440feb6c0b72af41e52d22c3e317295a0e3d9
-  data.tar.gz: 45718f0c45e6b896ee8d2e3127084adb652edb1587d0666deaf8c6c9d7829c1d7bd51498d89b7b0135af7a8c2d45338b736a90bd4e37c347e4c14ccd12d6cf2a
+  metadata.gz: 80831241dc6bebde63eb620b78c4c10cf2701e66867da8339f77ffcad40d8306126c7e84084980679929f7f765e3ba00bc3ddc78a9ba26fda82ba4a9528ff7d0
+  data.tar.gz: dc731f5bed637ed10b1beac0205b1199c5cf3a787d5139f30d38b0ff3d70aa86e0fe8557fe9bffeec55eef147e58b96ccffed3b27a8ab9f23043a7b9076768fe

data/README.md

@@ -27,6 +27,11 @@ Install the migration for Public Activity, which will provide logging:
 rails g public_activity:migration
 ```
 
+You can install the migrations (or update an existing installation) with this command:
+```bash
+rails stripe_model_callbacks:install:migrations
+```
+
 Do something like this in `config/routes.rb`:
 ```ruby
 Rails.application.routes.draw do
@@ -62,6 +67,16 @@ log and more automatically using the ActiveRecord models.
 
 You can use a service like Ultrahook to set up a proxy for your local development machine to test against.
 
+### Report errors
+
+```ruby
+StripeModelCallbacks.configure do |config|
+  config.on_error do |args:, error:|
+    PeakFlowUtils::Notifier.notify(error: error)
+  end
+end
+```
+
 ## Usage
 
 ### Queries with ActiveRecord
@@ -128,7 +143,7 @@ https://github.com/kaspernj/stripe_model_callbacks/tree/master/spec/factories
 
 You can mock Stripe events by using the helper method `mock_stripe_event` by including this helper:
 
-```
+```ruby
 RSpec.configure do |config|
   config.include StripeModelCallbacks::EventMocker
 end
@@ -149,5 +164,11 @@ mock_stripe_event("invoice.created", data: {object: {discount: {"customer": "cus
 ## Contributing
 Contribution directions go here.
 
+# Database schema changes
+  1. Add migration to `/db/migrate`: ```rails g migration AddStripeIdUniqToStripeInvoices```
+  2. Go to directory: `spec/dummy`
+  3. Run ```rails stripe_model_callbacks:install:migrations``` to copy missing migrations
+  4. Run `rails db:migrate` to apply schema changes
+
 ## License
 The gem is available as open source under the terms of the [MIT License](http://opensource.org/licenses/MIT).

data/app/models/stripe_model_callbacks/application_record.rb

@@ -6,7 +6,24 @@ class StripeModelCallbacks::ApplicationRecord < ActiveRecord::Base
 
   attr_writer :stripe_object
 
+  def self.check_object_is_stripe_class(object, allowed = nil)
+    raise "'stripe_class' not defined on #{name}" unless respond_to?(:stripe_class)
+
+    # Ignore general objects
+    return if object.class.name == "Stripe::StripeObject" # rubocop:disable Style/ClassEqualityComparison:
+
+    allowed ||= [stripe_class]
+
+    raise "Expected #{object.class.name} to be a #{allowed.map(&:name).join(", ")}" unless allowed.any? { |stripe_class| object.is_a?(stripe_class) }
+  end
+
+  def check_object_is_stripe_class(object, allowed = nil)
+    self.class.check_object_is_stripe_class(object, allowed)
+  end
+
   def self.create_from_stripe!(object)
+    check_object_is_stripe_class(object)
+
     model = new
     model.stripe_object = object
     model.assign_from_stripe(object)

data/app/services/stripe_model_callbacks/account/external_account/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Account::ExternalAccount::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     bank_account = StripeBankAccount.find_or_initialize_by(stripe_id: object.id)
     bank_account.assign_from_stripe(object)
 

data/app/services/stripe_model_callbacks/attributes_assigner_service.rb

@@ -7,7 +7,7 @@ class StripeModelCallbacks::AttributesAssignerService < ServicePattern::Service
     @stripe_model = stripe_model
   end
 
-  def execute
+  def perform
     attributes.each do |attribute|
       next unless stripe_model.respond_to?(attribute)
 

data/app/services/stripe_model_callbacks/base_service.rb

@@ -1,10 +1,10 @@
 class StripeModelCallbacks::BaseService < ServicePattern::Service
-  def self.reported_execute!(*args, &blk)
+  def self.reported_execute!(*args, **opts, &blk)
     with_exception_notifications do
-      response = execute(*args, &blk)
+      response = execute(*args, **opts, &blk)
       raise response.errors.join(". ") unless response.success?
 
-      return response
+      response
     end
   end
 
@@ -20,7 +20,40 @@ class StripeModelCallbacks::BaseService < ServicePattern::Service
       Rails.logger.error e.backtrace.join("\n")
     end
 
-    ExceptionNotifier.notify_exception(e) if Object.const_defined?("ExceptionNotifier")
+    ExceptionNotifier.notify_exception(e) if Object.const_defined?(:ExceptionNotifier)
+    PeakFlowUtils::Notifier.notify(error: e) if Object.const_defined?(:PeakFlowUtils)
     raise e
   end
+
+  def self.execute_with_advisory_lock!(*args, **opts, &blk)
+    # The difference between the stripe events is about a few milliseconds - with advisory_lock
+    # we will prevent from creating duplicated objects due to race condition.
+    # https://stripe.com/docs/webhooks/best-practices#event-ordering
+    with_exception_notifications do
+      StripeModelCallbacks::ApplicationRecord.with_advisory_lock(advisory_lock_name(*args, **opts)) do
+        response = execute(*args, **opts, &blk)
+        raise response.errors.join(". ") unless response.success?
+
+        response
+      end
+    end
+  end
+
+  def self.advisory_lock_name(event:)
+    stripe_event_data = event.data.object
+
+    ["stripe", stripe_event_data.object, "id", advisory_lock_id(stripe_event_data)].join("-")
+  end
+
+  def self.advisory_lock_id(stripe_event_data)
+    return stripe_event_data.id if stripe_event_data.respond_to?(:id)
+    return stripe_event_data.coupon.id if stripe_event_data.object == "discount"
+    return unless stripe_event_data.respond_to?(:customer)
+
+    if stripe_event_data.customer.is_a?(String)
+      stripe_event_data.customer
+    else
+      stripe_event_data.customer.id
+    end
+  end
 end

data/app/services/stripe_model_callbacks/charge/dispute_updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Charge::DisputeUpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     dispute.assign_from_stripe(object)
 
     if dispute.save

data/app/services/stripe_model_callbacks/charge/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Charge::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     charge.assign_from_stripe(object)
 
     if charge.save

data/app/services/stripe_model_callbacks/configure_service.rb

@@ -5,7 +5,7 @@ class StripeModelCallbacks::ConfigureService < StripeModelCallbacks::BaseEventSe
     @events = events
   end
 
-  def execute
+  def perform # rubocop:disable Metrics/AbcSize
     all_events
     charge_refund_events
     customer_bank_account_events
@@ -22,11 +22,14 @@ class StripeModelCallbacks::ConfigureService < StripeModelCallbacks::BaseEventSe
     recipient_events
     payout_events
     plan_events
+    price_events
     product_events
     review_events
     sku_events
     source_events
     subscription_events
+    subscription_schedule_events
+    tax_rate_events
     transfer_events
 
     succeed!
@@ -37,35 +40,35 @@ private
   def account_external_account_events
     %w[created deleted updated].each do |external_account_event|
       subscribe "account.external_account.#{external_account_event}" do |event|
-        StripeModelCallbacks::Account::ExternalAccount::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Account::ExternalAccount::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
 
   def all_events
     events.all do |event|
-      StripeModelCallbacks::NotifierService.reported_execute!(event: event)
+      StripeModelCallbacks::NotifierService.execute_with_advisory_lock!(event: event)
     end
   end
 
   def charge_events
     %w[captured failed pending refunded updated succeeded].each do |charge_event|
       subscribe "charge.#{charge_event}" do |event|
-        StripeModelCallbacks::Charge::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Charge::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
 
   def charge_refund_events
     subscribe "charge.refund.updated" do |event|
-      StripeModelCallbacks::Refund::UpdatedService.reported_execute!(event: event)
+      StripeModelCallbacks::Refund::UpdatedService.execute_with_advisory_lock!(event: event)
     end
   end
 
   def charge_dispute_events
     %w[closed created funds_reinstated funds_withdrawn updated].each do |charge_event|
       subscribe "charge.dispute.#{charge_event}" do |event|
-        StripeModelCallbacks::Charge::DisputeUpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Charge::DisputeUpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -73,21 +76,21 @@ private
   def coupon_events
     %w[created deleted updated].each do |coupon_event|
       subscribe "coupon.#{coupon_event}" do |event|
-        StripeModelCallbacks::Coupon::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Coupon::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
 
   def customer_bank_account_events
     subscribe "customer.bank_account.deleted" do |event|
-      StripeModelCallbacks::Customer::BankAccount::DeletedService.reported_execute!(event: event)
+      StripeModelCallbacks::Customer::BankAccount::DeletedService.execute_with_advisory_lock!(event: event)
     end
   end
 
   def customer_discount_events
     %w[created deleted updated].each do |customer_event|
       subscribe "customer.discount.#{customer_event}" do |event|
-        StripeModelCallbacks::Customer::DiscountUpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Customer::DiscountUpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -95,7 +98,7 @@ private
   def customer_events
     %w[created deleted updated].each do |customer_event|
       subscribe "customer.#{customer_event}" do |event|
-        StripeModelCallbacks::Customer::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Customer::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -103,7 +106,7 @@ private
   def customer_source_events
     %w[created deleted expiring updated].each do |customer_event|
       subscribe "customer.source.#{customer_event}" do |event|
-        StripeModelCallbacks::Customer::SourceUpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Customer::SourceUpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -112,9 +115,10 @@ private
     # Upcoming event doesnt send an invoice ID. Dunno what to do about it... Disabling for now
     # upcoming
 
-    %w[created payment_failed payment_succeeded sent updated].each do |invoice_event|
+    # https://stripe.com/docs/billing/invoices/overview#invoice-status-transition-endpoints-and-webhooks
+    %w[created deleted marked_uncollectible payment_failed payment_succeeded sent updated voided].each do |invoice_event|
       subscribe "invoice.#{invoice_event}" do |event|
-        StripeModelCallbacks::Invoice::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Invoice::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -122,7 +126,7 @@ private
   def invoice_item_events
     %w[created deleted updated].each do |event_type|
       subscribe "invoiceitem.#{event_type}" do |event|
-        StripeModelCallbacks::InvoiceItem::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::InvoiceItem::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -130,7 +134,7 @@ private
   def order_events
     %w[created updated].each do |order_event|
       subscribe "order.#{order_event}" do |event|
-        StripeModelCallbacks::Order::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Order::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -138,7 +142,7 @@ private
   def payout_events
     %w[canceled created failed paid updated].each do |payout_event|
       subscribe "payout.#{payout_event}" do |event|
-        StripeModelCallbacks::Payout::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Payout::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -146,7 +150,7 @@ private
   def recipient_events
     %w[created deleted updated].each do |recipient_event|
       subscribe "recipient.#{recipient_event}" do |event|
-        StripeModelCallbacks::Recipient::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Recipient::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -154,7 +158,7 @@ private
   def plan_events
     %w[created deleted updated].each do |plan_event|
       subscribe "plan.#{plan_event}" do |event|
-        StripeModelCallbacks::Plan::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Plan::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -162,7 +166,15 @@ private
   def sku_events
     %w[created deleted updated].each do |sku_event|
       subscribe "sku.#{sku_event}" do |event|
-        StripeModelCallbacks::Sku::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Sku::UpdatedService.execute_with_advisory_lock!(event: event)
+      end
+    end
+  end
+
+  def price_events
+    %w[created deleted updated].each do |price_event|
+      subscribe "price.#{price_event}" do |event|
+        StripeModelCallbacks::Price::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -170,7 +182,7 @@ private
   def product_events
     %w[created deleted updated].each do |product_event|
       subscribe "product.#{product_event}" do |event|
-        StripeModelCallbacks::Product::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Product::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -178,7 +190,7 @@ private
   def review_events
     %w[opened closed].each do |review_event|
       subscribe "review.#{review_event}" do |event|
-        StripeModelCallbacks::Review::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Review::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -186,7 +198,7 @@ private
   def source_events
     %w[canceled chargeable failed mandate_notification].each do |source_event|
       subscribe "source.#{source_event}" do |event|
-        StripeModelCallbacks::Source::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Source::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -202,7 +214,23 @@ private
   def subscription_events
     %w[created deleted trial_will_end updated].each do |subscription_event|
       subscribe "customer.subscription.#{subscription_event}" do |event|
-        StripeModelCallbacks::Customer::Subscription::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Customer::Subscription::UpdatedService.execute_with_advisory_lock!(event: event)
+      end
+    end
+  end
+
+  def subscription_schedule_events
+    %w[canceled created updated].each do |subscription_event|
+      subscribe "subscription_schedule.#{subscription_event}" do |event|
+        StripeModelCallbacks::SubscriptionSchedule::UpdatedService.execute_with_advisory_lock!(event: event)
+      end
+    end
+  end
+
+  def tax_rate_events
+    %w[created updated].each do |transfer_event|
+      subscribe "tax_rate.#{transfer_event}" do |event|
+        StripeModelCallbacks::TaxRate::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end
@@ -210,7 +238,7 @@ private
   def transfer_events
     %w[created reversed updated].each do |transfer_event|
       subscribe "transfer.#{transfer_event}" do |event|
-        StripeModelCallbacks::Transfer::UpdatedService.reported_execute!(event: event)
+        StripeModelCallbacks::Transfer::UpdatedService.execute_with_advisory_lock!(event: event)
       end
     end
   end

data/app/services/stripe_model_callbacks/coupon/sync_all.rb

@@ -0,0 +1,9 @@
+class StripeModelCallbacks::Coupon::SyncAll < StripeModelCallbacks::BaseService
+  def perform
+    Stripe::Coupon.list.each do |coupon|
+      StripeModelCallbacks::SyncFromStripe.execute!(stripe_object: coupon)
+    end
+
+    succeed!
+  end
+end

data/app/services/stripe_model_callbacks/coupon/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Coupon::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     coupon.assign_from_stripe(object)
     coupon.deleted_at ||= Time.zone.now if event.type == "coupon.deleted"
 

data/app/services/stripe_model_callbacks/customer/bank_account/deleted_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Customer::BankAccount::DeletedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     bank_account = StripeBankAccount.find_or_initialize_by(stripe_id: object.id)
     bank_account.assign_from_stripe(object)
 

data/app/services/stripe_model_callbacks/customer/deleted_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Customer::DeletedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     customer = StripeCustomer.find_or_initialize_by(stripe_id: object.id)
     customer.assign_from_stripe(object)
     customer.deleted_at = Time.zone.now

data/app/services/stripe_model_callbacks/customer/discount_updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Customer::DiscountUpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     discount.assign_from_stripe(object)
     discount.deleted_at ||= Time.zone.now if event&.type == "customer.discount.deleted"
 

data/app/services/stripe_model_callbacks/customer/source_updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Customer::SourceUpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     source = stripe_class.find_or_initialize_by(stripe_id: object.id)
     source.assign_from_stripe(object)
     source.deleted_at = Time.zone.now if event.type == "customer.source.deleted"

data/app/services/stripe_model_callbacks/customer/subscription/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Customer::Subscription::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     subscription.assign_from_stripe(object)
     subscription.deleted_at ||= Time.zone.now if event.type == "customer.subscription.deleted"
 

data/app/services/stripe_model_callbacks/customer/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Customer::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     customer = StripeCustomer.find_or_initialize_by(stripe_id: object.id)
     customer.assign_from_stripe(object)
     customer.deleted_at ||= Time.zone.now if event.type == "customer.deleted"

data/app/services/stripe_model_callbacks/event_mocker_service.rb

@@ -22,7 +22,7 @@ private
   end
 
   def fixture_path
-    @fixture_path ||= "#{File.dirname(__FILE__)}/../../../spec/fixtures/stripe_events/#{first_part}/#{name}.json"
+    @fixture_path ||= "#{File.dirname(__FILE__)}/../../../lib/stripe_model_callbacks/fixtures/stripe_events/#{first_part}/#{name}.json"
   end
 
   def payload

data/app/services/stripe_model_callbacks/invoice/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Invoice::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     invoice.assign_from_stripe(object)
 
     if invoice.save

data/app/services/stripe_model_callbacks/invoice_item/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::InvoiceItem::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     invoice_item = ::StripeInvoiceItem.find_or_initialize_by(stripe_id: object.id)
     invoice_item.assign_from_stripe(object)
     invoice_item.deleted_at = Time.zone.now if event.type == "invoiceitem.deleted"

data/app/services/stripe_model_callbacks/notifier_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::NotifierService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     Rails.logger.info "New Stripe event: #{event.type}"
     succeed!
   end

data/app/services/stripe_model_callbacks/order/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Order::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     order.assign_from_stripe(object)
 
     if order.save

data/app/services/stripe_model_callbacks/payout/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Payout::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     payout.assign_from_stripe(object)
 
     if payout.save

data/app/services/stripe_model_callbacks/plan/sync_all.rb

@@ -0,0 +1,9 @@
+class StripeModelCallbacks::Plan::SyncAll < StripeModelCallbacks::BaseService
+  def perform
+    Stripe::Plan.list.each do |plan|
+      StripeModelCallbacks::SyncFromStripe.execute!(stripe_object: plan)
+    end
+
+    succeed!
+  end
+end

data/app/services/stripe_model_callbacks/plan/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Plan::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     plan = StripePlan.find_or_initialize_by(stripe_id: object.id)
     plan.assign_from_stripe(object)
     plan.deleted_at ||= Time.zone.now if event.type == "plan.deleted"

data/app/services/stripe_model_callbacks/price/updated_service.rb

@@ -0,0 +1,14 @@
+class StripeModelCallbacks::Price::UpdatedService < StripeModelCallbacks::BaseEventService
+  def perform
+    price = StripePrice.find_or_initialize_by(stripe_id: object.id)
+    price.assign_from_stripe(object)
+    price.deleted_at ||= Time.zone.now if event.type == "price.deleted"
+
+    if price.save
+      price.create_activity :deleted if event.type == "price.deleted"
+      succeed!
+    else
+      fail! price.errors.full_messages
+    end
+  end
+end

data/app/services/stripe_model_callbacks/product/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Product::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     product = StripeProduct.find_or_initialize_by(stripe_id: object.id)
     product.assign_from_stripe(object)
     product.deleted_at ||= Time.zone.now if event.type == "product.deleted"

data/app/services/stripe_model_callbacks/recipient/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Recipient::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     recipient = StripeRecipient.find_or_initialize_by(stripe_id: object.id)
     recipient.assign_from_stripe(object)
     recipient.deleted_at ||= Time.zone.now if event.type == "recipient.deleted"

data/app/services/stripe_model_callbacks/refund/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Refund::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     refund = StripeRefund.find_or_initialize_by(stripe_id: object.id)
     refund.assign_from_stripe(object)
 

data/app/services/stripe_model_callbacks/review/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Review::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     review = StripeReview.find_or_initialize_by(stripe_id: object.id)
     review.assign_from_stripe(object)
 

data/app/services/stripe_model_callbacks/sku/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Sku::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     sku = StripeSku.find_or_initialize_by(stripe_id: object.id)
     sku.assign_from_stripe(object)
     sku.deleted_at ||= Time.zone.now if event.type == "sku.deleted"

data/app/services/stripe_model_callbacks/source/updated_service.rb

@@ -1,5 +1,5 @@
 class StripeModelCallbacks::Source::UpdatedService < StripeModelCallbacks::BaseEventService
-  def execute
+  def perform
     source.assign_from_stripe(object)
 
     if source.save

data/app/services/stripe_model_callbacks/subscription/state_checker_service.rb

@@ -6,13 +6,13 @@ class StripeModelCallbacks::Subscription::StateCheckerService < StripeModelCallb
     @state = state.to_s
   end
 
-  def execute
+  def perform
     if state.is_a?(Array)
       state.each do |state_i|
         response = Subscription::StateCheckerService.execute!(allowed: allowed, state: state_i)
         return response unless response.success?
       end
-    elsif !allowed.include?(state)
+    elsif allowed.exclude?(state)
       return fail! ["Not allowed: #{state}"]
     end