mercado-libre 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a5ef7b2444e036627b80fd4eec2e5378efac4ae0
-  data.tar.gz: 5c9632e9bdd341222ef09539cc12677dbac65249
+  metadata.gz: c7398d4a98adc3c4d26b5e2d16e1599e5314147e
+  data.tar.gz: b4f3da7ab182d0112d98990a98b6672d2854cb18
 SHA512:
-  metadata.gz: 3a9283f80c30e5274d430888bf67bcd30e7895ef055f74be8939374a259fd635e10e1bc3d742441a64ea0555fa3f9ae5eb6cc442844026dc2e27db3c8939c140
-  data.tar.gz: a9f97d6989e321607d4f48fd23db70e79535d009d435b1a85160152ced0567b51dd74cc2c032f71a64ddff51324e934c4f8001e7107d732945fbb6ee54a437f7
+  metadata.gz: 5632db2a3b336c971b876e59b3f7a043d684e0976f5e08e286ef09d3fb8831959353e34649797fca667b5745fb09296ccaea4c93369beb6528c1d6e9b32b8227
+  data.tar.gz: 3975e6d9097e6ca7e374cb789760041358022bc0311445cc10a76d40ab414687249cde1ed4a876fbda107580cde73a5c5f599753f04fa62cc21277e99f34027e

data/lib/mercado_libre/api.rb

@@ -5,9 +5,10 @@ module MercadoLibre
     attr_accessor :access_token
 
     def initialize(params)
-      @app_key    = params.fetch(:app_key)
-      @app_secret = params.fetch(:app_secret)
-      @host       = params.fetch(:host) { ENV.fetch('MERCADOLIBRE_API_HOST') }
+      @app_key      = params.fetch(:app_key)
+      @app_secret   = params.fetch(:app_secret)
+      @host         = params.fetch(:host) { ENV.fetch('MERCADOLIBRE_API_HOST') }
+      @retries_num  = Integer( params.fetch(:retries) { ENV.fetch('MERCADOLIBRE_HTTP_RETRIES') { 10 } } )
     end
 
     def publish_item(item)
@@ -24,11 +25,26 @@ module MercadoLibre
       Item.new(response)
     end
 
+    def pause_item(item_id)
+      url = "/items/#{item_id}"
+      authenticated_request(:put, url, {status: :paused}.to_json)
+    end
+
+    def unpause_item(item_id)
+      url = "/items/#{item_id}"
+      authenticated_request(:put, url, {status: :active}.to_json)
+    end
+
     def close_item(item_id)
       url = "/items/#{item_id}"
       authenticated_request(:put, url, {status: :closed}.to_json)
     end
-    
+   
+    def delete_item(item_id)
+      url = "/items/#{item_id}"
+      authenticated_request(:put, url, {deleted: true}.to_json)
+    end
+
     def relist_item(item_id, params)
       url = "/items/#{item_id}/relist"
       payload = {
@@ -49,31 +65,25 @@ module MercadoLibre
     end
 
     def authenticated_request(verb, url, data = {})
-      new_url = url_with_token(url)
-      begin
-        request(verb, new_url, data)
-      rescue Requests::Error
-        @retries ||=0
-        @retries +=1
-        if $!.message.strip == 'Forbidden'
-          @access_token = nil
-          authenticated_request(verb, url, data) if @retries < 10
-        else
-          raise $!
-        end
-      end
+      request(verb, url, data, authenticated: true)
     end
 
-    def request(verb, url, data)
-      url = "#{@host}#{url}"
+    def request(verb, url, data, authenticated: false)
+      full_url = "#{@host}#{url}"
+      full_url = url_with_token(full_url) if authenticated
       verb = verb.to_s.upcase
-      begin
-        response = Requests.request(verb, url, data: data)
-      rescue Requests::Error => e
-        new_message = [e.message.strip,e.response.body].join("\n")
-        raise $!, new_message, $!.backtrace
-      end
+      tries ||= @retries_num
+      response = Requests.request(verb, full_url, data: data)
+
       JSON.parse(response.body)
+
+    rescue *HTTPErrors
+      retry unless (tries -=1).zero?
+      raise
+    rescue Requests::Error
+      @access_token = nil if $!.message.strip == 'Forbidden' # Release access token
+      retry unless (tries -=1).zero?
+      raise $!, [$!.message.strip,$!.response.body].join("\n"), $!.backtrace #Verbose errors
     end
 
     def access_token
@@ -92,13 +102,13 @@ module MercadoLibre
     end
 
     def url_with_token(url)
-      require 'uri'
-      uri = URI(url)
-      params = URI.decode_www_form(uri.query || '')
-      params << ['access_token', access_token]
-      uri.query = URI.encode_www_form(params).to_s
-
-      uri.to_s
+      "#{url}#{url.include?('?') ? '&' : '?'}access_token=#{access_token}"
     end
   end
+end
+
+module MercadoLibre
+  HTTPErrors = [Timeout::Error, Errno::ETIMEDOUT, Errno::EINVAL, Errno::ECONNRESET,
+    Errno::ECONNREFUSED, EOFError, Net::HTTPBadResponse,
+    Net::HTTPHeaderSyntaxError, Net::ProtocolError]
 end

data/lib/mercado_libre/version.rb

@@ -1,3 +1,3 @@
 module MercadoLibre
-  VERSION = '0.0.4'
+  VERSION = '0.0.5'
 end

data/mercado-libre.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name              = "mercado-libre"
-  s.version           = "0.0.4"
+  s.version           = "0.0.5"
   s.summary           = "MercadoLibre API"
   s.description       = "mercado-libre"
   s.authors           = ["CarlosIPe"]

data/test/mercadolibre.rb

@@ -68,6 +68,21 @@ test "publish item returns item with permalink" do
   assert(published_item.permalink =~ /http/)
 end
 
+test "pauses item and status is paused" do
+  published_item = api.publish_item(item)
+  sleep(10) if ENV['TESTING_AGAINST_ML_SERVER']
+  api.pause_item(published_item.id)
+  assert_equal(api.get_item(published_item.id).status, 'paused')
+end
+
+test "unpauses item and status is active" do
+  published_item = api.publish_item(item)
+  sleep(10) if ENV['TESTING_AGAINST_ML_SERVER']
+  api.pause_item(published_item.id)
+  api.unpause_item(published_item.id)
+  assert_equal(api.get_item(published_item.id).status, 'active')
+end
+
 test "closes item and status is closed" do
   published_item = api.publish_item(item)
   sleep(10) if ENV['TESTING_AGAINST_ML_SERVER']

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mercado-libre
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - CarlosIPe