catarse_moip 1.0.9 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3192db8fdc1dc785ba708c1c0ecb1e083310dd76
-  data.tar.gz: 3219cc44b27bc5c88674aeb6a179f93915f529c9
+  metadata.gz: 4a467ec8f386bd6254b46c9052512702d5ef580e
+  data.tar.gz: 659e2d260728031a7703deb3ef421033e2ff53a7
 SHA512:
-  metadata.gz: 584a1d8e2f28b0571a0eaeab21146b8d5719212f36d14db063db59e7ada634d2a28b0f3274989f83a2b683663816bb6bb978281278dd05d0c48b2e53ad2b37b3
-  data.tar.gz: 01eac27fc7aac23e51b8bb875a257316accc18c1a3fd1c64b0f6a9321dc66303981d33411578de01ceb99f3a270703586694b58a9c4227ac0d53bf94a0d85cc5
+  metadata.gz: 290f04967235c67f428cd0e16a605dbdc2b1e28a1628c0add9886fc5e577e5b730417eff9d6606bee69743e665af6deb1ff7eaf5e916d9490849d5b3e561ef91
+  data.tar.gz: 72738a73a913e8c5b38e9778c534bf52b7f2295077d4e6fd3d65ff84d834665ee23e2feaea280bfe4a6604d21044a1667eb328ad276122be5c409d82f9af0532

data/Gemfile.lock

@@ -10,46 +10,44 @@ GIT
 PATH
   remote: .
   specs:
-    catarse_moip (1.0.9)
+    catarse_moip (2.0.0)
       libxml-ruby (~> 2.6.0)
-      rails (~> 3.2.12)
+      rails (~> 4.0)
 
 GEM
   remote: http://rubygems.org/
   specs:
-    actionmailer (3.2.13)
-      actionpack (= 3.2.13)
+    actionmailer (4.0.0)
+      actionpack (= 4.0.0)
       mail (~> 2.5.3)
-    actionpack (3.2.13)
-      activemodel (= 3.2.13)
-      activesupport (= 3.2.13)
-      builder (~> 3.0.0)
+    actionpack (4.0.0)
+      activesupport (= 4.0.0)
+      builder (~> 3.1.0)
       erubis (~> 2.7.0)
-      journey (~> 1.0.4)
-      rack (~> 1.4.5)
-      rack-cache (~> 1.2)
-      rack-test (~> 0.6.1)
-      sprockets (~> 2.2.1)
-    activemodel (3.2.13)
-      activesupport (= 3.2.13)
-      builder (~> 3.0.0)
-    activerecord (3.2.13)
-      activemodel (= 3.2.13)
-      activesupport (= 3.2.13)
-      arel (~> 3.0.2)
-      tzinfo (~> 0.3.29)
-    activeresource (3.2.13)
-      activemodel (= 3.2.13)
-      activesupport (= 3.2.13)
-    activesupport (3.2.13)
-      i18n (= 0.6.1)
-      multi_json (~> 1.0)
-    arel (3.0.2)
-    builder (3.0.4)
+      rack (~> 1.5.2)
+      rack-test (~> 0.6.2)
+    activemodel (4.0.0)
+      activesupport (= 4.0.0)
+      builder (~> 3.1.0)
+    activerecord (4.0.0)
+      activemodel (= 4.0.0)
+      activerecord-deprecated_finders (~> 1.0.2)
+      activesupport (= 4.0.0)
+      arel (~> 4.0.0)
+    activerecord-deprecated_finders (1.0.3)
+    activesupport (4.0.0)
+      i18n (~> 0.6, >= 0.6.4)
+      minitest (~> 4.2)
+      multi_json (~> 1.3)
+      thread_safe (~> 0.1)
+      tzinfo (~> 0.3.37)
+    arel (4.0.0)
+    atomic (1.1.13)
+    builder (3.1.4)
     crack (0.1.8)
-    database_cleaner (1.0.1)
+    database_cleaner (1.1.1)
     diff-lcs (1.2.4)
-    enumerate_it (1.0.3)
+    enumerate_it (1.1.1)
       activesupport (>= 3.0.0)
     erubis (2.7.0)
     factory_girl (4.2.0)
@@ -60,62 +58,59 @@ GEM
     hike (1.2.3)
     httparty (0.6.1)
       crack (= 0.1.8)
-    i18n (0.6.1)
-    journey (1.0.4)
-    json (1.8.0)
+    i18n (0.6.5)
     libxml-ruby (2.6.0)
     mail (2.5.4)
       mime-types (~> 1.16)
       treetop (~> 1.4.8)
-    mime-types (1.25)
-    multi_json (1.7.6)
-    nokogiri (1.5.9)
-    pg (0.15.1)
+    mime-types (1.24)
+    minitest (4.7.5)
+    multi_json (1.7.9)
+    nokogiri (1.5.10)
+    pg (0.16.0)
     polyglot (0.3.3)
-    rack (1.4.5)
-    rack-cache (1.2)
-      rack (>= 0.4)
-    rack-ssl (1.3.3)
-      rack
+    rack (1.5.2)
     rack-test (0.6.2)
       rack (>= 1.0)
