stripe 3.0.3 → 3.1.0

data/test/stripe/product_test.rb

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

data/test/stripe/recipient_card_test.rb

@@ -2,11 +2,8 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class RecipientCardTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:source)
-
     setup do
-      @recipient =
-        Stripe::Recipient.retrieve(API_FIXTURES.fetch(:transfer_recipient)[:id])
+      @recipient = Stripe::Recipient.retrieve("rp_123")
     end
 
     should "be listable" do
@@ -17,24 +14,31 @@ module Stripe
 
     should "be creatable" do
       card = @recipient.cards.create(
-        card: API_FIXTURES.fetch(:token)[:id]
+        card: "tok_123"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/recipients/#{@recipient.id}/cards"
       assert card.kind_of?(Stripe::Card)
     end
 
     should "be deletable" do
-      card = Stripe::Card.construct_from(FIXTURE.merge(recipient: @recipient.id))
+      card = Stripe::Card.construct_from({
+        id: "card_123",
+        recipient: @recipient.id
+      })
       card = card.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/recipients/#{@recipient.id}/cards/#{FIXTURE[:id]}"
+      assert_requested :delete, "#{Stripe.api_base}/v1/recipients/#{@recipient.id}/cards/card_123"
       assert card.kind_of?(Stripe::Card)
     end
 
     should "be saveable" do
-      card = Stripe::Card.construct_from(FIXTURE.merge(recipient: @recipient.id))
+      card = Stripe::Card.construct_from({
+        id: "card_123",
+        metadata: {},
+        recipient: @recipient.id
+      })
       card.metadata['key'] = 'value'
       card.save
-      assert_requested :post, "#{Stripe.api_base}/v1/recipients/#{@recipient.id}/cards/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/recipients/#{@recipient.id}/cards/card_123"
     end
   end
 end

data/test/stripe/recipient_test.rb

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

data/test/stripe/refund_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class RefundTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:refund)
-
     should "be listable" do
       refunds = Stripe::Refund.list
       assert_requested :get, "#{Stripe.api_base}/v1/refunds"
@@ -12,27 +10,27 @@ module Stripe
     end
 
     should "be retrievable" do
-      refund = Stripe::Refund.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/refunds/#{FIXTURE[:id]}"
+      refund = Stripe::Refund.retrieve("re_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/refunds/re_123"
       assert refund.kind_of?(Stripe::Refund)
     end
 
     should "be creatable" do
-      refund = Stripe::Refund.create(:charge => API_FIXTURES[:charge][:id])
+      refund = Stripe::Refund.create(:charge => "ch_123")
       assert_requested :post, "#{Stripe.api_base}/v1/refunds"
       assert refund.kind_of?(Stripe::Refund)
     end
 
     should "be saveable" do
-      refund = Stripe::Refund.retrieve(FIXTURE[:id])
+      refund = Stripe::Refund.retrieve("re_123")
       refund.metadata['key'] = 'value'
       refund.save
-      assert_requested :post, "#{Stripe.api_base}/v1/refunds/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/refunds/#{refund.id}"
     end
 
     should "be updateable" do
-      refund = Stripe::Refund.update(FIXTURE[:id], metadata: { key: 'value' })
-      assert_requested :post, "#{Stripe.api_base}/v1/refunds/#{FIXTURE[:id]}"
+      refund = Stripe::Refund.update("re_123", metadata: { key: 'value' })
+      assert_requested :post, "#{Stripe.api_base}/v1/refunds/re_123"
       assert refund.kind_of?(Stripe::Refund)
     end
   end

data/test/stripe/reversal_test.rb

@@ -2,10 +2,8 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class ReversalTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:transfer_reversal)
-
     setup do
-      @transfer = Stripe::Transfer.retrieve(API_FIXTURES.fetch(:transfer)[:id])
+      @transfer = Stripe::Transfer.retrieve("tr_123")
     end
 
     should "be listable" do
@@ -17,9 +15,9 @@ module Stripe
     end
 
     should "be retrievable" do
-      reversal = @transfer.reversals.retrieve(FIXTURE[:id])
+      reversal = @transfer.reversals.retrieve("trr_123")
       assert_requested :get,
-        "#{Stripe.api_base}/v1/transfers/#{@transfer.id}/reversals/#{FIXTURE[:id]}"
+        "#{Stripe.api_base}/v1/transfers/#{@transfer.id}/reversals/trr_123"
       assert reversal.kind_of?(Stripe::Reversal)
     end
 
