stripe 3.0.3 → 3.1.0

data/test/stripe/dispute_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class DisputeTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:dispute)
-
     should "be listable" do
       disputes = Stripe::Dispute.list
       assert_requested :get, "#{Stripe.api_base}/v1/disputes"
@@ -12,30 +10,30 @@ module Stripe
     end
 
     should "be retrievable" do
-      dispute = Stripe::Dispute.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}"
+      dispute = Stripe::Dispute.retrieve("dp_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/disputes/dp_123"
       assert dispute.kind_of?(Stripe::Dispute)
     end
 
     should "be saveable" do
-      dispute = Stripe::Dispute.retrieve(FIXTURE[:id])
+      dispute = Stripe::Dispute.retrieve("dp_123")
       dispute.metadata['key'] = 'value'
       dispute.save
-      assert_requested :post, "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/disputes/#{dispute.id}"
     end
 
     should "be updateable" do
-      dispute = Stripe::Dispute.update(FIXTURE[:id], metadata: { key: 'value' })
-      assert_requested :post, "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}"
+      dispute = Stripe::Dispute.update("dp_123", metadata: { key: 'value' })
+      assert_requested :post, "#{Stripe.api_base}/v1/disputes/dp_123"
       assert dispute.kind_of?(Stripe::Dispute)
     end
 
     context "#close" do
       should "be closeable" do
-        dispute = Stripe::Dispute.retrieve(FIXTURE[:id])
+        dispute = Stripe::Dispute.retrieve("dp_123")
         dispute.close
         assert_requested :post,
-          "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}/close"
+          "#{Stripe.api_base}/v1/disputes/#{dispute.id}/close"
       end
     end
   end

data/test/stripe/file_upload_test.rb

@@ -8,7 +8,7 @@ module Stripe
     # a separate spec for it though, so it's high effort with low reward for
     # the time being.
     FIXTURE = {
-      id: "fil_15ABpV2eZvKYlo2C7vu7XS5l",
+      id: "file_123",
       object: "file_upload",
     }.freeze
 
@@ -26,10 +26,10 @@ module Stripe
     end
 
     should "be retrievable" do
-      stub_request(:get, "#{Stripe.uploads_base}/v1/files/#{FIXTURE[:id]}").
+      stub_request(:get, "#{Stripe.uploads_base}/v1/files/file_123").
         to_return(body: JSON.generate(FIXTURE))
 
-      file = Stripe::FileUpload.retrieve(FIXTURE[:id])
+      file = Stripe::FileUpload.retrieve("file_123")
       assert file.kind_of?(Stripe::FileUpload)
     end
 

data/test/stripe/invoice_item_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class InvoiceItemTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:invoice_item)
-
     should "be listable" do
       invoiceitems = Stripe::InvoiceItem.list
       assert_requested :get, "#{Stripe.api_base}/v1/invoiceitems"
@@ -12,9 +10,9 @@ module Stripe
     end
 
     should "be retrievable" do
-      item = Stripe::InvoiceItem.retrieve(FIXTURE[:id])
+      item = Stripe::InvoiceItem.retrieve("ii_123")
       assert_requested :get,
-        "#{Stripe.api_base}/v1/invoiceitems/#{FIXTURE[:id]}"
+        "#{Stripe.api_base}/v1/invoiceitems/ii_123"
       assert item.kind_of?(Stripe::InvoiceItem)
     end
 
@@ -22,7 +20,7 @@ module Stripe
       item = Stripe::InvoiceItem.create(
         amount: 100,
         currency: "USD",
-        customer: API_FIXTURES[:customer][:id]
+        customer: "cus_123"
       )
       assert_requested :post,
         "#{Stripe.api_base}/v1/invoiceitems"
@@ -30,25 +28,25 @@ module Stripe
     end
 
     should "be saveable" do
-      item = Stripe::InvoiceItem.retrieve(FIXTURE[:id])
+      item = Stripe::InvoiceItem.retrieve("ii_123")
       item.metadata['key'] = 'value'
       item.save
       assert_requested :post,
-        "#{Stripe.api_base}/v1/invoiceitems/#{FIXTURE[:id]}"
+        "#{Stripe.api_base}/v1/invoiceitems/#{item.id}"
     end
 
     should "be updateable" do
-      item = Stripe::InvoiceItem.update(FIXTURE[:id], metadata: { key: 'value' })
+      item = Stripe::InvoiceItem.update("ii_123", metadata: { key: 'value' })
       assert_requested :post,