-    rails (3.2.13)
-      actionmailer (= 3.2.13)
-      actionpack (= 3.2.13)
-      activerecord (= 3.2.13)
-      activeresource (= 3.2.13)
-      activesupport (= 3.2.13)
-      bundler (~> 1.0)
-      railties (= 3.2.13)
-    railties (3.2.13)
-      actionpack (= 3.2.13)
-      activesupport (= 3.2.13)
-      rack-ssl (~> 1.3.2)
+    rails (4.0.0)
+      actionmailer (= 4.0.0)
+      actionpack (= 4.0.0)
+      activerecord (= 4.0.0)
+      activesupport (= 4.0.0)
+      bundler (>= 1.3.0, < 2.0)
+      railties (= 4.0.0)
+      sprockets-rails (~> 2.0.0)
+    railties (4.0.0)
+      actionpack (= 4.0.0)
+      activesupport (= 4.0.0)
       rake (>= 0.8.7)
-      rdoc (~> 3.4)
-      thor (>= 0.14.6, < 2.0)
-    rake (10.0.4)
-    rdoc (3.12.2)
-      json (~> 1.4)
-    rspec-core (2.13.1)
-    rspec-expectations (2.13.0)
+      thor (>= 0.18.1, < 2.0)
+    rake (10.1.0)
+    rspec-core (2.14.5)
+    rspec-expectations (2.14.2)
       diff-lcs (>= 1.1.3, < 2.0)
-    rspec-mocks (2.13.1)
-    rspec-rails (2.13.2)
+    rspec-mocks (2.14.3)
+    rspec-rails (2.14.0)
       actionpack (>= 3.0)
       activesupport (>= 3.0)
       railties (>= 3.0)
-      rspec-core (~> 2.13.0)
-      rspec-expectations (~> 2.13.0)
-      rspec-mocks (~> 2.13.0)
-    sprockets (2.2.2)
+      rspec-core (~> 2.14.0)
+      rspec-expectations (~> 2.14.0)
+      rspec-mocks (~> 2.14.0)
+    sprockets (2.10.0)
       hike (~> 1.2)
       multi_json (~> 1.0)
       rack (~> 1.0)
       tilt (~> 1.1, != 1.3.0)
+    sprockets-rails (2.0.0)
+      actionpack (>= 3.0)
+      activesupport (>= 3.0)
+      sprockets (~> 2.8)
     thor (0.18.1)
+    thread_safe (0.1.2)
+      atomic
     tilt (1.4.1)
-    treetop (1.4.15)
+    treetop (1.4.14)
       polyglot
       polyglot (>= 0.3.1)
     tzinfo (0.3.37)
@@ -130,4 +125,4 @@ DEPENDENCIES
   factory_girl_rails
   moip!
   pg
-  rspec-rails
+  rspec-rails (~> 2.14.0)

data/README.md

@@ -18,6 +18,12 @@ Configure the routes for your Catarse application. Add the following lines in th
 
     mount CatarseMoip::Engine => "/", :as => "catarse_moip"
 
+## Rails 3.2.x and Rails 4 support
+
+If you are using the Rails 3.2.x on Catarse's code, you can use the version `1.0.0`.
+
+For Rails 4 support use the `2.0.0` version.
+
 ### Configurations
 
 Create this configurations into Catarse database:

data/app/assets/javascripts/catarse_moip/moip_form.js

