amazon_seller_central 0.2.0 → 0.2.1

data/README.md

@@ -1,7 +1,7 @@
 # amazon\_seller\_central
 
-This gem is intended to wrap Amazon's SellerCentral pages with a Ruby
-API. Currently this gem supports accessing buyer feedback, accessing
+This gem wraps Amazon's SellerCentral pages with a Ruby API.
+Currently this gem supports accessing buyer feedback, accessing
 current inventory listings, and simple updates to those listings.
 
 ## Setup

data/VERSION

@@ -1 +1 @@
-0.2.0
+0.2.1

data/amazon_seller_central.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{amazon_seller_central}
-  s.version = "0.2.0"
+  s.version = "0.2.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["optoro"]
-  s.date = %q{2011-09-01}
+  s.date = %q{2011-09-08}
   s.description = %q{This gem is intended to wrap Amazon's SellerCentral pages with a Ruby API. Currently this gem supports accessing buyer feedback only.}
   s.email = %q{dev@optoro.com}
   s.extra_rdoc_files = [

data/lib/amazon_seller_central.rb

@@ -18,6 +18,6 @@ module AmazonSellerCentral
   end
 
   def self.mechanizer
-    AmazonSellerCentral::Mechanizer.instance
+    AmazonSellerCentral::Mechanizer.new
   end
 end

data/lib/amazon_seller_central/feedback_page.rb

@@ -9,8 +9,8 @@ module AmazonSellerCentral
     def next_page
       @next_page ||= begin
                        raise NoNextPageAvailableError unless has_next?
-                       page = AmazonSellerCentral.mechanizer.follow_link_with(:text => 'Next')
-                       FeedbackPage.new(:page => page)
+                       page = @agent.follow_link_with(:text => 'Next')
+                       FeedbackPage.new(:page => page, :agent => @agent)
                      end
     end
 
@@ -25,10 +25,11 @@ module AmazonSellerCentral
     module ClassMethods
 
       def load_first_page
-        AmazonSellerCentral.mechanizer.login_to_seller_central
-        feedback_home = AmazonSellerCentral.mechanizer.follow_link_with(:text => "Feedback")
-        feedback_page = AmazonSellerCentral.mechanizer.follow_link_with(:text => "View all your feedback")
-        FeedbackPage.new(:page => feedback_page)
+        mech = AmazonSellerCentral.mechanizer
+        mech.login_to_seller_central
+        feedback_home = mech.follow_link_with(:text => "Feedback")
+        feedback_page = mech.follow_link_with(:text => "View all your feedback")
+        FeedbackPage.new(:page => feedback_page, :agent => mech)
       end
 
       def load_all_pages

data/lib/amazon_seller_central/inventory.rb

@@ -2,9 +2,10 @@ module AmazonSellerCentral
   class Inventory
     module ClassMethods
       def load_first_page
-        AmazonSellerCentral.mechanizer.login_to_seller_central
-        manage_inventory = AmazonSellerCentral.mechanizer.follow_link_with(:text => "Manage Inventory")
-        InventoryPage.new( :page => manage_inventory, :page_no => 1, :uri_base => manage_inventory.uri.to_s )
+        mech = AmazonSellerCentral.mechanizer
+        mech.login_to_seller_central
+        manage_inventory = mech.follow_link_with(:text => "Manage Inventory")
+        InventoryPage.new( :page => manage_inventory, :page_no => 1, :uri_base => manage_inventory.uri.to_s, :agent => mech )
       end
 
       def load_all_pages

data/lib/amazon_seller_central/inventory_page.rb

@@ -15,11 +15,12 @@ module AmazonSellerCentral
       @next_page ||= begin
                        raise NoNextPageAvailableError unless has_next?
 
-                       next_page = AmazonSellerCentral.mechanizer.agent.get("#{@uri_base}&searchPageOffset=#{@page_no + 1}")
+                       next_page = @agent.agent.get("#{@uri_base}&searchPageOffset=#{@page_no + 1}")
                        InventoryPage.new(
                          :page => next_page,
                          :page_no => (@page_no + 1),
-                         :uri_base => @uri_base
+                         :uri_base => @uri_base,
+                         :agent => @agent
                        )
                      end
     end

data/lib/amazon_seller_central/mechanizer.rb

@@ -1,8 +1,6 @@
 require 'mechanize'
-require 'singleton'
 module AmazonSellerCentral
   class Mechanizer
-    include Singleton
     MASQUERADE_AGENTS = ['Mac Safari', 'Mac FireFox', 'Linux Firefox']
 
     attr_reader :agent

data/lib/amazon_seller_central/page.rb

@@ -3,8 +3,9 @@ module AmazonSellerCentral
     attr_accessor :body
 
     def initialize(options={})
-      @page = options.delete(:page)
-      @body = @page ? @page.body : ""
+      @page  = options.delete(:page)
+      @agent = options.delete(:agent)
+      @body  = @page ? @page.body : ""
     end
 
     def has_next?

data/spec/amazon_seller_central_spec.rb

@@ -26,7 +26,7 @@ describe "AmazonSellerCentral" do
     AmazonSellerCentral.configuration.login_password.should == @password
   end
 
-  it "allows access to a singleton mechanizer" do
+  it "allows access to a mechanizer" do
     mech = AmazonSellerCentral.mechanizer
     mech.should be_instance_of(AmazonSellerCentral::Mechanizer)
   end

data/spec/lib/feedback_page_spec.rb

@@ -2,7 +2,6 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 describe "FeedbackPage" do
   before :all do
-    #AmazonSellerCentral.mechanizer.reset!
     @first_page_test_regex  = FEEDBACK_FIRST_PAGE_TEST_REGEX
     @second_page_test_regex = FEEDBACK_SECOND_PAGE_TEST_REGEX
     @last_page_test_regex   = FEEDBACK_LAST_PAGE_TEST_REGEX
@@ -22,14 +21,12 @@ describe "FeedbackPage" do
 
   describe "ClassMethods" do
     it "loads the first page of feedback data" do
-      AmazonSellerCentral.mechanizer.reset!
       page = AmazonSellerCentral::FeedbackPage.load_first_page
       page.should be_kind_of(AmazonSellerCentral::FeedbackPage)
       page.body.should =~ @first_page_test_regex
     end
 
     it "loads all feedback pages" do
-      AmazonSellerCentral.mechanizer.reset!
       pages = AmazonSellerCentral::FeedbackPage.load_all_pages
       pages.size.should == 3
       pages.first.body.should =~ @first_page_test_regex

data/spec/lib/inventory_page_spec.rb

@@ -87,7 +87,7 @@ describe "InventoryPage" do
 
     @first_page.apply_listings(listings).should be_true
 
-    (AmazonSellerCentral.mechanizer.last_page.parser.css('div#msg_saveSuccess')[0]['style'] !~ /display: none/).should be_true
+    (@first_page.instance_variable_get('@agent').last_page.parser.css('div#msg_saveSuccess')[0]['style'] !~ /display: none/).should be_true
 
     FakeWeb.register_uri(:get, 'https://sellercentral.amazon.com/gp/ezdpc-gui/inventory-status/status.html/ref=ag_invmgr_mmap_home', :response => mock_pages[:update_inventory_result_from_page_1])
     listing = AmazonSellerCentral::Inventory.load_first_page.listings[0]

data/spec/lib/mechanizer_spec.rb

@@ -30,7 +30,7 @@ describe "Mechanizer" do
 
     mech = AmazonSellerCentral.mechanizer
     mech.login_to_seller_central
-    AmazonSellerCentral.mechanizer.reset!
+    mech.reset!
     lambda {
       mech.follow_link_with(:text => "Feedback")
     }.should raise_exception(AmazonSellerCentral::Mechanizer::AgentResetError)

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: amazon_seller_central
 version: !ruby/object:Gem::Version 
-  hash: 23
+  hash: 21
   prerelease: 
   segments: 
   - 0
   - 2
-  - 0
-  version: 0.2.0
+  - 1
+  version: 0.2.1
 platform: ruby
 authors: 
 - optoro
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-09-01 00:00:00 -04:00
+date: 2011-09-08 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency