pag_seguro 0.1.2 → 0.1.3

data/lib/pag_seguro/errors/unknown_error.rb

@@ -1,8 +1,8 @@
 module PagSeguro
   module Errors
-    class UnknownError
+    class UnknownError < Exception
       def initialize(response)
-        super("Unknown response code (#{response.code}):\n#{reponse.body}")
+        super("Unknown response code (#{response.code}):\n#{response.body}")
       end
     end
   end

data/lib/pag_seguro/payment.rb

@@ -2,7 +2,7 @@ module PagSeguro
   class Payment
     include ActiveModel::Validations
         
-    attr_accessor :id, :email, :token, :items, :sender, :shipping, :extra_amount, :redirect_url, :max_uses, :max_age
+    attr_accessor :id, :email, :token, :items, :sender, :shipping, :extra_amount, :redirect_url, :max_uses, :max_age, :response
     
     validates_presence_of :email, :token
     validates_format_of :extra_amount, with: /^\d+\.\d{2}$/, message: " must be a decimal and have 2 digits after the dot", allow_blank: true
@@ -36,16 +36,17 @@ module PagSeguro
     end
     
     def code
-      response = send_checkout
-      if response.code == 200
-        Nokogiri::XML(response.body).css("checkout code").first.content
-      elsif response.code == 401
-        raise Errors::Unauthorized
-      elsif response.code == 400
-        raise Errors::InvalidData.new(response.body)
-      else
-        raise Errors::UnknownError.new(response)
-      end
+      @response ||= parse_checkout_response
+      parse_code
+    end
+    
+    def date
+      @response ||= parse_checkout_response
+      parse_date
+    end
+    
+    def reset!
+      @response = nil
     end
     
     protected
@@ -60,5 +61,26 @@ module PagSeguro
       def send_checkout
         RestClient.post(checkout_url_with_params, checkout_xml, content_type: "application/xml"){|response, request, result| response }
       end
+      
+      def parse_checkout_response
+        response = send_checkout
+        if response.code == 200
+          response.body
+        elsif response.code == 401
+          raise Errors::Unauthorized
+        elsif response.code == 400
+          raise Errors::InvalidData.new(response.body)
+        else
+          raise Errors::UnknownError.new(response)
+        end        
+      end
+      
+      def parse_date
+        Nokogiri::XML(@response.body).css("checkout date").first.content
+      end
+      
+      def parse_code
+        Nokogiri::XML(@response.body).css("checkout code").first.content
+      end
   end
 end

data/lib/pag_seguro/version.rb

@@ -1,3 +1,3 @@
 module PagSeguro
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/spec/pag_seguro/errors/invalid_data_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+def invalid_data_xml
+<<XML
+<errors>
+  <error>
+    <code>404</code>
+    <message>Not Found</message>
+  </error>
+  <error>
+    <code>422</code>
+    <message>Unauthorized</message>
+  </error>
+</errors>
+XML
+end
+
+describe PagSeguro::Errors::InvalidData do
+  it "should be able to parse an error xml and raise the error codes" do
+    lambda { raise PagSeguro::Errors::InvalidData.new(invalid_data_xml) }.should raise_error(PagSeguro::Errors::InvalidData, "404: Not Found\n422: Unauthorized\n")
+  end
+end

data/spec/pag_seguro/errors/unauthorized_spec.rb

@@ -0,0 +1,7 @@
+require "spec_helper"
+
+describe PagSeguro::Errors::Unauthorized do
+  it "should be able to raise an unauthorized error" do
+    lambda { raise PagSeguro::Errors::Unauthorized.new }.should raise_error(PagSeguro::Errors::Unauthorized, "Credentials provided (e-mail and token) failed to authenticate")
+  end
+end

data/spec/pag_seguro/errors/unknown_error_spec.rb

@@ -0,0 +1,17 @@
+require "spec_helper"
+
+class MockResponse
+  def code
+    10000
+  end
+  
+  def body
+    " error description"
+  end
+end
+
+describe PagSeguro::Errors::UnknownError do
+  it "should be able to raise an unknown error" do
+    lambda { raise PagSeguro::Errors::UnknownError.new(MockResponse.new) }.should raise_error(PagSeguro::Errors::UnknownError, "Unknown response code (10000):\n error description")
+  end
+end

data/spec/pag_seguro/integration/checkout_spec.rb

@@ -23,7 +23,7 @@ describe "PagSeguro::Payment.code" do
     payment = create_valid_payment
     payment.code.size.should == 32
   end
