saucy 0.2.17 → 0.2.18

data/app/controllers/billings_controller.rb

@@ -15,7 +15,7 @@ class BillingsController < ApplicationController
 
   def update
     @account = current_account
-    if @account.save_braintree!(params[:account])
+    if @account.save_customer_and_subscription!(params[:account])
       redirect_to account_billing_path(@account), :notice => t('.update.notice', :default => "Billing information updated successfully")
     else
       render :edit

data/app/controllers/plans_controller.rb

@@ -13,7 +13,7 @@ class PlansController < ApplicationController
   def update
     @plans = Plan.ordered
     @account = current_account
-    if @account.save_braintree!(params[:account])
+    if @account.save_customer_and_subscription!(params[:account])
       redirect_to edit_account_path(@account), :notice => t('.update.notice', :default => "Plan changed successfully")
     else
       render :edit

data/lib/saucy.rb

@@ -1,9 +1,9 @@
 require 'saucy/user'
 require 'saucy/account'
+require 'saucy/subscription'
 require 'saucy/project'
 require 'saucy/plan'
 require 'saucy/account_authorization'
 require 'saucy/configuration'
 require 'saucy/engine'
 require 'saucy/projects_controller'
-

data/lib/saucy/account.rb

@@ -3,17 +3,8 @@ module Saucy
     extend ActiveSupport::Concern
 
     included do
-      require "rubygems"
-      require "braintree"
+      include Saucy::Subscription
 
-      CUSTOMER_ATTRIBUTES = { :cardholder_name => :cardholder_name, 
-                              :billing_email => :email, 
-                              :card_number => :number, 
-                              :expiration_month => :expiration_month, 
-                              :expiration_year => :expiration_year, 
-                              :verification_code => :cvv }
-
-      extend ActiveSupport::Memoizable
       has_many :memberships, :dependent => :destroy
       has_many :users, :through => :memberships
       has_many :projects, :dependent => :destroy
@@ -28,20 +19,11 @@ module Saucy
       validates_uniqueness_of :name, :keyword
       validates_presence_of :name, :keyword, :plan_id
 
-      attr_accessor *CUSTOMER_ATTRIBUTES.keys
-
       attr_accessible :name, :keyword
 
       validates_format_of :keyword,
                           :with    => %r{^[a-z0-9]+$},
                           :message => "must be only lower case letters."
-
-      before_create :create_customer
-      before_create :create_subscription, :if => :billed?
-      after_destroy :destroy_customer
-
-      memoize :customer
-      memoize :subscription
     end
 
     module InstanceMethods
@@ -68,169 +50,6 @@ module Saucy
       def memberships_by_name
         memberships.by_name
       end
-
-      def customer
-        Braintree::Customer.find(customer_token) if customer_token
-      end
-
-      def credit_card
-        customer.credit_cards[0] if customer && customer.credit_cards.any?
-      end
-
-      def subscription
-        Braintree::Subscription.find(subscription_token) if subscription_token
-      end
-
-      def save_braintree!(attributes)
-        successful = true
-        self.plan = ::Plan.find(attributes[:plan_id]) if changing_plan?(attributes)
-        if changing_braintree_attributes?(attributes)
-          successful = update_braintree_customer(attributes)
-        end
-        if successful && changing_plan?(attributes)
-          save_subscription
-          flush_cache :subscription
-        end
-        successful && save
-      end
-
-      def can_change_plan_to?(new_plan)
-        plan.limits.where(:value_type => :number).all? do |limit|
-          new_plan.limit(limit.name).value >= send(limit.name).count
-        end
-      end
-
-      def past_due?
-        subscription_status == Braintree::Subscription::Status::PastDue
-      end
-
-      private
-
-      def changing_plan?(attributes)
-        attributes[:plan_id].present?
-      end
-
-      def changing_braintree_attributes?(attributes)
-        CUSTOMER_ATTRIBUTES.keys.any? { |attribute| attributes[attribute].present? }
-      end
-
-      def set_braintree_attributes(attributes)
-        CUSTOMER_ATTRIBUTES.keys.each do |attribute|
-          send("#{attribute}=", attributes[attribute]) if attributes[attribute].present?
-        end
-      end
-
-      def update_braintree_customer(attributes)
-        set_braintree_attributes(attributes)
-        result = Braintree::Customer.update(customer_token, customer_attributes)
-        handle_customer_result(result)
-        result.success?
-      end
-
-      def save_subscription
-        if subscription 
-          Braintree::Subscription.update(subscription_token, :plan_id => plan_id)
-        elsif plan.billed?
-          create_subscription
-        end
-      end
-
-      def customer_attributes
-        {
-          :email => billing_email,
-          :credit_card => credit_card_attributes
-        }
-      end
-
-      def credit_card_attributes
-        if plan.billed?
-          card_attributes = { :cardholder_name => cardholder_name,
-                              :number => card_number,
-                              :expiration_month => expiration_month,
-                              :expiration_year => expiration_year,
-                              :cvv => verification_code }
-          if credit_card
-            card_attributes.merge!(:options => credit_card_options)
-          end
-          card_attributes
-        else 
-          {}
-        end
-      end
-
-      def credit_card_options
-        if customer && customer.credit_cards.any?
-          { :update_existing_token => credit_card.token }
-        else
-          {}
-        end
-      end
-
-      def create_customer
-        result = Braintree::Customer.create(customer_attributes)
-        handle_customer_result(result)
-      end
-
-      def destroy_customer
-        Braintree::Customer.delete(customer_token)
-      end
-
-      def handle_customer_result(result)
-        if result.success?
-          self.customer_token = result.customer.id
-          flush_cache :customer
-        else
-          verification = result.credit_card_verification
-          if verification && verification.status == "processor_declined"
-            errors[:card_number] << "was denied by the payment processor with the message: #{verification.processor_response_text}"
-          elsif verification && verification.status == "gateway_rejected"
-            errors[:verification_code] << "did not match"
-          elsif result.errors.any?
-            result.errors.each do |error|
-              if error.attribute == "number"
-                errors[:card_number] << error.message.gsub("Credit card number ", "")
-              elsif error.attribute == "CVV"
-                errors[:verification_code] << error.message.gsub("CVV ", "")
-              elsif error.attribute == "expiration_month"
-                errors[:expiration_month] << error.message.gsub("Expiration month ", "")
-              elsif error.attribute == "expiration_year"
-                errors[:expiration_year] << error.message.gsub("Expiration year ", "")
-              end
-            end
-          end
-        end
-        result.success?
-      end
-
-      def create_subscription
-        result = Braintree::Subscription.create(
-          :payment_method_token => credit_card.token,
-          :plan_id => plan_id
-        )
-        if result.success?
-          self.subscription_token = result.subscription.id
-          self.next_billing_date = result.subscription.next_billing_date
-          self.subscription_status = result.subscription.status
-        else
-          false
-        end
-      end
-    end
-
-    module ClassMethods
-      def update_subscriptions!
-        recently_billed = where("next_billing_date <= '#{Time.now}'")
-        recently_billed.each do |account|
-          account.subscription_status = account.subscription.status
-          account.next_billing_date = account.subscription.next_billing_date
-          account.save!
-          if account.past_due?
-            BillingMailer.problem(account, account.subscription.transactions.last).deliver!
-          else
-            BillingMailer.receipt(account, account.subscription.transactions.last).deliver!
-          end
-        end
-      end
     end
   end
 end

