mws-orders 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ffeac9dc8722894336970c9aff3a24f421cdf781299f4b71f7a17337666e29ee
-  data.tar.gz: f6696c0924e4f4d4e1a1ea39c01e74e8638a57e5fd12aa74d6c3eb8969087b3b
+  metadata.gz: 5024482ae6bb34ce7a6066333ab17e2bad0c8cd98330ef2d17710c843114c1bb
+  data.tar.gz: 2d2235a1ccd194a58cfb162d4cd89990e3878a232c62b0c7325f30d595dd0cf4
 SHA512:
-  metadata.gz: 49b0857eaafcecd6f75220b6961b9a7e82b4dc3a5f53ed2f635f63979b7991781325be5e1b363f664576426b67af88d72d8c7cddd475c629803dbde4f16669b0
-  data.tar.gz: f00ddc5e0649389dcc953c09725d5bec260261832e11848a5b0e05425fe1b0dc85ecfea895a5affa3b9a17ca80767914c028965e670f75fa7cd386ca135fb80c
+  metadata.gz: a0e92ba47bfc2feb5513324f22a60abd078528c327e1be3e22a0d27940f7f45d5917e3708cbda335967d3cb2cbffe9364a4677f359af01d698c897d6c4b52883
+  data.tar.gz: 75677d6f12e108daf1331ba3d18b5d2c54717abd79a85999e68545a13f18773a8804577d5d063710e2fdba7a6b8a151d4b3035f795774f9c3d55bf86d43b9747

data/README.md

@@ -11,9 +11,9 @@ To use Amazon MWS, you must have an eligible seller account.
 Create a client:
 
 ```ruby
-require "mws-orders"
-client = MWS.orders(marketplace: "ATVPDKIKX0DER",
-                    merchant_id: "123")
+require 'mws/orders/parser'
+client = MWS.orders(marketplace: 'ATVPDKIKX0DER',
+                    merchant_id: '123')
 ```
 
 Set up credentials [when instantiating or with environment variables](https://github.com/hakanensari/peddler#usage).
@@ -39,7 +39,7 @@ client.list_orders_by_next_token(orders.next_token).parse
 Get one or more orders based on their order numbers:
 
 ```ruby
-response = client.get_order("123-1234567-1234567")
+response = client.get_order('123-1234567-1234567')
 order = response.parse
 puts order # => #<MWS::Orders::Order amazon_order_id="123...
 ```
@@ -49,7 +49,7 @@ puts order # => #<MWS::Orders::Order amazon_order_id="123...
 List order items:
 
 ```ruby
-response = client.list_order_items("123-1234567-1234567")
+response = client.list_order_items('123-1234567-1234567')
 order_items = response.parse
 ```
 

data/lib/mws/orders/collection.rb

@@ -15,12 +15,12 @@ module MWS
         count.zero?
       end
 
+      def attributes
+        map(&:attributes)
+      end
+
       def inspect
-        "#<#{self.class} #{if count > 3
-                             "[#{take(3).map(&:inspect).join(', ')}...]"
-                           else
-                             "[#{map(&:inspect).join(', ')}]"
-                           end}>"
+        "#<#{self.class.name} #{to_a}>"
       end
 
       alias to_s inspect

data/lib/mws/orders/{service_status_message.rb → message.rb}

@@ -4,7 +4,7 @@ require 'mws/orders/entity'
 
 module MWS
   module Orders
-    class ServiceStatusMessage < Entity
+    class Message < Entity
       attribute(:locale) do
         text_at_xpath('Locale')
       end

data/lib/mws/orders/order.rb

@@ -112,6 +112,10 @@ module MWS
       attribute(:latest_delivered_at) do
         time_at_xpath('LatestDeliveryDate')
       end
+
+      def to_s
+        amazon_order_id
+      end
     end
   end
 end

data/lib/mws/orders/order_item.rb

@@ -105,6 +105,10 @@ module MWS
       attribute(:scheduled_delivery_starts_at) do
         time_at_xpath('ScheduledDeliveryStartDate')
       end
+
+      def to_s
+        order_item_id
+      end
     end
   end
 end

data/lib/mws/orders/parser.rb

@@ -4,6 +4,7 @@ require 'nokogiri'
 require 'mws/orders/orders'
 require 'mws/orders/order_items'
 require 'mws/orders/service_status'
+require 'peddler'
 
 module MWS
   module Orders
@@ -15,19 +16,12 @@ module MWS
       end
 
       def parse
-        node = find_result_node
-
-        case node.name
-        when /GetServiceStatus/
-          ServiceStatus.new(node)
-        when /GetOrder/
-          Orders.new(node).first
-        when /ListOrders/
-          Orders.new(node)
-        when /ListOrderItems/
-          OrderItems.new(node)
-        else
-          raise NotImplementedError
+        case result_node.name
+        when /GetOrder/         then order
+        when /GetServiceStatus/ then service_status
+        when /ListOrders/       then orders
+        when /ListOrderItems/   then order_items
+        else raise NotImplementedError
         end
       end
 