-  
+    
   it "should tell me when the email and token are invalid" do
     payment = PagSeguro::Payment.new("not_a_user@not_an_email.com", "NOTATOKEN7F048A09A8AEFDD1E5A7B91")
     lambda { payment.code }.should raise_error(PagSeguro::Errors::Unauthorized)
@@ -38,4 +38,36 @@ describe "PagSeguro::Payment.code" do
     payment = create_valid_payment
     RestClient.get(payment.checkout_payment_url).code.should == 200
   end
+end
+
+describe "PagSeguro::Payment.date" do
+  it "should send a request to pagseguro" do
+    payment = create_valid_payment
+    payment.date.should match(/^\d{4}\-\d{2}\-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}[-+]\d{2}:\d{2}$/)
+  end
+end
+
+describe "PagSeguro::Payment parse_checkout_response" do
+  before do
+    @payment = create_valid_payment
+    @payment.stub(:parse_code)
+    @payment.stub(:parse_date)
+    @payment.stub(:parse_checkout_response){ "some response" }
+  end
+  
+  it "should not make a request to pagseguro more than once" do
+    @payment.should_receive(:parse_checkout_response).exactly(1).times
+    
+    @payment.code
+    @payment.code
+    @payment.date
+  end
+  
+  it "should be make more than one request to pag seguro if the payment is reset" do
+    @payment.should_receive(:parse_checkout_response).exactly(2).times
+
+    @payment.code
+    @payment.reset!
+    @payment.date
+  end
 end

data/spec/pag_seguro/notification_spec.rb

@@ -55,4 +55,14 @@ describe PagSeguro::Notification do
     @items[1].quantity.should == "1"
     @items[1].amount.should == "25600.00"
   end
+  
+  it "should be_available if its status is 4" do
+    @notification.stub(:status){ 4 }
+    @notification.should be_available
+  end
+  
+  it "should be_approved if its status is 3" do
+    @notification.stub(:status){ 3 }
+    @notification.should be_approved
+  end
 end

data/spec/pag_seguro/payment_spec.rb

@@ -15,6 +15,15 @@ describe PagSeguro::Payment do
       it { @payment.should have_attribute_accessor(:redirect_url) }
       it { @payment.should have_attribute_accessor(:max_uses) }
       it { @payment.should have_attribute_accessor(:max_age) }
+      it { @payment.should have_attribute_accessor(:response) }
+
+      it "should respond to :code" do
+        @payment.respond_to?(:code).should be_true
+      end
+
+      it "should respond to :date" do
+        @payment.respond_to?(:date).should be_true
+      end
 
       it "should have items" do
         @payment.items.should be_instance_of(Array)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pag_seguro
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-07 00:00:00.000000000 Z
+date: 2012-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &70199441235740 !ruby/object:Gem::Requirement
+  requirement: &2152283540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70199441235740
+  version_requirements: *2152283540
 - !ruby/object:Gem::Dependency
   name: haml
-  requirement: &70199441234400 !ruby/object:Gem::Requirement
+  requirement: &2152283060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70199441234400
+  version_requirements: *2152283060
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70199441233240 !ruby/object:Gem::Requirement
+  requirement: &2152282300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70199441233240
+  version_requirements: *2152282300
 - !ruby/object:Gem::Dependency
   name: rest-client
-  requirement: &70199441231700 !ruby/object:Gem::Requirement
+  requirement: &2152281580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,7 +54,7 @@ dependencies:
         version: 1.6.7
   type: :runtime
   prerelease: false
-  version_requirements: *70199441231700
+  version_requirements: *2152281580
 description: 
 email:
 - stefano.diem@gmail.com
@@ -81,6 +81,9 @@ files:
 - lib/pag_seguro/version.rb
 - pag_seguro.gemspec
 - spec/pag_seguro/checkout_xml_spec.rb
+- spec/pag_seguro/errors/invalid_data_spec.rb
+- spec/pag_seguro/errors/unauthorized_spec.rb
+- spec/pag_seguro/errors/unknown_error_spec.rb
 - spec/pag_seguro/integration/checkout_spec.rb
 - spec/pag_seguro/integration/config.yml
 - spec/pag_seguro/item_spec.rb
@@ -117,6 +120,9 @@ specification_version: 3
 summary: A ruby gem to handle PagSeguro's API version 2
 test_files:
 - spec/pag_seguro/checkout_xml_spec.rb
+- spec/pag_seguro/errors/invalid_data_spec.rb
+- spec/pag_seguro/errors/unauthorized_spec.rb
+- spec/pag_seguro/errors/unknown_error_spec.rb
 - spec/pag_seguro/integration/checkout_spec.rb
 - spec/pag_seguro/integration/config.yml
 - spec/pag_seguro/item_spec.rb