peddler 1.3.0 → 1.4.0

data/test/integration/test_recommendations.rb

@@ -1,13 +1,11 @@
-require 'helper'
+require 'integration_helper'
 require 'mws/recommendations'
 
 class TestRecommendations < IntegrationTest
   def test_lists_recommendations
     clients.each do |client|
       res = client.list_recommendations
-      content = res.parse
-      skip unless content
-      refute_empty content
+      refute_empty res.parse
     end
   end
 

data/test/integration/test_reports.rb

@@ -1,4 +1,4 @@
-require 'helper'
+require 'integration_helper'
 require 'mws/reports'
 
 class TestReports < IntegrationTest

data/test/integration/test_sellers.rb

@@ -1,4 +1,4 @@
-require 'helper'
+require 'integration_helper'
 require 'mws/sellers'
 
 class TestSellers < IntegrationTest

data/test/integration/test_subscriptions.rb

@@ -1,4 +1,4 @@
-require 'helper'
+require 'integration_helper'
 require 'mws/subscriptions'
 
 class TestSubscriptions < IntegrationTest

data/test/integration_helper.rb

@@ -0,0 +1,41 @@
+require 'helper'
+require 'recorder'
+require 'dig_rb'
+
+%w(mws.yml mws.yml.example).each do |path|
+  file = File.expand_path("../#{path}", __FILE__)
+  if File.exist?(file)
+    $mws = YAML.load_file(file)
+    break
+  end
+end
+
+class IntegrationTest < MiniTest::Test
+  include Recorder
+
+  private
+
+  def clients
+    api = @api || test_name
+    $mws.map do |record|
+      MWS.const_get("#{api}::Client").new(record)
+    end
+  end
+end
+
+::Peddler::VCRMatcher.ignore_seller!
+
+VCR.configure do |c|
+  c.before_record do |interaction|
+    %w(
+      BuyerName BuyerEmail Name AddressLine1 PostalCode Phone Amount
+    ).each do |key|
+      interaction.response.body.gsub!(/<#{key}>[^<]+</, "<#{key}>FILTERED<")
+    end
+  end
+
+  $mws.each do |record|
+    c.filter_sensitive_data('FILTERED') { record['merchant_id'] }
+    c.filter_sensitive_data('FILTERED') { record['aws_access_key_id'] }
+  end
+end

data/test/mws.yml

@@ -1,16 +1,36 @@
 - primary_marketplace_id: A2EUQ1WTGCTBG2
-  aws_access_key_id: AKIAITHQMZUJROSOP27Q
-  aws_secret_access_key: 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
+  aws_access_key_id: AKIAILJW7BWJ6YBK4JUQ
+  aws_secret_access_key: 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
   merchant_id: A2A7WNXBU01UJW
+- primary_marketplace_id: A1AM78C64UM0Y8
+  aws_access_key_id: AKIAIHE4A3HDKICFHP4A
+  aws_secret_access_key: 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
+  merchant_id: A1PCFAUFK3U94S
+- primary_marketplace_id: A13V1IB3VIYZZH
+  aws_access_key_id: AKIAIOYKJRT4WIH3P72A
+  aws_secret_access_key: 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
+  merchant_id: A2DP6GIDYN6OTH
+- primary_marketplace_id: APJ6JRA9NG5V4
+  aws_access_key_id: AKIAIHE4YSF4UF7CDR6Q
+  aws_secret_access_key: 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
+  merchant_id: A2APQUVDBVWV7E
+- primary_marketplace_id: A1RKKUPIHCS9HS
+  aws_access_key_id: AKIAJAMAR367FRSYNOPA
+  aws_secret_access_key: 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
+  merchant_id: A2NEU7G6RGUKF
+- primary_marketplace_id: A1F83G8C2ARO7P
+  aws_access_key_id: AKIAJCJNB4MASX5JUMUA
+  aws_secret_access_key: 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
+  merchant_id: A2D9DZ30GEWP6N
+- primary_marketplace_id: A1VC38T7YXB528
+  aws_access_key_id: AKIAJZ6EOLNDLSOEOLBA
+  aws_secret_access_key: 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
+  merchant_id: A7BU5WGNVGQ1I
 - primary_marketplace_id: A1PA6795UKMFR9
   aws_access_key_id: AKIAIORT7554FEBW7UBQ
   aws_secret_access_key: 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
   merchant_id: A7ZXDJZD3UIRG
-- primary_marketplace_id: A1VC38T7YXB528
-  aws_access_key_id: AKIAJ5RDDSRROM6GXQ5Q
-  aws_secret_access_key: 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
-  merchant_id: A7BU5WGNVGQ1I
 - primary_marketplace_id: ATVPDKIKX0DER
-  aws_access_key_id: AKIAJ767ZET2BZR4UDDQ
-  aws_secret_access_key: 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
+  aws_access_key_id: AKIAJ25X7BKNXKVCYJXA
+  aws_secret_access_key: hguX3TMfKMrqKm71/dQLKqbv2VGCyZANnCXeHAsC
   merchant_id: A2H6NH4SQYFZ4M

data/test/mws.yml.example

@@ -1,8 +1,20 @@
+- primary_marketplace_id: A1AM78C64UM0Y8
+  aws_access_key_id: AWS_ACCESS_KEY_ID
+  aws_secret_access_key: AWS_SECRET_ACCESS_KEY
+  merchant_id: MERCHANT_ID
+- primary_marketplace_id: A13V1IB3VIYZZH
+  aws_access_key_id: AWS_ACCESS_KEY_ID
+  aws_secret_access_key: AWS_SECRET_ACCESS_KEY
+  merchant_id: MERCHANT_ID
+- primary_marketplace_id: A1RKKUPIHCS9HS
+  aws_access_key_id: AWS_ACCESS_KEY_ID
+  aws_secret_access_key: AWS_SECRET_ACCESS_KEY
+  merchant_id: MERCHANT_ID
 - primary_marketplace_id: A2EUQ1WTGCTBG2
   aws_access_key_id: AWS_ACCESS_KEY_ID
   aws_secret_access_key: AWS_SECRET_ACCESS_KEY
   merchant_id: MERCHANT_ID
-- primary_marketplace_id: A1PA6795UKMFR9
+- primary_marketplace_id: APJ6JRA9NG5V4
   aws_access_key_id: AWS_ACCESS_KEY_ID
   aws_secret_access_key: AWS_SECRET_ACCESS_KEY
   merchant_id: MERCHANT_ID
@@ -10,6 +22,14 @@
   aws_access_key_id: AWS_ACCESS_KEY_ID
   aws_secret_access_key: AWS_SECRET_ACCESS_KEY
   merchant_id: MERCHANT_ID
+- primary_marketplace_id: A1F83G8C2ARO7P
+  aws_access_key_id: AWS_ACCESS_KEY_ID
+  aws_secret_access_key: AWS_SECRET_ACCESS_KEY
+  merchant_id: MERCHANT_ID
+- primary_marketplace_id: A1PA6795UKMFR9
+  aws_access_key_id: AWS_ACCESS_KEY_ID
+  aws_secret_access_key: AWS_SECRET_ACCESS_KEY
+  merchant_id: MERCHANT_ID
 - primary_marketplace_id: ATVPDKIKX0DER
   aws_access_key_id: AWS_ACCESS_KEY_ID
   aws_secret_access_key: AWS_SECRET_ACCESS_KEY

data/test/recorder.rb

@@ -0,0 +1,34 @@
+require 'peddler/vcr_matcher'
+require 'yaml'
+require 'vcr'
+
+VCR.configure do |c|
+  c.hook_into :excon
+  c.cassette_library_dir = 'test/vcr_cassettes'
+
+  c.default_cassette_options = {
+    match_requests_on: [::Peddler::VCRMatcher],
+    record: ENV['RECORD'] ? :new_episodes : :none
+  }
+
+  # c.before_record do |interaction|
+  #   code = interaction.response.status.code
+  #   interaction.ignore! if code >= 400 && code != 414
+  # end
+end
+
+module Recorder
+  def setup
+    ENV['LIVE'] ? VCR.turn_off! : VCR.insert_cassette(test_name)
+  end
+
+  def teardown
+    VCR.eject_cassette if VCR.turned_on?
+  end
+
+  private
+
+  def test_name
+    self.class.name.sub('Test', '')
+  end
+end

data/test/unit/mws/test_fulfillment_inbound_shipment_client.rb

@@ -165,7 +165,8 @@ class TestMWSFulfillmentInboundShipmentClient < MiniTest::Test
     @client.stub(:run, nil) do
       @client.list_inbound_shipments(
         shipment_status_list: ['Foo'],
-        shipment_id_list: ['Bar'])
+        shipment_id_list: ['Bar']
+      )
     end
 
     assert_equal operation, @client.operation