@@ -14,7 +14,7 @@ App.addChild('MoipForm', {
           that.checkoutFailure({Code: 0, Mensagem: response.get_token_response.message});
         }
         else{
-          $('#catarse_moip_form').prepend(response.widget_tag);
+          that.createMoipWidget(response);
           if(_.isFunction(onSuccess)){
             onSuccess(response);
           }
@@ -23,6 +23,17 @@ App.addChild('MoipForm', {
     }
   },
 
+  createMoipWidget: function(data) {
+    widget_tag = $("<div/>").attr({
+      id: data.widget_tag.tag_id,
+      'data-token': data.widget_tag.token,
+      'callback-method-success': data.widget_tag.callback_success,
+      'callback-method-error': data.widget_tag.callback_error,
+    });
+
+    $("#catarse_moip_form").prepend(widget_tag);
+  },
+
   checkoutFailure: function(data) {
     this.loader.hide();
     var response_data = (data.length > 0 ? data[0] : data);
@@ -69,6 +80,10 @@ App.addChild('MoipForm', {
 
     this.loader = this.$('.loader');
 
+    //this.paymentChoice = new CATARSE.PaymentChoice();
+    //this.paymentCard = new CATARSE.PaymentCard({moipForm: this});
+    //this.paymentSlip = new CATARSE.PaymentSlip({moipForm: this});
+    //this.paymentAccount = new CATARSE.PaymentAccount({moipForm: this});
     window.checkoutSuccessful = _.bind(this.checkoutSuccessful, this);
     window.checkoutFailure = _.bind(this.checkoutFailure, this);
   }

data/app/assets/javascripts/catarse_moip/user_document.js

@@ -1,7 +1,7 @@
 App.views.MoipForm.UserDocument = {
   onContentClick: function(e){
     window.setTimeout(function(){
-      app.moipForm.checkoutSuccessful({'StatusPagamento': 'Success'});
+      this.moipForm.checkoutSuccessful({'StatusPagamento': 'Success'});
     }, 2000);
   },
 

data/app/controllers/catarse_moip/moip_controller.rb

@@ -22,7 +22,7 @@ module CatarseMoip
 
     def create_notification
       @backer = PaymentEngines.find_payment key: params[:id_transacao]
-      process_moip_message
+      process_moip_message(params)
       return render :nothing => true, :status => 200
     rescue Exception => e
       return render :text => "#{e.inspect}: #{e.message} recebemos: #{params}", :status => 422
@@ -46,7 +46,7 @@ module CatarseMoip
 
     def moip_response
       @backer = PaymentEngines.find_payment id: params[:id], user_id: current_user.id
-      first_update_backer unless params[:response]['StatusPagamento'] == 'Falha'
+      process_moip_message params[:response] unless params[:response]['StatusPagamento'] == 'Falha'
       render nothing: true, status: 200
     end
 
@@ -85,49 +85,49 @@ module CatarseMoip
 
       backer.update_column :payment_token, response[:token] if response and response[:token]
 
-      render json: { get_token_response: response, moip: @moip, widget_tag: @moip.widget_tag('checkoutSuccessful', 'checkoutFailure'), javascript_tag: @moip.javascript_tag }
+      render json: {
+        get_token_response: response,
+        moip: @moip,
+        widget_tag: {
+          tag_id: 'MoipWidget',
+          token: response[:token],
+          callback_success: 'checkoutSuccessful',
+          callback_error: 'checkoutFailure'
+        }
+      }
     end
 
-    def first_update_backer
-      response = ::MoIP.query(backer.payment_token)
-      if response && response["Autorizacao"]
-        params = response["Autorizacao"]["Pagamento"]
-        params = params.first unless params.respond_to?(:key)
-
-        backer.with_lock do
-          if params["Status"] == "Autorizado"
-            backer.confirm!
-          elsif backer.pending?
-            backer.waiting! 
-          end
-
-          backer.update_attributes({
-            :payment_id => params["CodigoMoIP"],
-            :payment_choice => params["FormaPagamento"],
-            :payment_service_fee => params["TaxaMoIP"]
-          }) if params
+    def update_backer params = nil
+      unless params && params["CodigoMoIP"] && params["TaxaMoIP"]
+        response = ::MoIP.query(backer.payment_token)
+        if response && response["Autorizacao"]
+          params = response["Autorizacao"]["Pagamento"]
+          params = params.first unless params.respond_to?(:key)
         end
       end
+
+      if params
+        backer.update_attributes({
+          :payment_id => params["CodigoMoIP"],
+          :payment_choice => params["FormaPagamento"],
+          :payment_service_fee => params["TaxaMoIP"]
+        })
+      end
     end
 
-    def process_moip_message
+    def process_moip_message params
+      PaymentEngines.create_payment_notification backer_id: backer.id, extra_data: JSON.parse(params.to_json.force_encoding('iso-8859-1').encode('utf-8'))
       backer.with_lock do
-        PaymentEngines.create_payment_notification backer_id: backer.id, extra_data: JSON.parse(params.to_json.force_encoding('iso-8859-1').encode('utf-8'))
-        payment_id = (backer.payment_id.gsub(".", "").to_i rescue 0)
-
-        if payment_id <= params[:cod_moip].to_i
-          backer.update_attributes payment_id: params[:cod_moip]
-
-          case params[:status_pagamento].to_i
-          when TransactionStatus::AUTHORIZED
-            backer.confirm! unless backer.confirmed?
-          when TransactionStatus::WRITTEN_BACK, TransactionStatus::REFUNDED
-            backer.refund! unless backer.refunded?
-          when TransactionStatus::CANCELED
-            backer.cancel! unless backer.canceled?
-          else
-            backer.waiting! if backer.pending?
-          end
+        update_backer if backer.payment_id.nil?
+        case params[:status_pagamento].to_i
+        when TransactionStatus::AUTHORIZED
+          backer.confirm! unless backer.confirmed?
+        when TransactionStatus::WRITTEN_BACK, TransactionStatus::REFUNDED
+          backer.refund! unless backer.refunded?
+        when TransactionStatus::CANCELED
+          backer.cancel! unless backer.canceled?
+        else
+          backer.waiting! if backer.pending?
         end
       end
     end

data/catarse_moip.gemspec

@@ -17,10 +17,10 @@ Gem::Specification.new do |s|
   s.files      = `git ls-files`.split($\)
   s.test_files = s.files.grep(%r{^(test|spec|features)/})
 
-  s.add_dependency "rails", "~> 3.2.12"
+  s.add_dependency "rails", "~> 4.0"
   s.add_dependency('libxml-ruby', '~> 2.6.0')
 
-  s.add_development_dependency "rspec-rails"
+  s.add_development_dependency "rspec-rails", "~> 2.14.0"
   s.add_development_dependency "factory_girl_rails"
   s.add_development_dependency "database_cleaner"
 end

data/config/routes.rb

@@ -5,9 +5,9 @@ CatarseMoip::Engine.routes.draw do
       get 'js'
     end
     member do
-      match :moip_response
-      match :review
-      match :get_moip_token
+      post :moip_response
+      get :review
+      post :get_moip_token
     end
   end
 end

data/lib/catarse_moip/version.rb

@@ -1,3 +1,3 @@
 module CatarseMoip
-  VERSION = "1.0.9"
+  VERSION = "2.0.0"
 end

data/spec/controllers/catarse_moip/moip_controller_spec.rb

@@ -6,13 +6,13 @@ describe CatarseMoip::MoipController do
 
   let(:get_token_response){{:status=>:fail, :code=>"171", :message=>"TelefoneFixo do endereço deverá ser enviado obrigatorio", :id=>"201210192052439150000024698931"}}
   let(:backer){ double('backer', {
-    id: 1,
-    key: 'backer key',
-    payment_id: 'payment id',
-    project: project,
-    pending?: false,
-    value: 10,
-    user: user,
+    id: 1, 
+    key: 'backer key', 
+    payment_id: 'payment id', 
+    project: project, 
+    pending?: false, 
+    value: 10, 
+    user: user, 
     payer_name: 'foo',
     payer_email: 'foo@bar.com',
     address_street: 'test',
@@ -22,13 +22,7 @@ describe CatarseMoip::MoipController do
     address_city: '123',
     address_state: '123',
     address_zip_code: '123',
-    address_phone_number: '123',
-    confirmed?: true,
-    confirm!: true,
-    canceled?: true,
-    cancel!: true,
-    refunded?: true,
-    refund!: true
+    address_phone_number: '123'
   }) }
 
   let(:user){ double('user', id: 1) }
@@ -59,69 +53,8 @@ describe CatarseMoip::MoipController do
 
     context "when we seach for an existing backer" do
       before do
-        controller.stub(:params).and_return({:id_transacao =>backer.key, :controller => "catarse_moip/moip", :action => "create_notification"})
         PaymentEngines.should_receive(:find_payment).with(key: backer.key).and_return(backer)
-        controller.should_receive(:process_moip_message)
-        post :create_notification, {:id_transacao => backer.key, :use_route => 'catarse_moip'}
-      end
-
-      its(:body){ should == ' ' }
-      its(:status){ should == 200 }
-      it("should assign backer"){ assigns(:backer).should == backer }
-    end
-
-    context "when receive ordered notification for backer" do
-      before do
-        controller.stub(:params).and_return({:cod_moip => 125, :id_transacao =>backer.key, :controller => "catarse_moip/moip", :action => "create_notification", :status_pagamento => 5})
-        backer.stub(:payment_id).and_return('123')
-
-        controller.should_receive(:process_moip_message).and_call_original
-        backer.should_receive(:update_attributes).with(payment_id: 125)
-        post :create_notification, {:id_transacao => backer.key, :use_route => 'catarse_moip'}
-      end
-
-      its(:body){ should == ' ' }
-      its(:status){ should == 200 }
-      it("should assign backer"){ assigns(:backer).should == backer }
-    end
-
-    context "when we receive a notification with the same payment id but with another status" do
-      before do
-        controller.stub(:params).and_return({:cod_moip => 123, :id_transacao =>backer.key, :controller => "catarse_moip/moip", :action => "create_notification", :status_pagamento => 1})
-        backer.stub(:payment_id).and_return('123')
-
-        controller.should_receive(:process_moip_message).and_call_original
-        backer.should_receive(:update_attributes).with(payment_id: 123)
-        post :create_notification, {:id_transacao => backer.key, :use_route => 'catarse_moip'}
-      end
-
-      its(:body){ should == ' ' }
-      its(:status){ should == 200 }
-      it("should assign backer"){ assigns(:backer).should == backer }
-    end
-
-    context "when receive a unordered notification for backer" do
-      before do
-        controller.stub(:params).and_return({:cod_moip => 122, :id_transacao =>backer.key, :controller => "catarse_moip/moip", :action => "create_notification", :status_pagamento => 5})
-        backer.stub(:payment_id).and_return('123')
-
-        controller.should_receive(:process_moip_message).and_call_original
-        backer.should_not_receive(:update_attributes).with(payment_id: 122)
-        post :create_notification, {:id_transacao => backer.key, :use_route => 'catarse_moip'}
-      end
-
-      its(:body){ should == ' ' }
-      its(:status){ should == 200 }
-      it("should assign backer"){ assigns(:backer).should == backer }
-    end
-
-    context "when backer payment_id is null" do
-      before do
-        controller.stub(:params).and_return({:cod_moip => 122, :id_transacao =>backer.key, :controller => "catarse_moip/moip", :action => "create_notification", :status_pagamento => 5})
-        backer.stub(:payment_id).and_return(nil)
-
-        controller.should_receive(:process_moip_message).and_call_original
-        backer.should_receive(:update_attributes).with(payment_id: 122)
+        controller.should_receive(:process_moip_message).with({"id_transacao"=>backer.key, "controller"=>"catarse_moip/moip", "action"=>"create_notification"})
         post :create_notification, {:id_transacao => backer.key, :use_route => 'catarse_moip'}
       end
 
@@ -136,7 +69,7 @@ describe CatarseMoip::MoipController do
 
     context "when the content of get_javascript_url raises an error" do
       before do
-        controller.should_receive(:open).at_least(3).times.and_raise('error')
+        controller.should_receive(:open).at_least(3).times.and_raise('error') 
         ->{
           get :js, locale: :pt, use_route: 'catarse_moip'
         }.should raise_error('error')
@@ -146,7 +79,7 @@ describe CatarseMoip::MoipController do
 
     context "when the content of get_javascript_url is read without errors" do
       before do
-        controller.should_receive(:open).and_return(file)
+        controller.should_receive(:open).and_return(file) 
         file.should_receive(:set_encoding).and_return(file)
         file.should_receive(:read).and_return(file)
         file.should_receive(:encode).and_return('js content')
@@ -160,7 +93,7 @@ describe CatarseMoip::MoipController do
   describe "POST moip_response" do
     let(:processor){ double('moip processor') }
     before do
-      controller.should_receive(:first_update_backer)
+      controller.should_receive(:process_moip_message)
       post :moip_response, id: backer.id, response: {StatusPagamento: 'Sucesso'}, use_route: 'catarse_moip'
     end
 
@@ -173,22 +106,35 @@ describe CatarseMoip::MoipController do
     end
 
     its(:status){ should == 200 }
-    its(:body){ should == "{\"get_token_response\":{\"status\":\"fail\",\"code\":\"171\",\"message\":\"TelefoneFixo do endereço deverá ser enviado obrigatorio\",\"id\":\"201210192052439150000024698931\"},\"moip\":\"{}\",\"widget_tag\":\"<div id='MoipWidget'\\n          data-token=''\\n          callback-method-success='checkoutSuccessful' \\n          callback-method-error='checkoutFailure'>\\n    </div>\",\"javascript_tag\":\"<script type='text/javascript' src='https://www.moip.com.br/transparente/MoipWidget-v2.js' charset='ISO-8859-1'></script>\"}" }
+    its(:body){ should == "{\"get_token_response\":{\"status\":\"fail\",\"code\":\"171\",\"message\":\"TelefoneFixo do endereço deverá ser enviado obrigatorio\",\"id\":\"201210192052439150000024698931\"},\"moip\":\"{}\",\"widget_tag\":{\"tag_id\":\"MoipWidget\",\"token\":null,\"callback_success\":\"checkoutSuccessful\",\"callback_error\":\"checkoutFailure\"}}" }
   end
 
-  describe "#first_update_backer" do
+  describe "#update_backer" do
     before do
       controller.stub(:backer).and_return(backer)
       backer.stub(:payment_token).and_return('token')
     end
 
+    context "with parameters containing CodigoMoIP and TaxaMoIP" do
+      let(:payment){ {"Status" => "Autorizado","Codigo" => "0","CodigoRetorno" => "0","TaxaMoIP" => "1.54","StatusPagamento" => "Sucesso","CodigoMoIP" => "18093844","Mensagem" => "Requisição processada com sucesso","TotalPago" => "25.00","url" => "https => //www.moip.com.br/Instrucao.do?token=R2W0N123E005F2A911V6O2I0Y3S7M4J853H0S0F0T0D044T8F4H4E9G0I3W8"} }
+      before do
+        MoIP.should_not_receive(:query)
+        backer.should_receive(:update_attributes).with({
+          payment_id: payment["CodigoMoIP"],
+          payment_choice: payment["FormaPagamento"],
+          payment_service_fee: payment["TaxaMoIP"]
+        })
+      end
+      it("should call update attributes but not call MoIP.query"){ controller.update_backer payment }
+    end
+
     context "with no response from moip" do
       let(:moip_query_response) { nil }
       before do
         MoIP.should_receive(:query).with(backer.payment_token).and_return(moip_query_response)
         backer.should_not_receive(:update_attributes)
       end
-      it("should never call update attributes"){ controller.first_update_backer }
+      it("should never call update attributes"){ controller.update_backer }
     end
 
     context "with an incomplete transaction" do
@@ -199,7 +145,7 @@ describe CatarseMoip::MoipController do
         MoIP.should_receive(:query).with(backer.payment_token).and_return(moip_query_response)
         backer.should_not_receive(:update_attributes)
       end
-      it("should never call update attributes"){ controller.first_update_backer }
+      it("should never call update attributes"){ controller.update_backer }
     end
 
     context "with a real data set that works for some cases" do
@@ -209,14 +155,13 @@ describe CatarseMoip::MoipController do
       before do
         MoIP.should_receive(:query).with(backer.payment_token).and_return(moip_query_response)
         payment = moip_query_response["Autorizacao"]["Pagamento"].first
-        backer.should_receive(:confirm!)
         backer.should_receive(:update_attributes).with({
           payment_id: payment["CodigoMoIP"],
           payment_choice: payment["FormaPagamento"],
           payment_service_fee: payment["TaxaMoIP"]
         })
       end
-      it("should call update attributes"){ controller.first_update_backer }
+      it("should call update attributes"){ controller.update_backer }
     end
   end
 
@@ -228,41 +173,57 @@ describe CatarseMoip::MoipController do
       controller.stub(:update_backer)
     end
 
+    context "when we already have the payment_id in backers table" do
+      before do
+        backer.stub(:payment_id).and_return('test')
+        controller.should_not_receive(:update_backer)
+      end
+
+      it 'should never call update_backer' do
+        controller.process_moip_message post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::AUTHORIZED})
+      end
+    end
+
+    context "when we still do not have the payment_id in backers table" do
+      before do
+        backer.stub(:payment_id).and_return(nil)
+        controller.should_receive(:update_backer)
+      end
+
+      it 'should call update_backer on the processor' do
+        controller.process_moip_message post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::AUTHORIZED})
+      end
+    end
+
     context "when there is a written back request and backer is not refunded" do
       before do
-        controller.stub(:params).and_return(post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::WRITTEN_BACK}))
         backer.stub(:refunded?).and_return(false)
         backer.should_receive(:refund!)