data/lib/saucy/subscription.rb

@@ -0,0 +1,213 @@
+module Saucy
+  module Subscription
+    extend ActiveSupport::Concern
+
+    included do
+      require "rubygems"
+      require "braintree"
+
+      extend ActiveSupport::Memoizable
+
+      CUSTOMER_ATTRIBUTES = { :cardholder_name => :cardholder_name, 
+                              :billing_email => :email, 
+                              :card_number => :number, 
+                              :expiration_month => :expiration_month, 
+                              :expiration_year => :expiration_year, 
+                              :verification_code => :cvv }
+
+      attr_accessor *CUSTOMER_ATTRIBUTES.keys
+
+      before_create :create_customer
+      before_create :create_subscription, :if => :billed?
+      after_destroy :destroy_customer
+
+      memoize :customer
+      memoize :subscription
+    end
+
+    module InstanceMethods
+      def customer
+        Braintree::Customer.find(customer_token) if customer_token
+      end
+
+      def credit_card
+        customer.credit_cards[0] if customer && customer.credit_cards.any?
+      end
+
+      def subscription
+        Braintree::Subscription.find(subscription_token) if subscription_token
+      end
+
+      def save_customer_and_subscription!(attributes)
+        successful = true
+        self.plan = ::Plan.find(attributes[:plan_id]) if changing_plan?(attributes)
+        if changing_customer_attributes?(attributes)
+          successful = update_customer(attributes)
+        end
+        if successful && past_due?
+          successful = retry_subscription_charge!
+        end
+        if successful && changing_plan?(attributes)
+          save_subscription
+          flush_cache :subscription
+        end
+        successful && save
+      end
+
+      def can_change_plan_to?(new_plan)
+        plan.limits.where(:value_type => :number).all? do |limit|
+          new_plan.limit(limit.name).value >= send(limit.name).count
+        end
+      end
+
+      def past_due?
+        subscription_status == Braintree::Subscription::Status::PastDue
+      end
+
+      private
+
+      def retry_subscription_charge!
+        authorized_transaction = Braintree::Subscription.retry_charge(subscription.id).transaction
+        result = Braintree::Transaction.submit_for_settlement(authorized_transaction.id)
+        handle_errors(authorized_transaction, result.errors) if !result.success?
+        update_subscription_cache!
+        result.success?
+      end
+
+      def update_subscription_cache!
+        flush_cache :subscription
+        update_attribute(:subscription_status, subscription.status)
+        update_attribute(:next_billing_date, subscription.next_billing_date)
+      end
+
+      def changing_plan?(attributes)
+        attributes[:plan_id].present?
+      end
+
+      def changing_customer_attributes?(attributes)
+        CUSTOMER_ATTRIBUTES.keys.any? { |attribute| attributes[attribute].present? }
+      end
+
+      def set_customer_attributes(attributes)
+        CUSTOMER_ATTRIBUTES.keys.each do |attribute|
+          send("#{attribute}=", attributes[attribute]) if attributes[attribute].present?
+        end
+      end
+
+      def update_customer(attributes)
+        set_customer_attributes(attributes)
+        result = Braintree::Customer.update(customer_token, customer_attributes)
+        handle_customer_result(result)
+        result.success?
+      end
+
+      def save_subscription
+        if subscription 
+          Braintree::Subscription.update(subscription_token, :plan_id => plan_id)
+        elsif plan.billed?
+          create_subscription
+        end
+      end
+
+      def customer_attributes
+        {
+          :email => billing_email,
+          :credit_card => credit_card_attributes
+        }
+      end
+
+      def credit_card_attributes
+        if plan.billed?
+          card_attributes = { :cardholder_name => cardholder_name,
+                              :number => card_number,
+                              :expiration_month => expiration_month,
+                              :expiration_year => expiration_year,
+                              :cvv => verification_code }
+          if credit_card
+            card_attributes.merge!(:options => credit_card_options)
+          end
+          card_attributes
+        else 
+          {}
+        end
+      end
+
+      def credit_card_options
+        if customer && customer.credit_cards.any?
+          { :update_existing_token => credit_card.token }
+        else
+          {}
+        end
+      end
+
+      def create_customer
+        result = Braintree::Customer.create(customer_attributes)
+        handle_customer_result(result)
+      end
+
+      def destroy_customer
+        Braintree::Customer.delete(customer_token)
+      end
+
+      def handle_customer_result(result)
+        if result.success?
+          self.customer_token = result.customer.id
+          flush_cache :customer
+        else
+          handle_errors(result.credit_card_verification, result.errors)
+        end
+        result.success?
+      end
+
+      def handle_errors(result, remote_errors)
+        if result && result.status == "processor_declined"
+          errors[:card_number] << "was denied by the payment processor with the message: #{result.processor_response_text}"
+        elsif result && result.status == "gateway_rejected"
+          errors[:verification_code] << "did not match"
+        elsif remote_errors.any?
+          remote_errors.each do |error|
+            if error.attribute == "number"
+              errors[:card_number] << error.message.gsub("Credit card number ", "")
+            elsif error.attribute == "CVV"
+              errors[:verification_code] << error.message.gsub("CVV ", "")
+            elsif error.attribute == "expiration_month"
+              errors[:expiration_month] << error.message.gsub("Expiration month ", "")
+            elsif error.attribute == "expiration_year"
+              errors[:expiration_year] << error.message.gsub("Expiration year ", "")
+            end
+          end
+        end
+      end
+
+      def create_subscription
+        result = Braintree::Subscription.create(
+          :payment_method_token => credit_card.token,
+          :plan_id => plan_id
+        )
+        if result.success?
+          self.subscription_token = result.subscription.id
+          self.next_billing_date = result.subscription.next_billing_date
+          self.subscription_status = result.subscription.status
+        else
+          false
+        end
+      end
+    end
+
+    module ClassMethods
+      def update_subscriptions!
+        recently_billed = where("next_billing_date <= '#{Time.now}'")
+        recently_billed.each do |account|
+          account.subscription_status = account.subscription.status
+          account.next_billing_date = account.subscription.next_billing_date
+          account.save!
+          if account.past_due?
+            BillingMailer.problem(account, account.subscription.transactions.last).deliver!
+          else
+            BillingMailer.receipt(account, account.subscription.transactions.last).deliver!
+          end
+        end
+      end
+    end
+  end
+end