-        "#{Stripe.api_base}/v1/invoiceitems/#{FIXTURE[:id]}"
+        "#{Stripe.api_base}/v1/invoiceitems/ii_123"
       assert item.kind_of?(Stripe::InvoiceItem)
     end
 
     should "be deletable" do
-      item = Stripe::InvoiceItem.retrieve(FIXTURE[:id])
+      item = Stripe::InvoiceItem.retrieve("ii_123")
       item = item.delete
       assert_requested :delete,
-        "#{Stripe.api_base}/v1/invoiceitems/#{FIXTURE[:id]}"
+        "#{Stripe.api_base}/v1/invoiceitems/#{item.id}"
       assert item.kind_of?(Stripe::InvoiceItem)
     end
   end

data/test/stripe/invoice_line_item_test.rb

@@ -2,6 +2,5 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class InvoiceLineItemTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:invoice_line_item)
   end
 end

data/test/stripe/invoice_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class InvoiceTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:invoice)
-
     should "be listable" do
       invoices = Stripe::Invoice.list
       assert_requested :get, "#{Stripe.api_base}/v1/invoices"
@@ -12,50 +10,50 @@ module Stripe
     end
 
     should "be retrievable" do
-      invoice = Stripe::Invoice.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/invoices/#{FIXTURE[:id]}"
+      invoice = Stripe::Invoice.retrieve("in_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/invoices/in_123"
       assert invoice.kind_of?(Stripe::Invoice)
     end
 
     should "be creatable" do
       invoice = Stripe::Invoice.create(
-        :customer => API_FIXTURES[:customer][:id]
+        :customer => "cus_123"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/invoices"
       assert invoice.kind_of?(Stripe::Invoice)
     end
 
     should "be saveable" do
-      invoice = Stripe::Invoice.retrieve(FIXTURE[:id])
+      invoice = Stripe::Invoice.retrieve("in_123")
       invoice.metadata['key'] = 'value'
       invoice.save
-      assert_requested :post, "#{Stripe.api_base}/v1/invoices/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/invoices/#{invoice.id}"
     end
 
     should "be updateable" do
-      invoice = Stripe::Invoice.update(FIXTURE[:id], metadata: { key: 'value' })
-      assert_requested :post, "#{Stripe.api_base}/v1/invoices/#{FIXTURE[:id]}"
+      invoice = Stripe::Invoice.update("in_123", metadata: { key: 'value' })
+      assert_requested :post, "#{Stripe.api_base}/v1/invoices/in_123"
       assert invoice.kind_of?(Stripe::Invoice)
     end
 
     context "#pay" do
       should "pay invoice" do
-        invoice = Stripe::Invoice.retrieve(FIXTURE[:id])
+        invoice = Stripe::Invoice.retrieve("in_123")
         invoice = invoice.pay
         assert_requested :post,
-          "#{Stripe.api_base}/v1/invoices/#{FIXTURE[:id]}/pay"
+          "#{Stripe.api_base}/v1/invoices/#{invoice.id}/pay"
         assert invoice.kind_of?(Stripe::Invoice)
       end
 
       should "pay invoice with a specific source" do
-        invoice = Stripe::Invoice.retrieve(FIXTURE[:id])
+        invoice = Stripe::Invoice.retrieve("in_123")
         invoice = invoice.pay(
-          source: API_FIXTURES[:customer][:sources][:data][0][:id]
+          source: 'src_123',
         )
         assert_requested :post,
-          "#{Stripe.api_base}/v1/invoices/#{FIXTURE[:id]}/pay",
+          "#{Stripe.api_base}/v1/invoices/#{invoice.id}/pay",
           body: {
-            source: API_FIXTURES[:customer][:sources][:data][0][:id]
+            source: 'src_123',
           }
         assert invoice.kind_of?(Stripe::Invoice)
       end
@@ -64,13 +62,33 @@ module Stripe
     context "#upcoming" do
       should "retrieve upcoming invoices" do
         invoice = Stripe::Invoice.upcoming(
-          customer: API_FIXTURES[:customer][:id],
-          subscription: API_FIXTURES[:subscription][:id]
+          customer: "cus_123",
+          subscription: "sub_123"
         )
         assert_requested :get, "#{Stripe.api_base}/v1/invoices/upcoming",
           query: {
-            customer: API_FIXTURES[:customer][:id],
-            subscription: API_FIXTURES[:subscription][:id]
+            customer: "cus_123",
+            subscription: "sub_123"
+          }
+        assert invoice.kind_of?(Stripe::Invoice)
+      end
+
+      should "retrieve upcoming invoices with items" do
+        items = [{
+          plan: 'gold',
+          quantity: 2
+        }]
+
+        invoice = Stripe::Invoice.upcoming(
+          customer: "cus_123",
+          subscription_items: items
+        )
+
+        assert_requested :get, "#{Stripe.api_base}/v1/invoices/upcoming",
+          query: {
+            customer: "cus_123",
+            :'subscription_items[][plan]' => 'gold',
+            :'subscription_items[][quantity]' =>  2
           }
         assert invoice.kind_of?(Stripe::Invoice)
       end
@@ -78,12 +96,12 @@ module Stripe
       should "be callable with an empty string" do
         invoice = Stripe::Invoice.upcoming(
           coupon: '',
-          customer: API_FIXTURES[:customer][:id]
+          customer: "cus_123"
         )
         assert_requested :get, "#{Stripe.api_base}/v1/invoices/upcoming",
           query: {
             coupon: '',
-            customer: API_FIXTURES[:customer][:id]
+            customer: "cus_123"
           }
         assert invoice.kind_of?(Stripe::Invoice)
       end

data/test/stripe/list_object_test.rb

@@ -9,7 +9,7 @@ module Stripe
 
     should "provide #count via enumerable" do
       list = Stripe::ListObject.construct_from({
-        data: [API_FIXTURES.fetch(:charge)]
+        data: [{ object: "charge" }]
       })
       assert_equal 1, list.count
     end

data/test/stripe/login_link_test.rb

@@ -2,18 +2,17 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class LoginLinkTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:login_link)
-
     setup do
-      account_fixture = API_FIXTURES.fetch(:account)
-      account_fixture = account_fixture.merge(
+      account_fixture = {
+        'id' => 'acct_123',
+        'object' => 'account',
         'login_links' => {
           'data' => [],
           'has_more' => false,
           'object' => 'list',
-          'url' =>  "/v1/accounts/#{account_fixture[:id]}/login_links"
+          'url' =>  "/v1/accounts/acct_123/login_links"
         }
-      )
+      }
       @account = Stripe::Account.construct_from(account_fixture)
     end
 
@@ -25,7 +24,7 @@ module Stripe
 
     should "be creatable" do
       stub_request(:post, "#{Stripe.api_base}/v1/accounts/#{@account.id}/login_links").
-        to_return(body: JSON.generate(FIXTURE))
+        to_return(body: JSON.generate({ object: 'login_link' }))
 
       login_link = @account.login_links.create
       assert_requested :post,

data/test/stripe/order_return_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class OrderReturnTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:order_return)
-
     should "be listable" do
       order_returns = Stripe::OrderReturn.list
       assert_requested :get, "#{Stripe.api_base}/v1/order_returns"