-        backer.should_receive(:update_attributes)
       end
 
       it 'should call refund!' do
-        controller.process_moip_message
+        controller.process_moip_message post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::WRITTEN_BACK})
       end
     end
 
     context "when there is an authorized request" do
       before do
-        controller.stub(:params).and_return(post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::AUTHORIZED}))
         backer.should_receive(:confirm!)
-        backer.should_receive(:update_attributes)
       end
 
       it 'should call confirm!' do
-        controller.process_moip_message
+        controller.process_moip_message post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::AUTHORIZED})
       end
     end
 
     context "when there is a refund request" do
       before do
-        controller.stub(:params).and_return(post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::REFUNDED}))
         backer.stub(:refunded?).and_return(false)
         backer.should_receive(:refund!)
-        backer.should_receive(:update_attributes)
       end
 
       it 'should mark refunded to true' do
-        controller.process_moip_message
+        controller.process_moip_message post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::REFUNDED})
       end
     end
   end

data/test/dummy/config/application.rb

@@ -2,8 +2,9 @@ require File.expand_path('../boot', __FILE__)
 
 require 'rails/all'
 
-Bundler.require(*Rails.groups)
-require "catarse_moip"
+# Require the gems listed in Gemfile, including any gems
+# you've limited to :test, :development, or :production.
+Bundler.require(:default, Rails.env)
 
 module Dummy
   class Application < Rails::Application