data/spec/models/account_spec.rb

@@ -65,291 +65,4 @@ describe Account do
 
     result.should == expected
   end
-
-  it "manifests braintree processor_declined errors as errors on number and doesn't save" do
-    FakeBraintree.failures["4111111111111112"] = { "message" => "Do Not Honor", "code" => "2000", "status" => "processor_declined" }
-    account = Factory.build(:account, 
-                            :cardholder_name => "Ralph Robot", 
-                            :billing_email => "ralph@example.com", 
-                            :card_number => "4111111111111112", 
-                            :expiration_month => 5, 
-                            :expiration_year => 2012,
-                            :plan => Factory(:paid_plan))
-    account.save.should_not be
-    FakeBraintree.customers.should be_empty
-    account.persisted?.should_not be
-    account.errors[:card_number].any? { |e| e =~ /denied/ }.should be
-  end
-
-  it "manifests braintree gateway_rejected errors as errors on number and doesn't save" do
-    FakeBraintree.failures["4111111111111112"] = { "message" => "Gateway Rejected: cvv", "code" => "N", "status" => "gateway_rejected" }
-    account = Factory.build(:account, 
-                            :cardholder_name => "Ralph Robot", 
-                            :billing_email => "ralph@example.com", 
-                            :card_number => "4111111111111112", 
-                            :expiration_month => 5, 
-                            :expiration_year => 2012,
-                            :verification_code => 200,
-                            :plan => Factory(:paid_plan))
-    account.save.should_not be
-    FakeBraintree.customers.should be_empty
-    account.persisted?.should_not be
-    account.errors[:verification_code].any? { |e| e =~ /did not match/ }.should be
-   end
-
-  it "manifests braintree gateway_rejected errors as errors on number and doesn't save" do
-    FakeBraintree.failures["4111111111111111"] = { "message" => "Credit card number is invalid.", "errors" => { "customer" => { "errors" => [], "credit-card" => { "errors" => [{ "message" => "Credit card number is invalid.", "code" => 81715, "attribute" => :number }] }}}}
-    account = Factory.build(:account, 
-                            :cardholder_name => "Ralph Robot", 
-                            :billing_email => "ralph@example.com", 
-                            :card_number => "4111111111111111", 
-                            :expiration_month => 5, 
-                            :expiration_year => 2012,
-                            :verification_code => 123,
-                            :plan => Factory(:paid_plan))
-    account.save.should_not be
-    FakeBraintree.customers.should be_empty
-    account.persisted?.should_not be
-    account.errors[:card_number].any? { |e| e =~ /is invalid/ }.should be
-   end
-end
-
-describe Account, "with a paid plan" do
-  subject do
-    Factory(:account, 
-            :cardholder_name => "Ralph Robot", 
-            :billing_email => "ralph@example.com", 
-            :card_number => "4111111111111111", 
-            :verification_code => "123",
-            :expiration_month => 5, 
-            :expiration_year => 2012,
-            :plan => Factory(:paid_plan))
-  end
-
-  it "has a customer_token" do
-    subject.customer_token.should_not be_nil
-  end
-
-  it "has a subscription_token" do
-    subject.subscription_token.should_not be_nil
-  end
-
-  it "has a customer" do
-    subject.customer.should_not be_nil
-  end
-
-  it "has a credit card" do
-    subject.credit_card.should_not be_nil
-  end
-
-  it "has a subscription" do
-    subject.subscription.should_not be_nil
-  end
-
-  it "has a next_billing_date" do
-    subject.next_billing_date.should_not be_nil
-  end
-
-  it "has an active subscription status" do
-    subject.subscription_status.should == Braintree::Subscription::Status::Active
-  end
-
-  it "is not past due" do
-    subject.past_due?.should_not be
-  end
-
-  it "creates a braintree customer, credit card, and subscription" do
-    FakeBraintree.customers[subject.customer_token].should_not be_nil
-    FakeBraintree.customers[subject.customer_token]["credit_cards"].first.should_not be_nil
-    FakeBraintree.subscriptions[subject.subscription_token].should_not be_nil
-  end
-
-  it "changes the subscription when the plan is changed" do
-    new_plan = Factory(:paid_plan, :name => "New Plan")
-    subject.save_braintree!(:plan_id => new_plan.id)
-    FakeBraintree.subscriptions[subject.subscription_token]["plan_id"].should == new_plan.id
-  end
-
-  it "updates the customer and credit card information when changed" do
-    subject.save_braintree!(:billing_email => "jrobot@example.com",
-                            :cardholder_name => "Jim Robot", 
-                            :card_number => "4111111111111115",
-                            :verification_code => "123",
-                            :expiration_month => 5, 
-                            :expiration_year => 2013)
-    subject.customer.email.should == "jrobot@example.com"
-    subject.credit_card.cardholder_name.should == "Jim Robot"
-  end
-
-  it "deletes the customer when deleted" do
-    subject.destroy
-    FakeBraintree.customers[subject.customer_token].should be_nil
-  end
-end
-
-describe Account, "with a free plan" do
-  subject do
-    Factory(:account, :plan => Factory(:plan))
-  end
-
-  it "has a customer_token" do
-    subject.customer_token.should_not be_nil
-  end
-
-  it "has a customer" do
-    subject.customer.should_not be_nil
-  end
-
-  it "doesn't have a credit_card" do
-    subject.credit_card.should be_nil
-  end
-
-  it "doesn't have a subscription_token" do
-    subject.subscription_token.should be_nil
-  end
-
-  it "doesn't have a subscription" do
-    subject.subscription.should be_nil
-  end
-
-  it "creates a braintree customer" do
-    FakeBraintree.customers[subject.customer_token].should_not be_nil
-  end
-
-  it "doesn't create a credit card, and subscription" do
-    FakeBraintree.customers[subject.customer_token]["credit_cards"].should be_nil
-    FakeBraintree.subscriptions[subject.subscription_token].should be_nil
-  end
-
-  it "creates a credit card, and subscription when the plan is changed to a paid plan and the billing info is supplied" do
-    new_plan = Factory(:paid_plan, :name => "New Plan")
-    subject.save_braintree!(:plan_id => new_plan.id,
-                            :cardholder_name => "Ralph Robot", 
-                            :billing_email => "ralph@example.com", 
-                            :card_number => "4111111111111111", 
-                            :verification_code => "123",
-                            :expiration_month => 5, 
-                            :expiration_year => 2012)
-
-    FakeBraintree.customers[subject.customer_token]["credit_cards"].first.should_not be_nil
-    FakeBraintree.subscriptions[subject.subscription_token].should_not be_nil
-    FakeBraintree.subscriptions[subject.subscription_token]["plan_id"].should == new_plan.id
-    subject.credit_card.should_not be_nil
-    subject.subscription.should_not be_nil
-  end
-
-  it "doesn't create a credit card, and subscription when the plan is changed to a different free plan" do
-    new_plan = Factory(:plan, :name => "New Plan")
-    subject.save_braintree!(:plan_id => new_plan.id)
-
-    subject.credit_card.should be_nil
-    subject.subscription.should be_nil
-  end
-end
-
-describe Account, "with a plan and limits, and other plans" do
-  subject { Factory(:account) }
-
-  before do
-    Factory(:limit, :name => "users", :value => 1, :plan => subject.plan)
-    Factory(:limit, :name => "projects", :value => 1, :plan => subject.plan)
-    Factory(:limit, :name => "ssl", :value => 1, :value_type => :boolean, :plan => subject.plan)
-    @can_switch = Factory(:plan)
-    Factory(:limit, :name => "users", :value => 1, :plan => @can_switch)
-    Factory(:limit, :name => "projects", :value => 1, :plan => @can_switch)
-    Factory(:limit, :name => "ssl", :value => 0, :value_type => :boolean, :plan => @can_switch)
-    @cannot_switch = Factory(:plan)
-    Factory(:limit, :name => "users", :value => 0, :plan => @cannot_switch)
-    Factory(:limit, :name => "projects", :value => 0, :plan => @cannot_switch)
-    Factory(:limit, :name => "ssl", :value => 1, :value_type => :boolean, :plan => @cannot_switch)
-
-    Factory(:membership, :account => subject)
-    Factory(:project, :account => subject)
-  end
-
-  it "indicates whether the account can switch to another plan" do
-    subject.can_change_plan_to?(@can_switch).should be
-    subject.can_change_plan_to?(@cannot_switch).should_not be
-  end
-end
-
-describe Account, "with a paid subscription" do
-  subject do
-    Factory(:account, 
-            :cardholder_name => "Ralph Robot", 
-            :billing_email => "ralph@example.com", 
-            :card_number => "4111111111111111", 
-            :verification_code => "123",
-            :expiration_month => 5, 
-            :expiration_year => 2012,
-            :plan => Factory(:paid_plan))
-  end
-
-  it "gets marked as past due and updates its next_billing_date when subscriptions are updated and it has been rejected by the gateway" do
-    subscription = FakeBraintree.subscriptions[subject.subscription_token]
-    subscription["status"] = Braintree::Subscription::Status::PastDue
-    subscription["next_billing_date"] = 2.months.from_now
-
-    Timecop.travel(subject.next_billing_date + 1.day) do
-      Account.update_subscriptions!
-      subject.reload.subscription_status.should == Braintree::Subscription::Status::PastDue
-      subject.next_billing_date.to_s.should == subscription["next_billing_date"].to_s
-      subject.past_due?.should be
-    end
-  end
-
-  it "gets marked as not past due and updates its next_billing_date when the subscription is active after its billing date" do
-    subscription = FakeBraintree.subscriptions[subject.subscription_token]
-    subscription["status"] = Braintree::Subscription::Status::Active
-    subscription["next_billing_date"] = 2.months.from_now
-    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Settled,
-                                  :subscription_id => subject.subscription_token }
-    subscription["transactions"] = [FakeBraintree.generated_transaction]
-
-    Timecop.travel(subject.next_billing_date + 1.day) do
-      Account.update_subscriptions!
-      subject.reload.subscription_status.should == Braintree::Subscription::Status::Active
-      subject.next_billing_date.to_s.should == subscription["next_billing_date"].to_s
-    end
-  end
-
-  it "receives a receipt email at it's billing email with transaction details" do
-    subscription = FakeBraintree.subscriptions[subject.subscription_token]
-    subscription["status"] = Braintree::Subscription::Status::Active
-    subscription["next_billing_date"] = 2.months.from_now
-    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Settled,
-                                  :subscription_id => subject.subscription_token }
-    subscription["transactions"] = [FakeBraintree.generated_transaction]
-
-    Timecop.travel(subject.next_billing_date + 1.day) do
-      ActionMailer::Base.deliveries.clear
-
-      Account.update_subscriptions!
-
-      ActionMailer::Base.deliveries.any? do |email|
-        email.to == [subject.billing_email] &&
-        email.subject =~ /receipt/i
-      end.should be
-    end
-  end
-
-  it "receives a receipt email at it's billing email with a notice that it failed when billing didn't work" do
-    subscription = FakeBraintree.subscriptions[subject.subscription_token]
-    subscription["status"] = Braintree::Subscription::Status::PastDue
-    subscription["next_billing_date"] = 2.months.from_now
-    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Failed,
-                                  :subscription_id => subject.subscription_token }
-    subscription["transactions"] = [FakeBraintree.generated_transaction]
-
-    Timecop.travel(subject.next_billing_date + 1.day) do
-      ActionMailer::Base.deliveries.clear
-
-      Account.update_subscriptions!
-
-      ActionMailer::Base.deliveries.any? do |email|
-        email.to == [subject.billing_email] &&
-        email.subject =~ /problem/i
-      end.should be
-    end
-  end
 end

