catarse_pagarme 2.4.10 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 167056c394c65f4907196b570a631367b03fe471
-  data.tar.gz: 6dedfa0adf8a6b398e5967d320851adcc21eb637
+  metadata.gz: e52946e7b76e2534a9d25953d63cb5630f6ef1a5
+  data.tar.gz: 824db7ed69c85c3f83bee7c916e7ebee07184b98
 SHA512:
-  metadata.gz: 0230b0a36302dace3afd5bdbeb2cc5e27a021260e096950b062353d5ec50479ae80eae3f74308b5e993e1ea00d8444dfa53df690d492a9f9d866ccc1965ccd58
-  data.tar.gz: f59f6667a1237763d93fc3dfec731562eb355a89f666777e677a4f1a82a1ea4735ffdc96593169f69fca1631d6645d6f29290e5d28680939ebecc5736b0f7322
+  metadata.gz: c7db17756e8ff0b80f92ad14e31e2ad05f0fde8e4be4bb0bb6e58363703730ebd08a2dcb8cb2e760394e5568657db7c8196e27e3c4c4ac47b5f4d330bc8eb690
+  data.tar.gz: ce404a55c7b605940d757e36e6199edd28d07127c7593b90330447fc2624a8daef2634c10d7e1897f28af3535f2aad063ba6847331f818aa0fef0ff3b6a63dbd

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    catarse_pagarme (2.4.10)
+    catarse_pagarme (2.5.0)
       pagarme (~> 1.9.8)
       rails (~> 4.0)
 
@@ -51,7 +51,7 @@ GEM
       treetop (~> 1.4.8)
     mime-types (1.25.1)
     minitest (5.4.0)
-    multi_json (1.10.1)
+    multi_json (1.11.0)
     netrc (0.10.3)
     pagarme (1.9.9)
       multi_json

data/README.md