@@ -11,16 +12,6 @@ module Dummy
     # Application configuration should go into files in config/initializers
     # -- all .rb files in that directory are automatically loaded.
 
-    # Custom directories with classes and modules you want to be autoloadable.
-    # config.autoload_paths += %W(#{config.root}/extras)
-
-    # Only load the plugins named here, in the order given (default is alphabetical).
-    # :all can be used as a placeholder for all plugins not explicitly named.
-    # config.plugins = [ :exception_notification, :ssl_requirement, :all ]
-
-    # Activate observers that should always be running.
-    # config.active_record.observers = :cacher, :garbage_collector, :forum_observer
-
     # Set Time.zone default to the specified zone and make Active Record auto-convert to this zone.
     # Run "rake -D time" for a list of tasks for finding time zone names. Default is UTC.
     # config.time_zone = 'Central Time (US & Canada)'
@@ -28,32 +19,5 @@ module Dummy
     # The default locale is :en and all translations from config/locales/*.rb,yml are auto loaded.
     # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
     # config.i18n.default_locale = :de
-
-    # Configure the default encoding used in templates for Ruby 1.9.
-    config.encoding = "utf-8"
-
-    # Configure sensitive parameters which will be filtered from the log file.
-    config.filter_parameters += [:password]
-
-    # Enable escaping HTML in JSON.
-    config.active_support.escape_html_entities_in_json = true
-
-    # Use SQL instead of Active Record's schema dumper when creating the database.
-    # This is necessary if your schema can't be completely dumped by the schema dumper,
-    # like if you have constraints or database-specific column types
-    # config.active_record.schema_format = :sql
-
-    # Enforce whitelist mode for mass assignment.
-    # This will create an empty whitelist of attributes available for mass-assignment for all models
-    # in your app. As such, your models will need to explicitly whitelist or blacklist accessible
-    # parameters by using an attr_accessible or attr_protected declaration.
-    config.active_record.whitelist_attributes = true
-
-    # Enable the asset pipeline
-    config.assets.enabled = true
-
-    # Version of your assets, change this if you want to expire all your assets
-    config.assets.version = '1.0'
   end
 end
