pag_seguro 0.5.5 → 0.5.6

data/spec/pag_seguro/pre_approval_spec.rb

@@ -2,63 +2,63 @@
 require "spec_helper"
 
 describe PagSeguro::PreApproval do
-  it { should have_attribute_accessor(:name) }
-  it { should have_attribute_accessor(:details) }
-  it { should have_attribute_accessor(:amount_per_payment) }
-  it { should have_attribute_accessor(:period) }
-  it { should have_attribute_accessor(:day_of_week) }
-  it { should have_attribute_accessor(:day_of_month) }
-  it { should have_attribute_accessor(:day_of_year) }
-  it { should have_attribute_accessor(:initial_date) }
-  it { should have_attribute_accessor(:final_date) }
-  it { should have_attribute_accessor(:max_amount_per_period) }
-  it { should have_attribute_accessor(:max_total_amount) }
-  it { should have_attribute_accessor(:review_URL) }
-
-  it { should validate_presence_of :name }
-  it { should validate_presence_of :period }
-  it { should validate_presence_of :final_date }
-  it { should validate_presence_of :max_amount_per_period }
-  it { should validate_presence_of :max_total_amount }
+  it { is_expected.to have_attribute_accessor(:name) }
+  it { is_expected.to have_attribute_accessor(:details) }
+  it { is_expected.to have_attribute_accessor(:amount_per_payment) }
+  it { is_expected.to have_attribute_accessor(:period) }
+  it { is_expected.to have_attribute_accessor(:day_of_week) }
+  it { is_expected.to have_attribute_accessor(:day_of_month) }
+  it { is_expected.to have_attribute_accessor(:day_of_year) }
+  it { is_expected.to have_attribute_accessor(:initial_date) }
+  it { is_expected.to have_attribute_accessor(:final_date) }
+  it { is_expected.to have_attribute_accessor(:max_amount_per_period) }
+  it { is_expected.to have_attribute_accessor(:max_total_amount) }
+  it { is_expected.to have_attribute_accessor(:review_URL) }
+
+  it { is_expected.to validate_presence_of :name }
+  it { is_expected.to validate_presence_of :period }
+  it { is_expected.to validate_presence_of :final_date }
+  it { is_expected.to validate_presence_of :max_amount_per_period }
+  it { is_expected.to validate_presence_of :max_total_amount }
 
   [:weekly, :monthly, :bimonthly, :trimonthly, :semiannually, :yearly].each do |period_type|
-    it { should allow_value(period_type).for(:period) }
-    it { should allow_value(period_type.to_s).for(:period) }
-    it { should allow_value(period_type.to_s.upcase).for(:period) }
+    it { is_expected.to allow_value(period_type).for(:period) }
+    it { is_expected.to allow_value(period_type.to_s).for(:period) }
+    it { is_expected.to allow_value(period_type.to_s.upcase).for(:period) }
   end
-  it { should_not allow_value(:some_other_period_type).for(:period) }
+  it { is_expected.not_to allow_value(:some_other_period_type).for(:period) }
 
-  it { should allow_value( nil ).for(:initial_date) }
-  it { should_not allow_value( Time.now - 10.minutes ).for(:initial_date) }
-  it { should allow_value( Time.now ).for(:initial_date) }
-  it { should allow_value( (PagSeguro::PreApproval::DATE_RANGE - 5.minutes).from_now ).for(:initial_date) }
-  it { should_not allow_value( PagSeguro::PreApproval::DATE_RANGE.from_now + 5.minutes ).for(:initial_date) }
+  it { is_expected.to allow_value( nil ).for(:initial_date) }
+  it { is_expected.not_to allow_value( Time.now - 10.minutes ).for(:initial_date) }
+  it { is_expected.to allow_value( Time.now ).for(:initial_date) }
+  it { is_expected.to allow_value( (PagSeguro::PreApproval::DATE_RANGE - 5.minutes).from_now ).for(:initial_date) }
+  it { is_expected.not_to allow_value( PagSeguro::PreApproval::DATE_RANGE.from_now + 5.minutes ).for(:initial_date) }
 