data/test/unit/mws/test_fulfillment_outbound_shipment_client.rb

@@ -112,9 +112,16 @@ class TestMWSFulfillmentOutboundShipmentClient < MiniTest::Test
   end
 
   def test_gets_package_tracking_details
-    assert_raises(NotImplementedError) do
-      @client.get_package_tracking_details
+    operation = {
+      'Action' => 'GetPackageTrackingDetails',
+      'PackageNumber' => '1'
+    }
+
+    @client.stub(:run, nil) do
+      @client.get_package_tracking_details('1')
     end
+
+    assert_equal operation, @client.operation
   end
 
   def test_cancels_fulfillment_order

data/test/unit/mws/test_orders_client.rb

@@ -9,6 +9,7 @@ class TestMWSOrdersClient < MiniTest::Test
   def test_lists_orders
     operation = {
       'Action' => 'ListOrders',
+      'CreatedAfter' => '2016-01-01',
       'OrderStatus.Status.1' => '1',
       'MarketplaceId.Id.1' => '1',
       'PaymentMethod.1' => '1',
@@ -19,6 +20,7 @@ class TestMWSOrdersClient < MiniTest::Test
     @client.stub(:run, nil) do
       @client.list_orders(
         marketplace_id: '1',
+        created_after: '2016-01-01',
         order_status: '1',
         tfm_shipment_status: '1',
         payment_method: '1',
@@ -29,6 +31,16 @@ class TestMWSOrdersClient < MiniTest::Test
     assert_equal operation, @client.operation
   end
 
+  def test_requires_start_time_keyword_when_listing_orders
+    @client.stub(:run, nil) do
+      assert_raises(ArgumentError) do
+        @client.list_orders
+      end
+      @client.list_orders(created_after: '2016-01-01')
+      @client.list_orders(last_updated_after: '2016-01-01')
+    end
+  end
+
   def test_lists_orders_by_next_token
     operation = {
       'Action' => 'ListOrdersByNextToken',

data/test/unit/mws/test_products_client.rb

@@ -136,6 +136,43 @@ class TestMWSProductsClient < MiniTest::Test
     assert_equal operation, @client.operation
   end
 
+  def test_gets_my_fees_estimate
+    operation = {
+      'Action' => 'GetMyFeesEstimate',
+      'FeesEstimateRequestList.FeesEstimateRequest.1.MarketplaceId' => '123',
+      'FeesEstimateRequestList.FeesEstimateRequest.1.IdType' => 'ASIN',
+      'FeesEstimateRequestList.FeesEstimateRequest.1.IdValue' => '123',
+      'FeesEstimateRequestList.FeesEstimateRequest.1.PriceToEstimateFees.ListingPrice.CurrencyCode' => 'USD',
+      'FeesEstimateRequestList.FeesEstimateRequest.1.PriceToEstimateFees.ListingPrice.Amount' => 30.00,
+      'FeesEstimateRequestList.FeesEstimateRequest.1.PriceToEstimateFees.Shipping.CurrencyCode' => 'USD',
+      'FeesEstimateRequestList.FeesEstimateRequest.1.PriceToEstimateFees.Shipping.Amount' => 3.99,
+      'FeesEstimateRequestList.FeesEstimateRequest.1.PriceToEstimateFees.Points.PointsNumber' => 0
+    }
+
+    @client.stub(:run, nil) do
+      @client.get_my_fees_estimate(
+        marketplace_id: '123',
+        id_type: 'ASIN',
+        id_value: '123',
+        price_to_estimate_fees: {
+          listing_price: {
+            currency_code: 'USD',
+            amount: 30.00
+          },
+          shipping: {
+            currency_code: 'USD',
+            amount: 3.99
+          },
+          points: {
+            points_number: 0
+          }
+        }
+      )
+    end
+
+    assert_equal operation, @client.operation
+  end
+
   def test_gets_my_price_for_sku
     operation = {
       'Action' => 'GetMyPriceForSKU',

data/test/unit/peddler/errors/test_builder.rb

@@ -0,0 +1,16 @@
+require 'helper'
+require 'peddler/errors/builder'
+
+class TestPeddlerErrorsBuilder < MiniTest::Test
+  def test_builds_error_class
+    Peddler::Errors::Builder.build('Foo')
+    assert Peddler::Errors::Foo
+  end
+
+  def test_thread_safety
+    Peddler::Errors::Builder.build('Foo')
+    assert_output '', '' do
+      Peddler::Errors::Builder.build('Foo')
+    end
+  end
+end

data/test/unit/peddler/errors/test_error.rb

@@ -0,0 +1,16 @@
+require 'helper'
+require 'peddler/errors/error'
+
+class TestPeddlerErrorsError < MiniTest::Test
+  def setup
+    @error = Peddler::Errors::Error.new('message', 'cause')
+  end
+
+  def test_sets_message
+    assert_equal 'message', @error.message
+  end
+
+  def test_sets_cause
+    assert_equal 'cause', @error.cause
+  end
+end

data/test/unit/peddler/errors/test_handler.rb

@@ -0,0 +1,60 @@
+require 'helper'
+require 'peddler/errors/handler'
+
+class TestPeddlerErrorsHandler < MiniTest::Test
+  def setup
+    @error = Peddler::Errors::Handler.call(@cause)
+  rescue => @error
+  end
+
+  class CausedByHTTPStatusError < TestPeddlerErrorsHandler
+    def setup
+      @code = 'FeedProcessingResultNotReady'
+      @message = 'Feed Submission Result is not ready for Feed 123'
+      @cause = Excon::Error::NotFound.new(
+        'Expected(200) <=> Actual(404 Not Found)',
+        nil,
+        OpenStruct.new(code: @code, message: @message)
+      )
+      super
+    end
+
+    def test_generates_custom_error
+      assert_includes @error.class.name, @code
+    end
+
+    def test_provides_message
+      assert_equal @message, @error.message
+    end
+
+    def test_provides_cause
+      assert_equal @cause, @error.cause
+    end
+  end
+
+  class CausedByAnotherError < TestPeddlerErrorsHandler
+    def setup
+      @cause = StandardError.new
+      super
+    end
+
+    def test_returns_original_error
+      assert_equal @cause, @error
+    end
+  end
+
+  class CausedByInternalServerError < TestPeddlerErrorsHandler
+    def setup
+      @cause = Excon::Error::InternalServerError.new(
+        nil,
+        nil,
+        OpenStruct.new(code: '500', message: nil)
+      )
+      super
+    end
+
+    def test_returns_original_error
+      assert_equal @cause, @error
+    end
+  end
+end

data/test/unit/peddler/{test_error_parser.rb → errors/test_parser.rb}

@@ -1,7 +1,7 @@
 require 'helper'
-require 'peddler/error_parser'
+require 'peddler/errors/parser'
 
-class TestPeddlerErrorParser < MiniTest::Test
+class TestPeddlerErrorsParser < MiniTest::Test
   def setup
     body = <<-XML
       <ErrorResponse>
@@ -21,7 +21,7 @@ class TestPeddlerErrorParser < MiniTest::Test
       }
     )
 
-    @parser = Peddler::ErrorParser.new(res)
+    @parser = Peddler::Errors::Parser.new(res)
   end
 
   def test_parses_data

data/test/unit/peddler/test_client.rb

@@ -138,7 +138,7 @@ class TestPeddlerClient < MiniTest::Test
   def test_error_callback_on_class
     Excon.stub({}, status: 503)
 
-    assert_raises(Excon::Errors::ServiceUnavailable) do
+    assert_raises(Excon::Error::ServiceUnavailable) do
       @client.run
     end
 
@@ -153,7 +153,7 @@ class TestPeddlerClient < MiniTest::Test
   def test_error_callback_on_instance
     Excon.stub({}, status: 503)
 
-    assert_raises(Excon::Errors::ServiceUnavailable) do
+    assert_raises(Excon::Error::ServiceUnavailable) do
       @client.run
     end
 
@@ -177,7 +177,7 @@ class TestPeddlerClient < MiniTest::Test
     other_client = klass.new
     other_client.configure_with_mock_data!
     other_client.operation('Foo')
-    assert_raises(Excon::Errors::ServiceUnavailable) do
+    assert_raises(Excon::Error::ServiceUnavailable) do
       other_client.run
     end
 

data/test/unit/peddler/test_flat_file_parser.rb

@@ -2,76 +2,85 @@ require 'helper'
 require 'peddler/flat_file_parser'
 
 class TestPeddlerFlatFileParser < MiniTest::Test
-  def setup
+  def test_parses_data
     body = "Feed Processing Summary:\n\tNumber of records processed\t\t11006\n\tNumber of records successful\t\t11006\n\noriginal-record-number\tsku\terror-code\terror-type\terror-message\n1822\t85da472e-ba6c-11e3-95af-002590a74356\t5000\tWarning\tThe update for Sku '85da472e-ba6c-11e3-95af-002590a74356' was skipped because it is identical to the update in feed '9518995390'.\n"
     body.encode!('ASCII-8BIT')
-    headers = { 'Content-Type' => 'application/octet-stream', 'Content-MD5' => 'SmZIMTs2mCO8u8QzR5LE7Q==' }
-    res = OpenStruct.new(body: body, headers: headers)
-
-    @parser = Peddler::FlatFileParser.new(res, 'ISO-8859-1')
-  end
-
-  def test_parses_data
-    assert_kind_of CSV::Table, @parser.parse
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'ISO-8859-1')
+    assert_kind_of CSV::Table, parser.parse
   end
 
   def test_parses_data_a_line_at_a_time
+    body = "Feed Processing Summary:\n\tNumber of records processed\t\t11006\n\tNumber of records successful\t\t11006\n\noriginal-record-number\tsku\terror-code\terror-type\terror-message\n1822\t85da472e-ba6c-11e3-95af-002590a74356\t5000\tWarning\tThe update for Sku '85da472e-ba6c-11e3-95af-002590a74356' was skipped because it is identical to the update in feed '9518995390'.\n"
+    body.encode!('ASCII-8BIT')
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'ISO-8859-1')
     counter = 0