@@ -35,10 +29,38 @@ module MWS
         @response.headers
       end
 
+      def status_code
+        @response.status
+      end
+
+      def body
+        @response.body
+      end
+
       private
 
+      def order
+        orders.first
+      end
+
+      def orders
+        Orders.new(result_node)
+      end
+
+      def order_items
+        OrderItems.new(result_node)
+      end
+
+      def service_status
+        ServiceStatus.new(result_node)
+      end
+
+      def result_node
+        @result_node ||= find_result_node
+      end
+
       def find_result_node
-        xml = Nokogiri(@response.body)
+        xml = Nokogiri(body)
         root = xml.children.first
 
         root.children.find { |node| node.name.include?('Result') }

data/lib/mws/orders/service_status.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require 'mws/orders/entity'
-require 'mws/orders/service_status_message'
+require 'mws/orders/message'
 
 module MWS
   module Orders
@@ -19,7 +19,7 @@ module MWS
       end
 
       attribute(:messages) do
-        xpath('Messages/Message').map { |node| ServiceStatusMessage.new(node) }
+        xpath('Messages/Message').map { |node| Message.new(node) }
       end
     end
   end

data/lib/mws/orders/version.rb

@@ -2,6 +2,6 @@
 
 module MWS
   module Orders
-    VERSION = '0.3.1'
+    VERSION = '0.4.0'
   end
 end

data/test/mws/orders/test_entity.rb

@@ -30,10 +30,10 @@ class TestEntity < MiniTest::Test
     XML
     entity = build_entity(xml)
     money = entity.money_at_xpath('Price')
-    assert_equal '$10.00', money.format
+    assert_equal 1000, money.fractional
   end
 
-  def test_parses_japanese_yen
+  def test_parses_yen
     xml = <<-XML
     <Price xmlns="example">
       <CurrencyCode>JPY</CurrencyCode>
@@ -42,8 +42,7 @@ class TestEntity < MiniTest::Test
     XML
     entity = build_entity(xml)
     money = entity.money_at_xpath('Price')
-
-    assert_equal '¥1,000', money.format
+    assert_equal 1000, money.fractional
   end
 
   def test_parses_text

data/test/mws/orders/{test_service_status_message.rb → test_message.rb}

@@ -2,10 +2,10 @@
 
 require 'test_helper'
 
-class TestServiceStatusMessage < MiniTest::Test
+class TestMessage < MiniTest::Test
   def setup
-    node = load_xml_fixture('service_status').xpath('//xmlns:Message').first
-    @message = ServiceStatusMessage.new(node)
+    node = load_fixture('service_status').xpath('//xmlns:Message').first
+    @message = Message.new(node)
   end
 
   def test_locale

data/test/mws/orders/test_order.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TestOrder < MiniTest::Test
   def setup
-    node = load_xml_fixture('orders').at_xpath('//xmlns:Order')
+    node = load_fixture('orders').at_xpath('//xmlns:Order')
     @order = Order.new(node)
   end
 
@@ -92,4 +92,8 @@ class TestOrder < MiniTest::Test
   def test_latest_delivered_at
     assert_kind_of Time, @order.latest_delivered_at
   end
+
+  def test_to_s
+    assert_equal @order.amazon_order_id, @order.to_s
+  end
 end

data/test/mws/orders/test_order_item.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TestOrderItemTest < MiniTest::Test
   def setup
-    node = load_xml_fixture('order_items').at_xpath('//xmlns:OrderItem')
+    node = load_fixture('order_items').at_xpath('//xmlns:OrderItem')
     @order_item = OrderItem.new(node)
   end
 
@@ -67,4 +67,8 @@ class TestOrderItemTest < MiniTest::Test
   def test_gift_wrap_level
     assert_kind_of String, @order_item.gift_wrap_level
   end
+
+  def test_to_s
+    assert_equal @order_item.order_item_id, @order_item.to_s
+  end
 end

data/test/mws/orders/test_order_items.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TestOrderItems < MiniTest::Test
   def setup
-    node = load_xml_fixture('order_items').xpath('//xmlns:ListOrderItemsResult')
+    node = load_fixture('order_items').xpath('//xmlns:ListOrderItemsResult')
     @order_items = OrderItems.new(node)
   end
 

data/test/mws/orders/test_orders.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TestOrders < MiniTest::Test
   def setup
-    node = load_xml_fixture('orders').xpath('//xmlns:ListOrdersResult')
+    node = load_fixture('orders').xpath('//xmlns:ListOrdersResult')
     @orders = Orders.new(node)
   end
 

data/test/mws/orders/test_payment_execution_detail.rb

@@ -5,8 +5,8 @@ require 'test_helper'
 class TestPaymentExecutionDetail < MiniTest::Test
   def setup
     node =