@@ -33,11 +31,11 @@ module Stripe
     end
 
     should "be saveable" do
-      reversal = @transfer.reversals.retrieve(FIXTURE[:id])
+      reversal = @transfer.reversals.retrieve("trr_123")
       reversal.metadata['key'] = 'value'
       reversal.save
       assert_requested :post,
-        "#{Stripe.api_base}/v1/transfers/#{@transfer.id}/reversals/#{FIXTURE[:id]}"
+        "#{Stripe.api_base}/v1/transfers/#{@transfer.id}/reversals/#{reversal.id}"
     end
   end
 end

data/test/stripe/sku_test.rb

@@ -2,8 +2,6 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class SKUTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:sku)
-
     should "be listable" do
       skus = Stripe::SKU.list
       assert_requested :get, "#{Stripe.api_base}/v1/skus"
@@ -12,8 +10,8 @@ module Stripe
     end
 
     should "be retrievable" do
-      sku = Stripe::SKU.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/skus/#{FIXTURE[:id]}"
+      sku = Stripe::SKU.retrieve("sku_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/skus/sku_123"
       assert sku.kind_of?(Stripe::SKU)
     end
 
@@ -22,28 +20,28 @@ module Stripe
         currency: "USD",
         inventory: { type: "finite", quantity: 500 },
         price: 100,
-        product: API_FIXTURES.fetch(:product)[:id]
+        product: "prod_123"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/skus"
     end
 
     should "be saveable" do
-      sku = Stripe::SKU.retrieve(FIXTURE[:id])
+      sku = Stripe::SKU.retrieve("sku_123")
       sku.metadata['key'] = 'value'
       sku.save
-      assert_requested :post, "#{Stripe.api_base}/v1/skus/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/skus/#{sku.id}"
     end
 
     should "be updateable" do
-      sku = Stripe::SKU.update(FIXTURE[:id], metadata: {foo: 'bar'})
-      assert_requested :post, "#{Stripe.api_base}/v1/skus/#{FIXTURE[:id]}"
+      sku = Stripe::SKU.update("sku_123", metadata: {foo: 'bar'})
+      assert_requested :post, "#{Stripe.api_base}/v1/skus/sku_123"
       assert sku.kind_of?(Stripe::SKU)
     end
 
     should "be deletable" do
-      sku = Stripe::SKU.retrieve(FIXTURE[:id])
+      sku = Stripe::SKU.retrieve("sku_123")
       sku = sku.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/skus/#{FIXTURE[:id]}"
+      assert_requested :delete, "#{Stripe.api_base}/v1/skus/#{sku.id}"
       assert sku.kind_of?(Stripe::SKU)
     end
   end

data/test/stripe/source_test.rb

@@ -2,39 +2,37 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class SourceTest < Test::Unit::TestCase
-    FIXTURE = API_FIXTURES.fetch(:source)
-
     should "be retrievable" do
-      source = Stripe::Source.retrieve(FIXTURE[:id])
-      assert_requested :get, "#{Stripe.api_base}/v1/sources/#{FIXTURE[:id]}"
+      source = Stripe::Source.retrieve("src_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/sources/src_123"
       assert source.kind_of?(Stripe::Source)
     end
 
     should "be creatable" do
       source = Stripe::Source.create(
         type: 'card',
-        token: API_FIXTURES.fetch(:token)[:id]
+        token: "tok_123"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/sources"
       assert source.kind_of?(Stripe::Source)
     end
 
     should "be saveable" do
-      source = Stripe::Source.retrieve(FIXTURE[:id])
+      source = Stripe::Source.retrieve("src_123")
       source.metadata['key'] = 'value'
       source.save
-      assert_requested :post, "#{Stripe.api_base}/v1/sources/#{FIXTURE[:id]}"
+      assert_requested :post, "#{Stripe.api_base}/v1/sources/#{source.id}"
     end
 
     should "be updateable" do
-      source = Stripe::Source.update(FIXTURE[:id], metadata: {foo: 'bar'})
-      assert_requested :post, "#{Stripe.api_base}/v1/sources/#{FIXTURE[:id]}"
+      source = Stripe::Source.update("src_123", metadata: {foo: 'bar'})
+      assert_requested :post, "#{Stripe.api_base}/v1/sources/src_123"
       assert source.kind_of?(Stripe::Source)
     end
 
     context "#delete" do
       should "not be deletable when unattached" do
-        source = Stripe::Source.retrieve(FIXTURE[:id])
+        source = Stripe::Source.retrieve("src_123")
 
         assert_raises NotImplementedError do
           source.delete
@@ -42,10 +40,13 @@ module Stripe
       end
 
       should "be deletable when attached to a customer" do
-        customer_id = API_FIXTURES.fetch(:customer)[:id]
-        source = Stripe::Source.construct_from(FIXTURE.merge(customer: customer_id))
-        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer_id}/sources/#{FIXTURE[:id]}").
-          to_return(body: JSON.generate(FIXTURE))
+        source = Stripe::Source.construct_from({
+          customer: "cus_123",
+          id: "src_123",
+          object: "source"
+        })
+        source = source.delete
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/cus_123/sources/src_123"
         assert source.kind_of?(Stripe::Source)
       end
     end
@@ -58,7 +59,7 @@ module Stripe
 
     context "#verify" do
       should "verify the source" do
-        source = Stripe::Source.retrieve(FIXTURE[:id])
+        source = Stripe::Source.retrieve("src_123")
         source = source.verify(:values => [1,2])
         assert source.kind_of?(Stripe::Source)
       end

data/test/stripe/stripe_client_test.rb

@@ -121,7 +121,7 @@ module Stripe
         should "support literal headers" do
           stub_request(:post, "#{Stripe.api_base}/v1/account").
             with(headers: { "Stripe-Account" => "bar" }).
-            to_return(body: JSON.generate(API_FIXTURES.fetch(:account)))
+            to_return(body: JSON.generate({ object: "account" }))
 
           client = StripeClient.new
           client.execute_request(:post, '/v1/account',
@@ -132,7 +132,7 @@ module Stripe
         should "support RestClient-style header keys" do
           stub_request(:post, "#{Stripe.api_base}/v1/account").
             with(headers: { "Stripe-Account" => "bar" }).
-            to_return(body: JSON.generate(API_FIXTURES.fetch(:account)))
+            to_return(body: JSON.generate({ object: "account" }))
 
           client = StripeClient.new
           client.execute_request(:post, '/v1/account',
@@ -141,6 +141,158 @@ module Stripe
         end
       end
 
+      context "logging" do
+        setup do
+          # Freeze time for the purposes of the `elapsed` parameter that we
+          # emit for responses. I didn't want to bring in a new dependency for
+          # this, but Mocha's `anything` parameter can't match inside of a hash
+          # and is therefore not useful for this purpose. If we switch over to
+          # rspec-mocks at some point, we can probably remove Timecop from the
+          # project.
+          Timecop.freeze(Time.local(1990))
+        end
+
+        teardown do
+          Timecop.return
+        end
+
+        should "produce appropriate logging" do
+          body = JSON.generate({ object: "account" })
+
+          Util.expects(:log_info).with("Request to Stripe API",
+            api_version: '2010-11-12',
+            idempotency_key: "abc",
+            method: :post,
+            path: "/v1/account"
+          )
+          Util.expects(:log_debug).with("Request details",
+            body: '',
+            idempotency_key: "abc"
+          )
+
+          Util.expects(:log_info).with("Response from Stripe API",
+            api_version: '2010-11-12',
+            elapsed: 0.0,
+            idempotency_key: "abc",
+            method: :post,
+            path: "/v1/account",
+            request_id: "req_123",
+            status: 200
+          )
+          Util.expects(:log_debug).with("Response details",
+            body: body,
+            idempotency_key: "abc",
+            request_id: "req_123"
+          )
+          Util.expects(:log_debug).with("Dashboard link for request",
+            idempotency_key: "abc",
+            request_id: "req_123",
+            url: Util.request_id_dashboard_url("req_123", Stripe.api_key)
+          )
+
+          stub_request(:post, "#{Stripe.api_base}/v1/account").
+            to_return(
+              body: body,
+              headers: {
+                "Idempotency-Key" => "abc",
+                "Request-Id" => "req_123",
+                "Stripe-Version" => "2010-11-12"
+              }
+            )
+
+          client = StripeClient.new
+          client.execute_request(:post, '/v1/account',
+            headers: {
+              "Idempotency-Key" => "abc",
+              "Stripe-Version" => "2010-11-12"
+            }
+          )
+        end
+
+        should "produce logging on API error" do
+          Util.expects(:log_info).with("Request to Stripe API",
+            api_version: nil,
+            idempotency_key: nil,
+            method: :post,
+            path: "/v1/account"
+          )
+          Util.expects(:log_info).with("Response from Stripe API",
+            api_version: nil,
+            elapsed: 0.0,
+            idempotency_key: nil,
+            method: :post,
+            path: "/v1/account",
+            request_id: nil,
+            status: 500
+          )
+
+          error = {
+            code: 'code',
+            message: 'message',
+            param: 'param',
+            type: 'type',
+          }
+          Util.expects(:log_info).with('Stripe API error',
+            status: 500,
+            error_code: error['code'],
+            error_message: error['message'],
+            error_param: error['param'],
+            error_type: error['type'],
+            idempotency_key: nil,
+            request_id: nil
+          )
+
+          stub_request(:post, "#{Stripe.api_base}/v1/account").
+            to_return(
+              body: JSON.generate({ :error => error }),
+              status: 500
+            )
+
+          client = StripeClient.new
+          assert_raises Stripe::APIError do
+            client.execute_request(:post, '/v1/account')
+          end
+        end
+
+        should "produce logging on OAuth error" do
+          Util.expects(:log_info).with("Request to Stripe API",
+            api_version: nil,
+            idempotency_key: nil,
+            method: :post,
+            path: "/oauth/token"
+          )
+          Util.expects(:log_info).with("Response from Stripe API",
+            api_version: nil,
+            elapsed: 0.0,
+            idempotency_key: nil,
+            method: :post,
+            path: "/oauth/token",
+            request_id: nil,
+            status: 400
+          )
+
+          Util.expects(:log_info).with('Stripe OAuth error',
+            status: 400,
+            error_code: "invalid_request",
+            error_description: "No grant type specified",
+            idempotency_key: nil,
+            request_id: nil
+          )
+
+          stub_request(:post, "#{Stripe.connect_base}/oauth/token").
+            to_return(body: JSON.generate({
+              error: "invalid_request",
+              error_description: "No grant type specified",
+            }), status: 400)
+
+          client = StripeClient.new
+          opts = {api_base: Stripe.connect_base}
+          assert_raises Stripe::OAuth::InvalidRequestError do
+            client.execute_request(:post, '/oauth/token', opts)
+          end
+        end
+      end
+
       context "Stripe-Account header" do
         should "use a globally set header" do
           begin
@@ -149,7 +301,7 @@ module Stripe
 
             stub_request(:post, "#{Stripe.api_base}/v1/account").
               with(headers: {"Stripe-Account" => Stripe.stripe_account}).
-              to_return(body: JSON.generate(API_FIXTURES.fetch(:account)))
+              to_return(body: JSON.generate({ object: "account" }))
 
             client = StripeClient.new
             client.execute_request(:post, '/v1/account')
@@ -162,7 +314,7 @@ module Stripe
           stripe_account = "acct_0000"
           stub_request(:post, "#{Stripe.api_base}/v1/account").
             with(headers: {"Stripe-Account" => stripe_account}).
-            to_return(body: JSON.generate(API_FIXTURES.fetch(:account)))
+            to_return(body: JSON.generate({ object: "account" }))
 
           client = StripeClient.new
           client.execute_request(:post, '/v1/account',
@@ -174,7 +326,7 @@ module Stripe
           stub_request(:post, "#{Stripe.api_base}/v1/account").
             with { |req|
               req.headers["Stripe-Account"].nil?
-            }.to_return(body: JSON.generate(API_FIXTURES.fetch(:account)))
+            }.to_return(body: JSON.generate({ object: "account" }))
 
           client = StripeClient.new
           client.execute_request(:post, '/v1/account')
@@ -208,7 +360,7 @@ module Stripe
                 }, data[:application])
 
                 true
-              }.to_return(body: JSON.generate(API_FIXTURES.fetch(:account)))
+              }.to_return(body: JSON.generate({ object: "account" }))
 
             client = StripeClient.new
             client.execute_request(:post, '/v1/account')
