stripe-ruby-mock 1.8.4.5 → 1.8.4.6

data/README.md

@@ -7,7 +7,7 @@
 
 In your gemfile:
 
-    gem 'stripe-ruby-mock', '>= 1.8.4.4'
+    gem 'stripe-ruby-mock', '>= 1.8.4.6'
 
 ## Features
 

data/lib/stripe_mock/data.rb

@@ -1,7 +1,7 @@
 module StripeMock
   module Data
 
-    def self.test_customer(cards, params)
+    def self.mock_customer(cards, params)
       cus_id = params[:id] || "test_cus_default"
       cards.each {|card| card[:customer] = cus_id}
       {
@@ -25,7 +25,7 @@ module StripeMock
       }.merge(params)
     end
 
-    def self.test_charge(params={})
+    def self.mock_charge(params={})
       {
         id: "ch_1fD6uiR9FAA2zc",
         object: "charge",
@@ -67,7 +67,7 @@ module StripeMock
       }.merge(params)
     end
 
-    def self.test_charge_array
+    def self.mock_charge_array
       {
         :data => [test_charge, test_charge, test_charge],
         :object => 'list',
@@ -75,7 +75,7 @@ module StripeMock
       }
     end
 
-    def self.test_card(params={})
+    def self.mock_card(params={})
       {
         id: "test_cc_default",
         object: "card",
@@ -99,7 +99,7 @@ module StripeMock
       }.merge(params)
     end
 