-

data/test/dummy/config/environments/development.rb

@@ -1,37 +1,29 @@
 Dummy::Application.configure do
-  # Settings specified here will take precedence over those in config/application.rb
+  # Settings specified here will take precedence over those in config/application.rb.
 
   # In the development environment your application's code is reloaded on
   # every request. This slows down response time but is perfect for development
   # since you don't have to restart the web server when you make code changes.
   config.cache_classes = false
 
-  # Log error messages when you accidentally call methods on nil.
-  config.whiny_nils = true
+  # Do not eager load code on boot.
+  config.eager_load = false
 
-  # Show full error reports and disable caching
+  # Show full error reports and disable caching.
   config.consider_all_requests_local       = true
   config.action_controller.perform_caching = false
 
-  # Don't care if the mailer can't send
+  # Don't care if the mailer can't send.
   config.action_mailer.raise_delivery_errors = false
 
-  # Print deprecation notices to the Rails logger
+  # Print deprecation notices to the Rails logger.
   config.active_support.deprecation = :log
 
-  # Only use best-standards-support built into browsers
-  config.action_dispatch.best_standards_support = :builtin
+  # Raise an error on page load if there are pending migrations
+  config.active_record.migration_error = :page_load
 
-  # Raise exception on mass assignment protection for Active Record models
-  config.active_record.mass_assignment_sanitizer = :strict
-
-  # Log the query plan for queries taking more than this (works
-  # with SQLite, MySQL, and PostgreSQL)
-  config.active_record.auto_explain_threshold_in_seconds = 0.5
-
-  # Do not compress assets
-  config.assets.compress = false
-
-  # Expands the lines which load the assets
+  # Debug mode disables concatenation and preprocessing of assets.
+  # This option may cause significant delays in view rendering with a large
+  # number of complex assets.
   config.assets.debug = true
 end

data/test/dummy/config/environments/production.rb

@@ -1,67 +1,80 @@
 Dummy::Application.configure do
-  # Settings specified here will take precedence over those in config/application.rb
+  # Settings specified here will take precedence over those in config/application.rb.
 
-  # Code is not reloaded between requests
+  # Code is not reloaded between requests.
   config.cache_classes = true
 
-  # Full error reports are disabled and caching is turned on
+  # Eager load code on boot. This eager loads most of Rails and
+  # your application in memory, allowing both thread web servers
+  # and those relying on copy on write to perform better.
+  # Rake tasks automatically ignore this option for performance.
+  config.eager_load = true
+
+  # Full error reports are disabled and caching is turned on.
   config.consider_all_requests_local       = false
   config.action_controller.perform_caching = true
 
