stripe_model_callbacks 0.1.7 → 0.1.8

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

@@ -4,7 +4,7 @@ class StripeModelCallbacks::Review::UpdatedService < StripeModelCallbacks::BaseE
     review.assign_from_stripe(object)
 
     if review.save
-      review.create_activity :closed if event.type == "review.closed"
+      review.create_audit!(action: :closed) if event.type == "review.closed"
       succeed!
     else
       fail! review.errors.full_messages

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

@@ -0,0 +1,8 @@
+class StripeModelCallbacks::SetupIntent::UpdatedService < StripeModelCallbacks::BaseEventService
+  def perform
+    refund = StripeSetupIntent.find_or_initialize_by(stripe_id: object.id)
+    refund.assign_from_stripe(object)
+    save_models_or_fail(refund)
+    succeed!
+  end
+end

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

@@ -5,7 +5,7 @@ class StripeModelCallbacks::Sku::UpdatedService < StripeModelCallbacks::BaseEven
     sku.deleted_at ||= Time.zone.now if event.type == "sku.deleted"
 
     if sku.save
-      sku.create_activity :deleted if event.type == "sku.deleted"
+      sku.create_audit!(action: :deleted) if event.type == "sku.deleted"
       succeed!
     else
       fail! sku.errors.full_messages

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

@@ -15,13 +15,13 @@ private
   def create_activity
     case event.type
     when "source.canceled"
-      source.create_activity :canceled
+      source.create_audit!(action: :canceled)
     when "source.chargeable"
-      source.create_activity :chargeable
+      source.create_audit!(action: :chargeable)
     when "source.failed"
-      source.create_activity :failed
+      source.create_audit!(action: :failed)
     when "source.mandate_notification"
-      source.create_activity :mandate_notification
+      source.create_audit!(action: :mandate_notification)
     end
   end
 

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

@@ -9,7 +9,7 @@ class StripeModelCallbacks::Subscription::StateCheckerService < StripeModelCallb
   def perform
     if state.is_a?(Array)
       state.each do |state_i|
-        response = Subscription::StateCheckerService.execute!(allowed: allowed, state: state_i)
+        response = Subscription::StateCheckerService.execute!(allowed:, state: state_i)
         return response unless response.success?
       end
     elsif allowed.exclude?(state)

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

@@ -22,7 +22,7 @@ private
   def create_activity
     return unless tracked_activities[event.type]
 
-    subscription_schedule.create_activity(tracked_activities[event.type])
+    subscription_schedule.create_audit!(action: tracked_activities[event.type])
   end
 
   def set_cancellation_date

data/app/services/stripe_model_callbacks/sync_everything.rb

@@ -1,8 +1,23 @@
 class StripeModelCallbacks::SyncEverything < StripeModelCallbacks::BaseService
+  def self.stripe_classes
+    [Stripe::Customer, Stripe::Coupon, Stripe::Invoice, Stripe::Plan, Stripe::PaymentIntent, Stripe::Subscription]
+  end
+
   def perform
-    StripeModelCallbacks::Coupon::SyncAll.execute!
-    StripeModelCallbacks::Plan::SyncAll.execute!
+    StripeModelCallbacks::SyncEverything.stripe_classes.each do |stripe_class|
+      stripe_class.list.each do |stripe_object|
+        StripeModelCallbacks::SyncFromStripe.execute!(stripe_object:)
+
+        sync_stripe_objects(Stripe::PaymentMethod.list(customer: stripe_object.id)) if stripe_class == Stripe::Customer
+      end
+    end
 
     succeed!
   end
+
+  def sync_stripe_objects(stripe_objects)
+    stripe_objects.each do |stripe_object|
+      StripeModelCallbacks::SyncFromStripe.execute!(stripe_object:)
+    end
+  end
 end

data/app/services/stripe_model_callbacks/sync_from_stripe.rb

@@ -16,7 +16,7 @@ class StripeModelCallbacks::SyncFromStripe < StripeModelCallbacks::BaseService
       model = model_class.create_from_stripe!(stripe_object)
     end
 
