pay-lago 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9c2b824c5412e64bb1f0c2efed6c5532265b9ff4978c8368d98d3fc7cba2cc2
-  data.tar.gz: fee44626ac0dcc1082cc688f74d56c74146fa33b3ee001c35ce1b3a3f729991b
+  metadata.gz: '09ba7ef76f7ab46e24c8a7602c22eaba9705551eb7406304ff9b3533fcda48a4'
+  data.tar.gz: a5c1b1cfee26e0afd36431a5e2480807ee03a378bf01829facbf0bcf9dc271de
 SHA512:
-  metadata.gz: 76cefb859ce9d0fed7736cf6a699517efbe36125d15be22334508fdd3daba204b9e283eae982889f2e68c0f35e88b258e3a5fdef856733f18ff6faff748d697c
-  data.tar.gz: 507d1b89c54951857c29f8d728d20ad902bd907943243a6349189197a1a56915a1309434c4efc24e3ee997e0905f0e245420341bfe38f46bd835c41a24073271
+  metadata.gz: 427777503e12fbddfc7b63ec98d945f98695dfb7f3aee42cfed8a3acd9002cf892c2fc0cce52c46b63cfdfda596da479a7862eb706e05b680a0fa81d97d6be80
+  data.tar.gz: c579510c4be89f0e92b30ce057722561c5fed149c9808f77a1518f867dbebde53560de1662885f85f1e7d401313658914653eaf018382b7dae55b40e42cd89b6

data/CHANGELOG.md

@@ -1,2 +1,14 @@
+## [0.1.1] - 2023-12-11
+### Added
+- Scope for Lago payment methods with a given provider.
+- Method to find a Lago payment method by provider, and provider id.
+- Added `pull!` and `push!` methods to billable, and payment method, to better control syncing with Lago.
+- Method to retrieve a payment method on billable.
+
+### Changed
+- Dependency `lago-ruby-client` was upgraded to version 0.52.2.pre.beta.
+- Improved API for adding a payment method to billable.
+- Method `subscription` on subscription can be passed `reload: true` to reload the subscription data.
+
 ## [0.1.0] - 2023-10-23
 - Initial release

data/Gemfile.lock

@@ -11,7 +11,7 @@ PATH
   remote: .
   specs:
     pay-lago (0.1.0)
-      lago-ruby-client (= 0.50.0.pre.beta)
+      lago-ruby-client (= 0.53.0.pre.beta)
       pay (~> 6.8)
 
 GEM
@@ -119,7 +119,7 @@ GEM
       reline (>= 0.3.8)
     json (2.6.3)
     jwt (2.7.1)
-    lago-ruby-client (0.50.0.pre.beta)
+    lago-ruby-client (0.53.0.pre.beta)
       jwt
       openssl
     language_server-protocol (3.17.0.3)

data/README.md