data/spec/models/subscription_spec.rb

@@ -0,0 +1,390 @@
+require 'spec_helper'
+
+describe Account do
+  subject { Factory(:account) }
+
+  it "manifests braintree processor_declined errors as errors on number and doesn't save" do
+    FakeBraintree.failures["4111111111111112"] = { "message" => "Do Not Honor", "code" => "2000", "status" => "processor_declined" }
+    account = Factory.build(:account, 
+                            :cardholder_name => "Ralph Robot", 
+                            :billing_email => "ralph@example.com", 
+                            :card_number => "4111111111111112", 
+                            :expiration_month => 5, 
+                            :expiration_year => 2012,
+                            :plan => Factory(:paid_plan))
+    account.save.should_not be
+    FakeBraintree.customers.should be_empty
+    account.persisted?.should_not be
+    account.errors[:card_number].any? { |e| e =~ /denied/ }.should be
+  end
+
+  it "manifests braintree gateway_rejected errors as errors on number and doesn't save" do
+    FakeBraintree.failures["4111111111111112"] = { "message" => "Gateway Rejected: cvv", "code" => "N", "status" => "gateway_rejected" }
+    account = Factory.build(:account, 
+                            :cardholder_name => "Ralph Robot", 
+                            :billing_email => "ralph@example.com", 
+                            :card_number => "4111111111111112", 
+                            :expiration_month => 5, 
+                            :expiration_year => 2012,
+                            :verification_code => 200,
+                            :plan => Factory(:paid_plan))
+    account.save.should_not be
+    FakeBraintree.customers.should be_empty
+    account.persisted?.should_not be
+    account.errors[:verification_code].any? { |e| e =~ /did not match/ }.should be
+   end
+
+  it "manifests braintree gateway_rejected errors as errors on number and doesn't save" do
+    FakeBraintree.failures["4111111111111111"] = { "message" => "Credit card number is invalid.", "errors" => { "customer" => { "errors" => [], "credit-card" => { "errors" => [{ "message" => "Credit card number is invalid.", "code" => 81715, "attribute" => :number }] }}}}
+    account = Factory.build(:account, 
+                            :cardholder_name => "Ralph Robot", 
+                            :billing_email => "ralph@example.com", 
+                            :card_number => "4111111111111111", 
+                            :expiration_month => 5, 
+                            :expiration_year => 2012,
+                            :verification_code => 123,
+                            :plan => Factory(:paid_plan))
+    account.save.should_not be
+    FakeBraintree.customers.should be_empty
+    account.persisted?.should_not be
+    account.errors[:card_number].any? { |e| e =~ /is invalid/ }.should be
+   end
+end
+
+describe Account, "with a paid plan" do
+  subject do
+    Factory(:account, 
+            :cardholder_name => "Ralph Robot", 
+            :billing_email => "ralph@example.com", 
+            :card_number => "4111111111111111", 
+            :verification_code => "123",
+            :expiration_month => 5, 
+            :expiration_year => 2012,
+            :plan => Factory(:paid_plan))
+  end
+
+  it "has a customer_token" do
+    subject.customer_token.should_not be_nil
+  end
+
+  it "has a subscription_token" do
+    subject.subscription_token.should_not be_nil
+  end
+
+  it "has a customer" do
+    subject.customer.should_not be_nil
+  end
+
+  it "has a credit card" do
+    subject.credit_card.should_not be_nil
+  end
+
+  it "has a subscription" do
+    subject.subscription.should_not be_nil
+  end
+
+  it "has a next_billing_date" do
+    subject.next_billing_date.should_not be_nil
+  end
+
+  it "has an active subscription status" do
+    subject.subscription_status.should == Braintree::Subscription::Status::Active
+  end
+
+  it "is not past due" do
+    subject.past_due?.should_not be
+  end
+
+  it "creates a braintree customer, credit card, and subscription" do
+    FakeBraintree.customers[subject.customer_token].should_not be_nil
+    FakeBraintree.customers[subject.customer_token]["credit_cards"].first.should_not be_nil
+    FakeBraintree.subscriptions[subject.subscription_token].should_not be_nil
+  end
+
+  it "changes the subscription when the plan is changed" do
+    new_plan = Factory(:paid_plan, :name => "New Plan")
+    subject.save_customer_and_subscription!(:plan_id => new_plan.id)
+    FakeBraintree.subscriptions[subject.subscription_token]["plan_id"].should == new_plan.id
+  end
+
+  it "updates the customer and credit card information when changed" do
+    subject.save_customer_and_subscription!(:billing_email => "jrobot@example.com",
+                                            :cardholder_name => "Jim Robot", 
+                                            :card_number => "4111111111111115",
+                                            :verification_code => "123",
+                                            :expiration_month => 5, 
+                                            :expiration_year => 2013)
+    subject.customer.email.should == "jrobot@example.com"
+    subject.credit_card.cardholder_name.should == "Jim Robot"
+  end
+
+  it "deletes the customer when deleted" do
+    subject.destroy
+    FakeBraintree.customers[subject.customer_token].should be_nil
+  end
+end
+
+describe Account, "with a free plan" do
+  subject do
+    Factory(:account, :plan => Factory(:plan))
+  end
+
+  it "has a customer_token" do
+    subject.customer_token.should_not be_nil
+  end
+
+  it "has a customer" do
+    subject.customer.should_not be_nil
+  end
+
+  it "doesn't have a credit_card" do
+    subject.credit_card.should be_nil
+  end
+
+  it "doesn't have a subscription_token" do
+    subject.subscription_token.should be_nil
+  end
+
+  it "doesn't have a subscription" do
+    subject.subscription.should be_nil
+  end
+
+  it "creates a braintree customer" do
+    FakeBraintree.customers[subject.customer_token].should_not be_nil
+  end
+
+  it "doesn't create a credit card, and subscription" do
+    FakeBraintree.customers[subject.customer_token]["credit_cards"].should be_nil
+    FakeBraintree.subscriptions[subject.subscription_token].should be_nil
+  end
+
+  it "creates a credit card, and subscription when the plan is changed to a paid plan and the billing info is supplied" do
+    new_plan = Factory(:paid_plan, :name => "New Plan")
+    subject.save_customer_and_subscription!(:plan_id => new_plan.id,
+                                            :cardholder_name => "Ralph Robot", 
+                                            :billing_email => "ralph@example.com", 
+                                            :card_number => "4111111111111111", 
+                                            :verification_code => "123",
+                                            :expiration_month => 5, 
+                                            :expiration_year => 2012)
+
+    FakeBraintree.customers[subject.customer_token]["credit_cards"].first.should_not be_nil
+    FakeBraintree.subscriptions[subject.subscription_token].should_not be_nil
+    FakeBraintree.subscriptions[subject.subscription_token]["plan_id"].should == new_plan.id
+    subject.credit_card.should_not be_nil
+    subject.subscription.should_not be_nil
+  end
+
+  it "doesn't create a credit card, and subscription when the plan is changed to a different free plan" do
+    new_plan = Factory(:plan, :name => "New Plan")
+    subject.save_customer_and_subscription!(:plan_id => new_plan.id)
+
+    subject.credit_card.should be_nil
+    subject.subscription.should be_nil
+  end
+end
+
+describe Account, "with a plan and limits, and other plans" do
+  subject { Factory(:account) }
+
+  before do
+    Factory(:limit, :name => "users", :value => 1, :plan => subject.plan)
+    Factory(:limit, :name => "projects", :value => 1, :plan => subject.plan)
+    Factory(:limit, :name => "ssl", :value => 1, :value_type => :boolean, :plan => subject.plan)
+    @can_switch = Factory(:plan)
+    Factory(:limit, :name => "users", :value => 1, :plan => @can_switch)
+    Factory(:limit, :name => "projects", :value => 1, :plan => @can_switch)
+    Factory(:limit, :name => "ssl", :value => 0, :value_type => :boolean, :plan => @can_switch)
+    @cannot_switch = Factory(:plan)
+    Factory(:limit, :name => "users", :value => 0, :plan => @cannot_switch)
+    Factory(:limit, :name => "projects", :value => 0, :plan => @cannot_switch)
+    Factory(:limit, :name => "ssl", :value => 1, :value_type => :boolean, :plan => @cannot_switch)
+
+    Factory(:membership, :account => subject)
+    Factory(:project, :account => subject)
+  end
+
+  it "indicates whether the account can switch to another plan" do
+    subject.can_change_plan_to?(@can_switch).should be
+    subject.can_change_plan_to?(@cannot_switch).should_not be
+  end
+end
+
+describe Account, "with a paid subscription" do
+  subject do
+    Factory(:account, 
+            :cardholder_name => "Ralph Robot", 
+            :billing_email => "ralph@example.com", 
+            :card_number => "4111111111111111", 
+            :verification_code => "123",
+            :expiration_month => 5, 
+            :expiration_year => 2012,
+            :plan => Factory(:paid_plan))
+  end
+
+  it "gets marked as past due and updates its next_billing_date when subscriptions are updated and it has been rejected by the gateway" do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::PastDue
+    subscription["next_billing_date"] = 2.months.from_now
+
+    Timecop.travel(subject.next_billing_date + 1.day) do
+      Account.update_subscriptions!
+      subject.reload.subscription_status.should == Braintree::Subscription::Status::PastDue
+      subject.next_billing_date.to_s.should == subscription["next_billing_date"].to_s
+      subject.past_due?.should be
+    end
+  end
+
+  it "gets marked as not past due and updates its next_billing_date when the subscription is active after its billing date" do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::Active
+    subscription["next_billing_date"] = 2.months.from_now
+    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Settled,
+                                  :subscription_id => subject.subscription_token }
+    subscription["transactions"] = [FakeBraintree.generated_transaction]
+
+    Timecop.travel(subject.next_billing_date + 1.day) do
+      Account.update_subscriptions!
+      subject.reload.subscription_status.should == Braintree::Subscription::Status::Active
+      subject.next_billing_date.to_s.should == subscription["next_billing_date"].to_s
+    end
+  end
+
+  it "receives a receipt email at it's billing email with transaction details" do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::Active
+    subscription["next_billing_date"] = 2.months.from_now
+    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Settled,
+                                  :subscription_id => subject.subscription_token }
+    subscription["transactions"] = [FakeBraintree.generated_transaction]
+
+    Timecop.travel(subject.next_billing_date + 1.day) do
+      ActionMailer::Base.deliveries.clear
+
+      Account.update_subscriptions!
+
+      ActionMailer::Base.deliveries.any? do |email|
+        email.to == [subject.billing_email] &&
+        email.subject =~ /receipt/i
+      end.should be
+    end
+  end
+
+  it "receives a receipt email at it's billing email with a notice that it failed when billing didn't work" do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::PastDue
+    subscription["next_billing_date"] = 2.months.from_now
+    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Failed,
+                                  :subscription_id => subject.subscription_token }
+    subscription["transactions"] = [FakeBraintree.generated_transaction]
+
+    Timecop.travel(subject.next_billing_date + 1.day) do
+      ActionMailer::Base.deliveries.clear
+
+      Account.update_subscriptions!
+
+      ActionMailer::Base.deliveries.any? do |email|
+        email.to == [subject.billing_email] &&
+        email.subject =~ /problem/i
+      end.should be
+    end
+  end
+end
+
+describe Account, "with a paid subscription that is past due" do
+  subject do
+    Factory(:account, 
+            :cardholder_name => "Ralph Robot", 
+            :billing_email => "ralph@example.com", 
+            :card_number => "4111111111111111", 
+            :verification_code => "123",
+            :expiration_month => 5, 
+            :expiration_year => 2012,
+            :plan => Factory(:paid_plan))
+  end
+
+  before do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::PastDue
+    subscription["next_billing_date"] = 2.months.from_now
+
+    Timecop.travel(subject.next_billing_date + 1.day) do
+      Account.update_subscriptions!
+    end
+    subject.reload
+  end
+
+  it "retries the subscription charge and updates the subscription when the billing information is correctly updated" do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::Active
+    subscription["next_billing_date"] = 2.months.from_now
+    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Settled,
+                                  :subscription_id => subject.subscription_token }
+    transaction = FakeBraintree.generated_transaction
+    subscription["transactions"] = [transaction]
+
+    retry_transaction = stub(:id => "12345")
+    retry_authorization = stub(:transaction => retry_transaction)
+
+    Braintree::Subscription.expects(:retry_charge).with(subject.subscription_token).returns(retry_authorization)
+    Braintree::Transaction.expects(:submit_for_settlement).with(retry_transaction.id).returns(stub(:success? => true))
+
+    subject.save_customer_and_subscription!(:card_number => "4111111111111111",
+                                            :verification_code => "124",
+                                            :expiration_month => 6,
+                                            :expiration_year => 2012).should be
+
+    subject.reload.subscription_status.should == Braintree::Subscription::Status::Active
+    subject.next_billing_date.to_s.should == subscription["next_billing_date"].to_s
+  end
+
+  it "retries the subscription charge and updates the subscription when the payment processing fails" do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::PastDue
+    subscription["next_billing_date"] = 1.day.from_now
+    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Failed,
+                                  :subscription_id => subject.subscription_token }
+    transaction = FakeBraintree.generated_transaction
+    subscription["transactions"] = [transaction]
+
+    retry_transaction = stub(:id => "12345", :status => "processor_declined", "processor_response_text" => "no good")
+    retry_authorization = stub(:transaction => retry_transaction)
+
+    Braintree::Subscription.expects(:retry_charge).with(subject.subscription_token).returns(retry_authorization)
+    Braintree::Transaction.expects(:submit_for_settlement).with(retry_transaction.id).returns(stub(:success? => false, :errors => []))
+
+    subject.save_customer_and_subscription!(:card_number => "4111111111",
+                                            :verification_code => "124",
+                                            :expiration_month => 6,
+                                            :expiration_year => 2012).should_not be
+
+    subject.errors[:card_number].should include("was denied by the payment processor with the message: no good")
+    subject.reload.subscription_status.should == Braintree::Subscription::Status::PastDue
+    subject.next_billing_date.to_s.should == subscription["next_billing_date"].to_s
+  end
+
+  it "retries the subscription charge and updates the subscription when the settlement fails" do
+    subscription = FakeBraintree.subscriptions[subject.subscription_token]
+    subscription["status"] = Braintree::Subscription::Status::PastDue
+    subscription["next_billing_date"] = 1.day.from_now
+    FakeBraintree.transaction = { :status => Braintree::Transaction::Status::Failed,
+                                  :subscription_id => subject.subscription_token }
+    transaction = FakeBraintree.generated_transaction
+    subscription["transactions"] = [transaction]
+
+    retry_transaction = stub(:id => "12345", :status => "")
+    retry_authorization = stub(:transaction => retry_transaction)
+
+    Braintree::Subscription.expects(:retry_charge).with(subject.subscription_token).returns(retry_authorization)
+    Braintree::Transaction.expects(:submit_for_settlement).with(retry_transaction.id).returns(stub(:success? => false, :errors => [stub(:attribute => 'number', :message => 'no good')]))
+
+    subject.save_customer_and_subscription!(:card_number => "4111111111",
+                                            :verification_code => "124",
+                                            :expiration_month => 6,
+                                            :expiration_year => 2012).should_not be
+
+    subject.errors[:card_number].should include("no good")
+    subject.reload.subscription_status.should == Braintree::Subscription::Status::PastDue
+    subject.next_billing_date.to_s.should == subscription["next_billing_date"].to_s
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: saucy
 version: !ruby/object:Gem::Version 
