pagseguro-oficial 2.4.0 → 2.5.0

data/spec/pagseguro/payment_request_spec.rb

@@ -1,6 +1,7 @@
 require "spec_helper"
 
 describe PagSeguro::PaymentRequest do
+  it_assigns_attribute :primary_receiver
   it_assigns_attribute :currency
   it_assigns_attribute :redirect_url
   it_assigns_attribute :extra_amount
@@ -20,6 +21,61 @@ describe PagSeguro::PaymentRequest do
     expect(payment.sender).to eql(sender)
   end
 
+  describe '#register split payment' do
+    before do
+      FakeWeb.register_uri(
+        :post,
+        'https://ws.pagseguro.uol.com.br/v2/checkouts?appId=id&appKey=key',
+        body: ""
+      )
+    end
+
+
+    let(:subject) do
+      PagSeguro::PaymentRequest.new(
+        receivers: receivers,
+        credentials: credentials,
+        primary_receiver: 'primary@example.com',
+        sender: sender
+      )
+    end
+
+    let(:credentials) do
+      PagSeguro::ApplicationCredentials.new('id', 'key')
+    end
+
+    let(:sender) do
+      PagSeguro::Sender.new(phone: PagSeguro::Phone.new(area_code: 1, number: 2345))
+    end
+
+    let(:receivers) do
+      [
+        { email: 'a@example.com', split: { amount: 1 } },
+        { email: 'b@example.com', split: { amount: 1 } }
+      ]
+    end
+
+    context 'ensure receivers' do
+      it 'are PagSeguro::Receiver' do
+        subject.receivers.each do |receiver|
+          expect(receiver).to be_a(PagSeguro::Receiver)
+        end
+      end
+
+      it 'have correct keys' do
+        expect(subject.receivers[0].email).to eq 'a@example.com'
+      end
+    end
+
+    it "changes url to checkouts" do
+      expect(PagSeguro::Request).to receive(:post_xml).with(
+        'checkouts', 'v2', credentials, a_string_matching(/<checkout>/)
+      )
+
+      subject.register
+    end
+  end
+
   it "sets the items" do
     payment = PagSeguro::PaymentRequest.new
     expect(payment.items).to be_a(PagSeguro::Items)
@@ -35,8 +91,8 @@ describe PagSeguro::PaymentRequest do
     before { FakeWeb.register_uri :any, %r[.*?], body: "" }
 
     it "serializes payment request" do
-      PagSeguro::PaymentRequest::Serializer
-        .should_receive(:new)
+      allow_any_instance_of(PagSeguro::PaymentRequest::RequestSerializer)
+        .to receive(:new)
         .with(payment)
         .and_return(double.as_null_object)
 
@@ -44,12 +100,12 @@ describe PagSeguro::PaymentRequest do
     end
 
     it "performs request" do
-      params = double
+      params = double(:Params)
 
-      PagSeguro::PaymentRequest::Serializer.any_instance.stub to_params: params
+      allow_any_instance_of(PagSeguro::PaymentRequest::RequestSerializer).to receive(:to_params).and_return(params)
 
-      PagSeguro::Request
-        .should_receive(:post)
+      expect(PagSeguro::Request)
+        .to receive(:post)
         .with("checkout", "v2", params)
 
       payment.register
@@ -57,10 +113,10 @@ describe PagSeguro::PaymentRequest do
 
     it "initializes response" do
       response = double
-      PagSeguro::Request.stub post: response
+      allow(PagSeguro::Request).to receive(:post).and_return(response)
 
-      PagSeguro::PaymentRequest::Response
-        .should_receive(:new)
+      expect(PagSeguro::PaymentRequest::Response)
+        .to receive(:new)
         .with(response)
 
       payment.register
@@ -68,7 +124,7 @@ describe PagSeguro::PaymentRequest do
 
     it "returns response" do
       response = double
-      PagSeguro::PaymentRequest::Response.stub new: response
+      allow(PagSeguro::PaymentRequest::Response).to receive(:new).and_return(response)
 
       expect(payment.register).to eql(response)
     end
@@ -76,7 +132,7 @@ describe PagSeguro::PaymentRequest do
 
   describe "#extra_params" do
     it "is empty before initialization" do
-      expect(subject.extra_params).to eql([])
+      expect(subject.extra_params).to be_empty
     end
 
     it "allows extra parameter addition" do

data/spec/pagseguro/person_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+RSpec.describe PagSeguro::Person do
+  it_assigns_attribute :name
+  it_assigns_attribute :birth_date
+
+  it_ensures_type PagSeguro::Address, :address
+  it_ensures_collection_type PagSeguro::Document, :documents, [{type: 'CPF', value: 12312312312}]
+  it_ensures_collection_type PagSeguro::Phone, :phones, [{type: 'HOME', area_code: 11, number: 12312312312}]
+end