@@ -63,9 +63,6 @@ for direct use of the [Lago API](https://docs.getlago.com/api-reference/intro).
 
 The client instance can be accessed from `Pay::Lago.client`.
 
-#### Please Note:
-For any API calls that would use a GET, the external_id must be [URI encoded](https://ruby-doc.org/current/stdlibs/uri/URI.html#method-c-encode_www_form_component), as the lago-ruby-client gem does not currently do this itself. See [this issue](https://github.com/getlago/lago-ruby-client/issues/136).
-
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
@@ -74,7 +71,7 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/pay-lago. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/[USERNAME]/pay-lago/blob/master/CODE_OF_CONDUCT.md).
+Bug reports and pull requests are welcome on GitHub at https://github.com/demingfactor/pay-lago. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/demingfactor/pay-lago/blob/master/CODE_OF_CONDUCT.md).
 
 ## License
 
@@ -82,4 +79,4 @@ The gem is available as open source under the terms of the [MIT License](https:/
 
 ## Code of Conduct
 
-Everyone interacting in the Pay::Lago project's codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/[USERNAME]/pay-lago/blob/master/CODE_OF_CONDUCT.md).
+Everyone interacting in the Pay::Lago project's codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/demingfactor/pay-lago/blob/master/CODE_OF_CONDUCT.md).

data/app/models/concerns/pay/lago/pay_payment_method_extensions.rb

@@ -0,0 +1,23 @@
+module Pay
+  module Lago
+    module PayPaymentMethodExtensions
+      extend ActiveSupport::Concern
+
+      included do
+        scope :with_provider, ->(provider) { where("pay_payment_methods.data->>'payment_provider' = ?", provider) }
+      end
+
+      def provider
+        data["payment_provider"].to_sym
+      rescue
+        nil
+      end
+
+      class_methods do
+        def find_by_lago_provider_and_id(provider, provider_id)
+          with_provider(provider).find_by_processor_and_id(:lago, provider_id)
+        end
+      end
+    end
+  end
+end

data/docs/4_payment_methods.md

@@ -0,0 +1,40 @@
+# Payment Methods
+
+## Adding a Payment Method to a Customer
+
+Lago uses the default method of payment for a given customer. As such, Payment Methods in this gem actually
+store customer information of the given provider, rather than the payment method itself.
+
+```ruby
+# Adding a Stripe customer as a Payment Method
+customer = Pay::Customer.find(1234)
+customer.add_payment_method(:stripe, "cus_1234")
+```
+
+```ruby
+# Adding a Stripe customer as a Payment Method, but make it not the default.
+customer = Pay::Customer.find(1234)
+customer.add_payment_method(:stripe, "cus_1234", default: false)
+```
+
+```ruby
+# Adding a Stripe customer as a Payment Method, but make it not sync with Lago.
+# NOTE: Syncing with Lago will only work anyways if default is true (which it is by default).
+customer = Pay::Customer.find(1234)
+customer.add_payment_method(:stripe, "cus_1234", sync: false)
+```
+
+## Retrieving a Payment Method
+
+```ruby
+# Retrieve a Payment Method with a given ID.
+customer = Pay::Customer.find(1234)
+customer.get_payment_method(:stripe, "cus_1234")
+```
+
+```ruby
+# If no Payment Method ID is given, it will return either the first Payment Method
+# for the customer with the given provider, or the default Payment Method (if provider matches).
+customer = Pay::Customer.find(1234)
+customer.get_payment_method(:stripe)
+```

data/lib/pay/lago/billable.rb

@@ -39,21 +39,36 @@ module Pay
 
       def customer
         begin
-          lago_customer = Lago.client.customers.get(uri_escape(pay_external_id))
+          lago_customer = Lago.client.customers.get(pay_external_id)
         rescue ::Lago::Api::HttpError
           lago_customer = Lago.client.customers.create(customer_attributes(pay_external_id))
         end
-        pay_customer.update!(processor_id: lago_customer.external_id) unless processor_id == lago_customer.external_id
+        pull!(lago_customer)
         lago_customer
       rescue ::Lago::Api::HttpError => e
         raise Pay::Lago::Error, e
       end
 
-      def update_customer!(**attributes)
+      def update_customer!(attributes = {})
+        attributes = symbolize_recursive(attributes)
         new_attributes = Lago.openstruct_to_h(customer).except(:lago_id)
-        Lago.client.customers.create(
+        lago_customer = Lago.client.customers.create(
           new_attributes.merge(attributes.except(:external_id))
         )
+        pull!(lago_customer)
+        lago_customer
+      end
+
+      def pull!(lago_customer = customer, force: false)
+        if pay_customer.processor_id.present? && pay_customer.processor_id != lago_customer.external_id && !force
+          raise Pay::Lago::Error.new("Error syncing Lago Customer: IDs do not match!")
+        end
+        new_data = Pay::Lago.openstruct_to_h(lago_customer)
+        pay_customer.update!(processor_id: lago_customer.external_id, data: new_data)
+      end
+
+      def push!
+        update_customer!(pay_customer.data)
       end
 
       def charge(amount = nil, addon: nil, options: {})
@@ -91,7 +106,7 @@ module Pay
 
         begin
           subscription = Lago.client.subscriptions.create(attributes)
-        rescue ::Lago::Api::HttpError
+        rescue ::Lago::Api::HttpError => e
           pay_subscription.destroy!
           raise Pay::Lago::Error, e
         end
@@ -100,12 +115,38 @@ module Pay
         Pay::Lago::Subscription.sync(lago_customer.external_id, external_id, object: subscription)
       end
 
-      def add_payment_method(_token = nil, default: true)
-        Pay::Lago::PaymentMethod.sync(pay_customer: pay_customer)
+      def get_payment_method(provider, provider_id = nil)
+        raise Pay::Lago::Error.new("Invalid provider!") unless Pay::Lago::PaymentMethod.valid_provider?(provider)
+        return Pay::PaymentMethod.find_by_lago_provider_and_id(provider, provider_id) if provider_id.present?
+        return pay_customer.default_payment_method if pay_customer.default_payment_method&.provider == provider
+        Pay::PaymentMethod.with_provider(provider).first
+      end
+
+      def add_payment_method(provider, provider_id, default: true, sync: true, options: {})
+        raise Pay::Lago::Error.new("Invalid provider!") unless Pay::Lago::PaymentMethod.valid_provider?(provider)
+
+        payment_method = pay_customer.payment_methods.with_provider(provider).where(processor_id: provider_id).first
+        payment_method ||= Pay::PaymentMethod.create(
+          processor_id: provider_id,
+          type: "card",
+          customer: pay_customer, data: options.merge(payment_provider: provider, provider_customer_id: provider_id)
+        )
+
+        payment_method.make_default! if default
+        payment_method.payment_processor.push! if sync && default
+
+        payment_method
+      end
+
+      def clear_payment_method!
+        data = (pay_customer.data || {}).merge(
+          "billing_configuration": {payment_provider: nil, payment_method_id: nil, provider_customer_id: nil, provider_payment_methods: nil}
+        )
+        update_customer!(data)
       end
 
-      def processor_subscription(subscription_id, options = {})
-        Pay::Lago::Subscription.get_subscription(processor_id, subscription_id)
+      def processor_subscription(subscription_id, **_options)
+        Pay::Lago.client.subscriptions.get(subscription_id)
       end
 
       def trial_end_date(subscription)
@@ -139,9 +180,9 @@ module Pay
         raise Pay::Lago::Error, e
       end
 
-      def uri_escape(uri)
-        return "" unless uri.is_a? String
-        URI.encode_www_form_component uri
+      def symbolize_recursive(hash)
+        return hash unless hash.is_a? Hash
+        hash.map { |key, value| [key.is_a?(String) ? key.to_sym : key, symbolize_recursive(value)] }.to_h
       end
     end
   end

data/lib/pay/lago/engine.rb

@@ -3,7 +3,7 @@ module Pay
     class Engine < ::Rails::Engine
       engine_name "pay_lago"
 
-      initializer "pay_lago.processors" do |app|
+      initializer "pay_lago.processors", after: "pay.processors" do |app|
         ActiveSupport.on_load(:active_record) do
           include Pay::Lago::Attributes
         end
@@ -18,6 +18,7 @@ module Pay
         Pay::Customer.include Pay::Lago::PayCustomerExtensions
         Pay::Charge.include Pay::Lago::PayExtensions
         Pay::Subscription.include Pay::Lago::PayExtensions
+        Pay::PaymentMethod.include Pay::Lago::PayPaymentMethodExtensions
 
         # Prepend Pay::Lago extensions
         Pay::Webhook.prepend Pay::Lago::WebhookExtensions

data/lib/pay/lago/payment_method.rb

@@ -1,23 +1,40 @@
 module Pay
   module Lago
     class PaymentMethod
+      VALID_PROVIDERS = %i[stripe adyen gocardless]
       attr_reader :pay_payment_method
 
       delegate :customer, :processor_id, to: :pay_payment_method
 
-      # Lago doesn't provide PaymentMethod IDs, so we have to lookup via the Customer
-      def self.sync(pay_customer:)
-        payment_method = pay_customer.default_payment_method || pay_customer.build_default_payment_method
-        lago_customer = pay_customer.customer
+      def self.valid_provider?(provider)
+        VALID_PROVIDERS.include?(provider.to_s.to_sym)
+      end
+
+      # Lago doesn't provide PaymentMethod IDs, so we have to lookup via the Customer.
+      # The most recently synced payment method is *always* the default.
+      def self.sync(lago_customer)
+        pay_customer = Pay::Customer.find_by(processor_id: lago_customer.external_id)
+        if pay_customer.blank?
+          Rails.logger.debug "Pay::Customer #{lago_customer.external_id} is not in the database while syncing Payment Method
+          #{lago_customer.provider_customer_id}"
+          return
+        end
+
         payment_data = lago_customer.billing_configuration
+        provider_id = payment_data.provider_customer_id
+
+        payment_method = pay_customer.payment_methods.find_by(processor_id: provider_id)
+        payment_method ||= pay_customer.build_default_payment_method
 
         payment_method_attr = {
-          processor_id: payment_data.provider_customer_id || NanoId.generate,
+          processor_id: provider_id,
           type: "card",
           data: Lago.openstruct_to_h(payment_data)
         }
 
         payment_method.update!(payment_method_attr)
+        payment_method.make_default!
+        
         payment_method
       rescue ::Lago::Api::HttpError => e
         raise Pay::Lago::Error, e
@@ -27,10 +44,31 @@ module Pay
         @pay_payment_method = pay_payment_method
       end
 
+      def pull!
+        return false unless pay_payment_method.default?
+        customer.pull!
+        pay_payment_method.update!(data: customer.data["billing_configuration"])
+      end
+
+      def push!
+        return false unless pay_payment_method.default?
+        update_customer!
+        customer.push!
+      end
+
       def make_default!
+        update_customer!
       end
 
-      def detach
+      def detach!
+        customer.clear_payment_method!
+      end
+
+      private
+
+      def update_customer!
+        data = (customer.data || {}).merge("billing_configuration": pay_payment_method.data)
+        customer.update!(data: data)
       end
     end
   end

data/lib/pay/lago/subscription.rb

@@ -23,7 +23,7 @@ module Pay
         to: :pay_subscription
 
       def self.sync(customer_id, subscription_id, object: nil, try: 0, retries: 1)
-        object ||= get_subscription(customer_id, subscription_id)
+        object ||= Pay::Lago.client.subscriptions.get(subscription_id)
 
         pay_customer = Pay::Customer.find_by(processor: :lago, processor_id: customer_id)
         if pay_customer.blank?
@@ -60,25 +60,18 @@ module Pay
         end
       end
 
-      def self.get_subscription(customer_id, external_id)
-        result = Lago.client.subscriptions.get_all(external_customer_id: customer_id, per_page: 9223372036854775807)
-        result.subscriptions.each { |sub| return sub if sub.external_id == external_id }
-        raise Pay::Lago::Error.new("No subscription could be found.")
-      rescue ::Lago::Api::HttpError => e
-        raise Pay::Lago::Error, e
-      end
-
       def initialize(pay_subscription)
         @pay_subscription = pay_subscription
       end
 
-      def subscription
-        @lago_subscription ||= self.class.get_subscription(pay_subscription.customer.processor_id, processor_id)
+      def subscription(reload: false)
+        @lago_subscription = nil if reload
+        @lago_subscription ||= Pay::Lago.client.subscriptions.get(processor_id)
       end
 
       def cancel(**options)
         customer_id = subscription.external_customer_id
-        response = Lago.client.subscriptions.destroy(URI.encode_www_form_component(processor_id), options: options)
+        response = Lago.client.subscriptions.destroy(processor_id, options: options)
         self.class.sync(customer_id, processor_id, object: response)
       rescue ::Lago::Api::HttpError => e
         raise Pay::Lago::Error, e
@@ -117,7 +110,7 @@ module Pay
         customer_id = subscription.external_customer_id
         attributes = options.merge(
           external_customer_id: customer_id,
-          external_id: processor_id, plan_code: plan
+          external_id: processor_id, plan_code: plan.to_s
         )
         new_subscription = Lago.client.subscriptions.create(attributes)
         self.class.sync(customer_id, processor_id, object: new_subscription)

data/lib/pay/lago/version.rb

@@ -2,6 +2,6 @@
 
 module Pay
   module Lago
-    VERSION = "0.1.0"
+    VERSION = "0.1.1"
   end
 end

data/lib/pay/lago/webhooks/customer_payment_provider_created.rb

@@ -4,13 +4,7 @@ module Pay
       class CustomerPaymentProviderCreated
         def call(event)
           customer = event.customer
-          pay_customer = Pay::Customer.find_by(processor_id: customer.external_id)
-          unless pay_customer.present?
-            Rails.logger.debug "Pay::Customer #{customer.external_id} is not in the database while syncing Payment Information"
-            return false
-          end
-
-          pay_customer.update!(data: Pay::Lago.openstruct_to_h(customer))
+          PaymentMethod.sync(customer)
         end
       end
     end

data/pay-lago.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |spec|
 
   # Uncomment to register a new dependency of your gem
   spec.add_dependency "pay", "~> 6.8"
-  spec.add_dependency "lago-ruby-client", "0.50.0-beta"
+  spec.add_dependency "lago-ruby-client", "0.53.0-beta"
 
   # For more information and examples about making a new gem, check out our
   # guide at: https://bundler.io/guides/creating_gem.html

data/test/dummy/db/migrate/1_create_pay_tables.rb

@@ -0,0 +1,93 @@
+class CreatePayTables < ActiveRecord::Migration[6.0]
+  def change
+    primary_key_type, foreign_key_type = primary_and_foreign_key_types
+
+    create_table :pay_customers, id: primary_key_type do |t|
+      t.belongs_to :owner, polymorphic: true, index: false, type: foreign_key_type
+      t.string :processor, null: false
+      t.string :processor_id
+      t.boolean :default
+      t.public_send Pay::Adapter.json_column_type, :data
+      t.datetime :deleted_at
+      t.string :stripe_account
+      t.timestamps
+    end
+    add_index :pay_customers, [:owner_type, :owner_id, :deleted_at], name: :pay_customer_owner_index, unique: true
+    add_index :pay_customers, [:processor, :processor_id], unique: true
+
+    create_table :pay_merchants, id: primary_key_type do |t|
+      t.belongs_to :owner, polymorphic: true, index: false, type: foreign_key_type
+      t.string :processor, null: false
+      t.string :processor_id
+      t.boolean :default
+      t.public_send Pay::Adapter.json_column_type, :data
+      t.timestamps
+    end
+    add_index :pay_merchants, [:owner_type, :owner_id, :processor]
+
+    create_table :pay_payment_methods, id: primary_key_type do |t|
+      t.belongs_to :customer, foreign_key: {to_table: :pay_customers}, null: false, index: false, type: foreign_key_type
+      t.string :processor_id, null: false
+      t.boolean :default
+      t.string :type
+      t.public_send Pay::Adapter.json_column_type, :data
+      t.timestamps
+    end
+    add_index :pay_payment_methods, [:customer_id, :processor_id], unique: true
+
+    create_table :pay_subscriptions, id: primary_key_type do |t|
+      t.belongs_to :customer, foreign_key: {to_table: :pay_customers}, null: false, index: false, type: foreign_key_type
+      t.string :name, null: false
+      t.string :processor_id, null: false
+      t.string :processor_plan, null: false
+      t.integer :quantity, default: 1, null: false
+      t.string :status, null: false
+      t.datetime :current_period_start
+      t.datetime :current_period_end
+      t.datetime :trial_ends_at
+      t.datetime :ends_at
+      t.boolean :metered
+      t.string :pause_behavior
+      t.datetime :pause_starts_at
+      t.datetime :pause_resumes_at
+      t.decimal :application_fee_percent, precision: 8, scale: 2
+      t.public_send Pay::Adapter.json_column_type, :metadata
+      t.public_send Pay::Adapter.json_column_type, :data
+      t.timestamps
+    end
+    add_index :pay_subscriptions, [:customer_id, :processor_id], unique: true
+    add_index :pay_subscriptions, [:metered]
+    add_index :pay_subscriptions, [:pause_starts_at]
+
+    create_table :pay_charges, id: primary_key_type do |t|
+      t.belongs_to :customer, foreign_key: {to_table: :pay_customers}, null: false, index: false, type: foreign_key_type
+      t.belongs_to :subscription, foreign_key: {to_table: :pay_subscriptions}, null: true, type: foreign_key_type
+      t.string :processor_id, null: false
+      t.integer :amount, null: false
+      t.string :currency
+      t.integer :application_fee_amount
+      t.integer :amount_refunded
+      t.public_send Pay::Adapter.json_column_type, :metadata
+      t.public_send Pay::Adapter.json_column_type, :data
+      t.timestamps
+    end
+    add_index :pay_charges, [:customer_id, :processor_id], unique: true
+
+    create_table :pay_webhooks, id: primary_key_type do |t|
+      t.string :processor
+      t.string :event_type
+      t.public_send Pay::Adapter.json_column_type, :event
+      t.timestamps
+    end
+  end
+
+  private
+
+  def primary_and_foreign_key_types
+    config = Rails.configuration.generators
+    setting = config.options[config.orm][:primary_key_type]
+    primary_key_type = setting || :primary_key
+    foreign_key_type = setting || :bigint
+    [primary_key_type, foreign_key_type]
+  end
+end

data/test/dummy/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.1].define(version: 2022_08_31_153001) do
+ActiveRecord::Schema[7.1].define(version: 2017_02_05_000000) do
   create_table "accounts", force: :cascade do |t|
     t.string "email"
     t.string "merchant_processor"
@@ -18,8 +18,8 @@ ActiveRecord::Schema[7.1].define(version: 2022_08_31_153001) do
   end
 
   create_table "pay_charges", force: :cascade do |t|
-    t.integer "customer_id", null: false
-    t.integer "subscription_id"
+    t.bigint "customer_id", null: false
+    t.bigint "subscription_id"
     t.string "processor_id", null: false
     t.integer "amount", null: false
     t.string "currency"
@@ -40,10 +40,11 @@ ActiveRecord::Schema[7.1].define(version: 2022_08_31_153001) do
     t.string "processor_id"
     t.boolean "default"
     t.json "data"
-    t.datetime "deleted_at"
+    t.datetime "deleted_at", precision: nil
+    t.string "stripe_account"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["owner_type", "owner_id", "deleted_at", "default"], name: "pay_customer_owner_index"
+    t.index ["owner_type", "owner_id", "deleted_at"], name: "pay_customer_owner_index", unique: true
     t.index ["processor", "processor_id"], name: "index_pay_customers_on_processor_and_processor_id", unique: true
   end
 
@@ -60,7 +61,7 @@ ActiveRecord::Schema[7.1].define(version: 2022_08_31_153001) do
   end
 
   create_table "pay_payment_methods", force: :cascade do |t|
-    t.integer "customer_id", null: false
+    t.bigint "customer_id", null: false
     t.string "processor_id", null: false
     t.boolean "default"
     t.string "type"
@@ -71,25 +72,25 @@ ActiveRecord::Schema[7.1].define(version: 2022_08_31_153001) do
   end
 
   create_table "pay_subscriptions", force: :cascade do |t|
-    t.integer "customer_id", null: false
+    t.bigint "customer_id", null: false
     t.string "name", null: false
     t.string "processor_id", null: false
     t.string "processor_plan", null: false
     t.integer "quantity", default: 1, null: false
     t.string "status", null: false
-    t.datetime "current_period_start"
-    t.datetime "current_period_end"
-    t.datetime "trial_ends_at"
-    t.datetime "ends_at"
+    t.datetime "current_period_start", precision: nil
+    t.datetime "current_period_end", precision: nil
+    t.datetime "trial_ends_at", precision: nil
+    t.datetime "ends_at", precision: nil
+    t.boolean "metered"
+    t.string "pause_behavior"
+    t.datetime "pause_starts_at", precision: nil
+    t.datetime "pause_resumes_at", precision: nil
     t.decimal "application_fee_percent", precision: 8, scale: 2
     t.json "metadata"
     t.json "data"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.boolean "metered"
-    t.string "pause_behavior"
-    t.datetime "pause_starts_at"
-    t.datetime "pause_resumes_at"
     t.index ["customer_id", "processor_id"], name: "index_pay_subscriptions_on_customer_id_and_processor_id", unique: true
     t.index ["metered"], name: "index_pay_subscriptions_on_metered"
     t.index ["pause_starts_at"], name: "index_pay_subscriptions_on_pause_starts_at"
@@ -107,7 +108,7 @@ ActiveRecord::Schema[7.1].define(version: 2022_08_31_153001) do
     t.string "email"
     t.string "name"
     t.string "owner_type"
-    t.bigint "owner_id"
+    t.integer "owner_id"
     t.index ["owner_type", "owner_id"], name: "index_teams_on_owner_type_and_owner_id"
   end
 

data/test/fixtures/pay/customers.yml

@@ -5,3 +5,4 @@ lago:
   processor: lago
   processor_id: gid://dummy/Pay::Customer/772847388
   default: true
+  data: {}

data/test/pay/lago/billable_test.rb

@@ -33,7 +33,7 @@ class Pay::Lago::Billable::Test < ActiveSupport::TestCase
 
   test "add payment method to lago" do
     assert_difference "Pay::PaymentMethod.count" do
-      @pay_customer.add_payment_method("x", default: true)
+      @pay_customer.add_payment_method(:stripe, "x", default: true)
     end
 
     payment_method = @pay_customer.default_payment_method

data/test/pay/lago/payment_method_test.rb

@@ -0,0 +1,34 @@
+require "test_helper"
+require "uri"
+
+class Pay::Lago::PaymentMethodTest < ActiveSupport::TestCase
+  setup do
+    @pay_customer = pay_customers(:lago)
+  end
+
+  test "Lago sync returns Pay::PaymentMethod" do
+    lago_customer = OpenStruct.new(
+      external_id: @pay_customer.processor_id,
+      billing_configuration: OpenStruct.new(
+        payment_provider: "stripe",
+        provider_customer_id: "x"
+      )
+    )
+    pay_payment_method = Pay::Lago::PaymentMethod.sync(lago_customer)
+    assert pay_payment_method.is_a?(Pay::PaymentMethod)
+    assert pay_payment_method.processor_id == "x"
+    assert @pay_customer.default_payment_method == pay_payment_method
+  end
+
+  test "PaymentMethod does not push or pull if not default" do
+    pay_payment_method = @pay_customer.add_payment_method(:stripe, "test_1234", default: false, sync: false).payment_processor
+    assert pay_payment_method.push! == false
+    assert pay_payment_method.pull! == false
+  end
+
+  test "PaymentMethod can be made default, updating Lago" do
+    pay_payment_method = @pay_customer.add_payment_method(:stripe, "test_1234_a", sync: true)
+    lago_customer = Pay::Lago.client.customers.get(@pay_customer.pay_external_id)
+    assert lago_customer.billing_configuration.provider_customer_id == pay_payment_method.processor_id
+  end
+end