-  # Disable Rails's static asset server (Apache or nginx will already do this)
+  # Enable Rack::Cache to put a simple HTTP cache in front of your application
+  # Add `rack-cache` to your Gemfile before enabling this.
+  # For large-scale production use, consider using a caching reverse proxy like nginx, varnish or squid.
+  # config.action_dispatch.rack_cache = true
+
+  # Disable Rails's static asset server (Apache or nginx will already do this).
   config.serve_static_assets = false
 
-  # Compress JavaScripts and CSS
-  config.assets.compress = true
+  # Compress JavaScripts and CSS.
+  config.assets.js_compressor = :uglifier
+  # config.assets.css_compressor = :sass
 
-  # Don't fallback to assets pipeline if a precompiled asset is missed
+  # Do not fallback to assets pipeline if a precompiled asset is missed.
   config.assets.compile = false
 
-  # Generate digests for assets URLs
+  # Generate digests for assets URLs.
   config.assets.digest = true
 
-  # Defaults to nil and saved in location specified by config.assets.prefix
-  # config.assets.manifest = YOUR_PATH
+  # Version of your assets, change this if you want to expire all your assets.
+  config.assets.version = '1.0'
 
-  # Specifies the header that your server uses for sending files
+  # Specifies the header that your server uses for sending files.
   # config.action_dispatch.x_sendfile_header = "X-Sendfile" # for apache
   # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for nginx
 
   # Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies.
   # config.force_ssl = true
 
-  # See everything in the log (default is :info)
-  # config.log_level = :debug
+  # Set to :debug to see everything in the log.
+  config.log_level = :info
 
-  # Prepend all log lines with the following tags
+  # Prepend all log lines with the following tags.
   # config.log_tags = [ :subdomain, :uuid ]
 
-  # Use a different logger for distributed setups
+  # Use a different logger for distributed setups.
   # config.logger = ActiveSupport::TaggedLogging.new(SyslogLogger.new)
 
-  # Use a different cache store in production
+  # Use a different cache store in production.
   # config.cache_store = :mem_cache_store
 
-  # Enable serving of images, stylesheets, and JavaScripts from an asset server
+  # Enable serving of images, stylesheets, and JavaScripts from an asset server.
   # config.action_controller.asset_host = "http://assets.example.com"
 
-  # Precompile additional assets (application.js, application.css, and all non-JS/CSS are already added)
+  # Precompile additional assets.
+  # application.js, application.css, and all non-JS/CSS in app/assets folder are already added.
   # config.assets.precompile += %w( search.js )
 
-  # Disable delivery errors, bad email addresses will be ignored
+  # Ignore bad email addresses and do not raise email delivery errors.
+  # Set this to true and configure the email server for immediate delivery to raise delivery errors.
   # config.action_mailer.raise_delivery_errors = false
 
-  # Enable threaded mode
-  # config.threadsafe!
-
   # Enable locale fallbacks for I18n (makes lookups for any locale fall back to
-  # the I18n.default_locale when a translation can not be found)
+  # the I18n.default_locale when a translation can not be found).
   config.i18n.fallbacks = true
 
-  # Send deprecation notices to registered listeners
+  # Send deprecation notices to registered listeners.
   config.active_support.deprecation = :notify
 
-  # Log the query plan for queries taking more than this (works
-  # with SQLite, MySQL, and PostgreSQL)
-  # config.active_record.auto_explain_threshold_in_seconds = 0.5
+  # Disable automatic flushing of the log to improve performance.
+  # config.autoflush_log = false
+
+  # Use default logging formatter so that PID and timestamp are not suppressed.
+  config.log_formatter = ::Logger::Formatter.new
 end

data/test/dummy/config/environments/test.rb

@@ -1,5 +1,5 @@
 Dummy::Application.configure do
-  # Settings specified here will take precedence over those in config/application.rb
+  # Settings specified here will take precedence over those in config/application.rb.
 
   # The test environment is used exclusively to run your application's
   # test suite. You never need to work with it otherwise. Remember that
@@ -7,31 +7,30 @@ Dummy::Application.configure do
   # and recreated between test runs. Don't rely on the data there!
   config.cache_classes = true
 
-  # Configure static asset server for tests with Cache-Control for performance
-  config.serve_static_assets = true
-  config.static_cache_control = "public, max-age=3600"
+  # Do not eager load code on boot. This avoids loading your whole application
+  # just for the purpose of running a single test. If you are using a tool that
+  # preloads Rails for running tests, you may have to set it to true.
+  config.eager_load = false
 
-  # Log error messages when you accidentally call methods on nil
-  config.whiny_nils = true
+  # Configure static asset server for tests with Cache-Control for performance.
+  config.serve_static_assets  = true
+  config.static_cache_control = "public, max-age=3600"
 
-  # Show full error reports and disable caching
+  # Show full error reports and disable caching.
   config.consider_all_requests_local       = true
   config.action_controller.perform_caching = false
 
-  # Raise exceptions instead of rendering exception templates
+  # Raise exceptions instead of rendering exception templates.
   config.action_dispatch.show_exceptions = false
 
-  # Disable request forgery protection in test environment
-  config.action_controller.allow_forgery_protection    = false
+  # Disable request forgery protection in test environment.
+  config.action_controller.allow_forgery_protection = false
 
   # Tell Action Mailer not to deliver emails to the real world.
   # The :test delivery method accumulates sent emails in the
   # ActionMailer::Base.deliveries array.
   config.action_mailer.delivery_method = :test
 