-  hash: 53
+  hash: 51
   prerelease: false
   segments: 
   - 0
   - 2
-  - 17
-  version: 0.2.17
+  - 18
+  version: 0.2.18
 platform: ruby
 authors: 
 - thoughtbot, inc.
@@ -17,14 +17,13 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-02-01 00:00:00 -05:00
+date: 2011-02-09 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  type: :runtime
-  prerelease: false
   name: formtastic
-  version_requirements: &id001 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -34,12 +33,12 @@ dependencies:
         - 1
         - 2
         version: "1.2"
-  requirement: *id001
-- !ruby/object:Gem::Dependency 
   type: :runtime
-  prerelease: false
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
   name: railties
-  version_requirements: &id002 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -50,12 +49,12 @@ dependencies:
         - 0
         - 3
         version: 3.0.3
-  requirement: *id002
-- !ruby/object:Gem::Dependency 
   type: :runtime
-  prerelease: false
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
   name: braintree
-  version_requirements: &id003 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -66,12 +65,12 @@ dependencies:
         - 6
         - 2
         version: 2.6.2
-  requirement: *id003
-- !ruby/object:Gem::Dependency 
   type: :runtime
-  prerelease: false
+  version_requirements: *id003
+- !ruby/object:Gem::Dependency 
   name: sham_rack