-  it { should_not allow_value( nil ).for(:final_date) }
-  it { should_not allow_value( Time.now - 10.minutes ).for(:final_date) }
-  it { should allow_value( Time.now ).for(:final_date) }
-  it { should allow_value( (PagSeguro::PreApproval::DATE_RANGE - 5.minutes).from_now ).for(:final_date) }
-  it { should_not allow_value( PagSeguro::PreApproval::DATE_RANGE.from_now + 5.minutes ).for(:final_date) }
+  it { is_expected.not_to allow_value( nil ).for(:final_date) }
+  it { is_expected.not_to allow_value( Time.now - 10.minutes ).for(:final_date) }
+  it { is_expected.to allow_value( Time.now ).for(:final_date) }
+  it { is_expected.to allow_value( (PagSeguro::PreApproval::DATE_RANGE - 5.minutes).from_now ).for(:final_date) }
+  it { is_expected.not_to allow_value( PagSeguro::PreApproval::DATE_RANGE.from_now + 5.minutes ).for(:final_date) }
 
   context "with an initial date" do
     subject { PagSeguro::PreApproval.new(initial_date: 5.days.from_now) }
 
-    it { should_not allow_value( nil ).for(:final_date) }
-    it { should_not allow_value( Time.now - 10.minutes + 5.days ).for(:final_date) }
-    it { should allow_value( Time.now + 5.days ).for(:final_date) }
-    it { should allow_value( (PagSeguro::PreApproval::DATE_RANGE - 5.minutes + 5.days).from_now ).for(:final_date) }
-    it { should_not allow_value( PagSeguro::PreApproval::DATE_RANGE.from_now + 5.minutes + 5.days ).for(:final_date) }
+    it { is_expected.not_to allow_value( nil ).for(:final_date) }
+    it { is_expected.not_to allow_value( Time.now - 10.minutes + 5.days ).for(:final_date) }
+    it { is_expected.to allow_value( Time.now + 5.days ).for(:final_date) }
+    it { is_expected.to allow_value( (PagSeguro::PreApproval::DATE_RANGE - 5.minutes + 5.days).from_now ).for(:final_date) }
+    it { is_expected.not_to allow_value( PagSeguro::PreApproval::DATE_RANGE.from_now + 5.minutes + 5.days ).for(:final_date) }
   end
 
   describe "initialized with minimum attributes" do
     subject{ build :minimum_pre_approval }
-    it { should be_valid }
+    it { is_expected.to be_valid }
   end
 
   describe "initialized with attributes" do
     subject{ build :pre_approval }
 
-    it { should be_valid }
+    it { is_expected.to be_valid }
     its(:name){ should == "Super seguro para notebook" }
     its(:details){ should == "Toda segunda feira será cobrado o valor de R$150,00 para o seguro do notebook" }
     its(:amount_per_payment){ should == '150.00' }
@@ -71,43 +71,43 @@ describe PagSeguro::PreApproval do
     context "weekly" do
       subject{ build :weekly_pre_approval }
 
-      it { should ensure_inclusion_of(:day_of_week).in_array(%w(monday tuesday wednesday thursday friday saturday sunday)) }
+      it { is_expected.to ensure_inclusion_of(:day_of_week).in_array(%w(monday tuesday wednesday thursday friday saturday sunday)) }
 
       its(:period){ should == 'weekly' }
       its(:day_of_week){ should == 'monday' }
-      it { should be_weekly }
-      it { should_not be_monthly }
-      it { should_not be_yearly }
+      it { is_expected.to be_weekly }
+      it { is_expected.not_to be_monthly }
+      it { is_expected.not_to be_yearly }
 
     end
 
     context "monthly" do
       subject{ build :monthly_pre_approval }
 
-      it { should ensure_inclusion_of(:day_of_month).in_range(1..28) }
+      it { is_expected.to ensure_inclusion_of(:day_of_month).in_range(1..28) }
 
       its(:period){ should == 'monthly' }
       its(:day_of_month){ should == 3 }
-      it { should_not be_weekly }
-      it { should be_monthly }
-      it { should_not be_yearly }
+      it { is_expected.not_to be_weekly }
+      it { is_expected.to be_monthly }
+      it { is_expected.not_to be_yearly }
     end
 
     context "yearly" do
       subject{ build :yearly_pre_approval }
 
-      it { should validate_presence_of(:day_of_year) }
-      it { should allow_value('10-22').for(:day_of_year) }
-      it { should allow_value('01-01').for(:day_of_year) }
-      it { should allow_value(PagSeguro::DayOfYear.new(month: 1, day: 1)).for(:day_of_year) }
-      it { should_not allow_value('1-1').for(:day_of_year) }
-      it { should_not allow_value("10-22\nanything").for(:day_of_year) }
+      it { is_expected.to validate_presence_of(:day_of_year) }
+      it { is_expected.to allow_value('10-22').for(:day_of_year) }
+      it { is_expected.to allow_value('01-01').for(:day_of_year) }
+      it { is_expected.to allow_value(PagSeguro::DayOfYear.new(month: 1, day: 1)).for(:day_of_year) }
+      it { is_expected.not_to allow_value('1-1').for(:day_of_year) }
+      it { is_expected.not_to allow_value("10-22\nanything").for(:day_of_year) }
 
       its(:period){ should == 'yearly' }
       its(:day_of_year){ should == '03-01' }