-  # Raise exception on mass assignment protection for Active Record models
-  config.active_record.mass_assignment_sanitizer = :strict
-
-  # Print deprecation notices to the stderr
+  # Print deprecation notices to the stderr.
   config.active_support.deprecation = :stderr
 end

data/test/dummy/config/initializers/filter_parameter_logging.rb

@@ -0,0 +1,4 @@
+# Be sure to restart your server when you modify this file.
+
+# Configure sensitive parameters which will be filtered from the log file.
+Rails.application.config.filter_parameters += [:password]

data/test/dummy/config/initializers/inflections.rb

@@ -1,15 +1,16 @@
 # Be sure to restart your server when you modify this file.
 
-# Add new inflection rules using the following format
-# (all these examples are active by default):
-# ActiveSupport::Inflector.inflections do |inflect|
+# Add new inflection rules using the following format. Inflections
+# are locale specific, and you may define rules for as many different
+# locales as you wish. All of these examples are active by default:
+# ActiveSupport::Inflector.inflections(:en) do |inflect|
 #   inflect.plural /^(ox)$/i, '\1en'
 #   inflect.singular /^(ox)en/i, '\1'
 #   inflect.irregular 'person', 'people'
 #   inflect.uncountable %w( fish sheep )
 # end
-#
+
 # These inflection rules are supported but not enabled by default:
-# ActiveSupport::Inflector.inflections do |inflect|
+# ActiveSupport::Inflector.inflections(:en) do |inflect|
 #   inflect.acronym 'RESTful'
 # end

data/test/dummy/config/initializers/secret_token.rb

@@ -1,7 +1,12 @@
 # Be sure to restart your server when you modify this file.
 
-# Your secret key for verifying the integrity of signed cookies.
+# Your secret key is used for verifying the integrity of signed cookies.
 # If you change this key, all old signed cookies will become invalid!
+
 # Make sure the secret is at least 30 characters and all random,
 # no regular words or you'll be exposed to dictionary attacks.
-Dummy::Application.config.secret_token = '21c461065b81f8bee3308c8773a98feacd4ea509c775b99e402b8bb08bdfc10334b69e2a19ce26ee8b053287947887e6d30b2ac53601fec09db1108d50b34182'
+# You can use `rake secret` to generate a secure secret key.
+
+# Make sure your secret_key_base is kept private
+# if you're sharing your code publicly.
+Dummy::Application.config.secret_key_base = '56b32f3c2786d84f9dc2a481c726e5a6675e0466e0bd7031322d20df94ec5010c8a488a29f4f0856a10bb02b4117f80ac70429f9926efe0583a1472857bb6421'

data/test/dummy/config/initializers/session_store.rb

@@ -1,8 +1,3 @@
 # Be sure to restart your server when you modify this file.
 
 Dummy::Application.config.session_store :cookie_store, key: '_dummy_session'
-
-# Use the database for sessions instead of the cookie-based default,
-# which shouldn't be used to store highly confidential information
-# (create the session table with "rails generate session_migration")
-# Dummy::Application.config.session_store :active_record_store

data/test/dummy/config/initializers/wrap_parameters.rb

@@ -1,14 +1,14 @@
 # Be sure to restart your server when you modify this file.
-#
+
 # This file contains settings for ActionController::ParamsWrapper which
 # is enabled by default.
 
 # Enable parameter wrapping for JSON. You can disable this by setting :format to an empty array.
 ActiveSupport.on_load(:action_controller) do
-  wrap_parameters format: [:json]
+  wrap_parameters format: [:json] if respond_to?(:wrap_parameters)
 end
 
-# Disable root element in JSON by default.
-ActiveSupport.on_load(:active_record) do
-  self.include_root_in_json = false
-end
+# To enable root element in JSON for ActiveRecord objects.
+# ActiveSupport.on_load(:active_record) do
+#  self.include_root_in_json = true
+# end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: catarse_moip
 version: !ruby/object:Gem::Version
-  version: 1.0.9
+  version: 2.0.0
 platform: ruby
 authors:
 - Antônio Roberto Silva
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-20 00:00:00.000000000 Z
+date: 2013-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -18,14 +18,14 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 3.2.12
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 3.2.12
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: libxml-ruby
   requirement: !ruby/object:Gem::Requirement
@@ -44,16 +44,16 @@ dependencies:
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.14.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.14.0
 - !ruby/object:Gem::Dependency
   name: factory_girl_rails
   requirement: !ruby/object:Gem::Requirement
@@ -150,6 +150,7 @@ files:
 - test/dummy/config/environments/production.rb
 - test/dummy/config/environments/test.rb
 - test/dummy/config/initializers/backtrace_silencers.rb
+- test/dummy/config/initializers/filter_parameter_logging.rb
 - test/dummy/config/initializers/inflections.rb
 - test/dummy/config/initializers/mime_types.rb
 - test/dummy/config/initializers/secret_token.rb
@@ -216,6 +217,7 @@ test_files:
 - test/dummy/config/environments/production.rb
 - test/dummy/config/environments/test.rb
 - test/dummy/config/initializers/backtrace_silencers.rb
+- test/dummy/config/initializers/filter_parameter_logging.rb
 - test/dummy/config/initializers/inflections.rb
 - test/dummy/config/initializers/mime_types.rb
 - test/dummy/config/initializers/secret_token.rb