Additions from Blurb

.gitignore

@@ -5,4 +5,6 @@ pkg/*
 .DS_Store
 spec/credentials.yml
 spec/fixtures/ephemeral_response/*
-.rvmrc
+.rvmrc
+.idea
+.idea/*

Gemfile

@@ -12,4 +12,4 @@ gem 'rash'
 gem 'rspec'
 gem 'ephemeral_response'
 gem 'awesome_print'
-gem 'pry'
+gem 'pry'

README.markdown

@@ -14,6 +14,10 @@ Quick Start
 
 To quickly test your connection to the service without credentials, you can ping the server, which returns server time in UTC:
 
+Start console(You have to be on the ruby-mws path)
+
+    rake console
+
     MWS::Base.server_time
 
 ### Initialization
@@ -114,4 +118,22 @@ This object can be used to access all API services. Below are examples on how to
 
 * GetReport - Used to request a report by report ID. All reports are currently returned as a flat file string.
 
-    `@mws.reports.get_report :report_id => '11223344'`
+    `@mws.reports.get_report :report_id => '11223344'`
+
+Testing
+-------
+
+Add the file `spec/credentials.yml` that looks like
+
+```
+aws_access_key_id: '[access key]'
+secret_access_key: '[secret access key]'
+seller_id: '[seller id]'
+marketplace_id: '[marketplace id]'
+```
+
+Then run `bundle exec rspec spec/`
+
+To use binding.pry in a spec, be sure to require it with `require 'pry'`.
+
+There's still work and research to be done on how to integrate our changes in with the author's. Because we're using different credentials than the author and there are hardcoded order ids in the specs we'll never have all the specs passing for all of us.

Rakefile

@@ -1 +1,6 @@
 require "bundler/gem_tasks"
+
+desc "Open an irb session preloaded with this library"
+task :console do
+  sh "irb -rubygems -I lib -r ruby-mws.rb"
+end

lib/ruby-mws.rb

@@ -37,11 +37,11 @@ def symbolize_keys
 
 class String
 
-  def camelize(first_letter_in_uppercase = true)
+  def ruby_mws_camelize(first_letter_in_uppercase = true)
     if first_letter_in_uppercase
       self.to_s.gsub(/\/(.?)/) { "::#{$1.upcase}" }.gsub(/(?:^|_)(.)/) { $1.upcase }
     else
-      self.to_s[0].chr.downcase + camelize(lower_case_and_underscored_word)[1..-1]
+      self.to_s[0].chr.downcase + ruby_mws_camelize(lower_case_and_underscored_word)[1..-1]
     end
   end
 end
@@ -56,7 +56,9 @@ def camelize(first_letter_in_uppercase = true)
 require 'ruby-mws/api/base'
 require 'ruby-mws/api/inventory'
 require 'ruby-mws/api/order'
+require 'ruby-mws/api/product'
+require 'ruby-mws/api/feed'
 require 'ruby-mws/api/report'
 require 'ruby-mws/api/query'
 require 'ruby-mws/api/response'
-require 'ruby-mws/api/binary_response'
+require 'ruby-mws/api/binary_response'

lib/ruby-mws/api/base.rb

@@ -37,11 +37,13 @@ def send_request(name, params, options={})
         params = [default_params(name), params, options, @connection.to_hash].inject :merge
 
         params[:lists] ||= {}
-        params[:lists][:marketplace_id] = "MarketplaceId.Id"
 
         query = Query.new params
         resp = self.class.send(params[:verb], query.request_uri)
 
+        @connection.call_communication_callbacks(:params => query.safe_params,
+                                                 :response => resp)
+
         @response = Response.parse resp, name, params
 
         if @response.respond_to?(:next_token) and @next[:token] = @response.next_token  # modifying, not comparing
@@ -52,7 +54,7 @@ def send_request(name, params, options={})
 
       def default_params(name)
         {
-          :action            => name.to_s.camelize,
+          :action            => name.to_s.ruby_mws_camelize,
           :signature_method  => 'HmacSHA256',
           :signature_version => '2',
           :timestamp         => Time.now.iso8601,
@@ -76,4 +78,4 @@ def inspect
     end
 
   end
-end
+end