mercadopago-rb 0.1.7 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8548490d8d225a618f65bda7bd8e9dfd651bead1
-  data.tar.gz: c50298287d02294b84bbad3434f3623f266f2b9f
+  metadata.gz: dab77349ceb162c112466e1edc44ca0e882eb81c
+  data.tar.gz: 971c2b88b0796ff50ff6f65fd3e3aab17818df22
 SHA512:
-  metadata.gz: 0b5ec17109934e9cb184bba01c298edbeedc1cfb21223fd22b9ed13b0f486fd5edbd28c259e8c0a49cd8000837212aa077df5715be89e2f2e0be1fd2af514ed3
-  data.tar.gz: 8fff268f191d919bcc7d36995f119862aadfb25295fa04cc4ae67d0ca906c3f331972f7ca2b3686530143cee788ab87ec2252d072e0a24810408b9a2e4c85acf
+  metadata.gz: 477f98911cd8c4cb07eb9497c703cac41c00293c69513747af1aec78bd81512657530fad45f4227d1eb0f0861e9776e527d4b95c7e316bd94b34e96afaa925e4
+  data.tar.gz: e35f6265cd7719ebc114757a0d7cdfb6c4f545b4505b03b93a28f09d1a2c0e6a04715fd4185f386064542b0e5060b33fedf21a87f50ec2831a91ca95f6ecb3fd

data/.travis.yml

@@ -4,9 +4,3 @@ rvm:
   - 2.1.10
   - 2.2.5
   - 2.3.1
-
-deploy:
-  provider: rubygems
-  api_key: "Trgb2IN5vfA9XLej/l7ywrY0JtrL1s/UeofMXrpLgF3Or/u0Xa0IjHifXGiaLwGLX1OuxspSZMvjyhs7b3ntSgezquW7o+en/DDaGptt45HO7JLvXIkyYqPGzagBoGRpUhZ61CZyDvK+GEqme0euJgcrvxXlz4o8OD7ySBUiJ7oWoG2BoPNfx8Z6yffJF8DB0qjw575Y8kWOafLqn9VPdKEKBcFP93PulmStGDe10Bff9k0Rpy3+HfGWMxz4uWEYRgO97UrzGoR/0XVtACVEJB4jlEUe72FnEFpiuM2mcBSH/oM9ycyC9b9tMpzqT2btmKlavdE+YEoEXpBwQee6XcDbSvrHV6eKKlg1+ZYxUcyQUDJ0It5aeFxCyS4xToxhBtP/Ch0RmaeLstntbZEu6Ba8nqyV3uzYAZ0u1SW/G4yjSH5wNYwsWj4fCmAMO1qcI2+11zrA4mVrZVs8tbIl0Bxni+gl9DGKtcqINJ6+Ue7i2m2pLmdnhD3YPjf3sCUVzpB3msmiIIBkAFXxS2yg4a4f0NKigpgWicKgfZa6ge9HJryacsha4x7VCh5BrJ7PD5AavBdNX0yUXU4GI9Qiw2AscLedJY57imLkfPIUV5bmbMwv5sIVI1/HG/wmGYe0uyxKd/hmxD7VLnG7Jxz6pcWHtp3ii2x3+gORKaAOC84="
-  gem: mercadopago
-  gemspec: mercadopago.gemspec

data/Gemfile.lock

@@ -1,12 +1,18 @@
 PATH
   remote: .
   specs:
-    mercadopago-rb (0.1.0)
+    mercadopago-rb (0.1.7)
 
 GEM
   remote: https://rubygems.org/
   specs:
+    addressable (2.5.0)
+      public_suffix (~> 2.0, >= 2.0.2)
+    crack (0.4.3)
+      safe_yaml (~> 1.0.0)
     diff-lcs (1.2.5)
+    hashdiff (0.3.1)
+    public_suffix (2.0.4)
     rake (10.5.0)
     rspec (3.4.0)
       rspec-core (~> 3.4.0)
@@ -21,6 +27,11 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.4.0)
     rspec-support (3.4.1)
+    safe_yaml (1.0.4)
+    webmock (2.3.1)
+      addressable (>= 2.3.6)
+      crack (>= 0.3.2)
+      hashdiff
 
 PLATFORMS
   ruby
@@ -30,6 +41,7 @@ DEPENDENCIES
   mercadopago-rb!
   rake (~> 10.0)
   rspec (~> 3.0)
+  webmock (~> 2.3.1)
 
 BUNDLED WITH
-   1.11.2
+   1.13.6

data/lib/mercadopago/client.rb