-      load_xml_fixture('orders').xpath('//xmlns:PaymentExecutionDetail')
-                                .first
+      load_fixture('orders').xpath('//xmlns:PaymentExecutionDetail')
+                            .first
     @ped = PaymentExecutionDetail.new(node)
   end
 

data/test/mws/orders/test_payment_execution_detail_item.rb

@@ -5,8 +5,7 @@ require 'test_helper'
 class TestPaymentExecutionDetailItem < MiniTest::Test
   def setup
     node =
-      load_xml_fixture('orders').xpath('//xmlns:PaymentExecutionDetailItem')
-                                .first
+      load_fixture('orders').xpath('//xmlns:PaymentExecutionDetailItem').first
     @pedi = PaymentExecutionDetailItem.new(node)
   end
 

data/test/mws/orders/test_service_status.rb

@@ -5,8 +5,8 @@ require 'test_helper'
 class TestServiceStatus < MiniTest::Test
   def setup
     node =
-      load_xml_fixture('service_status').xpath('//xmlns:GetServiceStatusResult')
-                                        .first
+      load_fixture('service_status').xpath('//xmlns:GetServiceStatusResult')
+                                    .first
     @status = ServiceStatus.new(node)
   end
 

data/test/mws/orders/test_shipping_address.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TestShippingAddress < MiniTest::Test
   def setup
-    node = load_xml_fixture('orders').xpath('//xmlns:ShippingAddress').first
+    node = load_fixture('orders').xpath('//xmlns:ShippingAddress').first
     @address = ShippingAddress.new(node)
   end
 

data/test/test_helper.rb

@@ -1,12 +1,7 @@
 # frozen_string_literal: true
 
 require 'minitest/autorun'
-require 'mws-orders'
-
-begin
-  require 'pry'
-rescue LoadError
-end
+require 'mws/orders/parser'
 
 I18n.enforce_available_locales = false
 
@@ -14,7 +9,7 @@ module MiniTest
   class Test
     include MWS::Orders
 
-    def load_xml_fixture(type)
+    def load_fixture(type)
       file = File.expand_path("../fixtures/#{type}.xml", __FILE__)
       body = File.read(file)
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mws-orders
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Hakan Ensari
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-01-17 00:00:00.000000000 Z
+date: 2019-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: money
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: peddler
   requirement: !ruby/object:Gem::Requirement
@@ -58,28 +58,70 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
-description: A rich Ruby interface to the Amazon MWS Orders API
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: 
 email:
-- hakanensari@gmail.com
+- me@hakanensari.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
 - LICENSE
 - README.md
-- lib/mws-orders.rb
 - lib/mws/orders/collection.rb
 - lib/mws/orders/document.rb
 - lib/mws/orders/entity.rb
 - lib/mws/orders/invoice_data.rb
+- lib/mws/orders/message.rb
 - lib/mws/orders/order.rb
 - lib/mws/orders/order_item.rb
 - lib/mws/orders/order_items.rb
@@ -88,7 +130,6 @@ files:
 - lib/mws/orders/payment_execution_detail.rb
 - lib/mws/orders/payment_execution_detail_item.rb
 - lib/mws/orders/service_status.rb
-- lib/mws/orders/service_status_message.rb
 - lib/mws/orders/shipping_address.rb
 - lib/mws/orders/tokenable.rb
 - lib/mws/orders/version.rb
@@ -96,6 +137,7 @@ files:
 - test/fixtures/orders.xml
 - test/fixtures/service_status.xml
 - test/mws/orders/test_entity.rb
+- test/mws/orders/test_message.rb
 - test/mws/orders/test_order.rb
 - test/mws/orders/test_order_item.rb
 - test/mws/orders/test_order_items.rb
@@ -104,7 +146,6 @@ files:
 - test/mws/orders/test_payment_execution_detail.rb
 - test/mws/orders/test_payment_execution_detail_item.rb
 - test/mws/orders/test_service_status.rb
-- test/mws/orders/test_service_status_message.rb
 - test/mws/orders/test_shipping_address.rb
 - test/mws/orders/test_tokenable.rb
 - test/test_helper.rb
@@ -120,25 +161,24 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.3'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
-summary: Wraps the Amazon MWS Orders API
+summary: A Ruby wrapper for the Amazon MWS Orders API
 test_files:
 - test/mws/orders/test_orders.rb
-- test/mws/orders/test_service_status_message.rb
 - test/mws/orders/test_order.rb
 - test/mws/orders/test_service_status.rb
 - test/mws/orders/test_shipping_address.rb
 - test/mws/orders/test_tokenable.rb
+- test/mws/orders/test_message.rb
 - test/mws/orders/test_payment_execution_detail_item.rb
 - test/mws/orders/test_order_items.rb
 - test/mws/orders/test_entity.rb

data/lib/mws-orders.rb

@@ -1,4 +0,0 @@
-# frozen_string_literal: true
-
-require 'peddler'
-require 'mws/orders/parser'