@@ -12,9 +10,9 @@ module Stripe
     end
 
     should "be retrievable" do
-      order_return = Stripe::OrderReturn.retrieve(FIXTURE[:id])
+      order_return = Stripe::OrderReturn.retrieve("orret_123")
       assert_requested :get,
-        "#{Stripe.api_base}/v1/order_returns/#{FIXTURE[:id]}"
+        "#{Stripe.api_base}/v1/order_returns/orret_123"
       assert order_return.kind_of?(Stripe::OrderReturn)
     end
   end

data/test/stripe/order_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class OrderTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:order)
-
     should "be listable" do
       orders = Stripe::Order.list
       assert_requested :get, "#{Stripe.api_base}/v1/orders"
@@ -12,8 +10,8 @@ module Stripe
     end
 
     should "be retrievable" do
-      order = Stripe::Order.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/orders/#{FIXTURE[:id]}"
+      order = Stripe::Order.retrieve("or_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/orders/or_123"
       assert order.kind_of?(Stripe::Order)
     end
 
@@ -26,31 +24,31 @@ module Stripe
     end
 
     should "be saveable" do
-      order = Stripe::Order.retrieve(FIXTURE[:id])
+      order = Stripe::Order.retrieve("or_123")
       order.metadata['key'] = 'value'
       order.save
-      assert_requested :post, "#{Stripe.api_base}/v1/orders/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}"
     end
 
     should "be updateable" do
-      order = Stripe::Order.update(FIXTURE[:id], metadata: { key: 'value' })
-      assert_requested :post, "#{Stripe.api_base}/v1/orders/#{FIXTURE[:id]}"
+      order = Stripe::Order.update("or_123", metadata: { key: 'value' })
+      assert_requested :post, "#{Stripe.api_base}/v1/orders/or_123"
       assert order.kind_of?(Stripe::Order)
     end
 
     context "#pay" do
       should "pay an order" do
-        order = Stripe::Order.retrieve(FIXTURE[:id])
-        order = order.pay(token: API_FIXTURES.fetch(:token)[:id])
+        order = Stripe::Order.retrieve("or_123")
+        order = order.pay(token: "tok_123")
         assert order.kind_of?(Stripe::Order)
       end
     end
 
     context "#return_order" do
       should "return an order" do
-        order = Stripe::Order.retrieve(FIXTURE[:id])
+        order = Stripe::Order.retrieve("or_123")
         order = order.return_order(:orders => [
-          { parent: API_FIXTURES.fetch(:sku)[:id] }
+          { parent: "sku_123" }
         ])
         assert order.kind_of?(Stripe::OrderReturn)
       end

data/test/stripe/payout_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class PayoutTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:payout)
-
     should "be listable" do
       payouts = Stripe::Payout.list
       assert_requested :get, "#{Stripe.api_base}/v1/payouts"