-    @parser.parse { counter += 1 }
+    parser.parse { counter += 1 }
     assert counter > 0
   end
 
   def test_summarises
-    refute_empty @parser.records_count
+    body = "Feed Processing Summary:\n\tNumber of records processed\t\t11006\n\tNumber of records successful\t\t11006\n\noriginal-record-number\tsku\terror-code\terror-type\terror-message\n1822\t85da472e-ba6c-11e3-95af-002590a74356\t5000\tWarning\tThe update for Sku '85da472e-ba6c-11e3-95af-002590a74356' was skipped because it is identical to the update in feed '9518995390'.\n"
+    body.encode!('ASCII-8BIT')
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'ISO-8859-1')
+    refute_empty parser.records_count
+  end
+
+  def test_summarises_non_english_reports
+    body = "Riepilogo elaborazione feed:\n\tNumero record elaborati\t\t1\n\tNumero record elaborati con successo\t\t1\n\n"
+    body.encode!('Cp1252')
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'ISO-8859-1')
+    refute_empty parser.records_count
   end
 
   def test_validates
-    assert @parser.valid?
+    body = "Feed Processing Summary:\n\tNumber of records processed\t\t11006\n\tNumber of records successful\t\t11006\n\noriginal-record-number\tsku\terror-code\terror-type\terror-message\n1822\t85da472e-ba6c-11e3-95af-002590a74356\t5000\tWarning\tThe update for Sku '85da472e-ba6c-11e3-95af-002590a74356' was skipped because it is identical to the update in feed '9518995390'.\n"
+    body.encode!('ASCII-8BIT')
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'ISO-8859-1')
+    assert parser.valid?
   end
 
   def test_handles_reports_without_a_summary
