bcarpenter-active_shipping 0.0.2

data/test/remote/usps_test.rb

@@ -0,0 +1,184 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class USPSTest < Test::Unit::TestCase
+  
+  def setup
+    @packages  = TestFixtures.packages
+    @locations = TestFixtures.locations
+    @carrier   = USPS.new(fixtures(:usps))
+  end
+  
+  def test_machinable_rate_discrepancy
+    assert_nothing_raised do
+      default_machinable_response = @carrier.find_rates(
+                                      Location.new(:zip => 83843),
+                                      Location.new(:zip => 70001),
+                                      Package.new(32, [12,6,2], :units => :imperial)
+                                    )
+      assert default_machinable_response.request =~ /<Machinable>TRUE<\/Machinable>/
+      
+      explicit_non_machinable_response = @carrier.find_rates(
+                                           Location.new(:zip => 83843),
+                                           Location.new(:zip => 70001),
+                                           Package.new(32, [12,6,2], :units => :imperial, :machinable => false)
+                                         )
+      assert explicit_non_machinable_response.request =~ /<Machinable>FALSE<\/Machinable>/
+      
+      assert_not_equal default_machinable_response.rates.map(&:price),
+                        explicit_non_machinable_response.rates.map(&:price)
+    end
+  end
+  
+  def test_zip_to_zip
+    assert_nothing_raised do
+      response = @carrier.find_rates(
+                   Location.new(:zip => 40524),
+                   Location.new(:zip => 40515),
+                   Package.new(16, [12,6,2], :units => :imperial)
+                 )
+    end
+  end
+  
+  def test_just_country_given
+    assert_nothing_raised do
+      response = @carrier.find_rates(
+                   @locations[:beverly_hills],
+                   Location.new(:country => 'CZ'),
+                   Package.new(100, [5,10,20])
+                 )
+    end
+  end
+  
+  def test_us_to_canada
+    response = nil
+    assert_nothing_raised do
+      response = @carrier.find_rates(
+                   @locations[:beverly_hills],
+                   @locations[:ottawa],
+                   @packages.values_at(:wii),
+                   :test => true
+                 )
+    assert_not_equal [], response.rates.length
+    end
+  end
+  
+  def test_domestic_rates_thoroughly
+    response = nil
+    assert_nothing_raised do
+      response = @carrier.find_rates(
+                   @locations[:new_york],
+                   @locations[:beverly_hills],
+                   @packages.values_at(:book,:wii),
+                   :test => true
+                 )
+    end
+    assert response.success?, response.message
+    assert_instance_of Hash, response.params
+    assert_instance_of String, response.xml
+    assert_instance_of Array, response.rates
+    assert_not_equal [], response.rates
+    
+    rate = response.rates.first
+    assert_equal 'USPS', rate.carrier
+    assert_equal 'USD', rate.currency
+    assert_instance_of Fixnum, rate.total_price
+    assert_instance_of Fixnum, rate.price
+    assert_instance_of String, rate.service_name
+    assert_instance_of String, rate.service_code
+    assert_instance_of Array, rate.package_rates
+    assert_equal @packages.values_at(:book, :wii), rate.packages
+    
+    package_rate = rate.package_rates.first
+    assert_instance_of Hash, package_rate
+    assert_instance_of Package, package_rate[:package]
+    assert_not_nil package_rate[:rate]
+    
+    other_than_two = response.rates.map(&:package_count).reject {|n| n == 2}
+    assert_equal [], other_than_two, "Some RateEstimates do not refer to the right number of packages (#{other_than_two.inspect})"
+    
+    
+  end
+  
+  def test_international_thoroughly
+    
+    response = nil
+    assert_nothing_raised do
+      response = @carrier.find_rates(
+                   @locations[:beverly_hills],
+                   @locations[:ottawa],
+                   @packages.values_at(:book, :wii),
+                   :test => true
+                 )
+    end
+    
+    assert response.success?, response.message
+    assert_instance_of Hash, response.params
+    assert_instance_of String, response.xml
+    assert_instance_of Array, response.rates
+    assert_not_equal [], response.rates
+    
+    rate = response.rates.first
+    assert_equal 'USPS', rate.carrier
+    assert_equal 'USD', rate.currency
+    assert_instance_of Fixnum, rate.total_price
+    assert_instance_of Fixnum, rate.price
+    assert_instance_of String, rate.service_name
+    assert_instance_of String, rate.service_code
+    assert_instance_of Array, rate.package_rates
+    assert_equal @packages.values_at(:book, :wii), rate.packages
+    
+    package_rate = rate.package_rates.first
+    assert_instance_of Hash, package_rate
+    assert_instance_of Package, package_rate[:package]
+    assert_not_nil package_rate[:rate]
+    
+    other_than_two = response.rates.map(&:package_count).reject {|n| n == 2}
+    assert_equal [], other_than_two, "Some RateEstimates do not refer to the right number of packages (#{other_than_two.inspect})"
+    
+  end
+  
+  def test_bare_packages
+    response = nil
+    p = Package.new(0,0)
+    response = begin
+      @carrier.find_rates(
+        @locations[:beverly_hills], # imperial (U.S. origin)
+        @locations[:ottawa],
+        p,
+        :test => true
+      )
+    rescue ResponseError => e
+      e.response
+    end
+    assert response.success?, response.message
+  end
+  
+  def test_valid_credentials
+    assert USPS.new(fixtures(:usps).merge(:test => true)).valid_credentials?
+  end
+  
+  # Uncomment and switch out SPECIAL_COUNTRIES with some other batch to see which
+  # countries are currently working. Commented out here just because it's a lot of
+  # hits to their server at once:
+  
+  # ALL_COUNTRIES = ActiveMerchant::Country.const_get('COUNTRIES').map {|c| c[:alpha2]}
+  # SPECIAL_COUNTRIES = USPS.const_get('COUNTRY_NAME_CONVERSIONS').keys.sort
+  # NORMAL_COUNTRIES = (ALL_COUNTRIES - SPECIAL_COUNTRIES)
+  # 
+  # SPECIAL_COUNTRIES.each do |code|
+  #   unless ActiveMerchant::Country.find(code).name == USPS.const_get('COUNTRY_NAME_CONVERSIONS')[code]
+  #     define_method("test_country_#{code}") do
+  #       response = nil
+  #       begin
+  #         response = @carrier.find_rates( @locations[:beverly_hills],
+  #                                         Location.new(:country => code),
+  #                                         @packages.values_at(:wii),
+  #                                         :test => true)
+  #       rescue Exception => e
+  #         flunk(e.inspect + "\nrequest: " + @carrier.last_request)
+  #       end
+  #       assert_not_equal [], response.rates.length
+  #     end
+  #   end
+  # end
+end

