stamps 0.3.3 → 0.3.4

data/lib/stamps/client/account.rb

@@ -21,12 +21,12 @@ module Stamps
         response[:errors].empty? ? response[:purchase_postage_response] : response
       end
       
-      # check the payment of purchase_pastage 
+      # Check the payment of purchase_postage 
       #
       def get_purchase_status(params = {})
         params[:authenticator] = authenticator_token
-        response = request('GetPurchaseStatuee', Stamps::Mapping::GetPurchaseStatus.new(params))
-        response[:errors].empty? ? response[:purchase_postage_response] : response
+        response = request('GetPurchaseStatus', Stamps::Mapping::GetPurchaseStatus.new(params))
+        response[:errors].empty? ? response[:get_purchase_status_response] : response
       end
 
       # Request carrier pickup

data/lib/stamps/client/rate.rb

@@ -25,7 +25,11 @@ module Stamps
           :rate          => Stamps::Mapping::Rate.new(params)
         })
         response = request('GetRates', rates)
-        response[:get_rates_response].nil? ? response : [response[:get_rates_response][:rates][:rate]].flatten
+        if response[:get_rates_response].nil?
+          response
+        else
+          response[:get_rates_response][:rates].nil? ? [] : [response[:get_rates_response][:rates][:rate]].flatten
+        end
       end
 
       def get_rate(params = {})

data/lib/stamps/mapping.rb

@@ -197,6 +197,11 @@ module Stamps
       property :PurchaseAmount, :from => :amount
       property :ControlTotal,   :from => :control_total
     end
+    
+    class GetPurchaseStatus < Hashie::Trash
+      property :Authenticator, :from => :authenticator
+      property :TransactionID, :from => :transaction_id
+    end
 
     class CancelStamp< Hashie::Trash
       property :Authenticator,  :from => :authenticator

data/lib/stamps/version.rb

@@ -1,3 +1,3 @@
 module Stamps
-  VERSION = "0.3.3"
+  VERSION = "0.3.4"
 end

data/test/client/rate_test.rb

@@ -41,5 +41,19 @@ class StampsTest < Test::Unit::TestCase
         assert_equal Hash, @rate[:add_ons].class
       end
     end
+
+    context '.get_rates with no rates available' do
+      setup do
+        stub_request(:post, Stamps.endpoint).
+          with(:headers => {"SoapAction" => "#{Stamps.namespace}/GetRates"}).
+          to_return(:body => fixture("GetRatesEmpty.xml"))
+        @rates = Stamps.get_rates
+      end
+
+      should 'return an empty array' do
+        assert_equal Array, @rates.class
+        assert_empty @rates
+      end
+    end
   end
 end

data/test/fixtures/GetRatesEmpty.xml

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<soap:Envelope xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
+  <soap:Body>
+    <GetRatesResponse xmlns="http://stamps.com/xml/namespace/2010/11/swsim/swsimv12">
+      <Authenticator>NNOUiINH+k4irLzmCD8AtTk17eA=</Authenticator>
+    </GetRatesResponse>
+  </soap:Body>
+</soap:Envelope>

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stamps
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.3.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-26 00:00:00.000000000 Z
+date: 2014-04-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simplecov
@@ -277,6 +277,7 @@ files:
 - test/fixtures/GetAccountInfo.xml
 - test/fixtures/GetRate.xml
 - test/fixtures/GetRates.xml
+- test/fixtures/GetRatesEmpty.xml
 - test/fixtures/InsufficientPostage.xml
 - test/fixtures/InvalidSoap.xml
 - test/fixtures/PurchasePostage.xml
@@ -325,6 +326,7 @@ test_files:
 - test/fixtures/GetAccountInfo.xml
 - test/fixtures/GetRate.xml
 - test/fixtures/GetRates.xml
+- test/fixtures/GetRatesEmpty.xml
 - test/fixtures/InsufficientPostage.xml
 - test/fixtures/InvalidSoap.xml
 - test/fixtures/PurchasePostage.xml