Merge pull request #83 from nickhammond/payment_profile

Update payment_profile on subscriptions to return active payment profile

lib/chargify_api_ares/resources/subscription.rb

@@ -34,7 +34,11 @@ def events(params = {})
     end
 
     def payment_profile
-      self.respond_to?('credit_card') ? credit_card : nil
+      if self.respond_to?('credit_card')
+        credit_card
+      elsif self.respond_to?('bank_account')
+        bank_account
+      end
     end
 
     # Perform a one-time charge on an existing subscription.

spec/resources/subscription_spec.rb

@@ -31,6 +31,27 @@
       lambda { subscription.save! }.should_not change(subscription, :attributes)
     end
   end
+
+  describe "points to the correct payment profile" do
+    before do
+      @subscription = build(:subscription)
+    end
+
+    it 'does not have a payment profile' do
+      @subscription.payment_profile.should be_nil
+    end
+
+    it 'returns credit_card details' do
+      @subscription.credit_card = "CREDIT CARD"
+      @subscription.payment_profile.should == "CREDIT CARD"
+    end
+
+    it 'returns bank_account details' do
+      @subscription.bank_account = "BANK ACCOUNT"
+      @subscription.payment_profile.should == "BANK ACCOUNT"
+    end
+
+  end
 
   it 'creates a one-time charge' do
     id = generate(:subscription_id)