catarse_moip 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ab03af734dc478b0be30cbdaed00b0bbf697078b
-  data.tar.gz: 85f97f93d4928661240b656353f8516f28a686ed
+  metadata.gz: d6298a7c73bdeb0c8eecca41f5082c44ff5c4215
+  data.tar.gz: 1c0b9f4b451fcef195c4db776e13217840dc220a
 SHA512:
-  metadata.gz: a7ba76a6ee80c22b5d1f0aa7fc1755fefa71822c7dc5eabef280812d3ed518f696938fb01a1fd9ab5fa3133e25109e1e7480d4c68c98f5da75edf552ff9247d2
-  data.tar.gz: 981bfd51b55ff412089c61682efe62703d5fcd83765e3376629eba4dbc263f4fbd37f512d4288db4e6d420b61e7053ebc50231a3df2713f9d7dd78c173ae0405
+  metadata.gz: 638beedc099e89fbc18a4e0c3237cec1ba6ca8ac039413d2b658680c2b0ab83c8fc6b3283b32a73dfc4102a9dce9a0737b926b008566f091566f0de48ab22c5d
+  data.tar.gz: 974327feb4232b0537c41fb94574e9d38df88c59f5218acec4c2c746d5213b125ce7fbb6b831b6cc3bcf024c19280517f791072dab03336c13398fc176d97981

data/Gemfile.lock

@@ -10,7 +10,7 @@ GIT
 PATH
   remote: .
   specs:
-    catarse_moip (1.0.1)
+    catarse_moip (1.0.2)
       libxml-ruby (~> 2.6.0)
       rails (~> 3.2.12)
 

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(params)
+      process_moip_message
       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
-      process_moip_message params[:response] unless params[:response]['StatusPagamento'] == 'Falha'
+      first_update_backer params[:response] unless params[:response]['StatusPagamento'] == 'Falha'
       render nothing: true, status: 200
     end
 
@@ -88,37 +88,36 @@ module CatarseMoip
       render json: { get_token_response: response, moip: @moip, widget_tag: @moip.widget_tag('checkoutSuccessful', 'checkoutFailure'), javascript_tag: @moip.javascript_tag }
     end
 
-    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
+    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.update_attributes({
           :payment_id => params["CodigoMoIP"],
           :payment_choice => params["FormaPagamento"],
           :payment_service_fee => params["TaxaMoIP"]
-        })
+        }) if params
       end
     end
 
-    def process_moip_message params
+    def process_moip_message
       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
-        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?
+        if backer.payment_id.gsub(".", "").to_i < 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
         end
       end
     end

data/lib/catarse_moip/version.rb

@@ -1,3 +1,3 @@
 module CatarseMoip
-  VERSION = "1.0.1"
+  VERSION = "1.0.2"
 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,7 +22,13 @@ describe CatarseMoip::MoipController do
     address_city: '123',
     address_state: '123',
     address_zip_code: '123',
-    address_phone_number: '123'
+    address_phone_number: '123',
+    confirmed?: true,
+    confirm!: true,
+    canceled?: true,
+    cancel!: true,
+    refunded?: true,
+    refund!: true
   }) }
 
   let(:user){ double('user', id: 1) }
@@ -53,8 +59,39 @@ 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).with({"id_transacao"=>backer.key, "controller"=>"catarse_moip/moip", "action"=>"create_notification"})
+        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 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
 
@@ -69,7 +106,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')
@@ -79,7 +116,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')
@@ -93,7 +130,7 @@ describe CatarseMoip::MoipController do
   describe "POST moip_response" do
     let(:processor){ double('moip processor') }
     before do
-      controller.should_receive(:process_moip_message)
+      controller.should_receive(:first_update_backer)
       post :moip_response, id: backer.id, response: {StatusPagamento: 'Sucesso'}, use_route: 'catarse_moip'
     end
 
@@ -109,32 +146,19 @@ describe CatarseMoip::MoipController do
     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>\"}" }
   end
 
-  describe "#update_backer" do
+  describe "#first_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.update_backer }
+      it("should never call update attributes"){ controller.first_update_backer }
     end
 
     context "with an incomplete transaction" do
@@ -145,7 +169,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.update_backer }
+      it("should never call update attributes"){ controller.first_update_backer }
     end
 
     context "with a real data set that works for some cases" do
@@ -161,7 +185,7 @@ describe CatarseMoip::MoipController do
           payment_service_fee: payment["TaxaMoIP"]
         })
       end
-      it("should call update attributes"){ controller.update_backer }
+      it("should call update attributes"){ controller.first_update_backer }
     end
   end
 
@@ -173,57 +197,41 @@ 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 post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::WRITTEN_BACK})
+        controller.process_moip_message
       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 post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::AUTHORIZED})
+        controller.process_moip_message
       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 post_moip_params.merge!({:id_transacao => backer.key, :status_pagamento => CatarseMoip::MoipController::TransactionStatus::REFUNDED})
+        controller.process_moip_message
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: catarse_moip
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Antônio Roberto Silva
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-21 00:00:00.000000000 Z
+date: 2013-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -183,7 +183,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.7
+rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
 summary: MoIP integration with Catarse