-      it { should_not be_weekly }
-      it { should_not be_monthly }
-      it { should be_yearly }
+      it { is_expected.not_to be_weekly }
+      it { is_expected.not_to be_monthly }
+      it { is_expected.to be_yearly }
     end
   end
 end

data/spec/pag_seguro/query_spec.rb

@@ -6,7 +6,7 @@ describe PagSeguro::Query do
   let(:transactions_xml){ File.open( File.expand_path( File.dirname(__FILE__) + '/../fixtures/transaction_history.xml') ) }
 
   describe "instance" do
-    before { PagSeguro::Query.any_instance.stub(transaction_data: transaction_data) }
+    before { allow_any_instance_of(PagSeguro::Query).to receive_messages(transaction_data: transaction_data) }
     let(:transaction){ PagSeguro::Query.new("mail", "token", "trans_code") }
     subject{ PagSeguro::Query }
 
@@ -21,8 +21,8 @@ describe PagSeguro::Query do
     context "with abandoned set to false" do
       it "should try to fetch transactions" do
         params = double(:params)
-        PagSeguro::Query.stub(search_params: params)
-        RestClient.should_receive(:get).with("https://ws.pagseguro.uol.com.br/v2/transactions", params: params)
+        allow(PagSeguro::Query).to receive_messages(search_params: params)
+        expect(RestClient).to receive(:get).with("https://ws.pagseguro.uol.com.br/v2/transactions", params: params)
         PagSeguro::Query.find "email", "token"
       end
     end
@@ -30,61 +30,63 @@ describe PagSeguro::Query do
     context "with abandoned set to true" do
       it "should try to fetch abandoned transactions" do
         params = double(:params)
-        PagSeguro::Query.stub(search_params: params)
-        RestClient.should_receive(:get).with("https://ws.pagseguro.uol.com.br/v2/transactions/abandoned", params: params)
+        allow(PagSeguro::Query).to receive_messages(search_params: params)
+        expect(RestClient).to receive(:get).with("https://ws.pagseguro.uol.com.br/v2/transactions/abandoned", params: params)
         PagSeguro::Query.find "email", "token", abandoned: true
       end
     end
 
     context "with a stubbed response" do
-      before { RestClient.stub(get: transactions_xml) }
+      before { allow(RestClient).to receive_messages(get: transactions_xml) }
       subject{ PagSeguro::Query.find "email", "token", options: true }
 
       it "calls search_url with received options" do
-        PagSeguro::Query.should_receive(:search_params).with("email", "token", {options: true})
+        expect(PagSeguro::Query).to receive(:search_params).with("email", "token", {options: true})
         PagSeguro::Query.find "email", "token", options: true
       end
 
-      it{ should be_a Array }
-      it{ should have(2).transactions }
-      it("should have transaction ids"){ subject.map(&:id).should == ["REF1234", "REF5678"] }
+      it{ is_expected.to be_a Array }
+      it'has 2 transactions' do
+        expect(subject.size).to eq(2)
+      end
+      it("should have transaction ids"){ expect(subject.map(&:id)).to eq(["REF1234", "REF5678"]) }
     end
   end
 
   describe "::search_params" do
-    it("returns an Hash"){ PagSeguro::Query.search_params("email", "token").should be_a Hash }
+    it("returns an Hash"){ expect(PagSeguro::Query.search_params("email", "token")).to be_a Hash }
 
     context "with default options" do
       let(:search_params) { PagSeguro::Query.search_params("email", "token") }
-      it { search_params.keys.should_not include :maxPageResults }
-      it { search_params.keys.should_not include :page }
-      it { search_params[:finalDate].should include Date.today.iso8601 }
-      it { search_params[:initialDate].should include Date.yesterday.iso8601 }
+      it { expect(search_params.keys).not_to include :maxPageResults }
+      it { expect(search_params.keys).not_to include :page }
+      it { expect(search_params[:finalDate]).to include Date.today.iso8601 }
+      it { expect(search_params[:initialDate]).to include Date.yesterday.iso8601 }
 
       it "should call parse_dates" do