@@ -24,6 +24,7 @@ create an (config/initializers/pagarme.rb) and configure with:
 ```
 CatarsePagarme.configure do |config|
   config.api_key = "API_KEY"
+  config.ecr_key = "ENCRYPTION KEY"
   config.slip_tax = "Slip payment tax"
   config.credit_card_tax = "Credit card transaction tax don't need to define the 0.39"
   config.interest_rate = "Interest rate for installments"

data/app/assets/javascripts/catarse_pagarme/credit_card.js

@@ -35,28 +35,6 @@ App.views.Pagarme.addChild('PaymentCard', _.extend({
     this.$('#payment_card_source').payment('formatCardCVC');
   },
 
-  getUrl: function(){
-    var that = this;
-    var url = '';
-
-    if(that.$('input#payment_save_card').prop('checked') || that.hasSelectedSomeCard()) {
-      url = '/payment/pagarme/'+that.parent.contributionId+'/pay_with_subscription';
-    } else {
-      url = '/payment/pagarme/'+that.parent.contributionId+'/pay_credit_card';
-    }
-
-    return url;
-  },
-
-  getAjaxType: function() {
-    var type = 'POST';
-
-    if(this.hasSelectedSomeCard()) {
-      type = 'PUT'
-    }
-    return type;
-  },
-
   selectedCard: function() {
     return this.$('input:radio[data-stored][name=payment_subscription_card]:checked');
   },
@@ -65,26 +43,6 @@ App.views.Pagarme.addChild('PaymentCard', _.extend({
     return this.selectedCard().length > 0;
   },
 
-  getPaymentData: function() {
-    var data = {};
-
-    if(this.hasSelectedSomeCard()) {
-      data = {
-        subscription_id: this.selectedCard().val(),
-        payment_card_installments: this.getInstallments() }
-    } else {
-      data = {
-        payment_card_number: this.$('input#payment_card_number').val(),
-        payment_card_name: this.$('input#payment_card_name').val(),
-        payment_card_date: this.$('input#payment_card_date').val(),
-        payment_card_source: this.$('input#payment_card_source').val(),
-        payment_card_installments: this.getInstallments()
-      }
-    }
-
-    return data;
-  },
-
   getInstallments: function() {
     if(this.hasSelectedSomeCard()) {
       return this.$('.my-credit-cards .selected select#payment_card_installments').val();
@@ -104,18 +62,77 @@ App.views.Pagarme.addChild('PaymentCard', _.extend({
     $(e.currentTarget).hide();
     that.parent.loader.show();
 
+    if( that.hasSelectedSomeCard() ) {
+      that.requestPayment({
+        card_id: this.selectedCard().val(),
+        payment_card_installments: that.getInstallments()
+      });
+    } else {
+      PagarMe.encryption_key = this.$('.pagarme-e-key').data('key');
+
+      var creditCard = this.newCreditCard();
+      var fieldErrors = creditCard.fieldErrors();
+
+      if(_.keys(fieldErrors).length > 0) {
+        this.displayErrors(fieldErrors);
+      } else {
+        this.generateCardHash(creditCard);
+      }
+    }
+
+    return false;
+  },
+
+  generateCardHash: function(creditCard){
+    var that = this;
+    creditCard.generateHash(function(cardHash) {
+      that.requestPayment({
+        card_hash: cardHash,
+        payment_card_installments: that.getInstallments(),
+        save_card: that.$('input#payment_save_card').is(':checked')
+      });
+    });
+  },
+
+  displayErrors: function(errors){
+    var msg = [];
+    this.parent.loader.hide();
+
+    $.each(errors, function(i, value){
+      msg.push(value)
+    });
+
+    this.message.find('.message-text').html(msg.join("<br/>"));
+    this.message.slideDown('slow');
+
+    $("#credit_card_submit").show();
+  },
+
+  newCreditCard: function(){
+    var creditCard = new PagarMe.creditCard();
+    creditCard.cardHolderName = this.$('input#payment_card_name').val();
+    creditCard.cardExpirationMonth = $.trim(this.$('input#payment_card_date').val().split('/')[0]);
+    creditCard.cardExpirationYear = $.trim(this.$('input#payment_card_date').val().split('/')[1]);
+    creditCard.cardNumber = this.$('input#payment_card_number').val();
+    creditCard.cardCVV = this.$('input#payment_card_source').val();
+    return creditCard;
+  },
+
+  requestPayment: function(data){
+    var that = this;
+
     $.ajax({
-      type: that.getAjaxType(),
-      url: that.getUrl(),
-      data: that.getPaymentData(),
+      type: 'POST',
+      url: '/payment/pagarme/'+that.parent.contributionId+'/pay_credit_card',
+      data: data,
       success: function(response){
         that.parent.loader.hide();
 
         if(response.payment_status == 'failed'){
           that.message.find('.message-text').html(response.message);
-          that.message.slideDown('slow')
+          that.message.slideDown('slow');
 
-          $(e.currentTarget).show();
+          $("#credit_card_submit").show();
         } else {
           var thank_you = $('#project_review').data('thank-you-path');
 
@@ -127,6 +144,7 @@ App.views.Pagarme.addChild('PaymentCard', _.extend({
         }
       }
     });
+
   },
 
   onKeyupPaymentCardNumber: function(e){

data/app/controllers/catarse_pagarme/application_controller.rb

@@ -5,7 +5,6 @@ module CatarsePagarme
 
     before_filter :authenticate_user!
     before_filter :configure_pagarme
-    skip_before_filter :force_http
     layout :false
 
     protected

data/app/controllers/catarse_pagarme/credit_cards_controller.rb

@@ -1,8 +1,55 @@
 module CatarsePagarme
-  class CreditCardsController < CreditCardBaseController
+  class CreditCardsController < CatarsePagarme::ApplicationController
 
     def create
-      charge_with_class CreditCardTransaction
+      transaction = CreditCardTransaction.new(credit_card_attributes, contribution).charge!
+
+      render json: { payment_status: transaction.status }
+    rescue Exception => e
+      render json: { payment_status: 'failed', message: e.message }
+    end
+
+    protected
+
+    def credit_card_attributes
+      hash = {
+        payment_method: 'credit_card',
+        amount: delegator.value_with_installment_tax(get_installment),
+        postback_url: ipn_pagarme_index_url(host: CatarsePagarme.configuration.host,
+                                            subdomain: CatarsePagarme.configuration.subdomain,
+                                            protocol: CatarsePagarme.configuration.protocol),
+        installments: get_installment,
+        customer: {
+          email: contribution.user.email,
+          name: contribution.user.name
+        },
+        metadata: {
+          key: contribution.key
+        }
+      }
+
+      if params[:card_hash].present?
+        hash[:card_hash] = params[:card_hash]
+      else
+        hash[:card_id] = params[:card_id]
+      end
+
+      if params[:save_card] === "true"
+        hash[:save_card] = true
+      end
+
+      hash
     end
+
+    def get_installment
+      if contribution.value.to_f < CatarsePagarme.configuration.minimum_value_for_installment.to_f
+        1
+      elsif params[:payment_card_installments].to_i > 0
+        params[:payment_card_installments].to_i
+      else
+        1
+      end
+    end
+
   end
 end

data/app/controllers/catarse_pagarme/notifications_controller.rb

@@ -1,7 +1,6 @@
 module CatarsePagarme
   class NotificationsController < CatarsePagarme::ApplicationController
     skip_before_filter :authenticate_user!
-    skip_before_filter :force_http
 
     def create
       if contribution

data/app/models/catarse_pagarme/credit_card_transaction.rb

@@ -1,15 +1,31 @@
 module CatarsePagarme
   class CreditCardTransaction < TransactionBase
+
     def charge!
+      save_card = self.attributes.delete(:save_card)
+
       self.transaction = PagarMe::Transaction.new(self.attributes)
       self.transaction.charge
 
+      change_contribution_state
+
       if self.transaction.status == 'refused'
         raise ::PagarMe::PagarMeError.new(I18n.t('projects.contributions.edit.transaction_error'))
       end
 
-      change_contribution_state
+      save_user_credit_card if save_card
       self.transaction
     end
+
+    def save_user_credit_card
+      card = self.transaction.card
+
+      credit_card = self.user.credit_cards.find_or_initialize_by(card_key: card.id)
+      credit_card.last_digits = card.last_digits
+      credit_card.card_brand = card.brand
+
+      credit_card.save!
+    end
+
   end
 end

data/app/models/catarse_pagarme/transaction_base.rb

@@ -1,7 +1,7 @@
 module CatarsePagarme
   class TransactionBase
     attr_accessor :attributes, :contribution,
-      :transaction, :user, :subscription
+      :transaction, :user
 
     def initialize(attributes, contribution)
       self.attributes = attributes

data/app/views/catarse_pagarme/pagarme/review.html.slim

@@ -1,6 +1,6 @@
-= javascript_include_tag 'catarse_pagarme'
-#catarse_pagarme_form
+= javascript_include_tag 'catarse_pagarme', 'https://pagar.me/assets/pagarme-v2.min.js'
 
+#catarse_pagarme_form
   .next_step_after_valid_document
     .w-row.list_payment.w-form
       .w-col.w-col-7.w-col-small-7.back-payment-moip-options-column
@@ -16,13 +16,14 @@
 
   #payment_type_credit_card_section.payment_section
     = form_tag 'javascript:void(0);', :class => 'pagarme' do
+      .pagarme-e-key data-key=CatarsePagarme.configuration.ecr_key
       div[class="my-credit-cards w-form back-payment-form-creditcard records-choice #{(current_user.credit_cards.present? ? '' : 'w-hidden')}"]
         - current_user.credit_cards.each_with_index do |credit_card, i|
-          = label_tag "payment_subscription_card_#{credit_card.subscription_id}" do
+          = label_tag "payment_subscription_card_#{credit_card.card_key}" do
             .w-row.creditcard-records
               .w-col.w-col-1
                 .w-radio.w-clearfix.back-payment-credit-card-radio-field
-                  = radio_button_tag :payment_subscription_card, credit_card.subscription_id, i == 0, {"data-stored" => true}
+                  = radio_button_tag :payment_subscription_card, credit_card.card_key, i == 0, {"data-stored" => true}
               .w-col.w-col-2
                 .fontsize-small.fontweight-semibold.text-success
                   = credit_card.card_brand.upcase

data/config/routes.rb

@@ -6,8 +6,6 @@ CatarsePagarme::Engine.routes.draw do
       put :second_slip, to: 'slip#update'
       post :pay_credit_card, to: 'credit_cards#create'
       post :pay_slip, to: 'slip#create'
-      post :pay_with_subscription, to: 'subscriptions#create'
-      put :pay_with_subscription, to: 'subscriptions#update'
     end
 
     collection do

data/lib/catarse_pagarme/configuration.rb

@@ -3,10 +3,11 @@ module CatarsePagarme
     attr_accessor :api_key, :slip_tax, :credit_card_tax, :interest_rate, :host, :subdomain, :protocol,
       :max_installments, :minimum_value_for_installment, :credit_card_cents_fee, :pagarme_tax, :stone_tax,
       :cielo_tax, :cielo_installment_diners_tax, :cielo_installment_not_diners_tax,
-      :cielo_installment_amex_tax, :cielo_installment_not_amex_tax
+      :cielo_installment_amex_tax, :cielo_installment_not_amex_tax, :ecr_key
 
     def initialize
       self.api_key = ''
+      self.ecr_key = ''
       self.slip_tax = 0
       self.credit_card_tax = 0
       self.interest_rate = 0

data/lib/catarse_pagarme/engine.rb

@@ -4,7 +4,6 @@ module CatarsePagarme
 
     config.to_prepare do
       ::Contribution.send(:include, CatarsePagarme::ContributionConcern)
-      ::CreditCard.send(:include, CatarsePagarme::CreditCardConcern)
     end
   end
 end

data/lib/catarse_pagarme/version.rb

@@ -1,3 +1,3 @@
 module CatarsePagarme
-  VERSION = "2.4.10"
+  VERSION = "2.5.0"
 end

data/spec/controllers/catarse_pagarme/credit_cards_controller_spec.rb

@@ -8,7 +8,6 @@ describe CatarsePagarme::CreditCardsController do
 
   let(:project) { create(:project, goal: 10_000, state: 'online') }
   let(:contribution) { create(:contribution, value: 10, project: project) }
-  let(:credit_card) { create(:credit_card, subscription_id: '1542')}
 
   describe 'pay with credit card' do
     context  'without an user' do
@@ -27,13 +26,7 @@ describe CatarsePagarme::CreditCardsController do
         before do
           post :create, {
             locale: :pt, project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme',
-            payment_card_date: '10/17', payment_card_number: '4012888888881881', payment_card_name: 'Foo bar',
-            payment_card_source: '574', payment_card_installments: '1' }
-
-        end
-
-        it 'payment_status should be filled' do
-          expect(ActiveSupport::JSON.decode(response.body)['payment_status']).not_to be_nil
+            card_hash: sample_card_hash }
         end
 
         it 'and payment_status is not failed' do
@@ -44,18 +37,12 @@ describe CatarsePagarme::CreditCardsController do
       context 'with invalid card data' do
         before do
           post :create, {
-            locale: :pt, project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme',
-            payment_card_date: '', payment_card_name: '',
-            payment_card_source: '143', payment_card_installments: '1' }
+            locale: :pt, project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme', card_hash: "abcd" }
         end
 
         it 'payment_status should be failed' do
           expect(ActiveSupport::JSON.decode(response.body)['payment_status']).to eq('failed')
         end
-
-        it 'message should be filled' do
-          expect(ActiveSupport::JSON.decode(response.body)['message']).not_to be_nil
-        end
       end
     end
   end

data/spec/spec_helper.rb

@@ -4,6 +4,7 @@ ENV["RAILS_ENV"] ||= 'test'
 require File.expand_path("../dummy/config/environment", __FILE__)
 
 require 'pagarme'
+require 'open-uri'
 require 'rspec/rails'
 require 'factory_girl'
 # Requires supporting ruby files with custom matchers and macros, etc,
@@ -34,3 +35,12 @@ RSpec.configure do |config|
     PaymentEngines.stub(:configuration).and_return({})
   end
 end
+
+def sample_card_hash
+  r = open("https://api.pagar.me/1/transactions/card_hash_key/?encryption_key=ek_test_zwAwnRvqD1c9GUERQ7oAP4FPyN9o2v").read
+  json = ActiveSupport::JSON.decode(r)
+  public_key = OpenSSL::PKey::RSA.new(json["public_key"])
+  encode_string = Base64.encode64(public_key.public_encrypt("card_number=4901720080344448&card_holder_name=Usuario de Teste&card_expiration_date=1220&card_cvv=314"))
+
+  "#{json['id']}_#{encode_string}"
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: catarse_pagarme
 version: !ruby/object:Gem::Version
-  version: 2.4.10
+  version: 2.5.0
 platform: ruby
 authors:
 - Antônio Roberto Silva
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-02 00:00:00.000000000 Z
+date: 2015-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -103,24 +103,18 @@ files:
 - app/assets/javascripts/catarse_pagarme/payment_slip.js
 - app/assets/stylesheets/catarse_pagarme/application.css
 - app/controllers/catarse_pagarme/application_controller.rb
-- app/controllers/catarse_pagarme/credit_card_base_controller.rb
 - app/controllers/catarse_pagarme/credit_cards_controller.rb
 - app/controllers/catarse_pagarme/notifications_controller.rb
 - app/controllers/catarse_pagarme/pagarme_controller.rb
 - app/controllers/catarse_pagarme/slip_controller.rb
-- app/controllers/catarse_pagarme/subscriptions_controller.rb
 - app/helpers/catarse_pagarme/application_helper.rb
 - app/models/catarse_pagarme/bank_account_concern.rb
 - app/models/catarse_pagarme/contribution_concern.rb
 - app/models/catarse_pagarme/contribution_delegator.rb
-- app/models/catarse_pagarme/credit_card_concern.rb
-- app/models/catarse_pagarme/credit_card_delegator.rb
 - app/models/catarse_pagarme/credit_card_transaction.rb
 - app/models/catarse_pagarme/fee_calculator_concern.rb
 - app/models/catarse_pagarme/payment_type.rb
-- app/models/catarse_pagarme/save_credit_card_transaction.rb
 - app/models/catarse_pagarme/slip_transaction.rb
-- app/models/catarse_pagarme/subscription_transaction.rb
 - app/models/catarse_pagarme/transaction_base.rb
 - app/views/catarse_pagarme/pagarme/_terms.html.slim
 - app/views/catarse_pagarme/pagarme/review.html.slim
@@ -139,7 +133,6 @@ files:
 - spec/controllers/catarse_pagarme/notifications_controller_spec.rb
 - spec/controllers/catarse_pagarme/pagarme_controller_spec.rb
 - spec/controllers/catarse_pagarme/slip_controller_spec.rb
-- spec/controllers/catarse_pagarme/subscriptions_controller_spec.rb
 - spec/dummy/README.rdoc
 - spec/dummy/Rakefile
 - spec/dummy/app/assets/images/.keep
@@ -226,7 +219,6 @@ test_files:
 - spec/controllers/catarse_pagarme/notifications_controller_spec.rb
 - spec/controllers/catarse_pagarme/pagarme_controller_spec.rb
 - spec/controllers/catarse_pagarme/slip_controller_spec.rb
-- spec/controllers/catarse_pagarme/subscriptions_controller_spec.rb
 - spec/dummy/README.rdoc
 - spec/dummy/Rakefile
 - spec/dummy/app/assets/images/.keep

data/app/controllers/catarse_pagarme/credit_card_base_controller.rb

@@ -1,53 +0,0 @@
-module CatarsePagarme
-  class CreditCardBaseController < CatarsePagarme::ApplicationController
-
-    protected
-
-    def charge_with_class(kclass)
-      transaction = kclass.new(credit_card_attributes, contribution).charge!
-
-      render json: { payment_status: transaction.status }
-    rescue Exception => e
-      render json: { payment_status: 'failed', message: e.message }
-    end
-
-    def splited_month_and_year
-      params[:payment_card_date].split('/')
-    rescue
-      [0, 0]
-    end
-
-    def get_installment
-      if contribution.value.to_f < CatarsePagarme.configuration.minimum_value_for_installment.to_f
-        1
-      elsif params[:payment_card_installments].to_i > 0
-        params[:payment_card_installments].to_i
-      else
-        1
-      end
-    end
-
-    def credit_card_attributes
-      {
-        payment_method: 'credit_card',
-        card_number: params[:payment_card_number],
-        card_holder_name: params[:payment_card_name],
-        card_expiration_month: splited_month_and_year[0],
-        card_expiration_year: splited_month_and_year[1],
-        card_cvv: params[:payment_card_source],
-        amount: delegator.value_with_installment_tax(get_installment),
-        postback_url: ipn_pagarme_index_url(host: CatarsePagarme.configuration.host,
-                                            subdomain: CatarsePagarme.configuration.subdomain,
-                                            protocol: CatarsePagarme.configuration.protocol),
-        installments: get_installment,
-        customer: {
-          email: contribution.user.email,
-          name: contribution.user.name
-        },
-        metadata: {
-          key: contribution.key 
-        }
-      }
-    end
-  end
-end

data/app/controllers/catarse_pagarme/subscriptions_controller.rb

@@ -1,27 +0,0 @@
-module CatarsePagarme
-  class SubscriptionsController < CreditCardBaseController
-    def create
-      charge_with_class SaveCreditCardTransaction
-    end
-
-    def update
-      charge_with_class SubscriptionTransaction
-    end
-
-    protected
-
-    def credit_card_attributes
-      attributes = super
-
-      if has_subscription?
-        attributes.update({subscription_id: params[:subscription_id]})
-      end
-
-      attributes
-    end
-
-    def has_subscription?
-      params[:subscription_id].present?
-    end
-  end
-end

data/app/models/catarse_pagarme/credit_card_concern.rb

@@ -1,9 +0,0 @@
-module CatarsePagarme::CreditCardConcern
-  extend ActiveSupport::Concern
-
-  included do
-    def pagarme_delegator
-      CatarsePagarme::CreditCardDelegator.new(self)
-    end
-  end
-end

data/app/models/catarse_pagarme/credit_card_delegator.rb

@@ -1,26 +0,0 @@
-module CatarsePagarme
-  class CreditCardDelegator
-    attr_accessor :credit_card, :subscription
-
-    def initialize(credit_card)
-      configure_pagarme
-      self.credit_card = credit_card
-    end
-
-    def cancel_subscription
-      get_subscription
-      self.subscription.cancel
-    end
-
-    def get_subscription
-      self.subscription ||= ::PagarMe::Subscription.find_by_id(self.credit_card.subscription_id)
-    end
-
-
-    protected
-
-    def configure_pagarme
-      ::PagarMe.api_key = CatarsePagarme.configuration.api_key
-    end
-  end
-end

data/app/models/catarse_pagarme/save_credit_card_transaction.rb

@@ -1,33 +0,0 @@
-module CatarsePagarme
-  class SaveCreditCardTransaction < TransactionBase
-    def charge!
-      installments = self.attributes.delete(:installments)
-      amount = self.attributes.delete(:amount)
-
-      self.subscription = PagarMe::Subscription.new(self.attributes)
-      self.subscription.create
-
-      self.subscription.charge(amount, installments)
-      self.transaction = subscription.current_transaction
-
-      save_user_credit_card
-      change_contribution_state
-
-
-      self.transaction
-    end
-
-    protected
-
-    def save_user_credit_card
-      credit_cards = self.user.credit_cards
-      unless credit_cards.where(subscription_id: self.subscription.id.to_s).present?
-        credit_cards.create!({
-          last_digits: self.subscription.card_last_digits,
-          card_brand: self.subscription.card_brand,
-          subscription_id: self.subscription.id
-        })
-      end
-    end
-  end
-end

data/app/models/catarse_pagarme/subscription_transaction.rb

@@ -1,21 +0,0 @@
-module CatarsePagarme
-  class SubscriptionTransaction < TransactionBase
-    def charge!
-      validate_subscription_id
-
-      self.subscription = PagarMe::Subscription.find_by_id(self.attributes[:subscription_id])
-      self.subscription.charge(self.attributes[:amount], self.attributes[:installments])
-      self.transaction = subscription.current_transaction
-
-      change_contribution_state
-
-      self.transaction
-    end
-
-    def validate_subscription_id
-      unless self.user.credit_cards.where(subscription_id: self.attributes[:subscription_id]).present?
-        raise PagarMe::PagarMeError.new('invalid subscription')
-      end
-    end
-  end
-end

data/spec/controllers/catarse_pagarme/subscriptions_controller_spec.rb

@@ -1,95 +0,0 @@
-require 'spec_helper'
-
-describe CatarsePagarme::SubscriptionsController do
-  before do
-    PaymentEngines.stub(:find_payment).and_return(contribution)
-    controller.stub(:current_user).and_return(user)
-  end
-
-  let(:project) { create(:project, goal: 10_000, state: 'online') }
-  let(:contribution) { create(:contribution, value: 10, project: project) }
-  let(:credit_card) { create(:credit_card, subscription_id: '1542')}
-
-  describe "pay with a saved credit card" do
-    context 'without an user' do
-      let(:user) { nil }
-
-      it 'should raise a error' do
-        expect {
-          post :create, { locale: :pt, project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme' }
-        }.to raise_error('invalid user')
-      end
-    end
-
-    context 'with an user' do
-      let(:user) { contribution.user }
-      context 'when subscription_id not associated with contribution user' do
-        before do
-          put :update, { locale: :pt, subscription_id: '1542', project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme' }
-        end
-
-        it 'payment_status should be failed' do
-          expect(ActiveSupport::JSON.decode(response.body)['payment_status']).to eq('failed')
-        end
-
-        it 'message should be filled' do
-          expect(ActiveSupport::JSON.decode(response.body)['message']).to eq('invalid subscription')
-        end
-      end
-    end
-  end
-
-  describe 'pay and save credit card' do
-    context 'without an user' do
-      let(:user) { nil }
-
-      it 'should raise a error' do
-        expect {
-          post :create, { locale: :pt, project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme' }
-        }.to raise_error('invalid user')
-      end
-    end
-
-    context "with an user" do
-      let(:user) { contribution.user }
-
-      context 'with invalid card data' do
-        before do
-          post :create, {
-            locale: :pt, project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme',
-            payment_card_date: '', payment_card_name: '',
-            payment_card_source: '143', payment_card_installments: '1' }
-        end
-
-        it 'payment_status should be failed' do
-          expect(ActiveSupport::JSON.decode(response.body)['payment_status']).to eq('failed')
-        end
-
-        it 'message should be filled' do
-          expect(ActiveSupport::JSON.decode(response.body)['message']).not_to be_nil
-        end
-      end
-
-      context "with valid card data" do
-        before do
-          post :create, {
-            locale: :pt, project_id: project.id, contribution_id: contribution.id, use_route: 'catarse_pagarme',
-            payment_card_date: '10/17', payment_card_number: '4901720080344448', payment_card_name: 'Foo bar',
-            payment_card_source: '143', payment_card_installments: '1' }
-        end
-
-        it "should save card data into user" do
-          expect(user.credit_cards.count).to eq(1)
-        end
-
-        it 'payment_status should be filled and not failed' do
-          expect(ActiveSupport::JSON.decode(response.body)['payment_status']).not_to be_nil
-        end
-
-        it 'and payment_status is not failed' do
-          expect(ActiveSupport::JSON.decode(response.body)['payment_status']).not_to eq('failed')
-        end
-      end
-    end
-  end
-end