@@ -413,30 +565,30 @@ module Stripe
 
         should 'not add an idempotency key to GET requests' do
           SecureRandom.expects(:uuid).times(0)
-          stub_request(:get, "#{Stripe.api_base}/v1/charges/#{API_FIXTURES.fetch(:charge)[:id]}").
+          stub_request(:get, "#{Stripe.api_base}/v1/charges/ch_123").
             with { |req|
               req.headers['Idempotency-Key'].nil?
-            }.to_return(body: JSON.generate(API_FIXTURES.fetch(:charge)))
+            }.to_return(body: JSON.generate({ object: "charge" }))
           client = StripeClient.new
-          client.execute_request(:get, "/v1/charges/#{API_FIXTURES.fetch(:charge)[:id]}")
+          client.execute_request(:get, "/v1/charges/ch_123")
         end
 
         should 'ensure there is always an idempotency_key on POST requests' do
           SecureRandom.expects(:uuid).at_least_once.returns("random_key")
           stub_request(:post, "#{Stripe.api_base}/v1/charges").
             with(headers: {"Idempotency-Key" => "random_key"}).
-            to_return(body: JSON.generate(API_FIXTURES.fetch(:charge)))
+            to_return(body: JSON.generate({ object: "charge" }))
           client = StripeClient.new
           client.execute_request(:post, "/v1/charges")
         end
 
         should 'ensure there is always an idempotency_key on DELETE requests' do
           SecureRandom.expects(:uuid).at_least_once.returns("random_key")