data/spec/pagseguro/phones_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+
+RSpec.describe PagSeguro::Phones do
+  [
+    :size,
+    :clear,
+    :empty?,
+    :any?,
+    :each,
+    :map
+  ].each do |method|
+    it { is_expected.to respond_to(method) }
+  end
+
+  it 'should initialize empty' do
+    expect(subject).to be_empty
+  end
+
+  context 'adding a new phone' do
+    let(:phone) { { type: 'HOME', area_code: 11, number: 11112222 } }
+
+    it "shouldn't add the same phone" do
+      subject << phone
+      expect{ subject << phone }.to_not change{ subject.size }
+    end
+
+    context 'ensures type the new phone' do
+      it 'can be a hash' do
+        expect{ subject << phone }.to change{ subject.size }.to(1)
+      end
+
+      it 'can be a PagSeguro::Phone' do
+        phone = PagSeguro::Phone.new(type: 'MOBILE', area_code: 11, number: 22221111)
+        expect{ subject << phone }.to change{ subject.size }.to(1)
+      end
+    end
+  end
+end

data/spec/pagseguro/receiver_spec.rb

@@ -0,0 +1,6 @@
+require "spec_helper"
+
+describe PagSeguro::Receiver do
+  it_assigns_attribute :email
+  it_ensures_type PagSeguro::ReceiverSplit, :split
+end

data/spec/pagseguro/receiver_split_spec.rb

@@ -0,0 +1,7 @@
+require "spec_helper"
+
+describe PagSeguro::ReceiverSplit do
+  it_assigns_attribute :amount
+  it_assigns_attribute :rate_percent
+  it_assigns_attribute :fee_percent
+end

data/spec/pagseguro/request_spec.rb

@@ -14,16 +14,16 @@ describe PagSeguro::Request do
     end
 
     it "includes credentials" do
-      PagSeguro.email = "EMAIL"
-      PagSeguro.token = "TOKEN"
+      PagSeguro.configuration.email = "EMAIL"
+      PagSeguro.configuration.token = "TOKEN"
       PagSeguro::Request.post("checkout", "v3")
 
       expect(FakeWeb.last_request.body).to include("email=EMAIL&token=TOKEN")
     end
 
     it "includes custom credentials" do
-      PagSeguro.email = "EMAIL"
-      PagSeguro.token = "TOKEN"
+      PagSeguro.configuration.email = "EMAIL"
+      PagSeguro.configuration.token = "TOKEN"
       PagSeguro::Request.post("checkout", "v3", email: 'foo', token: 'bar')
 
       expect(FakeWeb.last_request.body).to include("email=foo&token=bar")
@@ -43,6 +43,29 @@ describe PagSeguro::Request do
       expect(request["lib-description"]).to be
       expect(request["language-engine-description"]).to be
     end
+
+    context "when POST request with XML data" do
+      let(:credentials) { PagSeguro::ApplicationCredentials.new('app123', 'key123') }
+      let(:xml) { File.read('./spec/fixtures/authorization_request/authorization_request.xml') }
+
+      before do
+        PagSeguro::Request.post_xml('authorizations/request', 'v2', credentials, xml)
+      end
+
+      let(:request) { FakeWeb.last_request }
+
+      it 'include request headers' do
+        expect(request["Content-Type"]).to eq "application/xml; charset=#{PagSeguro.encoding}"
+      end
+
+      it 'include data xml' do
+        expect(request.body).to eq xml
+      end
+
+      it 'correct url' do
+        expect(request.path).to eq '/v2/authorizations/request?appId=app123&appKey=key123'
+      end
+    end
   end
 
   context "GET request" do

data/spec/pagseguro/session/response_spec.rb

@@ -40,9 +40,13 @@ RSpec.describe PagSeguro::Session::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_messages(
+          success?: false,
+          error?: true,
+          error: Aitch::BadRequestError
+        )
       end
+
       let(:raw_xml) { File.read("./spec/fixtures/invalid_code.xml") }
 
       it "change session errors" do

data/spec/pagseguro/session_spec.rb

@@ -25,8 +25,11 @@ describe PagSeguro::Session do |variable|
 
     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/transaction/response_spec.rb

@@ -38,11 +38,14 @@ describe PagSeguro::Transaction::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(true)
-        allow(http_response).to receive(:body).and_return(raw_xml)
+        allow(http_response).to receive_messages(
+          success?: false,
+          error?: true,
+          error: Aitch::NotFoundError,
+          body: raw_xml
+        )
       end
+
       let(:raw_xml) { File.read("./spec/fixtures/invalid_code.xml") }
 
       it "returns PagSeguro::Transaction::StatusCollection instance" do
@@ -77,11 +80,14 @@ describe PagSeguro::Transaction::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(:body).and_return(raw_xml)
+        allow(http_response).to receive_messages(
+          success?: false,
+          error?: true,
+          error: Aitch::BadRequestError,
+          body: raw_xml
+        )
       end