-    res = OpenStruct.new(body: "Foo\nBar\n")
-    parser = Peddler::FlatFileParser.new(res, 'ISO-8859-1')
-
+    response = OpenStruct.new(body: "Foo\nBar\n")
+    parser = Peddler::FlatFileParser.new(response, 'ISO-8859-1')
     refute_empty parser.content
   end
 
   def test_handles_japanese_flat_files
     body = "Foo\nこんにちは\n"
     body.encode!('SHIFT_JIS')
-    body.force_encoding('ASCII-8BIT')
-    res = OpenStruct.new(body: body)
-    parser = Peddler::FlatFileParser.new(res, 'Windows-31J')
-
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'Windows-31J')
     assert_equal 'こんにちは', parser.parse[0]['Foo']
   end
 
   def test_handles_japanese_curly_braces
     body = "Foo\n〝\n"
     body.encode!('Windows-31J')
-    body.force_encoding('ASCII-8BIT')
-    res = OpenStruct.new(body: body)
-    parser = Peddler::FlatFileParser.new(res, 'Windows-31J')
-
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'Windows-31J')
     assert_equal '〝', parser.parse[0]['Foo']
   end
 
   def test_handles_latin_1_flat_files
     body = "Foo\n™\n"
     body.encode!('Cp1252')
-    body.force_encoding('ASCII-8BIT')
-    res = OpenStruct.new(body: body)
-    parser = Peddler::FlatFileParser.new(res, 'Cp1252')
-
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'Cp1252')
     assert_equal '™', parser.parse['Foo'][0]
   end
 
   def test_handles_undefined_characters
     body = "Foo\n\xFF\n"
     body.force_encoding('ASCII-8BIT')
-    res = OpenStruct.new(body: body)
-    parser = Peddler::FlatFileParser.new(res, 'ASCII-8BIT')
-
+    parser = Peddler::FlatFileParser.new(build_mock_response(body), 'ASCII-8BIT')
     assert_equal '?', parser.parse['Foo'][0]
   end
+
+  private
+
+  def build_mock_response(body)
+    body.force_encoding('ASCII-8BIT')
+    headers = {
+      'Content-MD5' => Digest::MD5.base64digest(body)
+    }
+
+    OpenStruct.new(body: body, headers: headers)
+  end
 end

data/test/unit/peddler/test_structured_list.rb

@@ -16,6 +16,11 @@ class TestPeddlerStructuredList < MiniTest::Test
     assert_equal exp, @list.build(%w(foo bar))
   end
 
+  def test_flattens_nested_arrays_of_values
+    exp = { 'OrderStatus.Status.1' => 'foo', 'OrderStatus.Status.2' => 'bar' }
+    assert_equal exp, @list.build([%w(foo bar)])
+  end
+
   def test_handles_single_key
     list = Peddler::StructuredList.new('Foo')
     exp = { 'Foo.1' => 'bar' }

data/test/unit/peddler/test_vcr_matcher.rb

@@ -5,6 +5,10 @@ require 'peddler/vcr_matcher'
 class TestPeddlerVCRMatcher < MiniTest::Test
   include Recorder
 
+  def setup
+    VCR.insert_cassette(test_name, record: :none)
+  end
+
   def test_matches_recorded_post_without_body
     client.run
   end

data/test/unit/test_mws.rb

@@ -3,7 +3,7 @@ require 'mws'
 
 class Test < MiniTest::Test
   def test_delegates_to_apis
-    assert_equal MWS.methods(false).size, MWS.constants.size
+    assert_equal MWS.constants.size, MWS.methods(false).size
     assert_kind_of MWS::Products::Client, MWS.products
   end
 end