pagseguro-oficial 2.4.0 → 2.5.0

data/spec/pagseguro/features/create_transaction_request_spec.rb

@@ -37,8 +37,8 @@ RSpec.describe "Creating Transaction Request" do
         status: [400, "Bad Request"], body: body, content_type: "text/xml"
     end
 
-    it "returns false" do
-      expect(transaction.create).to be_falsey
+    it "does not change attributes" do
+      expect { transaction.create }.not_to change { transaction.code }
     end
 
     describe "#errors" do

data/spec/pagseguro/installment/response_spec.rb

@@ -39,9 +39,11 @@ RSpec.describe PagSeguro::Installment::Response do
 
     context "when request fails" do
       before do
-        allow(http_response).to receive(:success?).and_return(false)
-        allow(http_response).to receive(:bad_request?).and_return(true)
-        allow(http_response).to receive(:not_found?).and_return(false)
+        allow(http_response).to receive_messages(
+          success?: false,
+          error?: true,
+          error: Aitch::BadRequestError
+        )
       end
 
       let(:raw_xml) { File.read("./spec/fixtures/invalid_code.xml") }

data/spec/pagseguro/installment_spec.rb

@@ -33,8 +33,11 @@ describe PagSeguro::Installment do
 
     context "when request fails" do
       before do
-        allow(request).to receive(:success?).and_return(false)
-        allow(request).to receive(:bad_request?).and_return(true)
+        allow(request).to receive_messages(
+          success?: false,
+          error?: true,
+          error: Aitch::BadRequestError
+        )
       end
 
       let(:raw_xml) { File.read("./spec/fixtures/invalid_code.xml") }

data/spec/pagseguro/items_spec.rb

@@ -25,8 +25,8 @@ describe PagSeguro::Items do
   it "sets item from Hash" do
     item = PagSeguro::Item.new(id: 1234)
 
-    PagSeguro::Item
-      .should_receive(:new)
+    expect(PagSeguro::Item)
+      .to receive(:new)
       .with({id: 1234})
       .and_return(item)
 
@@ -36,16 +36,6 @@ describe PagSeguro::Items do
     expect(items).to include(item)
   end
 
-  it "increases quantity when adding duplicated item" do
-    item = PagSeguro::Item.new(id: 1234)
-    items = PagSeguro::Items.new
-    items << item
-    items << item
-
-    expect(items.size).to eql(1)
-    expect(item.quantity).to eql(2)
-  end
-
   it "empties items" do
     items = PagSeguro::Items.new
     items << {id: 1234}
@@ -53,4 +43,106 @@ describe PagSeguro::Items do
 
     expect(items).to be_empty
   end
+
+  context "#<<" do
+    before do
+      subject << item
+    end
+
+    subject do
+      PagSeguro::Items.new
+    end
+
+    let(:item) do
+      PagSeguro::Item.new(id: 1234, description: "Tea", quantity: 1, amount: 300.0)
+    end
+
+    context "when add same item" do
+      context "and no quantity is informed" do
+        let(:other_item) do
+          { id: 1234, description: "Tea", amount: 300.0 }
+        end
+
+        it "increases one in item quantity" do
+          expect { subject << other_item }.to change { item.quantity }.by(1)
+        end
+
+        it "does not increase items size" do
+          expect { subject << other_item }.not_to change { subject.size }
+        end
+      end
+
+      context "and quantity is informed" do
+        let(:other_item) do
+          { id: 1234, description: "Tea", quantity: 20, amount: 300.0 }
+        end
+
+        it "increases item quantity" do
+          expect { subject << other_item }.to change { item.quantity }.by(20)
+        end
+
+        it "does not increase items size" do
+          expect { subject << other_item }.not_to change { subject.size }
+        end
+      end
+    end
+
+    context "when add different item" do
+      context "with only description equals" do
+        let(:other_item) do
+          { id: 4321, description: "Tea" }
+        end
+
+        it "does not increase item quantity" do
+          expect { subject << other_item }.not_to change { item.quantity }
+        end
+
+        it "increases items size" do
+          expect { subject << other_item }.to change { subject.size }.by(1)
+        end
+      end
+
+      context "with only id equals" do
+        let(:other_item) do
+          { id: 1234, description: "Coffee" }
+        end
+
+        it "does not increase item quantity" do
+          expect { subject << other_item }.not_to change { item.quantity }
+        end
+
+        it "increases subject size" do
+          expect { subject << other_item }.to change { subject.size }.by(1)
+        end
+      end
+
+      context "with only amount equals" do
+        let(:other_item) do
+          { id: 4321, description: "Coffee", amount: 300.0 }
+        end
+
+        it "does not increase item quantity" do
+          expect { subject << other_item }.not_to change { item.quantity }
+        end
+
+        it "increases subject size" do
+          expect { subject << other_item }.to change { subject.size }.by(1)
+        end
+      end
+
+      context "with id and description different" do
+        let(:other_item) do
+          { id: 4321, description: "Coffee" }
+        end
+
+        it "does not increase item quantity" do
+          expect { subject << other_item }.not_to change { item.quantity }
+        end
+
+        it "increases subject size" do
+          expect { subject << other_item }.to change { subject.size }.by(1)
+        end
+      end
+    end
+  end
 end