data/test/test_helper.rb

@@ -0,0 +1,167 @@
+#!/usr/bin/env ruby
+$:.unshift(File.dirname(__FILE__) + '/../lib')
+
+require 'test/unit'
+require 'active_shipping'
+require 'mocha'
+
+module Test
+  module Unit
+    class TestCase
+      include ActiveMerchant::Shipping
+      
+      LOCAL_CREDENTIALS = ENV['HOME'] + '/.active_merchant/fixtures.yml' unless defined?(LOCAL_CREDENTIALS)
+      DEFAULT_CREDENTIALS = File.dirname(__FILE__) + '/fixtures.yml' unless defined?(DEFAULT_CREDENTIALS)
+      
+      MODEL_FIXTURES = File.dirname(__FILE__) + '/fixtures/' unless defined?(MODEL_FIXTURES)
+
+      def all_fixtures
+        @@fixtures ||= load_fixtures
+      end
+      
+      def fixtures(key)
+        data = all_fixtures[key] || raise(StandardError, "No fixture data was found for '#{key}'")
+        
+        data.dup
+      end
+          
+      def load_fixtures
+        file = File.exists?(LOCAL_CREDENTIALS) ? LOCAL_CREDENTIALS : DEFAULT_CREDENTIALS
+        yaml_data = YAML.load(File.read(file))
+        
+        model_fixtures = Dir.glob(File.join(MODEL_FIXTURES,'**','*.yml'))
+        model_fixtures.each do |file|
+          name = File.basename(file, '.yml')
+          yaml_data[name] = YAML.load(File.read(file))
+        end
+        
+        symbolize_keys(yaml_data)
+      
+        yaml_data
+      end
+      
+      def xml_fixture(path) # where path is like 'usps/beverly_hills_to_ottawa_response'
+        open(File.join(File.dirname(__FILE__),'fixtures','xml',"#{path}.xml")) {|f| f.read}
+      end
+      
+      def symbolize_keys(hash)
+        return unless hash.is_a?(Hash)
+        
+        hash.symbolize_keys!
+        hash.each{|k,v| symbolize_keys(v)}
+      end
+      
+    end
+  end
+end
+
+module ActiveMerchant
+  module Shipping
+    module TestFixtures
+      
+      mattr_reader :packages, :locations
+      
+      @@packages = {
+        :just_ounces => Package.new(16, nil, :units => :imperial),
+        :just_grams => Package.new(1000, nil),
+        :all_imperial => Package.new(16, [1,8,12], :units => :imperial),
+        :all_metric => Package.new(1000, [2,20,40]),
+        :book => Package.new(250, [14, 19, 2]),
+        :wii => Package.new((7.5 * 16), [15, 10, 4.5], :units => :imperial, :value => 269.99, :currency => 'GBP'),
+        :poster => Package.new(100, [93,10], :cylinder => true),
+        :small_half_pound => Package.new(8, [1,1,1], :units => :imperial),
+        :big_half_pound => Package.new((16 * 50), [24,24,36], :units => :imperial),
+        :chocolate_stuff => Package.new(80, [2,6,12], :units => :imperial)
+      }
+      
+      @@locations = {
+        :bare_ottawa => Location.new(:country => 'CA', :postal_code => 'K1P 1J1'),
+        :bare_beverly_hills => Location.new(:country => 'US', :zip => '90210'),
+        :ottawa => Location.new( :country => 'CA',
+                                      :province => 'ON',
+                                      :city => 'Ottawa',
+                                      :address1 => '110 Laurier Avenue West',
+                                      :postal_code => 'K1P 1J1',
+                                      :phone => '1-613-580-2400',
+                                      :fax => '1-613-580-2495'),
+        :beverly_hills => Location.new(
+                                      :country => 'US',
+                                      :state => 'CA',
+                                      :city => 'Beverly Hills',
+                                      :address1 => '455 N. Rexford Dr.',
+                                      :address2 => '3rd Floor',
+                                      :zip => '90210',
+                                      :phone => '1-310-285-1013',
+                                      :fax => '1-310-275-8159'),
+        :real_home_as_commercial => Location.new(
+                                      :country => 'US',
+                                      :city => 'Tampa',
+                                      :state => 'FL',
+                                      :address1 => '7926 Woodvale Circle',
+                                      :zip => '33615',
+                                      :address_type => 'commercial'), # means that UPS will default to commercial if it doesn't know
+        :fake_home_as_commercial => Location.new(
+                                      :country => 'US',
+                                      :state => 'FL',
+                                      :address1 => '123 fake st.',
+                                      :zip => '33615',
+                                      :address_type => 'commercial'),
+        :real_google_as_commercial => Location.new(
+                                      :country => 'US',
+                                      :city => 'Mountain View',
+                                      :state => 'CA',
+                                      :address1 => '1600 Amphitheatre Parkway',
+                                      :zip => '94043',
+                                      :address_type => 'commercial'),
+        :real_google_as_residential => Location.new(
+                                      :country => 'US',
+                                      :city => 'Mountain View',
+                                      :state => 'CA',
+                                      :address1 => '1600 Amphitheatre Parkway',
+                                      :zip => '94043',
+                                      :address_type => 'residential'), # means that will default to residential if it doesn't know
+        :fake_google_as_commercial => Location.new(
+                                      :country => 'US',
+                                      :city => 'Mountain View',
+                                      :state => 'CA',
+                                      :address1 => '123 bogusland dr.',
+                                      :zip => '94043',
+                                      :address_type => 'commercial'),
+        :fake_google_as_residential => Location.new(
+                                      :country => 'US',
+                                      :city => 'Mountain View',
+                                      :state => 'CA',
+                                      :address1 => '123 bogusland dr.',
+                                      :zip => '94043',
+                                      :address_type => 'residential'), # means that will default to residential if it doesn't know
+        :fake_home_as_residential => Location.new(
+                                      :country => 'US',
+                                      :state => 'FL',
+                                      :address1 => '123 fake st.',
+                                      :zip => '33615',
+                                      :address_type => 'residential'),
+        :real_home_as_residential => Location.new(
+                                      :country => 'US',
+                                      :city => 'Tampa',
+                                      :state => 'FL',
+                                      :address1 => '7926 Woodvale Circle',
+                                      :zip => '33615',
+                                      :address_type => 'residential'),
+        :london => Location.new(
+                                      :country => 'GB',
+                                      :city => 'London',
+                                      :address1 => '170 Westminster Bridge Rd.',
+                                      :zip => 'SE1 7RW'),
+        :new_york => Location.new(
+                                      :country => 'US',
+                                      :city => 'New York',
+                                      :state => 'NY',
+                                      :address1 => '780 3rd Avenue',
+                                      :address2 => 'Suite  2601',
+                                      :zip => '10017')
+                                      
+      }
+      
+    end
+  end
+end

data/test/unit/base_test.rb

@@ -0,0 +1,18 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class BaseTest < Test::Unit::TestCase
+  include ActiveMerchant::Shipping
+
+  def test_get_usps_by_string
+    assert_equal USPS, Base.carrier('usps')
+    assert_equal USPS, Base.carrier('USPS')
+  end
+
+  def test_get_usps_by_name
+    assert_equal USPS, Base.carrier(:usps)
+  end
+  
+  def test_get_unknown_carrier
+    assert_raise(NameError){ Base.carrier(:polar_north) }
+  end
+end

data/test/unit/carriers/fedex_test.rb

@@ -0,0 +1,78 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class FedExTest < Test::Unit::TestCase
+  def setup
+    @packages               = TestFixtures.packages
+    @locations              = TestFixtures.locations
+    @carrier                = FedEx.new(:key => '1111', :password => '2222', :account => '3333', :login => '4444')
+  end
+  
+  def test_initialize_options_requirements
+    assert_raises ArgumentError do FedEx.new end
+    assert_raises ArgumentError do FedEx.new(:login => '999999999') end
+    assert_raises ArgumentError do FedEx.new(:password => '7777777') end
+    assert_nothing_raised { FedEx.new(:key => '999999999', :password => '7777777', :account => '123', :login => '123')}
+  end
+  
+  # def test_no_rates_response
+  #   @carrier.expects(:commit).returns(xml_fixture('fedex/empty_response'))
+  # 
+  #   response = @carrier.find_rates(
+  #     @locations[:ottawa],                                
+  #     @locations[:beverly_hills],            
+  #     @packages.values_at(:book, :wii)
+  #   )
+  #   assert_equal "WARNING - 556: There are no valid services available. ", response.message
+  # end
+  
+  def test_find_tracking_info_should_return_a_tracking_response
+    @carrier.expects(:commit).returns(xml_fixture('fedex/tracking_response'))
+    assert_instance_of ActiveMerchant::Shipping::TrackingResponse, @carrier.find_tracking_info('077973360403984', :test => true)
+  end
+  
+  def test_find_tracking_info_should_parse_response_into_correct_number_of_shipment_events
+    @carrier.expects(:commit).returns(xml_fixture('fedex/tracking_response'))
+    response = @carrier.find_tracking_info('077973360403984', :test => true)
+    assert_equal 7, response.shipment_events.size
+  end
+  
+  def test_find_tracking_info_should_return_shipment_events_in_ascending_chronological_order
+    @carrier.expects(:commit).returns(xml_fixture('fedex/tracking_response'))
+    response = @carrier.find_tracking_info('077973360403984', :test => true)
+    assert_equal response.shipment_events.map(&:time).sort, response.shipment_events.map(&:time)
+  end
+  
+  def test_building_request_and_parsing_response
+    expected_request = xml_fixture('fedex/ottawa_to_beverly_hills_rate_request')
+    mock_response = xml_fixture('fedex/ottawa_to_beverly_hills_rate_response')
+    Time.any_instance.expects(:to_xml_value).returns("2009-07-20T12:01:55-04:00")
+    
+    @carrier.expects(:commit).with {|request, test_mode| Hash.from_xml(request) == Hash.from_xml(expected_request) && test_mode}.returns(mock_response)
+    response = @carrier.find_rates( @locations[:ottawa],
+                                    @locations[:beverly_hills],
+                                    @packages.values_at(:book, :wii), :test => true)
+    assert_equal ["FedEx Ground"], response.rates.map(&:service_name)
+    assert_equal [3836], response.rates.map(&:price)
+    
+    assert response.success?, response.message
+    assert_instance_of Hash, response.params
+    assert_instance_of String, response.xml
+    assert_instance_of Array, response.rates
+    assert_not_equal [], response.rates
+    
+    rate = response.rates.first
+    assert_equal 'FedEx', rate.carrier
+    assert_equal 'CAD', rate.currency
+    assert_instance_of Fixnum, rate.total_price
+    assert_instance_of Fixnum, rate.price
+    assert_instance_of String, rate.service_name
+    assert_instance_of String, rate.service_code
+    assert_instance_of Array, rate.package_rates
+    assert_equal @packages.values_at(:book, :wii), rate.packages
+    
+    package_rate = rate.package_rates.first
+    assert_instance_of Hash, package_rate
+    assert_instance_of Package, package_rate[:package]
+    assert_nil package_rate[:rate]
+  end
+end