catarse_moip 1.0.4 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d16a1550e626fb9eae05ff313dd0295f62a16e0d
-  data.tar.gz: 70ad09f8044ad602ccac430460e75be1fbe3dbf6
+  metadata.gz: c66d57d3bfcaab6661b6135c0889b8f70ce0a07d
+  data.tar.gz: 5525def340c92871f2c30b92e389a0b9947b6c95
 SHA512:
-  metadata.gz: 8c6ce4f3311fe998821ac03f3b16e1cda8a3f65aa6c465ca86a3f41744c682cb6a3f4e1999285f0e405a586d2cf1aac3f072a72a1349916856e60766c283b9a4
-  data.tar.gz: 67a3ce105e6d099fbd3f01d4bbf521c39ba94808edd5a19361c9dfb0a9e1b4c918cf068b5bc8bab99bed4fa043e5c6d8055ad75d857caf67aeae00eae85e129e
+  metadata.gz: d9497ecb61d0c0190caf4b8ba465723a9519e93a4be6c3e14e5778304d0ca66e176f58e8391e4ed925604cb7c2327a42a234e6d7a3a60429ce4add6501d8737d
+  data.tar.gz: faf0ad6b55baa625cd7c625a3ab79e387a92c62937272b744bc6d132cd07cd9e5980d8a91625ca861b15b7efd292c1e86a2f278e27884efa86bb765c120f6874

data/Gemfile.lock

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

data/app/controllers/catarse_moip/moip_controller.rb

@@ -110,7 +110,7 @@ module CatarseMoip
       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
-        if backer.payment_id.gsub(".", "").to_i < params[:cod_moip].to_i
+        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

data/lib/catarse_moip/version.rb

@@ -1,3 +1,3 @@
 module CatarseMoip
-  VERSION = "1.0.4"
+  VERSION = "1.0.5"
 end

data/spec/controllers/catarse_moip/moip_controller_spec.rb

@@ -85,6 +85,21 @@ describe CatarseMoip::MoipController do
       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})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: catarse_moip
 version: !ruby/object:Gem::Version
-  version: 1.0.4
+  version: 1.0.5
 platform: ruby
 authors:
 - Antônio Roberto Silva
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-10 00:00:00.000000000 Z
+date: 2013-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails