active_shipping 1.9.2 → 1.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6c27b2374a64a55c68a9b78eac1d7feb97550a41
-  data.tar.gz: 49e411902c19f7db17242a8ee2cd4d8e00a3be2c
+  metadata.gz: 6478be87d7d470eaec0ca8142a1bc13bd9385eec
+  data.tar.gz: 68dc6e8aa2821f4d86f15543eef680be9a34f798
 SHA512:
-  metadata.gz: f9ed243e3a7234568d0c67eb8c2d8c35f887222b69a422b9b4a77a2dc666182540f0d8949e43e9db9282772e0718d0555df33c9dcc50fc3023ad2236abfe2a66
-  data.tar.gz: ce6824f5dd320b1b94493d660bd0c970e02a1c5fc453fff0f584f4b713b034e78b1cb790715c24ad16facc59f4f7aea2ef4c504f11a654405cd6d7a74098d36e
+  metadata.gz: 7a9870708b91d2bbcf436d295cfef6d73537090f9b1de4d3e9372bf58759a977e1e79b1c735b0fbdd460fc25602bb5e40abc00ef405e53bf70c071e7b8d7ec62
+  data.tar.gz: f533ab6e93abd4395da473620be33f8330e2ab8c530a95c141b44dd3fc097cd60b85b80604e72229b2e45d0c3c64b1d5775e9ec1f130b26b99610cc3fd875766

data/active_shipping.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |s|
   s.add_dependency("quantified", "~> 1.0.1")
   s.add_dependency("activesupport", ">= 3.2", "< 5.1.0")
   s.add_dependency("active_utils", "~> 3.2.0")
-  s.add_dependency("nokogiri", ">= 1.6")
+  s.add_dependency("nokogiri", "= 1.6.8")
 
   s.add_development_dependency("minitest")
   s.add_development_dependency("rake")

data/lib/active_shipping.rb

@@ -23,3 +23,4 @@ require 'active_shipping/carriers'
 require 'active_shipping/errors'
 require 'active_shipping/external_return_label_request'
 require 'active_shipping/external_return_label_response'
+require 'active_shipping/address_validation_response'

data/lib/active_shipping/address_validation_response.rb

@@ -0,0 +1,30 @@
+module ActiveShipping
+
+  # Response object class for calls to {ActiveShipping::Carrier#validate_address}.
+  # 
+  # @!attribute location
+  #   The Location to be validated
+  #   @return [String]
+  class AddressValidationResponse < Response
+    attr_reader :validity, :classification, :candidate_addresses, :options, :params
+
+    def initialize(success, message, params = {}, options = {})
+      @validity = options[:validity]
+      @candidate_addresses = options[:candidate_addresses]
+      @classification = options[:classification]
+      super
+    end
+
+    def address_match?
+      @validity == :valid
+    end
+
+    def residential?
+      @classification == :residential
+    end
+
+    def commercial?
+      @classification == :commercial
+    end
+  end
+end

data/lib/active_shipping/carriers/fedex.rb

@@ -590,7 +590,12 @@ module ActiveShipping
       message = response_message(xml)
 
       if success
-        origin = nil
+        tracking_details_root = xml.at('CompletedTrackDetails')
+        success = response_success?(tracking_details_root)
+        message = response_message(tracking_details_root)
+      end
+
+      if success
         delivery_signature = nil
         shipment_events = []
 
@@ -599,13 +604,28 @@ module ActiveShipping
           when 1
             all_tracking_details.first
           when 0
-            raise ActiveShipping::Error, "The response did not contain tracking details"
+            message = "The response did not contain tracking details"
+            return TrackingResponse.new(
+              false,
+              message,
+              Hash.from_xml(response),
+              carrier: @@name,
+              xml: response,
+              request: last_request
+            )
           else
             all_unique_identifiers = xml.root.xpath('CompletedTrackDetails/TrackDetails/TrackingNumberUniqueIdentifier').map(&:text)
-            raise ActiveShipping::Error, "Multiple matches were found. Specify a unqiue identifier: #{all_unique_identifiers.join(', ')}"
+            message = "Multiple matches were found. Specify a unqiue identifier: #{all_unique_identifiers.join(', ')}"
+            return TrackingResponse.new(
+              false,
+              message,
+              Hash.from_xml(response),
+              carrier: @@name,
+              xml: response,
+              request: last_request
+            )
         end
 
-
         first_notification = tracking_details.at('Notification')
         severity = first_notification.at('Severity').text
         if severity == 'ERROR' || severity == 'FAILURE'
@@ -634,11 +654,11 @@ module ActiveShipping
           end
         end
 
-        if origin_node = tracking_details.at('OriginLocationAddress')
-          origin = Location.new(
-                :country =>     origin_node.at('CountryCode').text,
-                :province =>    origin_node.at('StateOrProvinceCode').text,
-                :city =>        origin_node.at('City').text
+        origin = if origin_node = tracking_details.at('OriginLocationAddress')
+          Location.new(
+            country: origin_node.at('CountryCode').text,
+            province: origin_node.at('StateOrProvinceCode').text,
+            city: origin_node.at('City').text
           )
         end
 
@@ -667,26 +687,29 @@ module ActiveShipping
 
           shipment_events << ShipmentEvent.new(description, zoneless_time, location, description, type_code)
         end
-        shipment_events = shipment_events.sort_by(&:time)
 
+        shipment_events = shipment_events.sort_by(&:time)
       end
 
-      TrackingResponse.new(success, message, Hash.from_xml(response),
-                           :carrier => @@name,
-                           :xml => response,
-                           :request => last_request,
-                           :status => status,
-                           :status_code => status_code,
-                           :status_description => status_description,
-                           :ship_time => ship_time,
-                           :scheduled_delivery_date => scheduled_delivery_time,
-                           :actual_delivery_date => actual_delivery_time,
-                           :delivery_signature => delivery_signature,
-                           :shipment_events => shipment_events,
-                           :shipper_address => (shipper_address.nil? || shipper_address.unknown?) ? nil : shipper_address,
-                           :origin => origin,
-                           :destination => destination,
-                           :tracking_number => tracking_number
+      TrackingResponse.new(
+        success,
+        message,
+        Hash.from_xml(response),
+        carrier: @@name,
+        xml: response,
+        request: last_request,
+        status: status,
+        status_code: status_code,
+        status_description: status_description,
+        ship_time: ship_time,
+        scheduled_delivery_date: scheduled_delivery_time,
+        actual_delivery_date: actual_delivery_time,
+        delivery_signature: delivery_signature,
+        shipment_events: shipment_events,
+        shipper_address: (shipper_address.nil? || shipper_address.unknown?) ? nil : shipper_address,
+        origin: origin,
+        destination: destination,
+        tracking_number: tracking_number
       )
     end
 
@@ -701,13 +724,13 @@ module ActiveShipping
     end
 
     def response_success?(document)
-      highest_severity = document.root.at('HighestSeverity')
+      highest_severity = document.at('HighestSeverity')
       return false if highest_severity.nil?
       %w(SUCCESS WARNING NOTE).include?(highest_severity.text)
     end
 
     def response_message(document)
-      notifications = document.root.at('Notifications')
+      notifications = document.at('Notifications')
       return "" if notifications.nil?
 
       "#{notifications.at('Severity').text} - #{notifications.at('Code').text}: #{notifications.at('Message').text}"

data/lib/active_shipping/carriers/ups.rb

@@ -18,7 +18,8 @@ module ActiveShipping
       :ship_confirm => 'ups.app/xml/ShipConfirm',
       :ship_accept => 'ups.app/xml/ShipAccept',
       :delivery_dates =>  'ups.app/xml/TimeInTransit',
-      :void =>  'ups.app/xml/Void'
+      :void =>  'ups.app/xml/Void',
+      :validate_address => 'ups.app/xml/XAV'
     }
 
     PICKUP_CODES = HashWithIndifferentAccess.new(
@@ -232,6 +233,21 @@ module ActiveShipping
       35
     end
 
+    # Validates a location with the Street Level Validation service
+    #
+    # @param location [Location] The Location to validate
+    # @return [ActiveShipping::AddressValidationResponse] The response from the validation endpoint. This
+    #   response will determine if the given address is valid or not, its commercial/residential classification,
+    #   and the cleaned-up address and/or potential candidate addresses if the passed location can't be found
+    def validate_address(location, options = {})
+      location = upsified_location(location)
+      options = @options.merge(options)
+      access_request = build_access_request
+      address_validation_request = build_address_validation_request(location, options)
+      response = commit(:validate_address, save_request(access_request + address_validation_request), options[:test])
+      parse_address_validation_response(location, response, options)
+    end
+
     protected
 
     def upsified_location(location)
@@ -963,6 +979,97 @@ module ActiveShipping
       end
     end
 
+    def build_address_validation_request(location, options = {})
+      xml_builder = Nokogiri::XML::Builder.new do |xml|
+        xml.AddressValidationRequest do
+          xml.Request do
+            xml.RequestAction('XAV')
+            xml.RequestOption('3')
+
+            if options[:customer_context]
+              xml.TransactionReference do
+                xml.CustomerContext(options[:customer_context])
+                xml.XpciVersion("1.0")
+              end
+            end
+          end
+
+          xml.AddressKeyFormat do
+            xml.AddressLine(location.address1)
+            if location.address2.present?
+              xml.AddressLine(location.address2)
+            end
+            xml.PoliticalDivision2(location.city)
+            xml.PoliticalDivision1(location.state)
+            xml.PostcodePrimaryLow(location.postal_code)
+            xml.CountryCode(location.country_code)
+          end
+        end
+      end
+
+      xml_builder.to_xml
+    end
+
+    def parse_address_validation_response(address, response, options={})
+      xml     = build_document(response, 'AddressValidationResponse')
+      success = response_success?(xml)
+      message = response_message(xml)
+
+      validity = nil
+      classification_code = nil
+      classification_description = nil
+      addresses = []
+
+      if success
+        if xml.at('AddressClassification/Code').present?
+          classification_code = xml.at('AddressClassification/Code').text
+        end
+        
+        classification = case classification_code
+        when "1"
+          :commercial
+        when "2"
+          :residential
+        else
+          :unknown
+        end
+
+        validity = if xml.at("ValidAddressIndicator").present?
+          :valid
+        elsif xml.at("AmbiguousAddressIndicator").present?
+          :ambiguous
+        elsif xml.at("NoCandidatesIndicator").present?
+          :invalid
+        else
+          :unknown
+        end
+
+        addresses = xml.css('AddressKeyFormat').collect { |node| location_from_address_key_format_node(node) }
+      end
+
+      params = Hash.from_xml(response).values.first
+      response = AddressValidationResponse.new(success, message, params, :validity => validity, :classification => classification, :candidate_addresses => addresses, :xml => response, :request => last_request)
+    end
+
+    # Converts from a AddressKeyFormat XML node to a Location
+    def location_from_address_key_format_node(address)
+      return nil unless address
+      country = address.at('CountryCode').try(:text)
+      country = 'US' if country == 'ZZ' # Sometimes returned by SUREPOST in the US
+      
+      address_lines = address.css('AddressLine')
+
+      Location.new(
+        :country     => country,
+        :postal_code => address.at('PostcodePrimaryLow').try(:text),
+        :province    => address.at('PoliticalDivision1').try(:text),
+        :city        => address.at('PoliticalDivision2').try(:text),
+        :address1    => address_lines[0].try(:text),
+        :address2    => address_lines[1].try(:text),
+        :address3    => address_lines[2].try(:text),
+      )
+    end
+
     def location_from_address_node(address)
       return nil unless address
       country = address.at('CountryCode').try(:text)

data/lib/active_shipping/version.rb

@@ -1,3 +1,3 @@
 module ActiveShipping
-  VERSION = "1.9.2"
+  VERSION = "1.10.1"
 end

data/test/fixtures/xml/fedex/tracking_response_unable_to_process.xml

@@ -0,0 +1,32 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<TrackReply xmlns="http://fedex.com/ws/track/v7" xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/">
+  <HighestSeverity>SUCCESS</HighestSeverity>
+  <Notifications>
+    <Severity>SUCCESS</Severity>
+    <Source>trck</Source>
+    <Code>0</Code>
+    <Message>Request was successfully processed.</Message>
+    <LocalizedMessage>Request was successfully processed.</LocalizedMessage>
+  </Notifications>
+  <TransactionDetail>
+    <CustomerTransactionId>ActiveShipping</CustomerTransactionId>
+  </TransactionDetail>
+  <Version>
+    <ServiceId>trck</ServiceId>
+    <Major>7</Major>
+    <Intermediate>0</Intermediate>
+    <Minor>0</Minor>
+  </Version>
+  <CompletedTrackDetails>
+    <HighestSeverity>FAILURE</HighestSeverity>
+    <Notifications>
+      <Severity>FAILURE</Severity>
+      <Source>trck</Source>
+      <Code>6330</Code>
+      <Message>Sorry, we are unable to process your tracking request.  Please retry later, or contact Customer Service at 1.800.Go.FedEx(R) 800.463.3339.</Message>
+      <LocalizedMessage>Sorry, we are unable to process your tracking request.  Please retry later, or contact Customer Service at 1.800.Go.FedEx(R) 800.463.3339.</LocalizedMessage>
+    </Notifications>
+    <DuplicateWaybill>false</DuplicateWaybill>
+    <MoreData>false</MoreData>
+  </CompletedTrackDetails>
+</TrackReply>

data/test/fixtures/xml/ups/address_validation_request.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0"?>
+<AddressValidationRequest>
+  <Request>
+    <RequestAction>XAV</RequestAction>
+    <RequestOption>3</RequestOption>
+  </Request>
+  <AddressKeyFormat>
+    <AddressLine>55 Glenlake Parkway NE</AddressLine>
+    <PoliticalDivision2>Atlanta</PoliticalDivision2>
+    <PoliticalDivision1>GA</PoliticalDivision1>
+    <PostcodePrimaryLow>30328</PostcodePrimaryLow>
+    <CountryCode>US</CountryCode>
+  </AddressKeyFormat>
+</AddressValidationRequest>

data/test/fixtures/xml/ups/address_validation_response.xml

@@ -0,0 +1,25 @@
+<AddressValidationResponse>
+  <Response>
+    <TransactionReference />
+    <ResponseStatusCode>1</ResponseStatusCode>
+    <ResponseStatusDescription>Success</ResponseStatusDescription>
+  </Response>
+  <ValidAddressIndicator />
+  <AddressClassification>
+    <Code>1</Code>
+    <Description>Commercial</Description>
+  </AddressClassification>
+  <AddressKeyFormat>
+    <AddressClassification>
+      <Code>1</Code>
+      <Description>Commercial</Description>
+    </AddressClassification>
+    <AddressLine>55 GLENLAKE PKWY</AddressLine>
+    <Region>ATLANTA GA 30328-3474</Region>
+    <PoliticalDivision2>ATLANTA</PoliticalDivision2>
+    <PoliticalDivision1>GA</PoliticalDivision1>
+    <PostcodePrimaryLow>30328</PostcodePrimaryLow>
+    <PostcodeExtendedLow>3474</PostcodeExtendedLow>
+    <CountryCode>US</CountryCode>
+  </AddressKeyFormat>
+</AddressValidationResponse>

data/test/fixtures/xml/ups/address_validation_response_ambiguous.xml

@@ -0,0 +1,38 @@
+<AddressValidationResponse>
+  <Response>
+    <TransactionReference />
+    <ResponseStatusCode>1</ResponseStatusCode>
+    <ResponseStatusDescription>Success</ResponseStatusDescription>
+  </Response>
+  <AmbiguousAddressIndicator />
+  <AddressClassification>
+    <Code>1</Code>
+    <Description>Commercial</Description>
+  </AddressClassification>
+  <AddressKeyFormat>
+    <AddressClassification>
+      <Code>1</Code>
+      <Description>Commercial</Description>
+    </AddressClassification>
+    <AddressLine>55 GLENLAKE PKWY</AddressLine>
+    <Region>ATLANTA GA 30328-3474</Region>
+    <PoliticalDivision2>ATLANTA</PoliticalDivision2>
+    <PoliticalDivision1>GA</PoliticalDivision1>
+    <PostcodePrimaryLow>30328</PostcodePrimaryLow>
+    <PostcodeExtendedLow>3474</PostcodeExtendedLow>
+    <CountryCode>US</CountryCode>
+  </AddressKeyFormat>
+  <AddressKeyFormat>
+    <AddressClassification>
+      <Code>2</Code>
+      <Description>Residential</Description>
+    </AddressClassification>
+    <AddressLine>555 GLENVALE</AddressLine>
+    <Region>ATLANTA GA 30328-3474</Region>
+    <PoliticalDivision2>ATLANTA</PoliticalDivision2>
+    <PoliticalDivision1>GA</PoliticalDivision1>
+    <PostcodePrimaryLow>30328</PostcodePrimaryLow>
+    <PostcodeExtendedLow>3474</PostcodeExtendedLow>
+    <CountryCode>US</CountryCode>
+  </AddressKeyFormat>
+</AddressValidationResponse>

data/test/fixtures/xml/ups/address_validation_response_no_candidates.xml

@@ -0,0 +1,8 @@
+<AddressValidationResponse>
+  <Response>
+    <TransactionReference />
+    <ResponseStatusCode>1</ResponseStatusCode>
+    <ResponseStatusDescription>Success</ResponseStatusDescription>
+  </Response>
+  <NoCandidatesIndicator />
+</AddressValidationResponse>

data/test/remote/fedex_test.rb

@@ -316,6 +316,8 @@ class RemoteFedExTest < Minitest::Test
     end
   end
 
+  ### create_shipment
+
   def test_cant_obtain_multiple_shipping_labels
     assert_raises(ActiveShipping::Error,"Multiple packages are not supported yet.") do
       @carrier.create_shipment(

data/test/unit/carriers/fedex_test.rb

@@ -318,15 +318,32 @@ class FedExTest < Minitest::Test
     @carrier.expects(:commit).returns(mock_response)
 
     assert_raises ActiveShipping::ResponseContentError do
-    @carrier.find_rates(
+      @carrier.find_rates(
         location_fixtures[:ottawa],
         location_fixtures[:beverly_hills],
         package_fixtures.values_at(:book, :wii),
-        :test => true
+        test: true
       )
     end
   end
 
+  def test_parsing_response_without_notifications
+    mock_response = xml_fixture('fedex/reply_without_notifications')
+
+    @carrier.expects(:commit).returns(mock_response)
+
+    response = @carrier.find_rates(
+      location_fixtures[:ottawa],
+      location_fixtures[:beverly_hills],
+      package_fixtures.values_at(:book, :wii),
+      test: true
+    )
+
+    assert_predicate response, :success?
+  end
+
+  ### find_tracking_info
+
   def test_response_transient_failure
     mock_response = xml_fixture('fedex/tracking_response_failure_code_9045')
     @carrier.expects(:commit).returns(mock_response)
@@ -339,7 +356,7 @@ class FedExTest < Minitest::Test
     assert_equal msg, error.message
   end
 
-  def test_response_terminal_failure
+  def test_response_with_failure_nested_in_track_details
     mock_response = xml_fixture('fedex/tracking_response_failure_code_9080')
     @carrier.expects(:commit).returns(mock_response)
 
@@ -351,23 +368,18 @@ class FedExTest < Minitest::Test
     assert_equal msg, error.message
   end
 
-  def test_parsing_response_without_notifications
-    mock_response = xml_fixture('fedex/reply_without_notifications')
-
+  def test_response_with_failure_nested_in_completed_track_details
+    mock_response = xml_fixture('fedex/tracking_response_unable_to_process')
     @carrier.expects(:commit).returns(mock_response)
 
-    response = @carrier.find_rates(
-      location_fixtures[:ottawa],
-      location_fixtures[:beverly_hills],
-      package_fixtures.values_at(:book, :wii),
-      :test => true
-    )
+    error = assert_raises(ActiveShipping::ResponseError) do
+      @carrier.find_tracking_info('123456789013')
+    end
 
-    assert response.success?
+    msg = 'FAILURE - 6330: Sorry, we are unable to process your tracking request.  Please retry later, or contact Customer Service at 1.800.Go.FedEx(R) 800.463.3339.'
+    assert_equal msg, error.message
   end
 
-  ### find_tracking_info
-
   def test_tracking_info_for_delivered_with_signature
     mock_response = xml_fixture('fedex/tracking_response_delivered_with_signature')
     @carrier.expects(:commit).returns(mock_response)
@@ -507,7 +519,7 @@ class FedExTest < Minitest::Test
     mock_response = xml_fixture('fedex/tracking_response_multiple_results')
     @carrier.expects(:commit).returns(mock_response)
 
-    error = assert_raises(ActiveShipping::Error) do
+    error = assert_raises(ActiveShipping::ResponseError) do
       @carrier.find_tracking_info('123456789012')
     end
 
@@ -574,6 +586,8 @@ class FedExTest < Minitest::Test
     assert_empty response.shipment_events
   end
 
+  ### create_shipment
+  
   def test_create_shipment
     confirm_response = xml_fixture('fedex/create_shipment_response')
     @carrier.stubs(:commit).returns(confirm_response)

data/test/unit/carriers/ups_test.rb

@@ -628,4 +628,32 @@ class UPSTest < Minitest::Test
     assert_equal 'OZS', request.search('/Package/PackageWeight/UnitOfMeasurement/Code').text
     assert_equal '8.0', request.search('/Package/PackageWeight/Weight').text
   end
+
+  def test_address_validation
+    location = Location.new(address1: "55 Glenlake Parkway", city: "Atlanta", state: "GA", zip: "30328", country: "US")
+    address_validation_response = xml_fixture('ups/address_validation_response')
+    @carrier.expects(:commit).returns(address_validation_response)
+    response = @carrier.validate_address(location)
+    assert_equal :commercial, response.classification
+    assert_equal true, response.address_match?
+  end
+
+  def test_address_validation_ambiguous
+    location = Location.new(address1: "55 Glen", city: "Atlanta", state: "GA", zip: "30328", country: "US")
+    address_validation_response = xml_fixture('ups/address_validation_response_ambiguous')
+    @carrier.expects(:commit).returns(address_validation_response)
+    response = @carrier.validate_address(location)
+    assert_equal false, response.address_match?
+    assert_equal :ambiguous, response.validity
+  end
+
+  def test_address_validation_no_candidates
+    location = Location.new(address1: "55 Glenblagahrhadd", city: "Atlanta", state: "GA", zip: "30321", country: "US")
+    address_validation_response = xml_fixture('ups/address_validation_response_no_candidates')
+    @carrier.expects(:commit).returns(address_validation_response)
+    response = @carrier.validate_address(location)
+    assert_equal false, response.address_match?
+    assert_equal :invalid, response.validity
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_shipping
 version: !ruby/object:Gem::Version
-  version: 1.9.2
+  version: 1.10.1
 platform: ruby
 authors:
 - Shopify
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-15 00:00:00.000000000 Z
+date: 2017-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: quantified
@@ -62,16 +62,16 @@ dependencies:
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: 1.6.8
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: 1.6.8
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -182,6 +182,7 @@ files:
 - gemfiles/activesupport50.gemfile
 - gemfiles/activesupport_master.gemfile
 - lib/active_shipping.rb
+- lib/active_shipping/address_validation_response.rb
 - lib/active_shipping/carrier.rb
 - lib/active_shipping/carriers.rb
 - lib/active_shipping/carriers/australia_post.rb
@@ -299,6 +300,7 @@ files:
 - test/fixtures/xml/fedex/tracking_response_multiple_results.xml
 - test/fixtures/xml/fedex/tracking_response_not_found.xml
 - test/fixtures/xml/fedex/tracking_response_shipment_exception.xml
+- test/fixtures/xml/fedex/tracking_response_unable_to_process.xml
 - test/fixtures/xml/fedex/tracking_response_with_blank_state.xml
 - test/fixtures/xml/fedex/unknown_fedex_document_reply.xml
 - test/fixtures/xml/kunaki/invalid_state_response.xml
@@ -328,6 +330,10 @@ files:
 - test/fixtures/xml/stamps/track_shipment_request.xml
 - test/fixtures/xml/stamps/track_shipment_response.xml
 - test/fixtures/xml/ups/access_request.xml
+- test/fixtures/xml/ups/address_validation_request.xml
+- test/fixtures/xml/ups/address_validation_response.xml
+- test/fixtures/xml/ups/address_validation_response_ambiguous.xml
+- test/fixtures/xml/ups/address_validation_response_no_candidates.xml
 - test/fixtures/xml/ups/delivered_shipment_with_refund.xml
 - test/fixtures/xml/ups/delivered_shipment_without_events_tracking_response.xml
 - test/fixtures/xml/ups/delivery_dates_response.xml
@@ -525,6 +531,7 @@ test_files:
 - test/fixtures/xml/fedex/tracking_response_multiple_results.xml
 - test/fixtures/xml/fedex/tracking_response_not_found.xml
 - test/fixtures/xml/fedex/tracking_response_shipment_exception.xml
+- test/fixtures/xml/fedex/tracking_response_unable_to_process.xml
 - test/fixtures/xml/fedex/tracking_response_with_blank_state.xml
 - test/fixtures/xml/fedex/unknown_fedex_document_reply.xml
 - test/fixtures/xml/kunaki/invalid_state_response.xml
@@ -554,6 +561,10 @@ test_files:
 - test/fixtures/xml/stamps/track_shipment_request.xml
 - test/fixtures/xml/stamps/track_shipment_response.xml
 - test/fixtures/xml/ups/access_request.xml
+- test/fixtures/xml/ups/address_validation_request.xml
+- test/fixtures/xml/ups/address_validation_response.xml
+- test/fixtures/xml/ups/address_validation_response_ambiguous.xml
+- test/fixtures/xml/ups/address_validation_response_no_candidates.xml
 - test/fixtures/xml/ups/delivered_shipment_with_refund.xml
 - test/fixtures/xml/ups/delivered_shipment_without_events_tracking_response.xml
 - test/fixtures/xml/ups/delivery_dates_response.xml