data/spec/pagseguro/pagseguro_spec.rb

@@ -7,12 +7,12 @@ describe PagSeguro do
       config.app_key = "APPTOKEN"
       config.email = "EMAIL"
       config.token = "TOKEN"
+      config.receiver_email = "RECEIVER_EMAIL"
     end
-    PagSeguro.receiver_email = "RECEIVER_EMAIL"
   end
 
   after do
-    PagSeguro.environment = :production
+    PagSeguro.configuration.environment = :production
   end
 
   it { expect(PagSeguro.email).to eql("EMAIL") }
@@ -33,8 +33,8 @@ describe PagSeguro do
   end
 
   context "default settings" do
-    it { expect(PagSeguro.encoding).to eql("UTF-8") }
-    it { expect(PagSeguro.environment).to eql(:production) }
+    it { expect(PagSeguro.configuration.encoding).to eql("UTF-8") }
+    it { expect(PagSeguro.configuration.environment).to eql(:production) }
     it { expect(PagSeguro.account_credentials.email).to eql("EMAIL") }
     it { expect(PagSeguro.account_credentials.token).to eql("TOKEN") }
     it { expect(PagSeguro.application_credentials.app_id).to eql("APP123") }
@@ -43,7 +43,7 @@ describe PagSeguro do
 
   describe ".api_url" do
     it "raises when environment has no endpoint" do