-  version_requirements: &id004 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -82,12 +81,12 @@ dependencies:
         - 3
         - 3
         version: 1.3.3
-  requirement: *id004
-- !ruby/object:Gem::Dependency 
   type: :runtime
-  prerelease: false
+  version_requirements: *id004
+- !ruby/object:Gem::Dependency 
   name: sinatra
-  version_requirements: &id005 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -98,12 +97,12 @@ dependencies:
         - 1
         - 2
         version: 1.1.2
-  requirement: *id005
+  type: :runtime
+  version_requirements: *id005
 - !ruby/object:Gem::Dependency 
-  type: :development
-  prerelease: false
   name: aruba
-  version_requirements: &id006 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -114,7 +113,8 @@ dependencies:
         - 2
         - 6
         version: 0.2.6
-  requirement: *id006
+  type: :development
+  version_requirements: *id006
 description: Clearance-based Rails engine for Software as a Service (Saas) that provides account and project management
 email: support@thoughtbot.com
 executables: []
@@ -205,6 +205,7 @@ files:
 - lib/saucy/project.rb
 - lib/saucy/projects_controller.rb
 - lib/saucy/routing_extensions.rb
+- lib/saucy/subscription.rb
 - lib/saucy/user.rb
 - lib/saucy.rb
 - lib/saucy/railties/tasks.rake
@@ -242,6 +243,7 @@ files:
 - spec/models/plan_spec.rb
 - spec/models/project_spec.rb
 - spec/models/signup_spec.rb
+- spec/models/subscription_spec.rb
 - spec/models/user_spec.rb
 - spec/route_extensions_spec.rb
 - spec/saucy_spec.rb