@@ -11,7 +11,6 @@ module Mercadopago
     def_delegators :@rest_client, :access_token, :public_key, :get, :post,
       :put, :delete
 
-
     def initialize(opts = {})
       @rest_client = RestClient.new(opts)
     end
@@ -26,6 +25,14 @@ module Mercadopago
       call_endpoint(Endpoint::PaymentMethods, method, data)
     end
 
+    # This method provides interaction with preference method's API endpoint
+    def preference(method, data = {})
+      call_endpoint(Endpoint::Preference, method, data)
+    end
+
+    def card_token(method, data = {})
+      call_endpoint(Endpoint::CardToken, method, data)
+    end
 
     private
     def call_endpoint(klazz, method, data)

data/lib/mercadopago/config.rb

@@ -4,7 +4,6 @@ module Mercadopago
   module Config
     extend self
     attr_accessor :access_token, :public_key
-    attr_reader :secrets
 
     def access_token
       @access_token ||= secrets('access_token') ||

data/lib/mercadopago/endpoint.rb

@@ -1,3 +1,5 @@
 require 'mercadopago/endpoint/base'
 require 'mercadopago/endpoint/payment_methods'
 require 'mercadopago/endpoint/payment'
+require 'mercadopago/endpoint/preference'
+require 'mercadopago/endpoint/card_token'

data/lib/mercadopago/endpoint/base.rb

@@ -17,7 +17,6 @@ module Mercadopago
         id = data[:id] || raise(StandardError.new('Unknown ID'))
         "#{self.class::ENDPOINT}/#{id}"
       end
-
     end
   end
 end

data/lib/mercadopago/endpoint/card_token.rb

@@ -0,0 +1,19 @@
+module Mercadopago
+  module Endpoint
+    class CardToken < Base
+      ENDPOINT = "/v1/card_tokens".freeze
+
+      def get
+        rest_client.get(member_endpoint)
+      end
+
+      def create
+        rest_client.post(ENDPOINT, data)
+      end
+
+      def update
+        rest_client.put(ENDPOINT, data)
+      end
+    end
+  end
+end

data/lib/mercadopago/endpoint/payment.rb

@@ -1,7 +1,7 @@
 module Mercadopago
   module Endpoint
     class Payment < Base
-      ENDPOINT = "/payments".freeze
+      ENDPOINT = "/v1/payments".freeze
 
       def get
         endpoint = data[:id] ? member_endpoint : ENDPOINT
@@ -23,7 +23,6 @@ module Mercadopago
       def search
         rest_client.get(ENDPOINT + "/search", data[:filters])
       end
-
     end
   end
 end

data/lib/mercadopago/endpoint/payment_methods.rb

@@ -1,12 +1,11 @@
 module Mercadopago
   module Endpoint
     class PaymentMethods < Base
-      ENDPOINT = "/payment_methods".freeze
+      ENDPOINT = "/v1/payment_methods".freeze
 
       def get
         rest_client.get(ENDPOINT)
       end
-
     end
   end
 end

data/lib/mercadopago/endpoint/preference.rb

@@ -0,0 +1,19 @@
+module Mercadopago
+  module Endpoint
+    class Preference < Base
+      ENDPOINT = "/checkout/preferences".freeze
+
+      def get
+        rest_client.get(member_endpoint)
+      end
+
+      def create
+        rest_client.post(ENDPOINT, data)
+      end
+
+      def update
+        rest_client.update(ENDPOINT, data)
+      end
+    end
+  end
+end

data/lib/mercadopago/response.rb

@@ -1,6 +1,6 @@
 module Mercadopago
   class Response
-    attr_reader :status, :response, :hash
+    attr_reader :status, :response, :content
 
     def initialize(status, response)
       @status = status
@@ -8,11 +8,11 @@ module Mercadopago
     end
 
     def [](attr)
-      hash[attr]
+      content[attr]
     end
 
     def to_h
-      hash
+      content
     end
 
     def method_missing(name, *args, &block)
@@ -27,8 +27,8 @@ module Mercadopago
       response.has_key?(name.to_sym) || response.has_key?(name.to_s) || super
     end
 
-    def hash
-      @hash ||= {status: status, response: response}
+    def content
+      @content ||= {status: status, response: response}
     end
   end
 end

data/lib/mercadopago/rest_client.rb

@@ -7,17 +7,12 @@ require 'mercadopago/response'
 module Mercadopago
   class RestClient
     attr_accessor :http, :api_base_uri
-    attr_reader :access_token, :public_key, :client_secret, :query
+    attr_reader :access_token, :public_key, :client_secret
 
     MIME_JSON = 'application/json'.freeze
     MIME_FORM = 'application/x-www-form-urlencoded'.freeze
     API_BASE_URL = 'https://api.mercadopago.com'.freeze
     MERCADO_PAGO_VERSION = '0.3.4'.freeze