-      PagSeguro.environment = :invalid
+      PagSeguro.configuration.environment = :invalid
 
       expect {
         PagSeguro.api_url("/")
@@ -55,7 +55,7 @@ describe PagSeguro do
     end
 
     it "returns sandbox api url when the environment is :sandbox" do
-      PagSeguro.environment = :sandbox
+      PagSeguro.configuration.environment = :sandbox
 
       expect(PagSeguro.api_url("v2/some/path")).to eql("https://ws.sandbox.pagseguro.uol.com.br/v2/some/path")
     end
@@ -63,7 +63,7 @@ describe PagSeguro do
 
   describe ".site_url" do
     it "raises when environment has no endpoint" do
-      PagSeguro.environment = :invalid
+      PagSeguro.configuration.environment = :invalid
 
       expect {
         PagSeguro.site_url("/")
@@ -75,7 +75,7 @@ describe PagSeguro do
     end
 
     it "returns sandbox site url when the environment is :sandbox" do
-      PagSeguro.environment = :sandbox
+      PagSeguro.configuration.environment = :sandbox
       expect(PagSeguro.site_url("v2/some/path")).to eql("https://sandbox.pagseguro.uol.com.br/v2/some/path")
     end
   end

data/spec/pagseguro/partner_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+
+RSpec.describe PagSeguro::Partner do
+  it_assigns_attribute :name
+  it_assigns_attribute :birth_date
+
+  it_ensures_collection_type PagSeguro::Document, :documents, [{type: 'CPF', value: 12312312312}]
+end

data/spec/pagseguro/payment_request/request_serializer_spec.rb

@@ -0,0 +1,251 @@
+require "spec_helper"
+
+describe PagSeguro::PaymentRequest::RequestSerializer do
+  context '#to_xml_params' do
+    subject { described_class.new(payment) }
+
+    let(:payment) { PagSeguro::PaymentRequest.new }
+
+    it 'should serializer primary receivers' do
+      payment.primary_receiver = 'a@example.com'
+      expect(subject.to_xml_params).to match %r[<primaryReceiver>.*<email>a@example.com<\/email>.*<\/primaryReceiver>]m
+    end
+
+    it 'should serializer other receivers' do
+      payment.receivers = [
+        {
+          email: 'a@example.com',
+          split: {
+            amount: 10,
+            fee_percent: 11,
+            rate_percent: 12
+          }
+        }
+      ]
+      expect(subject.to_xml_params).to match %r[<receivers>.*<receiver>.*<email>a@example.com<\/email>]m
+    end
+
+    it 'should serializer sender' do
+      payment.sender = PagSeguro::Sender.new(
+        name: 'Sender',
+        email: 'b@example.com',
+        phone: PagSeguro::Phone.new(area_code: 51, number: 2)
+      )
+      expect(subject.to_xml_params).to match %r[<sender>.*<name>Sender<\/name>.*<email>b@example.com<\/email>.*<phone>.*<areaCode>51<\/areaCode>.*<number>2<\/number>]m
+    end
+
+    it 'should serializer currency' do
+      payment.currency = 'BRL'
+      expect(subject.to_xml_params).to match %r[<currency>BRL</currency>]
+    end
+
+    it 'should serializer reference' do
+      payment.reference = 'order123'
+      expect(subject.to_xml_params).to match %r[<reference>order123</reference>]
+    end
+
+    it 'should serializer redirect url' do
+      payment.redirect_url = 'http://example.com'
+      expect(subject.to_xml_params).to match %r[<redirectURL>http://example.com</redirectURL>]
+    end
+
+    it 'should serializer notification url' do
+      payment.notification_url = 'http://example.com'
+      expect(subject.to_xml_params).to match %r[<notificationURL>http://example.com</notificationURL>]
+    end
+
+    context 'should serializer item' do
+      before do
+        payment.items << { id: 1, description: 'Description#1', quantity: 2, amount: 3, weight: 4, shipping_cost: 5 }
+      end
+
+      it 'to include id' do
+        expect(subject.to_xml_params).to match %r[<id>1</id>]
+      end
+
+      it 'to include description' do
+        expect(subject.to_xml_params).to match %r[<description>Description#1</description>]
+      end
+
+      it 'to include quantity' do
+        expect(subject.to_xml_params).to match %r[<quantity>2</quantity>]
+      end
+
+      it 'to include amount' do
+        expect(subject.to_xml_params).to match %r[<amount>3.00</amount>]
+      end
+
+      it 'to include weight' do
+        expect(subject.to_xml_params).to match %r[<weight>4</weight>]
+      end
+
+      it 'to include weight' do
+        expect(subject.to_xml_params).to match %r[<shippingCost>5.00</shippingCost>]
+      end
+    end
+  end
+
+  context '#to_params' do
+    let(:payment_request) { PagSeguro::PaymentRequest.new }
+    let(:params) { serializer.to_params }
+    subject(:serializer) { described_class.new(payment_request) }
+
+    context "global configuration serialization" do
+      before do
+        PagSeguro.configuration.receiver_email = "RECEIVER"
+      end
+
+      it { expect(params).to include(receiverEmail: PagSeguro.configuration.receiver_email) }
+    end
+
+    context "generic attributes serialization" do
+      let(:payment_request) do
+        double(:PaymentRequest,
+          currency: "BRL",
+          reference: "REF123",
+          extra_amount: 1234.50,
+          redirect_url: "REDIRECT_URL",
+          notification_url: "NOTIFICATION_URL",
+          abandon_url: "ABANDON_URL",
+          max_uses: 5,
+          max_age: 3600,
+        ).as_null_object
+      end
+
+      it { expect(params).to include(currency: "BRL") }
+      it { expect(params).to include(reference: "REF123") }
+      it { expect(params).to include(extraAmount: "1234.50") }
+      it { expect(params).to include(redirectURL: "REDIRECT_URL") }
+      it { expect(params).to include(notificationURL: "NOTIFICATION_URL") }
+      it { expect(params).to include(abandonURL: "ABANDON_URL") }
+      it { expect(params).to include(maxUses: 5) }
+      it { expect(params).to include(maxAge: 3600) }
+    end
+
+    context "shipping serialization" do
+      before do
+        payment_request.shipping = PagSeguro::Shipping.new({
+          type_id: 1,
+          cost: 1234.56
+        })
+      end
+
+      it { expect(params).to include(shippingType: 1) }
+      it { expect(params).to include(shippingCost: "1234.56") }
+    end
+
+    context "address serialization" do
+      let(:address) do
+        PagSeguro::Address.new(
+          street: "STREET",
+          state: "STATE",
+          city: "CITY",
+          postal_code: "POSTAL_CODE",
+          district: "DISTRICT",
+          number: "NUMBER",
+          complement: "COMPLEMENT"
+        )
+      end
+      let(:payment_request) do
+        double(:PaymentRequest, shipping: shipping).as_null_object
+      end
+      let(:shipping) do
+        double(:Shipping, address: address).as_null_object
+      end
+
+      it { expect(params).to include(shippingAddressStreet: "STREET") }
+      it { expect(params).to include(shippingAddressCountry: "BRA") }
+      it { expect(params).to include(shippingAddressState: "STATE") }
+      it { expect(params).to include(shippingAddressCity: "CITY") }
+      it { expect(params).to include(shippingAddressPostalCode: "POSTAL_CODE") }
+      it { expect(params).to include(shippingAddressDistrict: "DISTRICT") }
+      it { expect(params).to include(shippingAddressNumber: "NUMBER") }
+      it { expect(params).to include(shippingAddressComplement: "COMPLEMENT") }
+    end
+
+    context "sender serialization" do
+      before do
+        sender = PagSeguro::Sender.new({
+          email: "EMAIL",
+          name: "NAME",
+          cpf: "CPF"
+        })
+
+        allow(payment_request).to receive(:sender).and_return(sender)
+      end
+
+      it { expect(params).to include(senderEmail: "EMAIL") }
+      it { expect(params).to include(senderName: "NAME") }
+      it { expect(params).to include(senderCPF: "CPF") }
+    end
+
+    context "phone serialization" do
+      before do
+        sender = PagSeguro::Sender.new({
+          phone: {
+            area_code: "AREA_CODE",
+            number: "NUMBER"
+          }
+        })
+
+        allow(payment_request).to receive(:sender).and_return(sender)
+      end
+
+      it { expect(params).to include(senderAreaCode: "AREA_CODE") }
+      it { expect(params).to include(senderPhone: "NUMBER") }
+    end
+
+    context "items serialization" do
+      def build_item(index)
+        PagSeguro::Item.new({
+          id: "ID#{index}",
+          description: "DESC#{index}",
+          quantity: "QTY#{index}",
+          amount: index * 100 + 0.12,
+          weight: "WEIGHT#{index}",
+          shipping_cost: index * 100 + 0.34
+        })
+      end
+
+      shared_examples_for "item serialization" do |index|
+        it { expect(params).to include("itemId#{index}" => "ID#{index}") }
+        it { expect(params).to include("itemDescription#{index}" => "DESC#{index}") }
+        it { expect(params).to include("itemAmount#{index}" => "#{index}00.12") }
+        it { expect(params).to include("itemShippingCost#{index}" => "#{index}00.34") }
+        it { expect(params).to include("itemQuantity#{index}" => "QTY#{index}") }
+        it { expect(params).to include("itemWeight#{index}" => "WEIGHT#{index}") }
+      end
+
+      before do
+        payment_request.items << build_item(1)
+        payment_request.items << build_item(2)
+      end
+
+      it_behaves_like "item serialization", 1
+      it_behaves_like "item serialization", 2
+    end
+
+    context "credentials serialization" do
+      before do
+        credentials = PagSeguro::ApplicationCredentials.new(
+          "app123", "qwerty", "authocode"
+        )
+
+        payment_request.stub(credentials: credentials)
+      end
+    end
+
+    context "extra params serialization" do
+      let(:payment_request) do
+        double(:PaymentRequest,
+          extra_params: [
+            { extraParam: 'param_value' },
+            { newExtraParam: 'extra_param_value' }
+          ]).as_null_object
+      end
+
+      it { expect(params).to include(extraParam: 'param_value') }
+      it { expect(params).to include(newExtraParam: 'extra_param_value') }
+    end
+  end
+end