+
       let(:raw_xml) { File.read("./spec/fixtures/invalid_code.xml") }
 
       it "returns the transaction" do

data/spec/pagseguro/transaction/search_spec.rb

@@ -4,7 +4,14 @@ describe PagSeguro::Search do
   let(:search) { PagSeguro::Search.new("foo", "bar", 1) }
   let(:source) { File.read("./spec/fixtures/transactions/search.xml") }
   let(:xml) { Nokogiri::XML(source) }
-  let(:response) { double(:response, data: xml, unauthorized?: false, bad_request?: false, not_found?: false) }
+  let(:response) do
+    double(
+      :response,
+      data: xml,
+      error?: false,
+      error: nil
+    )
+  end
 
   context 'when being initialized' do
     it 'initializes with passed page number' do
@@ -57,19 +64,19 @@ describe PagSeguro::Search do
     context 'when there is a next page' do
       search = PagSeguro::Search.new('foo', 'bar', 0)
       it 'is page 0' do
-        expect(search.next_page?).to be_truthy
+        expect(search).to be_next_page
       end
 
       it 'is not page 0, but page < total_pages' do
         allow(search).to receive(:total_pages).and_return(3)
-        expect(search.next_page?).to be_truthy
+        expect(search).to be_next_page
       end
     end
 
     context 'when there is no next page' do
       it 'is not page 0, but page == total_pages' do
         allow(search).to receive(:total_pages).and_return(1)
-        expect(search.next_page?).to be_falsy
+        expect(search).not_to be_next_page
       end
     end
   end
@@ -77,11 +84,11 @@ describe PagSeguro::Search do
   describe '#previous_page?' do
     context 'when there is a previous page' do
       search = PagSeguro::Search.new('foo', 'bar', 2)
-      it { expect(search.previous_page?).to be_truthy }
+      it { expect(search).to be_previous_page }
     end
 
     context 'when there is no previous page' do
-      it { expect(search.previous_page?).to be_falsy }
+      it { expect(search).not_to be_previous_page }
     end
   end
 
@@ -115,12 +122,12 @@ describe PagSeguro::Search do
   describe '#valid?' do
     it 'is valid' do
       allow(search).to receive(:fetch).and_return(true)
-      expect(search.valid?).to be_truthy
+      expect(search).to be_valid
     end
 
     it "isn't valid" do
       allow(search).to receive(:fetch).and_return(false)
-      expect(search.valid?).to be_falsy
+      expect(search).not_to be_valid
     end
   end
 end

data/spec/pagseguro/transaction_cancellation/response_spec.rb

@@ -24,9 +24,12 @@ describe PagSeguro::TransactionCancellation::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(:body).and_return(raw_xml)
+        allow(http_response).to receive_messages(
+          success?: false,
+          error?: true,
+          error: Aitch::BadRequestError,
+          body: raw_xml
+        )
       end
       let(:raw_xml) { File.read("./spec/fixtures/invalid_code.xml") }
 

data/spec/pagseguro/transaction_cancellation_spec.rb

@@ -20,8 +20,8 @@ describe PagSeguro::TransactionCancellation do
     context "when request succeds" do
       let(:raw_xml) { File.read("./spec/fixtures/transaction_cancellation/success.xml") }
 
-      it "returns boolean" do
-        expect(subject.register).to be_truthy
+      it "returns a PagSeguro::TransactionCancellation" do
+        expect(subject.register).to be_a_kind_of PagSeguro::TransactionCancellation
       end
 
       it "does not add errors" do
@@ -35,9 +35,11 @@ describe PagSeguro::TransactionCancellation do
 
     context "when request fails" do
       before do
-        allow(http_request).to receive(:success?).and_return(false)
-        allow(http_request).to receive(:bad_request?).and_return(true)
-        allow(http_request).to receive(:not_found?).and_return(false)
+        allow(http_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/{refund → transaction_refund}/request_serializer_spec.rb

@@ -1,10 +1,10 @@
 require "spec_helper"
 
-describe PagSeguro::Refund::RequestSerializer do
-  subject(:serializer) { PagSeguro::Refund::RequestSerializer.new(refund) }
+describe PagSeguro::TransactionRefund::RequestSerializer do
+  subject(:serializer) { PagSeguro::TransactionRefund::RequestSerializer.new(refund) }
 
   let(:refund) do
-    PagSeguro::Refund.new transaction_code: "1234",
+    PagSeguro::TransactionRefund.new transaction_code: "1234",
       value: "100.50"
   end
 

data/spec/pagseguro/{refund → transaction_refund}/response_serializer_spec.rb

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe PagSeguro::Refund::ResponseSerializer do
+describe PagSeguro::TransactionRefund::ResponseSerializer do
   context "for existing refund response" do
     let(:source) { File.read("./spec/fixtures/refund/success.xml") }
     let(:xml) { Nokogiri::XML(source) }