pag_seguro 0.1.6 → 0.1.7

data/.gitignore

@@ -3,4 +3,4 @@
 Gemfile.lock
 pkg/*
 coverage/*
-spec/pag_seguro/integration/config.yml
+spec/pag_seguro/integration/config_example.yml

data/README.md

@@ -18,16 +18,17 @@ Além disso, é necessário que tenha uma conta no pag seguro, e que habilite as
     Em Integrações -> Pagamentos via API é necessário ativar a opção "Quero receber somente pagamentos via API."
     Em Integrações -> Notificação de transações é necessário ativar a notificação de transações e definir a url de retorno
 	
-## Documentação da API e dos attributos
+## Documentação
+### Classes e Atributos
 
-A nomenclatura dos atributos esperados pelo PagSeguro foram mantidas porém no padrão underscore (que é o padrão de ruby, ao invés do camelcase utilizado pelo pagseguro). Seguem os links das documentações dos attributos no pagseguro:
+A nomenclatura dos atributos e recursos (classes) esperados pelo PagSeguro foram mantidas porém usando o padrão de nomenclatura do ruby (ao invés do camelcase utilizado pelo pagseguro). Seguem os links das documentações dos atributos no pagseguro:
 
 * [API de pagamentos](https://pagseguro.uol.com.br/v2/guia-de-integracao/api-de-pagamentos.html#v2-item-api-de-pagamentos-parametros-api)
 * [API de notificação](https://pagseguro.uol.com.br/v2/guia-de-integracao/api-de-notificacoes.html)
 
-## Documentação
+###  API de Pagamento
 
-Segue um exemplo de uso desta gem (para mais exemplos, olhe o arquivo spec/integration/checkout_spec.rb):
+Segue um exemplo de uso para criação de um pagamento no PagSeguro:
 
     payment = PagSeguro::Payment.new(EMAIL, TOKEN)
     
@@ -38,7 +39,40 @@ Segue um exemplo de uso desta gem (para mais exemplos, olhe o arquivo spec/integ
     
     redirect_to_url = payment.checkout_payment_url
     
-Além dos items presentes no exemplo acima, é possível configurar `payment.sender` (com informações do usuário que está efetuando a compra) e `payment.shipping` ( com as informações de endereço ).
+O método checkout_payment_url envia as informações de `payment` ao PagSeguro e em caso de sucesso gera a url do PagSeguro para qual o comprador deverá ser redirecionado para efetuar a compra.
+
+Além dos items presentes no exemplo acima, é possível configurar `payment.sender` (com informações do usuário que está efetuando a compra), `payment.shipping` ( com as informações de endereço ), entre outras opções (para mais exemplos, olhe o arquivo spec/integration/checkout_spec.rb). Em especial, o attributo `payment.id` deve ser utilizado para referenciar um pagamento único no seu sistema.
+
+Com exceção do atributo response (que é utilizado para armazenar a resposta enviada pelo PagSeguro), todos os outros atributos podem ser inicialidos em formato hash na inicialização do `PagSeguro::Payment`:
+
+    payment = PagSeguro::Payment.new(EMAIL, TOKEN, id: 2, items: [ PagSeguro::Item.new(id: 17, description: "A pipe", amount: "3.00",  quantity: "89") ], extra_amount: '1.00' )
+
+### API de Notificação
+
+As notificações de alteração no status da compra no PagSeguro serão enviadas para a URL que tiver configurado na Notificação de transações (vide Instalação). Obs.: Até o momento o PagSeguro não permite configurar uma url dinâmica para envio das notificação ( e apenas permite uma url por conta ), então provavelemente será necessário que crie uma conta diferente no PagSeguro para cada sistema que desenvolver.
+
+O código da notificação é enviado pelo PagSeguro através do parâmentro `notificationCode` em uma requisição do tipo POST ( lembre-se de adicionar uma rota post respectiva ). Segue um exemplo de uso da notificação em uma aplicação rails:
+
+    class PagSeguroNotificationController < ApplicationController
+      def parse_notification
+        EMAIL = "seu_email_cadastrado@nopagseguro.com.br"
+        TOKEN = "SEU_TOKEN_GERADO_NO_PAG_SEGURO"
+        NOTIFICATION_CODE = params(:notificationCode)
+        
+        notification = PagSeguro::Notification.new(EMAIL, TOKEN, NOTIFICATION_CODE)
+        
+        if notification.approved?
+          # Idealmente faça alguns testes de sanidade, como notification.gross_amount, notification.item_count, etc
+          # notification.id referencia o id do payment, caso tenha sido configurado
+          # transacation_id identifica o código da transação no pag seguro
+          Invoice.find(notification.id).approve!(notification.transaction_id)
+        end
+        
+        if notification.cancelled? || notification.returned?
+          Invoice.find(notification.id).void!
+        end
+      end
+    end
 
 ## Validações
 
@@ -46,7 +80,7 @@ Os modelos utilizados nesta gem utilizam as validações do ActiveModel (semelha
 
 ## Testes
 
-Esta gem possui testes extensivos utilizando Rspec. Para rodar os estes, altere o arquivo spec/pag_seguro/integration/config.yml com seus dados no pag_seguro, entre na pasta onde a gem está instalada e execute:
+Esta gem possui testes extensivos utilizando Rspec. Para rodar os testes, altere o arquivo spec/pag_seguro/integration/config.yml com seus dados no pag_seguro, entre na pasta onde a gem está instalada e execute:
 
     bundle
     guard

data/lib/pag_seguro/checkout.xml.haml

@@ -39,7 +39,7 @@
 
   - if shipping.present?
     %shipping
-      - if shipping.type.present?
+      - if !shipping.type.zero?
         %type= shipping.type
       %address
         %country BRA

data/lib/pag_seguro/notification.rb

@@ -1,10 +1,25 @@
+# encoding: utf-8
 require 'net/https'
 
 module PagSeguro
   class Notification
     attr_accessor :data
-    PAGSEGURO_APPROVED = 3
-    PAGSEGURO_AVAILABLE = 4
+    
+    # possible status values
+    PAGSEGURO_PROCESSING        = 1
+    PAGSEGURO_IN_ANALYSIS       = 2
+    PAGSEGURO_APPROVED          = 3
+    PAGSEGURO_AVAILABLE         = 4
+    PAGSEGURO_DISPUTED          = 5
+    PAGSEGURO_RETURNED          = 6
+    PAGSEGURO_CANCELLED         = 7
+    
+    # possible type values
+    PAGSEGURO_PAYMENT           = 1
+    PAGSEGURO_TRANSFER          = 2
+    PAGSEGURO_ADDITION_OF_FUNDS = 3
+    PAGSEGURO_CHARGE            = 4
+    PAGSEGURO_BONUS             = 5
   
     def initialize(email = nil, token = nil, notification_code=nil)
       raise "Needs a notification code" if notification_code.blank?
@@ -13,25 +28,45 @@ module PagSeguro
       @data = Nokogiri::XML(notification_data(email, token, notification_code))
     end
       
-    def status
-      @data.css("status").first.content.to_i
-    end
-  
-    def approved?
-      PAGSEGURO_APPROVED == status
+    def id
+      @data.css("reference").first.content
     end
     
-    def available?
-      PAGSEGURO_AVAILABLE == status
+    def gross_amount
+      @data.css("grossAmount").first.content
     end
-  
-    def id
-      @data.css("reference").first.content
+
+    def discount_amount
+      @data.css("discountAmount").first.content
+    end
+
+    def fee_amount
+      @data.css("feeAmount").first.content
+    end
+
+    def net_amount
+      @data.css("feeAmount").first.content
+    end
+
+    def extra_amount
+      @data.css("feeAmount").first.content
+    end
+
+    def installment_count
+      @data.css("itemCount").first.content.to_i
+    end
+    
+    def item_count
+      @data.css("itemCount").first.content.to_i
     end
     
     def transaction_id
       @data.css("code").first.content
     end
+    
+    def date
+      @data.css("date").first.content
+    end
   
     def items
       @data.css("items item").map do |i|
@@ -39,6 +74,10 @@ module PagSeguro
       end
     end
     
+    def payment_method
+      pm = PaymentMethod.new(code: parse_css("paymentMethod code"), type: parse_css("paymentMethod type"))
+    end
+    
     def sender
       sn = Sender.new
       sn.name = parse_css("sender name")
@@ -62,6 +101,62 @@ module PagSeguro
       sh
     end
     
+    def status
+      @data.css("status").first.content.to_i
+    end
+
+    def type
+      @data.css("type").first.content.to_i
+    end
+
+    def processing?
+      PAGSEGURO_PROCESSING == status
+    end
+    
+    def in_analysis?
+      PAGSEGURO_IN_ANALYSIS == status
+    end
+
+    def approved?
+      PAGSEGURO_APPROVED == status
+    end
+    
+    def available?
+      PAGSEGURO_AVAILABLE == status
+    end
+    
+    def disputed?
+      PAGSEGURO_DISPUTED == status
+    end
+    
+    def returned?
+      PAGSEGURO_RETURNED == status
+    end
+    
+    def cancelled?
+      PAGSEGURO_CANCELLED == status
+    end
+    
+    def payment?
+      PAGSEGURO_PAYMENT == type
+    end
+    
+    def transfer?
+      PAGSEGURO_TRANSFER == type
+    end
+    
+    def addition_of_funds?
+      PAGSEGURO_ADDITION_OF_FUNDS == type
+    end
+    
+    def charge?
+      PAGSEGURO_CHARGE == type
+    end
+    
+    def bonus?
+      PAGSEGURO_BONUS == type
+    end
+  
     private
       def notification_data(email, token, notification_code)
         RestClient.get("https://ws.pagseguro.uol.com.br/v2/transactions/notifications/#{notification_code}?email=#{email}&token=#{token}")

data/lib/pag_seguro/payment.rb

@@ -63,7 +63,7 @@ module PagSeguro
       end
             
       def send_checkout
-        RestClient.post(checkout_url_with_params, checkout_xml, content_type: "application/xml"){|response, request, result| response }
+        RestClient.post(checkout_url_with_params, checkout_xml, content_type: "application/xml"){|resp, request, result| resp }
       end
       
       def parse_checkout_response
@@ -76,7 +76,7 @@ module PagSeguro
           raise Errors::InvalidData.new(response.body)
         else
           raise Errors::UnknownError.new(response)
-        end        
+        end
       end
       
       def parse_date

data/lib/pag_seguro/payment_method.rb

@@ -0,0 +1,72 @@
+# encoding: utf-8
+module PagSeguro
+  class PaymentMethod
+    attr_accessor :code, :type
+    
+    # Payment Method types
+    CREDIT_CARD        = 1
+    BANK_BILL          = 2
+    ONLINE_DEBIT       = 3
+    PAG_SEGURO_BALANCE = 4
+    OI_PAGGO           = 5
+    
+    def initialize(options = {})
+      @code = options[:code]
+      @type = options[:type]
+    end
+    
+    def code
+      @code.to_i
+    end
+    
+    def type
+      @type.to_i
+    end
+    
+    def credit_card?
+      CREDIT_CARD == type
+    end
+    
+    def bank_bill?
+      BANK_BILL == type
+    end
+    
+    def online_debit?
+      ONLINE_DEBIT == type
+    end
+    
+    def pag_seguro_balance?
+      PAG_SEGURO_BALANCE == type
+    end
+    
+    def oi_paggo?
+      OI_PAGGO == type
+    end
+    
+    def name
+      case code
+      when 101 then "Cartão de crédito Visa"
+      when 102 then "Cartão de crédito MasterCard"
+      when 103 then "Cartão de crédito American Express"
+      when 104 then "Cartão de crédito Diners"
+      when 105 then "Cartão de crédito Hipercard"
+      when 106 then "Cartão de crédito Aura"
+      when 107 then "Cartão de crédito Elo"
+      when 108 then "Cartão de crédito PLENOCard"
+      when 109 then "Cartão de crédito PersonalCard"
+      when 201 then "Boleto Bradesco"
+      when 202 then "Boleto Santander"
+      when 301 then "Débito online Bradesco"
+      when 302 then "Débito online Itaú"
+      when 303 then "Débito online Unibanco"
+      when 304 then "Débito online Banco do Brasil"
+      when 305 then "Débito online Banco Real"
+      when 306 then "Débito online Banrisul"        
+      when 307 then "Débito online HSBC"
+      when 401 then "Saldo PagSeguro"
+      when 501 then "Oi Paggo"
+      else "Desconhecido"
+      end
+    end
+  end
+end

data/lib/pag_seguro/shipping.rb

@@ -4,7 +4,7 @@ module PagSeguro
     
     PAC = 1
     SEDEX = 2
-    NOT_IDENTIFIED = 3
+    UNIDENTIFIED = 3
     
     validates_format_of :postal_code, with: /^\d{8}$/, message: " must be an integer with 8 digits", allow_blank: true
     
@@ -25,5 +25,21 @@ module PagSeguro
     def postal_code
       @postal_code if @postal_code.present? && @postal_code.to_s.size == 8
     end
+    
+    def type
+      @type.to_i
+    end
+    
+    def pac?
+      PAC == type
+    end
+    
+    def sedex?
+      SEDEX == type
+    end
+    
+    def unidentified?
+      UNIDENTIFIED == type
+    end
   end
 end

data/lib/pag_seguro/version.rb

@@ -1,3 +1,3 @@
 module PagSeguro
-  VERSION = "0.1.6"
+  VERSION = "0.1.7"
 end

data/lib/pag_seguro.rb

@@ -9,6 +9,7 @@ require "rest-client"
 # PagSeguro classes
 require "item"
 require "payment"
+require "payment_method"
 require "sender"
 require "shipping"
 require "notification"

data/spec/fixtures/notification.xml

@@ -0,0 +1,55 @@
+<?xml version="1.0" encoding="ISO-8859-1" standalone="yes"?>  
+<transaction>  
+  <date>2011-02-10T16:13:41.000-03:00</date>  
+  <code>9E884542-81B3-4419-9A75-BCC6FB495EF1</code>  
+  <reference>REF1234</reference>  
+  <type>1</type>  
+  <status>3</status>  
+  <paymentMethod>  
+    <type>1</type>  
+    <code>101</code>  
+  </paymentMethod>  
+  <grossAmount>49900.00</grossAmount>  
+  <discountAmount>0.00</discountAmount>  
+  <feeAmount>0.00</feeAmount>  
+  <netAmount>49900.00</netAmount>  
+  <extraAmount>0.00</extraAmount>  
+  <installmentCount>1</installmentCount>  
+  <itemCount>2</itemCount>  
+  <items>  
+    <item>  
+      <id>0001</id>  
+      <description>Notebook Prata</description>  
+      <quantity>1</quantity>  
+      <amount>24300.00</amount>  
+    </item>  
+    <item>  
+      <id>0002</id>  
+      <description>Notebook Rosa</description>  
+      <quantity>1</quantity>  
+      <amount>25600.00</amount>  
+    </item>  
+  </items>  
+  <sender>  
+    <name>Jos� Comprador</name>  
+    <email>comprador@uol.com.br</email>  
+    <phone>  
+      <areaCode>11</areaCode>  
+      <number>56273440</number>  
+    </phone>  
+  </sender>  
+  <shipping>  
+    <address>  
+      <street>Av. Brig. Faria Lima</street>  
+      <number>1384</number>  
+      <complement>5o andar</complement>  
+      <district>Jardim Paulistano</district>  
+      <postalCode>01452002</postalCode>  
+      <city>Sao Paulo</city>  
+      <state>SP</state>  
+      <country>BRA</country>  
+    </address>  
+    <type>1</type>  
+    <cost>21.50</cost>  
+  </shipping>  
+</transaction>

data/spec/pag_seguro/checkout_xml_spec.rb

@@ -102,7 +102,7 @@ describe PagSeguro::Payment do
       
       it "should have shipping type" do
         @xml_without_shipping_info.css("checkout shipping type").should be_empty
-        @xml.css("checkout shipping type").first.content.to_i.should == PagSeguro::Shipping::SEDEX
+        @xml.css("checkout shipping type").first.content.to_i.should == 2
       end
 
       it "should have state" do

data/spec/pag_seguro/integration/checkout_spec.rb

@@ -1,10 +1,6 @@
 # encoding: utf-8
 require 'spec_helper'
 
-config = YAML.load_file(File.dirname(__FILE__) + "/config.yml")
-EMAIL = config["email"]
-TOKEN = config["token"]
-
 def create_valid_payment
   payment = PagSeguro::Payment.new(EMAIL, TOKEN)
   payment.items = [
@@ -18,63 +14,73 @@ def create_valid_payment
   payment
 end
 
-describe "PagSeguro::Payment.code" do
-  it "should send a request to pagseguro" do
-    payment = create_valid_payment
-    payment.code.size.should == 32
+describe PagSeguro::Payment do
+  before do
+    if EMAIL == "seu_email_cadastrado@nopagseguro.com.br"
+      pending "You need to set your email for your PagSeguro account in spec/pag_seguro/integration/config.yml in order to run this spec"
+    elsif TOKEN == "SEU_TOKEN_GERADO_NO_PAG_SEGURO"
+      pending "You need to set your token for your PagSeguro account in spec/pag_seguro/integration/config.yml in order to run this spec"
+    end
   end
   
-  it "should be valid even without a sender and shipping information" do
-    payment = create_valid_payment
-    payment.sender = nil
-    payment.shipping = nil
-    payment.code.size.should == 32
-  end
+  describe "#code" do
+    it "should send a request to pagseguro" do
+      payment = create_valid_payment
+      payment.code.size.should == 32
+    end
+  
+    it "should be valid even without a sender and shipping information" do
+      payment = create_valid_payment
+      payment.sender = nil
+      payment.shipping = nil
+      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)
-  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)
+    end
     
-  it "should list errors given by pagseguro" do
-    payment = PagSeguro::Payment.new(EMAIL, TOKEN)
-    lambda { payment.code }.should raise_error(PagSeguro::Errors::InvalidData)
-  end
+    it "should list errors given by pagseguro" do
+      payment = PagSeguro::Payment.new(EMAIL, TOKEN)
+      lambda { payment.code }.should raise_error(PagSeguro::Errors::InvalidData)
+    end
   
-  it "should give a response code of 200 for the user pagseguro url" do
-    payment = create_valid_payment
-    RestClient.get(payment.checkout_payment_url).code.should == 200
+    it "should give a response code of 200 for the user pagseguro url" do
+      payment = create_valid_payment
+      RestClient.get(payment.checkout_payment_url).code.should == 200
+    end
   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}$/)
+  describe "#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
-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
+  describe "#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
+    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
+      @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
+    it "should 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
+      @payment.code
+      @payment.reset!
+      @payment.date
+    end
   end
 end

data/spec/pag_seguro/integration/config.yml

@@ -1,2 +1,3 @@
 email: seu_email_cadastrado@nopagseguro.com.br
-token: SEU_TOKEN_GERADO_NO_PAG_SEGURO
+token: SEU_TOKEN_GERADO_NO_PAG_SEGURO
+notification_code: SEU_CODIGO_DE_NOTIFICACAO

data/spec/pag_seguro/integration/notification_spec.rb

@@ -0,0 +1,44 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe PagSeguro::Notification do
+  before :all do
+    if EMAIL == "seu_email_cadastrado@nopagseguro.com.br"
+      pending "You need to set your email for your PagSeguro account in spec/pag_seguro/integration/config.yml in order to run this spec"
+    elsif TOKEN == "SEU_TOKEN_GERADO_NO_PAG_SEGURO"
+      pending "You need to set your token for your PagSeguro account in spec/pag_seguro/integration/config.yml in order to run this spec"
+    elsif NOTIFICATION_CODE == "SEU_CODIGO_DE_NOTIFICACAO"
+      pending "You need to set your notification token for your PagSeguro account in spec/pag_seguro/integration/config.yml in order to run this spec"
+    else
+      @notification = PagSeguro::Notification.new(EMAIL, TOKEN, NOTIFICATION_CODE)
+    end
+  end
+  
+  it { @notification.transaction_id.should be_present }
+  it { @notification.date.should be_present }
+  it { @notification.id.should be_present }
+  it { @notification.type.should be_present }
+  it { @notification.status.should be_present }
+  it { @notification.payment_method.type.should be_present }
+  it { @notification.payment_method.code.should be_present }
+  it { @notification.gross_amount.should be_present }
+  it { @notification.discount_amount.should be_present }
+  it { @notification.fee_amount.should be_present }
+  it { @notification.net_amount.should be_present }
+  it { @notification.extra_amount.should be_present }
+  it { @notification.installment_count.should be_present }
+  it { @notification.item_count.should be_present }
+  it { @notification.items.should be_present }
+
+  it "should have all required item attributes" do
+    @notification.items.each do |item|
+      item.id.should be_present
+      item.description.should be_present
+      item.amount.should be_present
+      item.quantity.should be_present
+    end
+  end
+  
+  it { @notification.sender.email.should be_present }
+  it { @notification.shipping.type.should be_present }
+end

data/spec/pag_seguro/notification_spec.rb

@@ -1,10 +1,11 @@
 # encoding: utf-8
 
 require 'spec_helper'
-require_relative 'notification_data_mock'
 
 describe PagSeguro::Notification do
   before do
+    notification_xml_mock = File.open( File.expand_path( File.dirname(__FILE__) + '/../fixtures/notification.xml') )
+    PagSeguro::Notification.any_instance.stub(:notification_data){ notification_xml_mock }
     @notification = PagSeguro::Notification.new("mail", "token", "not_code")
   end
   
@@ -16,6 +17,42 @@ describe PagSeguro::Notification do
     @notification.transaction_id.should == "9E884542-81B3-4419-9A75-BCC6FB495EF1"
   end
   
+  it "should have a gross amount" do
+    @notification.gross_amount.should be_present
+    @notification.gross_amount.should match(/^\d+\.\d{2}$/)
+  end
+
+  it "should have a discount amount" do
+    @notification.discount_amount.should be_present
+    @notification.discount_amount.should match(/^\d+\.\d{2}$/)
+  end
+
+  it "should have a fee amount" do
+    @notification.fee_amount.should be_present
+    @notification.fee_amount.should match(/^\d+\.\d{2}$/)
+  end
+  
+  it "should have a net amount" do
+    @notification.net_amount.should be_present
+    @notification.net_amount.should match(/^\d+\.\d{2}$/)
+  end
+
+  it "should have an extra amount" do
+    @notification.extra_amount.should be_present
+    @notification.extra_amount.should match(/^\d+\.\d{2}$/)
+  end
+
+  it "should have an installment count" do
+    @notification.installment_count.should be_present
+    @notification.installment_count.should be_an_integer
+  end
+  
+  it "should have an item count" do
+    @notification.item_count.should be_present
+    @notification.item_count.should be_an_integer
+    @notification.item_count.should == @notification.items.count
+  end
+  
   it "should be approved in this case" do
     @notification.should be_approved
   end
@@ -28,9 +65,14 @@ describe PagSeguro::Notification do
     @sender.phone_number == "56273440"
   end
   
+  it "should have a date" do
+    @notification.date.should be_present
+    @notification.date.should match(/^\d{4}\-\d{2}\-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}[-+]\d{2}:\d{2}$/)
+  end
+  
   it "should have a shipping" do
     @shipping = @notification.shipping
-    @shipping.type.should == "1"
+    @shipping.type.should == 1
     @shipping.cost.should == "21.50"
     @shipping.state.should == "SP"
     @shipping.city.should == "Sao Paulo"
@@ -41,6 +83,12 @@ describe PagSeguro::Notification do
     @shipping.complement.should == "5o andar"
   end
   
+  it "should have a payment method" do
+    @payment_method = @notification.payment_method
+    @payment_method.code.should == 101
+    @payment_method.type.should == 1
+  end
+  
   it "should have items" do
     @items = @notification.items
     @items.size.should == 2
@@ -56,13 +104,80 @@ describe PagSeguro::Notification do
     @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
+  describe "status" do
+    it "should have a status" do
+      @notification.status.should == 3
+    end
+    
+    it "should be processing if its status is 1" do
+      @notification.stub(:status){ 1 }
+      @notification.should be_processing
+    end
+
+    it "should be in analysis if its status is 2" do
+      @notification.stub(:status){ 2 }
+      @notification.should be_in_analysis
+    end
+
+    it "should be approved if its status is 3" do
+      @notification.stub(:status){ 3 }
+      @notification.should be_approved
+    end
+
+    it "should be available if its status is 4" do
+      @notification.stub(:status){ 4 }
+      @notification.should be_available
+    end
+
+    it "should be disputed if its status is 5" do
+      @notification.stub(:status){ 5 }
+      @notification.should be_disputed
+    end
+
+    it "should be disputed if its status is 5" do
+      @notification.stub(:status){ 5 }
+      @notification.should be_disputed
+    end
+
+    it "should be returned if its status is 6" do
+      @notification.stub(:status){ 6 }
+      @notification.should be_returned
+    end
+
+    it "should be cancelled if its status is 7" do
+      @notification.stub(:status){ 7 }
+      @notification.should be_cancelled
+    end
   end
   
-  it "should be_approved if its status is 3" do
-    @notification.stub(:status){ 3 }
-    @notification.should be_approved
+  describe "type" do
+    it "should have a type" do
+      @notification.type.should == 1
+    end
+    
+    it "should be payment if type is 1" do
+      @notification.stub(:type){ 1 }
+      @notification.should be_payment
+    end
+
+    it "should be transfer if type is 2" do
+      @notification.stub(:type){ 2 }
+      @notification.should be_transfer
+    end
+
+    it "should be addition of funds if type is 3" do
+      @notification.stub(:type){ 3 }
+      @notification.should be_addition_of_funds
+    end
+
+    it "should be charge if type is 4" do
+      @notification.stub(:type){ 4 }
+      @notification.should be_charge
+    end
+
+    it "should be bonus if type is 5" do
+      @notification.stub(:type){ 5 }
+      @notification.should be_bonus
+    end
   end
 end

data/spec/pag_seguro/payment_method_spec.rb

@@ -0,0 +1,76 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe PagSeguro::PaymentMethod do
+  context "instance" do
+    context "accessors" do
+      before { @payment_method = PagSeguro::PaymentMethod.new }
+      
+      it { @payment_method.should have_attribute_accessor(:code) }
+	    it { @payment_method.should have_attribute_accessor(:type) }      
+	  end
+    
+    it "should be able to initialize with all attributes" do
+      payment_method = PagSeguro::PaymentMethod.new(code: "101", type: "1")
+      payment_method.code.should == 101
+      payment_method.type.should == 1
+    end
+    
+    describe "types" do
+      before { @payment_method = PagSeguro::PaymentMethod.new }
+      
+      it "should be credit card if type is 1" do
+        @payment_method.stub(:type){ 1 }
+        @payment_method.should be_credit_card
+      end
+      
+      it "should be bank bill if type is 2" do
+        @payment_method.stub(:type){ 2 }
+        @payment_method.should be_bank_bill
+      end
+      
+      it "should be online debit if type is 3" do
+        @payment_method.stub(:type){ 3 }
+        @payment_method.should be_online_debit
+      end
+
+      it "should be PagSeguro balance if type is 4" do
+        @payment_method.stub(:type){ 4 }
+        @payment_method.should be_pag_seguro_balance
+      end
+      
+      it "should be oi paggo if type is 5" do
+        @payment_method.stub(:type){ 5 }
+        @payment_method.should be_oi_paggo
+      end
+    end
+    
+    describe "codes" do
+      def should_have_meaning_for_code(meaning, code)
+        PagSeguro::PaymentMethod.new(code: code).name.should == meaning
+      end
+      
+      it { should_have_meaning_for_code("Cartão de crédito Visa", 101) }
+      it { should_have_meaning_for_code("Cartão de crédito MasterCard", 102) }
+      it { should_have_meaning_for_code("Cartão de crédito American Express", 103) }
+      it { should_have_meaning_for_code("Cartão de crédito Diners", 104) }
+      it { should_have_meaning_for_code("Cartão de crédito Hipercard", 105) }
+      it { should_have_meaning_for_code("Cartão de crédito Aura", 106) }
+      it { should_have_meaning_for_code("Cartão de crédito Elo", 107) }
+      it { should_have_meaning_for_code("Cartão de crédito PLENOCard", 108) }
+      it { should_have_meaning_for_code("Cartão de crédito PersonalCard", 109) }
+      it { should_have_meaning_for_code("Boleto Bradesco", 201) }
+      it { should_have_meaning_for_code("Boleto Santander", 202) }
+      it { should_have_meaning_for_code("Débito online Bradesco", 301) }
+      it { should_have_meaning_for_code("Débito online Itaú", 302) }
+      it { should_have_meaning_for_code("Débito online Unibanco", 303) }
+      it { should_have_meaning_for_code("Débito online Banco do Brasil", 304) }
+      it { should_have_meaning_for_code("Débito online Banco Real", 305) }
+      it { should_have_meaning_for_code("Débito online Banrisul", 306) }
+      it { should_have_meaning_for_code("Débito online HSBC", 307) }
+      it { should_have_meaning_for_code("Saldo PagSeguro", 401) }
+      it { should_have_meaning_for_code("Oi Paggo", 501) }
+      it { should_have_meaning_for_code("Desconhecido", 0) }
+    end
+  end
+end

data/spec/pag_seguro/payment_spec.rb

@@ -107,29 +107,37 @@ describe PagSeguro::Payment do
     describe "parse_checkout_response" do
       before do
         @payment = PagSeguro::Payment.new("mymail", "mytoken")
-        @response = double("response")
-        @payment.stub(:send_checkout){ @response }
+      end
+
+      it "should not raise errors if response code is 200" do
+        PagSeguro::Payment.any_instance.stub_chain(:send_checkout, :code){ 200 }
+        PagSeguro::Payment.any_instance.stub_chain(:send_checkout, :body){ "some body info" }
+        lambda { @payment.send(:parse_checkout_response) }.should_not raise_error
       end
       
       it "should raise PagSeguro::Errors::InvalidData if response code is 400" do
-        @response.stub(code: 400, body: "some error description")
+        PagSeguro::Payment.any_instance.stub_chain(:send_checkout, :code){ 400 }
+        PagSeguro::Payment.any_instance.stub_chain(:send_checkout, :body){ "some error description" }
         lambda { @payment.send(:parse_checkout_response) }.should raise_error(PagSeguro::Errors::InvalidData)
       end
       
       it "should raise PagSeguro::Errors::Unauthorized if response code is 400" do
-        @response.stub(code: 401)
+        PagSeguro::Payment.any_instance.stub_chain(:send_checkout, :code){ 401 }
         lambda { @payment.send(:parse_checkout_response) }.should raise_error(PagSeguro::Errors::Unauthorized)
       end
-      
-      it "should not raise errors if response code is 200" do
-        @response.stub(code: 200, body: "some response body")
-        lambda { @payment.send(:parse_checkout_response) }.should_not raise_error
-      end
-      
+            
       it "should raise PagSeguro::Errors::UnknownError if response code is not 200, 400 or 401" do
-        @response.stub(code: 300, body: "some response body")
+        PagSeguro::Payment.any_instance.stub_chain(:send_checkout, :code){ 300 }
+        PagSeguro::Payment.any_instance.stub_chain(:send_checkout, :body){ "some response body" }
         lambda { @payment.send(:parse_checkout_response) }.should raise_error(PagSeguro::Errors::UnknownError)
       end
+      
+      it "should be able to reset response" do
+        @payment.response = "something"
+        @payment.response.should be_present
+        @payment.reset!
+        @payment.response.should be_nil
+      end
     end
     
   end

data/spec/pag_seguro/shipping_spec.rb

@@ -23,6 +23,23 @@ describe PagSeguro::Shipping do
     it { @shipping.should have_attribute_accessor(:street) }
     it { @shipping.should have_attribute_accessor(:number) }
     it { @shipping.should have_attribute_accessor(:complement) }
+    
+    describe "types" do
+      it "should be pac if type is 1" do
+        @shipping.stub( :type ){ 1 }
+        @shipping.should be_pac
+      end
+      
+      it "should be sedex if type is 2" do
+        @shipping.stub( :type ){ 2 }
+        @shipping.should be_sedex
+      end
+      
+      it "should be unidentified if type is 3" do
+        @shipping.stub( :type ){ 3 }
+        @shipping.should be_unidentified
+      end
+    end
   end
   
   it "should be able to initialize all attributes" do

data/spec/spec_helper.rb

@@ -6,6 +6,11 @@ end
 require 'yaml'
 require File.dirname(__FILE__) + "/../lib/pag_seguro"
 
+config = YAML.load_file(File.dirname(__FILE__) + "/pag_seguro/integration/config.yml")
+EMAIL = config["email"]
+TOKEN = config["token"]
+NOTIFICATION_CODE = config["notification_code"]
+
 class HaveAttributeAccessor
   def initialize(attribute)
     @attribute = attribute

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pag_seguro
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &2168863380 !ruby/object:Gem::Requirement
+  requirement: &2152303520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2168863380
+  version_requirements: *2152303520
 - !ruby/object:Gem::Dependency
   name: haml
-  requirement: &2168862900 !ruby/object:Gem::Requirement
+  requirement: &2152302620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2168862900
+  version_requirements: *2152302620
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &2168862140 !ruby/object:Gem::Requirement
+  requirement: &2152302080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2168862140
+  version_requirements: *2152302080
 - !ruby/object:Gem::Dependency
   name: rest-client
-  requirement: &2168861420 !ruby/object:Gem::Requirement
+  requirement: &2152300820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,7 +54,7 @@ dependencies:
         version: 1.6.7
   type: :runtime
   prerelease: false
-  version_requirements: *2168861420
+  version_requirements: *2152300820
 description: 
 email:
 - stefano.diem@gmail.com
@@ -76,19 +76,22 @@ files:
 - lib/pag_seguro/item.rb
 - lib/pag_seguro/notification.rb
 - lib/pag_seguro/payment.rb
+- lib/pag_seguro/payment_method.rb
 - lib/pag_seguro/sender.rb
 - lib/pag_seguro/shipping.rb
 - lib/pag_seguro/version.rb
 - pag_seguro.gemspec
+- spec/fixtures/notification.xml
 - 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/integration/notification_spec.rb
 - spec/pag_seguro/item_spec.rb
-- spec/pag_seguro/notification_data_mock.rb
 - spec/pag_seguro/notification_spec.rb
+- spec/pag_seguro/payment_method_spec.rb
 - spec/pag_seguro/payment_spec.rb
 - spec/pag_seguro/sender_spec.rb
 - spec/pag_seguro/shipping_spec.rb
@@ -119,15 +122,17 @@ signing_key:
 specification_version: 3
 summary: A ruby gem to handle PagSeguro's API version 2
 test_files:
+- spec/fixtures/notification.xml
 - 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/integration/notification_spec.rb
 - spec/pag_seguro/item_spec.rb
-- spec/pag_seguro/notification_data_mock.rb
 - spec/pag_seguro/notification_spec.rb
+- spec/pag_seguro/payment_method_spec.rb
 - spec/pag_seguro/payment_spec.rb
 - spec/pag_seguro/sender_spec.rb
 - spec/pag_seguro/shipping_spec.rb

data/spec/pag_seguro/notification_data_mock.rb

@@ -1,64 +0,0 @@
-# encoding: utf-8
-module PagSeguro
-  class Notification
-    def notification_data(email, token, notification_code)
-      <<-XML
-        <?xml version="1.0" encoding="ISO-8859-1" standalone="yes"?>  
-        <transaction>  
-          <date>2011-02-10T16:13:41.000-03:00</date>  
-          <code>9E884542-81B3-4419-9A75-BCC6FB495EF1</code>  
-          <reference>REF1234</reference>  
-          <type>1</type>  
-          <status>3</status>  
-          <paymentMethod>  
-            <type>1</type>  
-            <code>101</code>  
-          </paymentMethod>  
-          <grossAmount>49900.00</grossAmount>  
-          <discountAmount>0.00</discountAmount>  
-          <feeAmount>0.00</feeAmount>  
-          <netAmount>49900.00</netAmount>  
-          <extraAmount>0.00</extraAmount>  
-          <installmentCount>1</installmentCount>  
-          <itemCount>2</itemCount>  
-          <items>  
-            <item>  
-              <id>0001</id>  
-              <description>Notebook Prata</description>  
-              <quantity>1</quantity>  
-              <amount>24300.00</amount>  
-            </item>  
-            <item>  
-              <id>0002</id>  
-              <description>Notebook Rosa</description>  
-              <quantity>1</quantity>  
-              <amount>25600.00</amount>  
-            </item>  
-          </items>  
-          <sender>  
-            <name>José Comprador</name>  
-            <email>comprador@uol.com.br</email>  
-            <phone>  
-              <areaCode>11</areaCode>  
-              <number>56273440</number>  
-            </phone>  
-          </sender>  
-          <shipping>  
-            <address>  
-              <street>Av. Brig. Faria Lima</street>  
-              <number>1384</number>  
-              <complement>5o andar</complement>  
-              <district>Jardim Paulistano</district>  
-              <postalCode>01452002</postalCode>  
-              <city>Sao Paulo</city>  
-              <state>SP</state>  
-              <country>BRA</country>  
-            </address>  
-            <type>1</type>  
-            <cost>21.50</cost>  
-          </shipping>  
-        </transaction> 
-      XML
-    end
-  end
-end