-          stub_request(:delete, "#{Stripe.api_base}/v1/charges/#{API_FIXTURES.fetch(:charge)[:id]}").
+          stub_request(:delete, "#{Stripe.api_base}/v1/charges/ch_123").
             with(headers: {"Idempotency-Key" => "random_key"}).
-            to_return(body: JSON.generate(API_FIXTURES.fetch(:charge)))
+            to_return(body: JSON.generate({ object: "charge" }))
           client = StripeClient.new
-          client.execute_request(:delete, "/v1/charges/#{API_FIXTURES.fetch(:charge)[:id]}")
+          client.execute_request(:delete, "/v1/charges/ch_123")
         end
 
         should 'not override a provided idempotency_key' do
@@ -448,7 +600,7 @@ module Stripe
           # expected.
           stub_request(:post, "#{Stripe.api_base}/v1/charges").
             with(headers: {"Idempotency-Key" => "provided_key"}).
-            to_return(body: JSON.generate(API_FIXTURES.fetch(:charge)))
+            to_return(body: JSON.generate({ object: "charge" }))
 
           client = StripeClient.new
           client.execute_request(:post, "/v1/charges",
@@ -495,21 +647,33 @@ module Stripe
       context "params serialization" do
         should 'allows empty strings in params' do
           client = StripeClient.new
-          client.execute_request(:get, '/v1/invoices/upcoming',
-                                 params: { customer: API_FIXTURES[:customer][:id],
-                                           coupon: '' })
-          assert_requested(:get, "#{Stripe.api_base}/v1/invoices/upcoming?",
-                           query: { customer: API_FIXTURES[:customer][:id],
-                                    coupon: '' })
+          client.execute_request(:get, '/v1/invoices/upcoming', params: {
+             customer: 'cus_123',
+             coupon: ''
+          })
+          assert_requested(
+            :get,
+            "#{Stripe.api_base}/v1/invoices/upcoming?",
+            query: {
+              customer: 'cus_123',
+              coupon: ''
+            }
+          )
         end
 
         should 'filter nils in params' do
           client = StripeClient.new
-          client.execute_request(:get, '/v1/invoices/upcoming',
-                                 params: { customer: API_FIXTURES[:customer][:id],
-                                           coupon: nil })
-          assert_requested(:get, "#{Stripe.api_base}/v1/invoices/upcoming?",
-                           query: { customer: API_FIXTURES[:customer][:id] })
+          client.execute_request(:get, '/v1/invoices/upcoming', params: {
+             customer: 'cus_123',
+             coupon: nil
+          })
+          assert_requested(
+            :get,
+            "#{Stripe.api_base}/v1/invoices/upcoming?",
+            query: {
+              customer: 'cus_123'
+            }
+          )
         end
       end
     end
@@ -517,7 +681,7 @@ module Stripe
     context "#request" do
       should "return a result and response object" do
         stub_request(:post, "#{Stripe.api_base}/v1/charges").
-          to_return(body: JSON.generate(API_FIXTURES.fetch(:charge)))
+          to_return(body: JSON.generate({ object: "charge" }))
 
         client = StripeClient.new
         charge, resp = client.request { Charge.create }