-        PagSeguro::Query.should_receive(:parse_dates)
+        expect(PagSeguro::Query).to receive(:parse_dates)
         search_params
       end
     end
 
     context "with page number set to 2" do
       let(:search_params){ PagSeguro::Query.search_params("email", "token", page: 2) }
-      it { search_params[:page].should == 2 }
+      it { expect(search_params[:page]).to eq(2) }
     end
 
     context "with max_page_results set to 100" do
       let(:search_params){ PagSeguro::Query.search_params("email", "token", max_page_results: 100) }
-      it { search_params[:maxPageResults].should == 100 }
+      it { expect(search_params[:maxPageResults]).to eq(100) }
     end
 
     context "with initial_date set to 20 days ago" do
       let(:search_params){ PagSeguro::Query.search_params("email", "token", initial_date: 20.days.ago) }
-      it { search_params[:initialDate].should include 20.days.ago.iso8601 }
+      it { expect(search_params[:initialDate]).to include 20.days.ago.iso8601 }
     end
 
     context "with final_date set to 10 days ago" do
       let(:search_params){ PagSeguro::Query.search_params("email", "token", initial_date: 20.days.ago, final_date: 10.days.ago) }
-      it { search_params[:finalDate].should include 10.days.ago.iso8601 }
+      it { expect(search_params[:finalDate]).to include 10.days.ago.iso8601 }
     end
   end
 
@@ -115,21 +117,21 @@ describe PagSeguro::Query do
 
     context "first return argument" do
       it "returns yesterday if no initial date was given" do
-        PagSeguro::Query.parse_dates.first.should == (Time.now - 1.day).iso8601
+        expect(PagSeguro::Query.parse_dates.first).to eq((Time.now - 1.day).iso8601)
       end
 
       it "returns the initial date in iso8601 format if provided" do
-        PagSeguro::Query.parse_dates(initial_date: 3.days.ago).first.should == 3.days.ago.iso8601
+        expect(PagSeguro::Query.parse_dates(initial_date: 3.days.ago).first).to eq(3.days.ago.iso8601)
       end
     end
 
     context "second return argument" do
       it "returns initial_date + 1 day no final date was given" do
-        PagSeguro::Query.parse_dates(initial_date: 2.days.ago).last.should == 1.day.ago.iso8601
+        expect(PagSeguro::Query.parse_dates(initial_date: 2.days.ago).last).to eq(1.day.ago.iso8601)
       end
 
       it "returns the final date in iso8601 format if provided" do
-        PagSeguro::Query.parse_dates(initial_date: 3.days.ago, final_date: 2.days.ago).last.should == 2.days.ago.iso8601
+        expect(PagSeguro::Query.parse_dates(initial_date: 3.days.ago, final_date: 2.days.ago).last).to eq(2.days.ago.iso8601)
       end
     end
   end

data/spec/pag_seguro/sender_spec.rb

@@ -3,10 +3,10 @@ require "spec_helper"
 
 describe PagSeguro::Sender do
   context "instance" do
-    it { should have_attribute_accessor(:email) }
-    it { should have_attribute_accessor(:name) }
-    it { should have_attribute_accessor(:phone_ddd) }
-    it { should have_attribute_accessor(:phone_number) }
+    it { is_expected.to have_attribute_accessor(:email) }
+    it { is_expected.to have_attribute_accessor(:name) }
+    it { is_expected.to have_attribute_accessor(:phone_ddd) }
+    it { is_expected.to have_attribute_accessor(:phone_number) }
 
     context "initialized with all attributes" do
       subject { PagSeguro::Sender.new attributes_for(:sender) }
@@ -23,37 +23,37 @@ describe PagSeguro::Sender do
 
     context "with nil name" do
       subject { build :sender, name: nil }
-      it { should_not be_a_valid_name }
+      it { is_expected.not_to be_a_valid_name }
       its(:name){ should be_nil }
     end
 
     context "with name Joao" do
       subject { build :sender, name: "Joao" }
-      it { should_not be_a_valid_name }
+      it { is_expected.not_to be_a_valid_name }
       its(:name){ should be_nil }
     end
 
     context "with name Joao Paulo" do
       subject { build :sender, name: "Joao Paulo" }
-      it { should be_a_valid_name }
+      it { is_expected.to be_a_valid_name }
       its(:name){ should == "Joao Paulo" }
     end
 
     context "with name João Paulo" do
       subject { build :sender, name: "João Paulo" }
-      it { should be_a_valid_name }
+      it { is_expected.to be_a_valid_name }
       its(:name){ should == "João Paulo" }
     end
 
     context "with very big name" do
       subject { build :sender, name: ("a" * 50)+" "+("b" * 10) }