-    succeed!(model: model)
+    succeed!(model:)
   end
 
   def model_class_name

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

@@ -15,13 +15,13 @@ private
   def create_activity
     case event.type
     when "tax_rate.created"
-      tax_rate.create_activity :created
+      tax_rate.create_audit!(action: :created)
     when "tax_rate.updated"
-      tax_rate.create_activity :updated
+      tax_rate.create_audit!(action: :updated)
     end
   end
 
   def tax_rate
-    @tax_rate ||= StripeTaxRate.find_or_initialize_by(stripe_id: object.id)
+    @tax_rate ||= StripeTaxRate.find_or_initialize_by(stripe_id: object.is_a?(String) ? object : object.id)
   end
 end

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

@@ -3,7 +3,7 @@ class StripeModelCallbacks::Transfer::UpdatedService < StripeModelCallbacks::Bas
     transfer.assign_from_stripe(object)
 
     if transfer.save
-      transfer.create_activity :reversed if event.type == "transfer.reversed"
+      transfer.create_audit!(action: :reversed) if event.type == "transfer.reversed"
       succeed!
     else
       fail! transfer.errors.full_messages

data/config/rails_best_practices.yml

@@ -28,6 +28,7 @@ RemoveUnusedMethodsInHelpersCheck: { except_methods: [] }
 RemoveUnusedMethodsInModelsCheck: {
   except_methods: [
     "StripeModelCallbacks::ApplicationRecord#create_on_stripe!",
+    "StripeModelCallbacks::ApplicationRecord#inherited",
     "StripeModelCallbacks::ApplicationRecord#reload!",
     "StripeModelCallbacks::ApplicationRecord#update_on_stripe!",
     "StripeModelCallbacks::ApplicationRecord#destroy_on_stripe!"

data/db/migrate/20180206151132_change_activities_trackable_id_to_string.rb

@@ -1,5 +1,5 @@
 class ChangeActivitiesTrackableIdToString < ActiveRecord::Migration[5.1]
   def change
-    change_column :activities, :trackable_id, :string  # rubocop:disable Rails/ReversibleMigration
+    change_column :activities, :trackable_id, :string # rubocop:disable Rails/ReversibleMigration
   end
 end

data/db/migrate/20180208091647_change_stripe_subscription_items_quantity_to_integer.rb

@@ -1,5 +1,5 @@
 class ChangeStripeSubscriptionItemsQuantityToInteger < ActiveRecord::Migration[5.1]
   def change
-    change_column :stripe_subscription_items, :quantity, :integer  # rubocop:disable Rails/ReversibleMigration
+    change_column :stripe_subscription_items, :quantity, :integer # rubocop:disable Rails/ReversibleMigration
   end
 end

data/db/migrate/20181219121712_add_new_primary_id_and_rename_old.rb

@@ -17,7 +17,7 @@ class AddNewPrimaryIdAndRenameOld < ActiveRecord::Migration[5.2]
       change_column_null table_name, :stripe_id, false
       remove_column table_name, :id
 
-      add_column table_name, :id, :primary_key
+      add_column table_name, :id, :primary_key # rubocop:disable Rails/DangerousColumnNames
     end
   end
 
@@ -28,7 +28,7 @@ class AddNewPrimaryIdAndRenameOld < ActiveRecord::Migration[5.2]
       next unless column_exists?(table_name, :stripe_id)
 
       remove_column table_name, :id
-      add_column table_name, :id, :string, primary_key: true
+      add_column table_name, :id, :string, primary_key: true # rubocop:disable Rails/DangerousColumnNames
       execute "UPDATE #{table_name} SET id = stripe_id"
       change_column_null table_name, :id, false
       remove_column table_name, :stripe_id

data/db/migrate/20230210081405_create_stripe_setup_intents.rb

@@ -0,0 +1,28 @@
+class CreateStripeSetupIntents < ActiveRecord::Migration[7.0]
+  def change
+    create_table :stripe_setup_intents do |t|
+      t.string :stripe_id, index: {unique: true}, null: false
+      t.string :application
+      t.string :cancellation_reason
+      t.string :client_secret
+      t.datetime :created
+      t.string :customer, index: true
+      t.string :description
+      t.json :flow_directions
+      t.string :last_setup_error
+      t.string :latest_attempt
+      t.boolean :livemode
+      t.json :mandate
+      t.json :metadata
+      t.string :next_action
+      t.string :on_behalf_of
+      t.json :payment_method
+      t.json :payment_method_options
+      t.json :payment_method_types
+      t.string :single_use_mandate
+      t.string :status
+      t.string :usage
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20230331182902_add_amount_captured_to_stripe_charges.rb

@@ -0,0 +1,6 @@
+class AddAmountCapturedToStripeCharges < ActiveRecord::Migration[7.0]
+  def change
+    add_column :stripe_charges, :amount_captured_cents, :integer
+    add_column :stripe_charges, :amount_captured_currency, :string
+  end
+end

data/db/migrate/20230420092307_create_stripe_payment_methods.rb

@@ -0,0 +1,15 @@
+class CreateStripePaymentMethods < ActiveRecord::Migration[7.0]
+  def change
+    create_table :stripe_payment_methods do |t|
+      t.string :stripe_id, index: {unique: true}, null: false
+      t.json :billing_details
+      t.json :card
+      t.json :metadata
+      t.string :customer
+      t.string :stripe_type
+      t.boolean :livemode
+      t.integer :created
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20230421072509_create_stripe_payment_intents.rb

@@ -0,0 +1,44 @@
+class CreateStripePaymentIntents < ActiveRecord::Migration[7.0]
+  def change # rubocop:disable Metrics/AbcSize
+    create_table :stripe_payment_intents do |t|
+      t.string :stripe_id, index: {unique: true}, null: false
+      t.integer :amount
+      t.integer :amount_capturable
+      t.json :amount_details
+      t.integer :amount_received
+      t.string :application
+      t.integer :application_fee_amount
+      t.json :automatic_payment_methods
+      t.integer :canceled_at
+      t.string :cancellation_reason
+      t.string :capture_method
+      t.string :client_secret
+      t.string :confirmation_method
+      t.integer :created
+      t.string :currency
+      t.string :customer, index: true
+      t.text :description
+      t.string :invoice
+      t.json :last_payment_error
+      t.string :latest_charge, index: true
+      t.boolean :livemode
+      t.json :metadata
+      t.json :next_action
+      t.string :on_behalf_of, index: true
+      t.string :payment_method, index: true
+      t.json :payment_method_options
+      t.json :payment_method_types
+      t.json :processing
+      t.string :receipt_email
+      t.string :review, index: true
+      t.string :setup_future_usage
+      t.json :shipping
+      t.string :statement_descriptor
+      t.string :statement_descriptor_suffix
+      t.string :status
+      t.json :transfer_data
+      t.string :transfer_group
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20230422074329_add_payment_intent_to_stripe_charges.rb

@@ -0,0 +1,6 @@
+class AddPaymentIntentToStripeCharges < ActiveRecord::Migration[7.0]
+  def change
+    add_column :stripe_charges, :payment_intent, :string
+    add_index :stripe_charges, :payment_intent
+  end
+end

data/db/migrate/20230422175529_add_payment_intent_to_stripe_refunds.rb

@@ -0,0 +1,6 @@
+class AddPaymentIntentToStripeRefunds < ActiveRecord::Migration[7.0]
+  def change
+    add_column :stripe_refunds, :payment_intent, :string
+    add_index :stripe_refunds, :payment_intent
+  end
+end

data/db/migrate/20230422180602_change_stripe_refunds_stripe_charge_id_to_nullable.rb

@@ -0,0 +1,5 @@
+class ChangeStripeRefundsStripeChargeIdToNullable < ActiveRecord::Migration[7.0]
+  def change
+    change_column_null :stripe_refunds, :stripe_charge_id, true
+  end
+end

data/db/migrate/20230518072538_change_stripe_setup_intents_payment_method_to_string.rb

@@ -0,0 +1,7 @@
+class ChangeStripeSetupIntentsPaymentMethodToString < ActiveRecord::Migration[7.0]
+  def change
+    rename_column :stripe_setup_intents, :payment_method, :payment_method_old
+    add_column :stripe_setup_intents, :payment_method, :string
+    add_index :stripe_setup_intents, :payment_method
+  end
+end

data/lib/stripe_model_callbacks/configuration.rb

@@ -15,7 +15,7 @@ class StripeModelCallbacks::Configuration
     yield
   rescue => e # rubocop:disable Style/RescueStandardError
     @on_error_callbacks.each do |callback|
-      callback.call(args: args, error: e)
+      callback.call(args:, error: e)
     end
 
     raise e

data/lib/stripe_model_callbacks/event_mocker.rb

@@ -1,5 +1,5 @@
 module StripeModelCallbacks::EventMocker
   def mock_stripe_event(name, args = nil)
-    StripeModelCallbacks::EventMockerService.new(name: name, scope: self, args: args)
+    StripeModelCallbacks::EventMockerService.execute!(name:, scope: self, args:)
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_charges.rb

@@ -7,5 +7,15 @@ FactoryBot.define do
     livemode { false }
     paid { false }
     refunded { false }
+
+    trait :with_conditional_stripe_mock do
+      after :create do |stripe_charge|
+        stripe_charge.create_stripe_mock! if StripeMock.instance
+      end
+    end
+
+    trait :with_stripe_mock do
+      after :create, &:create_stripe_mock!
+    end
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_coupons.rb

@@ -2,20 +2,20 @@ FactoryBot.define do
   factory :stripe_coupon do
     sequence(:stripe_id) { |n| "stripe-coupon-#{n}" }
 
-    trait :with_stripe_mock do
+    trait :with_conditional_stripe_mock do
       duration { "repeating" }
       duration_in_months { 1 }
 
       after :create do |stripe_coupon|
-        mock_coupon = Stripe::Coupon.create(
-          duration: stripe_coupon.duration,
-          duration_in_months: stripe_coupon.duration_in_months,
-          id: stripe_coupon.stripe_id,
-          percent_off: stripe_coupon.percent_off
-        )
-        stripe_coupon.assign_from_stripe(mock_coupon)
-        stripe_coupon.save!
+        stripe_coupon.create_stripe_mock! if StripeMock.instance
       end
     end
+
+    trait :with_stripe_mock do
+      duration { "repeating" }
+      duration_in_months { 1 }
+
+      after :create, &:create_stripe_mock!
+    end
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_customers.rb

@@ -6,12 +6,14 @@ FactoryBot.define do
     delinquent { false }
     livemode { false }
 
-    trait :with_stripe_mock do
+    trait :with_conditional_stripe_mock do
       after :create do |stripe_customer|
-        mock_customer = Stripe::Customer.create(source: StripeMock.create_test_helper.generate_card_token)
-        stripe_customer.assign_from_stripe(mock_customer)
-        stripe_customer.save!
+        stripe_customer.create_stripe_mock! if StripeMock.instance
       end
     end
+
+    trait :with_stripe_mock do
+      after :create, &:create_stripe_mock!
+    end
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_payment_intents.rb

@@ -0,0 +1,17 @@
+FactoryBot.define do
+  factory :stripe_payment_intent do
+    amount { 125_00 }
+    currency { "usd" }
+    sequence(:stripe_id) { |n| "stripe-payment-intent-#{n}" }
+
+    trait :with_conditional_stripe_mock do
+      after :create do |stripe_payment_intent|
+        stripe_payment_intent.create_stripe_mock! if StripeMock.instance
+      end
+    end
+
+    trait :with_stripe_mock do
+      after :create, &:create_stripe_mock!
+    end
+  end
+end

data/lib/stripe_model_callbacks/factories/stripe_payment_methods.rb

@@ -0,0 +1,15 @@
+FactoryBot.define do
+  factory :stripe_payment_method do
+    sequence(:stripe_id) { |n| "stripe-payment-method-#{n}" }
+
+    trait :with_conditional_stripe_mock do
+      after :create do |stripe_payment_method|
+        stripe_payment_method.create_stripe_mock! if StripeMock.instance
+      end
+    end
+
+    trait :with_stripe_mock do
+      after :create, &:create_stripe_mock!
+    end
+  end
+end

data/lib/stripe_model_callbacks/factories/stripe_plans.rb

@@ -9,21 +9,18 @@ FactoryBot.define do
     livemode { false }
     stripe_product
 
-    trait :with_stripe_mock do
-      association :stripe_product, factory: [:stripe_product, :with_stripe_mock]
+    trait :with_conditional_stripe_mock do
+      association :stripe_product, factory: [:stripe_product, :with_conditional_stripe_mock]
 
       after :create do |stripe_plan|
-        mock_plan = Stripe::Plan.create(
-          id: stripe_plan.stripe_id,
-          amount: stripe_plan.amount_cents,
-          currency: stripe_plan.currency,
-          interval: stripe_plan.interval,
-          interval_count: stripe_plan.interval_count,
-          product: stripe_plan.stripe_product.stripe_id
-        )
-        stripe_plan.assign_from_stripe(mock_plan)
-        stripe_plan.save!
+        stripe_plan.create_stripe_mock! if StripeMock.instance
       end
     end
+
+    trait :with_stripe_mock do
+      association :stripe_product, factory: [:stripe_product, :with_stripe_mock]
+
+      after :create, &:create_stripe_mock!
+    end
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_products.rb

@@ -3,16 +3,14 @@ FactoryBot.define do
     sequence(:name) { |n| "Stripe product #{n}" }
     sequence(:stripe_id) { |n| "stripe-product-#{n}" }
 
-    trait :with_stripe_mock do
+    trait :with_conditional_stripe_mock do
       after :create do |stripe_product|
-        mock_product = Stripe::Product.create(
-          id: stripe_product.stripe_id,
-          name: stripe_product.name,
-          type: "service"
-        )
-        stripe_product.assign_from_stripe(mock_product)
-        stripe_product.save!
+        stripe_product.create_stripe_mock! if StripeMock.instance
       end
     end
+
+    trait :with_stripe_mock do
+      after :create, &:create_stripe_mock!
+    end
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_setup_intents.rb

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :stripe_setup_intent do
+    sequence(:stripe_id) { |n| "stripe-setup-intent-#{n}" }
+    stripe_customer
+    livemode { false }
+  end
+end

data/lib/stripe_model_callbacks/factories/stripe_subscription_items.rb

@@ -5,15 +5,14 @@ FactoryBot.define do
     stripe_plan
     quantity { 1 }
 
-    trait :with_stripe_mock do
+    trait :with_conditional_stripe_mock do
       after :build do |item|
-        mock_item = Stripe::SubscriptionItem.create(
-          subscription: item.stripe_subscription.stripe_id,
-          plan: item.stripe_plan.stripe_id,
-          quantity: item.quantity
-        )
-        item.assign_from_stripe(mock_item)
+        item.create_stripe_mock! if StripeMock.instance
       end
     end
+
+    trait :with_stripe_mock do
+      after :build, &:create_stripe_mock!
+    end
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_subscriptions.rb

@@ -19,22 +19,20 @@ FactoryBot.define do
       cancel_at_period_end { true }
     end
 
-    trait :with_stripe_mock do
-      association :stripe_customer, factory: [:stripe_customer, :with_stripe_mock]
-      association :stripe_plan, factory: [:stripe_plan, :with_stripe_mock]
+    trait :with_conditional_stripe_mock do
+      association :stripe_customer, factory: [:stripe_customer, :with_conditional_stripe_mock]
+      association :stripe_plan, factory: [:stripe_plan, :with_conditional_stripe_mock]
 
       after :create do |stripe_subscription|
-        cancel_at_period_end = stripe_subscription.cancel_at_period_end?
+        stripe_subscription.create_stripe_mock! if StripeMock.instance
+      end
+    end
 
-        mock_subscription = Stripe::Subscription.create(
-          customer: stripe_subscription.stripe_customer.stripe_id,
-          plan: stripe_subscription.stripe_plan.stripe_id
-        )
-        stripe_subscription.assign_from_stripe(mock_subscription)
-        stripe_subscription.save!
+    trait :with_stripe_mock do
+      association :stripe_customer, factory: [:stripe_customer, :with_stripe_mock]
+      association :stripe_plan, factory: [:stripe_plan, :with_stripe_mock]
 
-        stripe_subscription.cancel!(at_period_end: true) if cancel_at_period_end
-      end
+      after :create, &:create_stripe_mock!
     end
   end
 end

data/lib/stripe_model_callbacks/factories/stripe_tax_rates.rb

@@ -1,5 +1,15 @@
 FactoryBot.define do
   factory :stripe_tax_rate do
     sequence(:stripe_id) { |n| "stripe-tax-rate-#{n}" }
+
+    trait :with_conditional_stripe_mock do
+      after :create do |stripe_tax_rate|
+        stripe_tax_rate.create_stripe_mock! if StripeMock.instance
+      end
+    end
+
+    trait :with_stripe_mock do
+      after :create, &:create_stripe_mock!
+    end
   end
 end

data/lib/stripe_model_callbacks/fixtures/stripe_events/charge/charge.refunded.json

@@ -12,6 +12,7 @@
       "id": "ch_00000000000000",
       "object": "charge",
       "amount": 100,
+      "amount_captured": 100,
       "amount_refunded": 100,
       "application": null,
       "application_fee": null,

data/lib/stripe_model_callbacks/fixtures/stripe_events/payment_intent/payment_intent.amount_capturable_updated.json

@@ -0,0 +1,66 @@
+{
+  "id": "evt_1MxwhGIICJxvfdbRjLpH5qNb",
+  "object": "event",
+  "api_version": "2022-11-15",
+  "created": 1681755758,
+  "data": {
+    "object": {
+      "id": "pi_1JGpBw4Lf1hyuG3tSrdK3llw",
+      "object": "payment_intent",
+      "amount": 1000,
+      "amount_capturable": 0,
+      "amount_details": {
+        "tip": {}
+      },
+      "amount_received": 0,
+      "application": null,
+      "application_fee_amount": null,
+      "automatic_payment_methods": null,
+      "canceled_at": null,
+      "cancellation_reason": null,
+      "capture_method": "automatic",
+      "client_secret": "pi_1JGpBw4Lf1hyuG3tSrdK3llw_secret_LS2rgASDfUetReE3XLXayygdt",
+      "confirmation_method": "automatic",
+      "created": 1627149796,
+      "currency": "usd",
+      "customer": "cus_NiCY7UI5u0pbJH",
+      "description": "Created by stripe.com/docs demo",
+      "invoice": null,
+      "last_payment_error": null,
+      "latest_charge": null,
+      "livemode": false,
+      "metadata": {},
+      "next_action": null,
+      "on_behalf_of": null,
+      "payment_method": null,
+      "payment_method_options": {
+        "card": {
+          "installments": null,
+          "mandate_options": null,
+          "network": null,
+          "request_three_d_secure": "automatic"
+        }
+      },
+      "payment_method_types": [
+        "card"
+      ],
+      "processing": null,
+      "receipt_email": null,
+      "review": null,
+      "setup_future_usage": null,
+      "shipping": null,
+      "statement_descriptor": null,
+      "statement_descriptor_suffix": null,
+      "status": "requires_payment_method",
+      "transfer_data": null,
+      "transfer_group": null
+    }
+  },
+  "livemode": true,
+  "pending_webhooks": 1,
+  "request": {
+    "id": null,
+    "idempotency_key": "setatt_1MxwglIICJxvfdbR7jPCO6O3-src_1MxwgmIICJxvfdbRoaqKyWHH"
+  },
+  "type": "payment_intent.amount_capturable_updated"
+}