@@ -12,8 +10,8 @@ module Stripe
     end
 
     should "be retrievable" do
-      payout = Stripe::Payout.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/payouts/#{FIXTURE[:id]}"
+      payout = Stripe::Payout.retrieve("tr_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/payouts/tr_123"
       assert payout.kind_of?(Stripe::Payout)
     end
 
@@ -27,21 +25,21 @@ module Stripe
     end
 
     should "be saveable" do
-      payout = Stripe::Payout.retrieve(FIXTURE[:id])
+      payout = Stripe::Payout.retrieve("tr_123")
       payout.metadata['key'] = 'value'
       payout.save
-      assert_requested :post, "#{Stripe.api_base}/v1/payouts/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/payouts/#{payout.id}"
     end
 
     should "be updateable" do
-      payout = Stripe::Payout.update(FIXTURE[:id], metadata: {foo: 'bar'})
-      assert_requested :post, "#{Stripe.api_base}/v1/payouts/#{FIXTURE[:id]}"
+      payout = Stripe::Payout.update("tr_123", metadata: {foo: 'bar'})
+      assert_requested :post, "#{Stripe.api_base}/v1/payouts/tr_123"
       assert payout.kind_of?(Stripe::Payout)
     end
 
     context "#cancel" do
       should "cancel a payout" do
-        payout = Stripe::Payout.retrieve(FIXTURE[:id])
+        payout = Stripe::Payout.retrieve("tr_123")
         payout = payout.cancel
         assert payout.kind_of?(Stripe::Payout)
       end

data/test/stripe/plan_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class PlanTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:plan)
-
     should "be listable" do
       plans = Stripe::Plan.list
       assert_requested :get, "#{Stripe.api_base}/v1/plans"
@@ -12,8 +10,8 @@ module Stripe
     end
 
     should "be retrievable" do
-      plan = Stripe::Plan.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/plans/#{FIXTURE[:id]}"
+      plan = Stripe::Plan.retrieve("sapphire-elite")
+      assert_requested :get, "#{Stripe.api_base}/v1/plans/sapphire-elite"
       assert plan.kind_of?(Stripe::Plan)
     end
 
@@ -30,22 +28,22 @@ module Stripe
     end
 
     should "be saveable" do
-      plan = Stripe::Plan.retrieve(FIXTURE[:id])
+      plan = Stripe::Plan.retrieve("sapphire-elite")
       plan.metadata['key'] = 'value'
       plan.save
-      assert_requested :post, "#{Stripe.api_base}/v1/plans/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/plans/#{plan.id}"
     end
 
     should "be updateable" do
-      plan = Stripe::Plan.update(FIXTURE[:id], metadata: {foo: 'bar'})
-      assert_requested :post, "#{Stripe.api_base}/v1/plans/#{FIXTURE[:id]}"
+      plan = Stripe::Plan.update("sapphire-elite", metadata: {foo: 'bar'})
+      assert_requested :post, "#{Stripe.api_base}/v1/plans/sapphire-elite"
       assert plan.kind_of?(Stripe::Plan)
     end
 
     should "be deletable" do
-      plan = Stripe::Plan.retrieve(FIXTURE[:id])
+      plan = Stripe::Plan.retrieve("sapphire-elite")
       plan = plan.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/plans/#{FIXTURE[:id]}"
+      assert_requested :delete, "#{Stripe.api_base}/v1/plans/#{plan.id}"
       assert plan.kind_of?(Stripe::Plan)
     end
   end