-    def self.test_coupon(params={})
+    def self.mock_coupon(params={})
       {
         :duration => 'repeating',
         :duration_in_months => 3,
@@ -110,7 +110,7 @@ module StripeMock
     end
 
     #FIXME nested overrides would be better than hardcoding plan_id
-    def self.test_subscription(params={})
+    def self.mock_subscription(params={})
       StripeMock::Util.rmerge({
         :current_period_end => 1308681468,
         :status => "trialing",
@@ -133,7 +133,7 @@ module StripeMock
       }, params)
     end
 
-    def self.test_invoice(params={})
+    def self.mock_invoice(params={})
       {
         :id => 'in_test_invoice',
         :object => 'invoice',
@@ -173,7 +173,7 @@ module StripeMock
       }.merge(params)
     end
 
-    def self.test_paid_invoice
+    def self.mock_paid_invoice
       test_invoice.merge({
           :attempt_count => 1,
           :attempted => true,
@@ -185,7 +185,7 @@ module StripeMock
         })
     end
 
-    def self.test_invoice_customer_array
+    def self.mock_invoice_customer_array
       {
         :data => [test_invoice],
         :object => 'list',
@@ -193,7 +193,7 @@ module StripeMock
       }
     end
 
-    def self.test_plan(params={})
+    def self.mock_plan(params={})
       {
         interval: "month",
         name: "The Basic Plan",
@@ -207,7 +207,7 @@ module StripeMock
       }.merge(params)
     end
 
-    def self.test_recipient(params={})
+    def self.mock_recipient(params={})
       {
         :name => "Stripe User",
         :type => "individual",
@@ -225,7 +225,7 @@ module StripeMock
       }.merge(params)
     end
 
-    def self.test_recipient_array
+    def self.mock_recipient_array
       {
         :data => [test_recipient, test_recipient, test_recipient],
         :object => 'list',
@@ -233,7 +233,7 @@ module StripeMock
       }
     end
 
-    def self.test_transfer(params={})
+    def self.mock_transfer(params={})
       {
         :status => 'pending',
         :amount => 100,
@@ -254,7 +254,7 @@ module StripeMock
       }.merge(params)
     end
 
-    def self.test_transfer_array
+    def self.mock_transfer_array
       {
         :data => [test_transfer, test_transfer, test_transfer],
         :object => 'list',
@@ -262,7 +262,7 @@ module StripeMock
       }
     end
 
-    def self.test_invalid_api_key_error
+    def self.mock_invalid_api_key_error
       {
         "error" => {
           "type" => "invalid_request_error",
@@ -271,7 +271,7 @@ module StripeMock
       }
     end
 
-    def self.test_invalid_exp_year_error
+    def self.mock_invalid_exp_year_error
       {
         "error" => {
           "code" => "invalid_expiry_year",
@@ -282,7 +282,7 @@ module StripeMock
       }
     end
 
-    def self.test_missing_id_error
+    def self.mock_missing_id_error
       {
         :error => {
           :param => "id",
@@ -292,13 +292,13 @@ module StripeMock
       }
     end
 
-    def self.test_delete_subscription(params={})
+    def self.mock_delete_subscription(params={})
       {
         deleted: true
       }.merge(params)
     end
 
-    def self.test_api_error
+    def self.mock_api_error
       {
         :error => {
           :type => "api_error"
@@ -306,7 +306,7 @@ module StripeMock
       }
     end
 
-    def self.test_delete_discount_response
+    def self.mock_delete_discount_response
       {
         :deleted => true,
         :id => "di_test_coupon"

data/lib/stripe_mock/instance.rb

@@ -68,13 +68,13 @@ module StripeMock
     def generate_card_token(card_params)
       token = new_id 'tok'
       card_params[:id] = new_id 'cc'
-      @card_tokens[token] = Data.test_card(card_params)
+      @card_tokens[token] = Data.mock_card(card_params)
       token
     end
 
     def get_card_by_token(token)
       if token.nil? || @card_tokens[token].nil?
-        Data.test_card :id => new_id('cc')
+        Data.mock_card :id => new_id('cc')
       else
         @card_tokens.delete(token)
       end
@@ -87,7 +87,7 @@ module StripeMock
 
       if obj.nil?
         msg = message || "No such #{type}: #{id}"
-        raise Stripe::InvalidRequestError.new(msg, type.to_s, 400)
+        raise Stripe::InvalidRequestError.new(msg, type.to_s, 404)
       end
     end
 

data/lib/stripe_mock/request_handlers/charges.rb

@@ -9,13 +9,13 @@ module StripeMock
 
       def new_charge(route, method_url, params, headers)
         id = new_id('ch')
-        charges[id] = Data.test_charge(params.merge :id => id)
+        charges[id] = Data.mock_charge(params.merge :id => id)
       end
 
       def get_charge(route, method_url, params, headers)
         route =~ method_url
         assert_existance :charge, $1, charges[$1]
-        charges[$1] ||= Data.test_charge(:id => $1)
+        charges[$1] ||= Data.mock_charge(:id => $1)
       end
 
     end

data/lib/stripe_mock/request_handlers/customers.rb

@@ -18,7 +18,7 @@ module StripeMock
           cards << get_card_by_token(params.delete(:card))
           params[:default_card] = cards.first[:id]
         end
-        customers[ params[:id] ] = Data.test_customer(cards, params)
+        customers[ params[:id] ] = Data.mock_customer(cards, params)
       end
 
       def update_subscription(route, method_url, params, headers)
@@ -29,7 +29,7 @@ module StripeMock
         plan = plans[ params[:plan] ]
         assert_existance :plan, params[:plan], plan
 
-        sub = Data.test_subscription id: new_id('su'), plan: plan, customer: $1
+        sub = Data.mock_subscription id: new_id('su'), plan: plan, customer: $1
         customer[:subscription] = sub
       end
 
@@ -42,10 +42,12 @@ module StripeMock
         sub = customer[:subscription]
         assert_existance nil, nil, sub, "No active subscription for customer: #{$1}"
 
+        customer[:subscription] = nil
+
         plan = plans[ sub[:plan][:id] ]
         assert_existance :plan, params[:plan], plan
 
-        Data.test_delete_subscription(id: sub[:id])
+        Data.mock_delete_subscription(id: sub[:id])
       end
 
       def update_customer(route, method_url, params, headers)
@@ -53,11 +55,11 @@ module StripeMock
         assert_existance :customer, $1, customers[$1]
 
         card_id = new_id('cc') if params.delete(:card)
-        cus = customers[$1] ||= Data.test_customer([], :id => $1)
+        cus = customers[$1] ||= Data.mock_customer([], :id => $1)
         cus.merge!(params)
 
         if card_id
-          new_card = Data.test_card(id: card_id, customer: cus[:id])
+          new_card = Data.mock_card(id: card_id, customer: cus[:id])
 
           if cus[:cards][:count] == 0
             cus[:cards][:count] += 1
@@ -74,7 +76,7 @@ module StripeMock
       def get_customer(route, method_url, params, headers)
         route =~ method_url
         assert_existance :customer, $1, customers[$1]
-        customers[$1] ||= Data.test_customer([], :id => $1)
+        customers[$1] ||= Data.mock_customer([], :id => $1)
       end
 
       def list_customers(route, method_url, params, headers)

data/lib/stripe_mock/request_handlers/invoice_items.rb

@@ -7,7 +7,7 @@ module StripeMock
       end
 
       def new_invoice_item(route, method_url, params, headers)
-        Data.test_invoice(params)
+        Data.mock_invoice(params)
       end
 
     end

data/lib/stripe_mock/request_handlers/plans.rb

@@ -3,28 +3,35 @@ module StripeMock
     module Plans
 
       def Plans.included(klass)
-        klass.add_handler 'post /v1/plans',       :new_plan
-        klass.add_handler 'post /v1/plans/(.*)',  :update_plan
-        klass.add_handler 'get /v1/plans/(.*)',   :get_plan
-        klass.add_handler 'get /v1/plans',        :list_plans
+        klass.add_handler 'post /v1/plans',        :new_plan
+        klass.add_handler 'post /v1/plans/(.*)',   :update_plan
+        klass.add_handler 'get /v1/plans/(.*)',    :get_plan
+        klass.add_handler 'delete /v1/plans/(.*)', :delete_plan
+        klass.add_handler 'get /v1/plans',         :list_plans
       end
 
       def new_plan(route, method_url, params, headers)
         params[:id] ||= new_id('plan')
-        plans[ params[:id] ] = Data.test_plan(params)
+        plans[ params[:id] ] = Data.mock_plan(params)
       end
 
       def update_plan(route, method_url, params, headers)
         route =~ method_url
         assert_existance :plan, $1, plans[$1]
-        plans[$1] ||= Data.test_plan(:id => $1)
+        plans[$1] ||= Data.mock_plan(:id => $1)
         plans[$1].merge!(params)
       end
 
       def get_plan(route, method_url, params, headers)
         route =~ method_url
         assert_existance :plan, $1, plans[$1]
-        plans[$1] ||= Data.test_plan(:id => $1)
+        plans[$1] ||= Data.mock_plan(:id => $1)
+      end
+
+      def delete_plan(route, method_url, params, headers)
+        route =~ method_url
+        assert_existance :plan, $1, plans[$1]
+        plans.delete($1)
       end
 
       def list_plans(route, method_url, params, headers)

data/lib/stripe_mock/version.rb

@@ -1,4 +1,4 @@
 module StripeMock
   # stripe-ruby-mock version
-  VERSION = "1.8.4.5"
+  VERSION = "1.8.4.6"
 end

data/spec/shared_stripe_examples/charge_examples.rb

@@ -52,7 +52,7 @@ shared_examples 'Charge API' do
     expect { Stripe::Charge.retrieve('nope') }.to raise_error {|e|
       expect(e).to be_a Stripe::InvalidRequestError
       expect(e.param).to eq('charge')
-      expect(e.http_status).to eq(400)
+      expect(e.http_status).to eq(404)
     }
   end
 

data/spec/shared_stripe_examples/customer_examples.rb

@@ -68,7 +68,7 @@ shared_examples 'Customer API' do
     expect { Stripe::Customer.retrieve('nope') }.to raise_error {|e|
       expect(e).to be_a Stripe::InvalidRequestError
       expect(e.param).to eq('customer')
-      expect(e.http_status).to eq(400)
+      expect(e.http_status).to eq(404)
     }
   end
 
@@ -136,6 +136,8 @@ shared_examples 'Customer API' do
     result = customer.cancel_subscription
     expect(result.deleted).to eq(true)
     expect(result.id).to eq(sub.id)
+    customer = Stripe::Customer.retrieve('test_customer_sub')
+    expect(customer.subscription).to be_nil
   end
 
   it "cannot update to a plan that does not exist" do

data/spec/shared_stripe_examples/plan_examples.rb

@@ -74,10 +74,24 @@ shared_examples 'Plan API' do
     expect { Stripe::Plan.retrieve('nope') }.to raise_error {|e|
       expect(e).to be_a Stripe::InvalidRequestError
       expect(e.param).to eq('plan')
-      expect(e.http_status).to eq(400)
+      expect(e.http_status).to eq(404)
     }
   end
 
+  it "deletes a stripe plan" do
+    Stripe::Plan.create(id: 'super_member', amount: 111)
+
+    plan = Stripe::Plan.retrieve('super_member')
+    expect(plan).to_not be_nil
+
+    plan.delete
+
+    expect { Stripe::Plan.retrieve('super_member') }.to raise_error {|e|
+      expect(e).to be_a Stripe::InvalidRequestError
+      expect(e.param).to eq('plan')
+      expect(e.http_status).to eq(404)
+    }
+  end
 
   it "retrieves all plans" do
     Stripe::Plan.create({ id: 'Plan One', amount: 54321 })

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe-ruby-mock
 version: !ruby/object:Gem::Version
-  version: 1.8.4.5
+  version: 1.8.4.6
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-26 00:00:00.000000000 Z
+date: 2013-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: stripe