-      it { should be_a_valid_name }
+      it { is_expected.to be_a_valid_name }
       its(:name){ should == "a" * 50 }
     end
 
     context "with double spaces in name" do
       subject { build :sender, name: "Stefano   Benatti" }
-      it { should be_a_valid_name }
+      it { is_expected.to be_a_valid_name }
       its(:name){ should == "Stefano Benatti" }
     end
 

data/spec/pag_seguro/shipping_spec.rb

@@ -4,22 +4,22 @@ require "spec_helper"
 describe PagSeguro::Shipping do
   let(:shipping){ PagSeguro::Shipping.new }
 
-  it { should have_attribute_accessor(:type) }
-  it { should have_attribute_accessor(:state) }
-  it { should have_attribute_accessor(:city) }
-  it { should have_attribute_accessor(:postal_code) }
-  it { should have_attribute_accessor(:district) }
-  it { should have_attribute_accessor(:street) }
-  it { should have_attribute_accessor(:number) }
-  it { should have_attribute_accessor(:complement) }
-  it { should have_attribute_accessor(:cost) }
+  it { is_expected.to have_attribute_accessor(:type) }
+  it { is_expected.to have_attribute_accessor(:state) }
+  it { is_expected.to have_attribute_accessor(:city) }
+  it { is_expected.to have_attribute_accessor(:postal_code) }
+  it { is_expected.to have_attribute_accessor(:district) }
+  it { is_expected.to have_attribute_accessor(:street) }
+  it { is_expected.to have_attribute_accessor(:number) }
+  it { is_expected.to have_attribute_accessor(:complement) }
+  it { is_expected.to have_attribute_accessor(:cost) }
 
   its(:type){ should == PagSeguro::Shipping::UNIDENTIFIED }
 
   describe "instance" do
     subject{ build(:shipping) }
 
-    it { should be_valid }
+    it { is_expected.to be_valid }
     its(:cost){ should == "12.13" }
     its(:postal_code){ should == "05363000" }
 
@@ -30,17 +30,17 @@ describe PagSeguro::Shipping do
 
     context "with type 1" do
       subject{ build(:shipping, type: 1) }
-      it { should be_pac }
+      it { is_expected.to be_pac }
     end
 
     context "with type 2" do
       subject{ build(:shipping, type: 2) }
-      it { should be_sedex }
+      it { is_expected.to be_sedex }
     end
 
     context "with type 3" do
       subject{ build(:shipping, type: 3) }
-      it { should be_unidentified }
+      it { is_expected.to be_unidentified }
     end
   end
 end

data/spec/pag_seguro/url_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe PagSeguro::Url do
+
+  it "has a default environment" do
+    expect(PagSeguro::Url.environment).to eq :production
+  end
+
+  it "should change the environment" do
+    PagSeguro::Url.environment = :sandbox
+    expect(PagSeguro::Url.environment).to eq :sandbox
+  end
+
+  describe ".api_url" do
+    it "raises when environment has no endpoint" do
+      PagSeguro::Url.environment = :invalid
+
+      expect {
+        PagSeguro::Url.api_url("/")
+      }.to raise_exception(PagSeguro::Url::InvalidEnvironmentError)
+    end
+
+    it "returns api url" do
+      PagSeguro::Url.environment = :production
+      expect(PagSeguro::Url.api_url("/some/path")).to eql("https://ws.pagseguro.uol.com.br/v2/some/path")
+    end
+
+    it "returns sandbox api url" do
+      PagSeguro::Url.environment = :sandbox
+      expect(PagSeguro::Url.api_url("/some/path")).to eql("https://ws.sandbox.pagseguro.uol.com.br/v2/some/path")
+    end
+  end
+
+  describe ".site_url" do
+    it "raises when environment has no endpoint" do
+      PagSeguro::Url.environment = :invalid
+
+      expect {
+        PagSeguro::Url.site_url("/")
+      }.to raise_exception(PagSeguro::Url::InvalidEnvironmentError)
+    end
+
+    it "returns site url" do
+      PagSeguro::Url.environment = :production
+      expect(PagSeguro::Url.site_url("/some/path")).to eql("https://pagseguro.uol.com.br/v2/some/path")
+    end
+
+    it "returns sandbox site url" do
+      PagSeguro::Url.environment = :sandbox
+      expect(PagSeguro::Url.site_url("/some/path")).to eql("https://sandbox.pagseguro.uol.com.br/v2/some/path")
+    end
+  end
+end