-    API_VERSION = '/v1'.freeze
-    GET = "GET".freeze
-    POST = "POST".freeze
-    PUT = "PUT".freeze
-    DELETE = "DELETE".freeze
 
     def initialize(opts = {})
       @access_token = opts[:access_token] || Config.access_token
@@ -25,27 +20,27 @@ module Mercadopago
       @http = set_http
     end
 
-    def generate_uri(uri, params = {})
+    def request_uri(path, params = {})
       params['access_token'] = access_token
-      @query ||= (uri.include?("?") ? "&" : "?") +
-        URI.encode_www_form(params)
-      API_VERSION + uri + @query
+      uri = api_base_uri + path
+      uri.query = [uri.query, URI.encode_www_form(params)].compact.join('&')
+      uri.request_uri
     end
 
     def get(uri, params = {}, content_type = MIME_JSON)
-      request(GET, uri, params, content_type)
+      request(Net::HTTP::Get::METHOD, uri, params, content_type)
     end
 
     def post(uri, data, params = {}, content_type = MIME_JSON)
-      request(POST, uri, params, content_type, data)
+      request(Net::HTTP::Post::METHOD, uri, params, content_type, data)
     end
 
     def put(uri, data = nil, params = {}, content_type = MIME_JSON)
-      request(PUT, uri, params, content_type, data)
+      request(Net::HTTP::Put::METHOD, uri, params, content_type, data)
     end
 
     def delete(uri, params = {}, content_type = MIME_JSON)
-      request(DELETE, uri, params, content_type)
+      request(Net::HTTP::Delete::METHOD, uri, params, content_type)
     end
 
     def api_base_uri
@@ -53,11 +48,10 @@ module Mercadopago
     end
 
     def request(method, uri, params, content_type, data = {})
-      api_result = http.send_request(method, generate_uri(uri, params), data.to_json, headers(content_type))
+      api_result = http.send_request(method, request_uri(uri, params), data.to_json, headers(content_type))
       Response.new(api_result.code, JSON.parse(api_result.body))
     end
 
-    private
     def headers(content_type)
       {
         'User-Agent' => "MercadoPago Ruby SDK v" + MERCADO_PAGO_VERSION,
@@ -66,6 +60,7 @@ module Mercadopago
       }
     end
 
+    private
     def set_http
       http = Net::HTTP.new(api_base_uri.host, api_base_uri.port)
       if api_base_uri.scheme == "https"

data/lib/mercadopago/version.rb

@@ -1,3 +1,3 @@
 module Mercadopago
-  VERSION = "0.1.7"
+  VERSION = "0.2.0"
 end

data/mercadopago.gemspec

@@ -23,4 +23,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.11"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency "webmock", "~> 2.3.1"
 end

data/spec/mercadopago_client/card_token_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe "Mercadopago::Client" do
+  let(:client) { Mercadopago.client }
+  let(:api_endpoint) { :some_invalid_endpoint }
+  before(:each) do
+    Mercadopago.configure do |config|
+      config.access_token = 'foo_bar_access_token'
+      config.public_key = 'foo_bar_public_key'
+    end
+  end
+
+  describe 'card_token' do
+    context 'when API endpoint is invalid' do
+      it { expect{ client.card_token(:api_endpoint)}.to raise_error(Mercadopago::Errors::Endpoint)}
+    end
+  end
+end

data/spec/{payment_spec.rb → mercadopago_client/payment_spec.rb}

@@ -14,6 +14,5 @@ describe "Mercadopago::Client" do
     context 'when API endpoint is invalid' do
       it { expect{ client.payment(:api_endpoint)}.to raise_error(Mercadopago::Errors::Endpoint)}
     end
-
   end
 end

data/spec/mercadopago_client/preference_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe "Mercadopago::Client" do
+  let(:client) { Mercadopago.client }
+  let(:api_endpoint) { :some_invalid_endpoint }
+  before(:each) do
+    Mercadopago.configure do |config|
+      config.access_token = 'foo_bar_access_token'
+      config.public_key = 'foo_bar_public_key'
+    end
+  end
+
+  describe 'preference' do
+    context 'when API endpoint is invalid' do
+      it { expect{ client.preference(:api_endpoint)}.to raise_error(Mercadopago::Errors::Endpoint)}
+    end
+  end
+end

data/spec/mercadopago_spec.rb

@@ -4,6 +4,4 @@ describe Mercadopago do
   it 'has a version number' do
     expect(Mercadopago::VERSION).not_to be nil
   end
-
-  
 end

data/spec/rest_client_spec.rb

@@ -2,9 +2,9 @@ require 'spec_helper'
 
 describe "Mercadopago::RestClient" do
   let(:rest_client) { Mercadopago.client.rest_client }
-  let(:status_ok) { "200" }
-  let(:status_error) { "400" }
-  let(:empty_string_hash) { "{}" }
+  let(:endpoint) { "/v1/payments" }
+  let(:url) { MERCADOPAGO_API + rest_client.request_uri(endpoint) }
+  let(:headers) { rest_client.headers(Mercadopago::RestClient::MIME_JSON) }
 
   before(:each) do
     Mercadopago.configure do |config|
@@ -15,25 +15,21 @@ describe "Mercadopago::RestClient" do
 
   describe 'rest_client' do
     context 'when API endpoint\'s call response is ok' do
-      before do
-        http_stub = double
-        allow(http_stub).to receive(:code).and_return(status_ok)
-        allow(http_stub).to receive(:body).and_return(empty_string_hash)
-        allow(rest_client.http).to receive(:send_request) do |arg1, arg2, arg3, arg4|
-        end.and_return(http_stub)
+      it do
+        stub_request(:get, url)
+        .with( :body => "{}", :headers => headers )
+        .to_return(:status => 200, :body => "{}", :headers => {})
+        expect(rest_client.get(endpoint)[:status]).to eq("200")
       end
-      it { expect(rest_client.get("/v1/payments")[:status]).to eq(status_ok) }
     end
 
     context 'when API endpoint\'s call response is a bad request' do
-      before do
-        http_stub = double
-        allow(http_stub).to receive(:code).and_return(status_error)
-        allow(http_stub).to receive(:body).and_return(empty_string_hash)
-        allow(rest_client.http).to receive(:send_request) do |arg1, arg2, arg3, arg4|
-        end.and_return(http_stub)
+      it do
+        stub_request(:get, url)
+        .with( :body => "{}", :headers => headers )
+        .to_return(:status => 400, :body => "{}", :headers => {})
+        expect(rest_client.get(endpoint)[:status]).to eq("400")
       end
-      it { expect(rest_client.get("/v1/payments")[:status]).to eq(status_error) }
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,2 +1,5 @@
 $LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
 require 'mercadopago'
+require 'webmock/rspec'
+
+MERCADOPAGO_API = 'https://api.mercadopago.com'.freeze

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mercadopago-rb
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.2.0
 platform: ruby
 authors:
 - Federico Bonisconti
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-04 00:00:00.000000000 Z
+date: 2016-12-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -52,6 +52,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.3.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.3.1
 description: Object-oriented wrapper for mercadopago's API.
 email:
 - fedebonisconti26@gmail.com
@@ -77,8 +91,10 @@ files:
 - lib/mercadopago/config/locales/es.yml
 - lib/mercadopago/endpoint.rb
 - lib/mercadopago/endpoint/base.rb
+- lib/mercadopago/endpoint/card_token.rb
 - lib/mercadopago/endpoint/payment.rb
 - lib/mercadopago/endpoint/payment_methods.rb
+- lib/mercadopago/endpoint/preference.rb
 - lib/mercadopago/errors.rb
 - lib/mercadopago/errors/bad_request.rb
 - lib/mercadopago/errors/credential_not_found.rb
@@ -89,9 +105,11 @@ files:
 - lib/mercadopago/version.rb
 - mercadopago.gemspec
 - spec/config/config_spec.rb
+- spec/mercadopago_client/card_token_spec.rb
+- spec/mercadopago_client/payment_methods_spec.rb
+- spec/mercadopago_client/payment_spec.rb
+- spec/mercadopago_client/preference_spec.rb
 - spec/mercadopago_spec.rb
-- spec/payment_methods_spec.rb
-- spec/payment_spec.rb
 - spec/response_spec.rb
 - spec/rest_client_spec.rb
 - spec/spec_helper.rb
@@ -119,5 +137,13 @@ rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Object-oriented wrapper for mercadopago's API.
-test_files: []
-has_rdoc: 
+test_files:
+- spec/config/config_spec.rb
+- spec/mercadopago_client/card_token_spec.rb
+- spec/mercadopago_client/payment_methods_spec.rb
+- spec/mercadopago_client/payment_spec.rb
+- spec/mercadopago_client/preference_spec.rb
+- spec/mercadopago_spec.rb
+- spec/response_spec.rb
+- spec/rest_client_spec.rb
+- spec/spec_helper.rb