moip 1.0.4 → 2.0.0

data/Gemfile.lock

@@ -28,8 +28,5 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  activesupport (>= 2.3.2)
-  httparty (~> 0.6.1)
   moip!
-  nokogiri (~> 1.4.3)
   rspec (~> 2.1.0)

data/README.markdown

@@ -65,19 +65,13 @@ Após realizar os testes na SandBox, você poderá fazer a mudança para o ambie
 ###Checkout
 
     def checkout
-      response = MoIP::Client.checkout(boleto)
+      response = MoIP.checkout(boleto)
 
       # exibe o boleto para impressão
-      redirect_to MoIP::Client.moip_page(response["Token"])
+      redirect_to MoIP.moip_page(response["Token"])
     end
 
-###Erros
-
- - MoIP::MissingPaymentTypeError - Quando falta a razão do pagamento na requisição.
- - MoIP::MissingPayerError - Quando falta as informações do pagador na requisição.
- - MoIP::WebServerResponseError - Quando há algum erro ao se enviar a solicitação ao servidor. Normalmente a razão do erro vem como resposta da mensagem.
-
-## Futuras implementações
+### Futuras implementações
 
 * Pagamento Simples
 * Pagamento Recorrente

data/Rakefile

@@ -2,8 +2,6 @@
 require 'rubygems'
 require 'rake'
 
-require 'rspec/core/rake_task'
-
 begin
   require 'jeweler'
   Jeweler::Tasks.new do |gem|
@@ -15,7 +13,7 @@ begin
     gem.authors = ["Guilherme Nascimento"]
     gem.add_development_dependency "rspec", "~> 2.1.0"
     gem.add_dependency "nokogiri", "~> 1.4.3"
-    gem.add_dependency "httparty", "~> 0.8.1"
+    gem.add_dependency "httparty", "~> 0.6.1"
     gem.add_dependency "activesupport", '>= 2.3.2'
   end
   Jeweler::GemcutterTasks.new
@@ -23,9 +21,15 @@ rescue LoadError
   puts "Jeweler (or a dependency) not available. Install it with: gem install jeweler"
 end
 
-task :default => :spec
+require 'rspec/core/rake_task'
+
+desc "Run all examples"
 RSpec::Core::RakeTask.new
 
+task :spec => :check_dependencies
+
+task :default => :spec
+
 require 'rake/rdoctask'
 Rake::RDocTask.new do |rdoc|
   version = File.exist?('VERSION') ? File.read('VERSION') : ""

data/VERSION

@@ -1 +1 @@
-1.0.4
+2.0.0

data/lib/{moip/direct_payment.rb → direct_payment.rb}

@@ -1,29 +1,20 @@
 # encoding: utf-8
-require "nokogiri"
-
 module MoIP
 
-  class MissingPaymentTypeError < StandardError ; end
-  class MissingPayerError < StandardError ; end
-
   class DirectPayment
 
     class << self
 
       # Cria uma instrução de pagamento direto
       def body(attributes = {})
-        raise(MissinPaymentTypeError, "É necessário informar a razão do pagamento") if attributes[:razao].nil?
-        raise(MissingPayerError, "É obrigatório passar as informações do pagador") if attributes[:pagador].nil?
-
         builder = Nokogiri::XML::Builder.new(:encoding => "UTF-8") do |xml|
 
           # Identificador do tipo de instrução
           xml.EnviarInstrucao {
             xml.InstrucaoUnica {
-
               # Dados da transação
               xml.Razao {
-                xml.text attributes[:razao]
+                xml.text "Pagamento"
               }
               xml.Valores {
                 xml.Valor(:moeda => "BRL") {
@@ -52,6 +43,7 @@ module MoIP
                   xml.Instituicao {
                     xml.text attributes[:instituicao]
                   }
+
                   xml.CartaoCredito {
                     xml.Numero {
                       xml.text attributes[:numero]
@@ -89,6 +81,7 @@ module MoIP
               }
 
               # Dados do pagador
+              raise(StandardError, "É obrigatório passar as informações do pagador") if attributes[:pagador].nil?
               xml.Pagador {
                 xml.Nome { xml.text attributes[:pagador][:nome] }
                 xml.LoginMoIP { xml.text attributes[:pagador][:login_moip] }
@@ -124,23 +117,15 @@ module MoIP
                   }
                 }
               end
-              
-              if attributes[:url_retorno]
-                # URL de retorno
-                xml.URLRetorno {
-                  xml.text attributes[:url_retorno]
-                }
-              end
-                
+
             }
           }
         end
-
         builder.to_xml
       end
 
     end
 
   end
-
+  
 end

data/lib/moip.rb

@@ -1,47 +1,77 @@
 # encoding: utf-8
 require "rubygems"
 require 'active_support/core_ext/module/attribute_accessors'
-require 'active_support/deprecation'
+require 'httparty'
+require "nokogiri"
+
+require "direct_payment"
 
 module MoIP
-  autoload :DirectPayment, 'moip/direct_payment'
-  autoload :Client,        'moip/client'
+    include HTTParty
 
-  # URI para acessar o serviço
-  mattr_accessor :uri
-  @@uri = 'https://www.moip.com.br'
+    # URI para acessar o serviço
+    mattr_accessor :uri
+    @@uri = 'https://www.moip.com.br'
 
-  # Token de autenticação
-  mattr_accessor :token
+    # Token de autenticação
+    mattr_accessor :token
 
-  # Chave de acesso ao serviço
-  mattr_accessor :key
+    # Chave de acesso ao serviço
+    mattr_accessor :key
 
-  def self.setup
-    yield self
-  end
+    def self.setup
+      yield self
+    end
 
-  STATUS = {1 => "authorized", 2 => "started", 3 => "printed", 4 => "completed", 5 => "canceled", 6 => "analysing"}
+    STATUS = { 1 => "authorized", 2 => "started", 3 => "printed", 4 => "completed", 5 => "canceled", 6 => "analysing" }
 
-  class << self
-    def checkout(attributes = {})
-      ActiveSupport::Deprecation.warn("MoIP.checkout has been deprecated. Use MoIP::Client.checkout instead", caller)
-      MoIP::Client.checkout(attributes)
-    end
+    base_uri "#{self.uri}/ws/alpha"
+    basic_auth self.token, self.key
 
-    def query(token)
-      ActiveSupport::Deprecation.warn("MoIP.query has been deprecated. Use MoIP::Client.query instead", caller)
-      MoIP::Client.query(token)
-    end
+    class << self
+      # Envia uma instrução para pagamento único
+      def checkout(attributes = {})
+        full_data = peform_action!(:post, 'EnviarInstrucao/Unica', :body => DirectPayment.body(attributes))
 
-    def moip_page(token)
-      ActiveSupport::Deprecation.warn("MoIP.moip_page has been deprecated. use MoIP::Client.moip_page instead", caller)
-      MoIP::Client.moip_page(token)
-    end
+        get_response!(full_data["EnviarInstrucaoUnicaResponse"]["Resposta"])
+      end
+
+      # Consulta dos dados das autorizações e pagamentos associados à Instrução
+      def query(token)
+        full_data = peform_action!(:get, "ConsultarInstrucao/#{token}")
+
+        get_response!(full_data["ConsultarTokenResponse"]["RespostaConsultar"])
+      end
+
+      # Retorna a URL de acesso ao MoIP
+      def moip_page(token)
+        raise(StandardError, "É necessário informar um token para retornar os dados da transação") if token.nil?
+        "#{self.uri}/Instrucao.do?token=#{token}"
+      end
+
+      # Monta o NASP
+      def notification(params)
+        notification = {}
+        notification[:transaction_id] = params["id_transacao"]
+        notification[:amount]         = params["valor"]
+        notification[:status]         = STATUS[params["status_pagamento"].to_i]
+        notification[:code]           = params["cod_moip"]
+        notification[:payment_type]   = params["tipo_pagamento"]
+        notification[:email]          = params["email_consumidor"]
+        notification
+      end
+
+      private
+
+      def peform_action!(action_name, url, options = {})
+        response = self.send(action_name, url, options)
+        raise(StandardError, "Ocorreu um erro ao chamar o webservice") if response.nil?
+        response
+      end
 
-    def notification(params)
-      ActiveSupport::Deprecation.warn("MoIP.notification has been deprecated. use MoIP::Client.notification instead", caller)
-      MoIP::Client.moip_page(token)
+      def get_response!(data)
+        raise(StandardError, data["Erro"]) if data["Status"] == "Falha"
+        data
+      end
     end
-  end
 end

data/moip.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{moip}
-  s.version = "1.0.4"
+  s.version = "2.0.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Guilherme Nascimento"]
@@ -24,9 +24,8 @@ Gem::Specification.new do |s|
     "README.markdown",
     "Rakefile",
     "VERSION",
+    "lib/direct_payment.rb",
     "lib/moip.rb",
-    "lib/moip/client.rb",
-    "lib/moip/direct_payment.rb",
     "moip.gemspec",
     "spec/moip_spec.rb"
   ]
@@ -45,8 +44,10 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<rspec>, ["~> 2.1.0"])
       s.add_development_dependency(%q<rspec>, ["~> 2.1.0"])
       s.add_development_dependency(%q<rspec>, ["~> 2.1.0"])
+      s.add_development_dependency(%q<rspec>, ["~> 2.1.0"])
+      s.add_development_dependency(%q<rspec>, ["~> 2.1.0"])
       s.add_runtime_dependency(%q<nokogiri>, ["~> 1.4.3"])
-      s.add_runtime_dependency(%q<httparty>, ["~> 0.8.1"])
+      s.add_runtime_dependency(%q<httparty>, ["~> 0.6.1"])
       s.add_runtime_dependency(%q<activesupport>, [">= 2.3.2"])
     else
       s.add_dependency(%q<moip>, [">= 0"])
@@ -54,8 +55,10 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<rspec>, ["~> 2.1.0"])
       s.add_dependency(%q<rspec>, ["~> 2.1.0"])
       s.add_dependency(%q<rspec>, ["~> 2.1.0"])
+      s.add_dependency(%q<rspec>, ["~> 2.1.0"])
+      s.add_dependency(%q<rspec>, ["~> 2.1.0"])
       s.add_dependency(%q<nokogiri>, ["~> 1.4.3"])
-      s.add_dependency(%q<httparty>, ["~> 0.8.1"])
+      s.add_dependency(%q<httparty>, ["~> 0.6.1"])
       s.add_dependency(%q<activesupport>, [">= 2.3.2"])
     end
   else
@@ -64,8 +67,10 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<rspec>, ["~> 2.1.0"])
     s.add_dependency(%q<rspec>, ["~> 2.1.0"])
     s.add_dependency(%q<rspec>, ["~> 2.1.0"])
+    s.add_dependency(%q<rspec>, ["~> 2.1.0"])
+    s.add_dependency(%q<rspec>, ["~> 2.1.0"])
     s.add_dependency(%q<nokogiri>, ["~> 1.4.3"])
-    s.add_dependency(%q<httparty>, ["~> 0.8.1"])
+    s.add_dependency(%q<httparty>, ["~> 0.6.1"])
     s.add_dependency(%q<activesupport>, [">= 2.3.2"])
   end
 end

data/spec/moip_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 require "moip"
 
-require "digest/sha1"
-
 describe "Make payments with the MoIP API" do
 
   before :all do
@@ -28,233 +26,107 @@ describe "Make payments with the MoIP API" do
                 :cep => "70100-000",
                 :tel_fixo => "(61)3211-1221" }
 
-    @billet_without_razao = { :value => "8.90", :id_proprio => id,
-                              :forma => "BoletoBancario", :pagador => @pagador}
-    @billet = { :value => "8.90", :id_proprio => id,
-                :forma => "BoletoBancario", :pagador => @pagador ,
-                :razao=> "Pagamento" }
-
-    @debit = { :value => "8.90", :id_proprio => id, :forma => "DebitoBancario",
-               :instituicao => "BancoDoBrasil", :pagador => @pagador,
-               :razao => "Pagamento"}
-
-    @credit = { :value => "8.90", :id_proprio => id, :forma => "CartaoCredito",
-                :instituicao => "AmericanExpress",:numero => "345678901234564",
-                :expiracao => "08/11", :codigo_seguranca => "1234",
-                :nome => "João Silva", :identidade => "134.277.017.00",
-                :telefone => "(21)9208-0547", :data_nascimento => "25/10/1980",
-                :parcelas => "2", :recebimento => "AVista",
-                :pagador => @pagador, :razao => "Pagamento"}
+    @billet = { :value => "8.90", :id_proprio => id, :forma => "BoletoBancario", :pagador => @pagador }
+    @debit = { :value => "8.90", :id_proprio => id, :forma => "DebitoBancario", :instituicao => "BancoDoBrasil", :pagador => @pagador }
+    @credit = { :value => "8.90", :id_proprio => id, :forma => "CartaoCredito", :instituicao => "AmericanExpress", :numero => "345678901234564", :expiracao => "08/11", :codigo_seguranca => "1234", :nome => "João Silva", :identidade => "134.277.017.00", :telefone => "(21)9208-0547", :data_nascimento => "25/10/1980", :parcelas => "2", :recebimento => "AVista", :pagador => @pagador }
   end
 
-  context "checkout" do
-    before(:each) do
-      MoIP::Client.stub!(:post).
-          and_return("ns1:EnviarInstrucaoUnicaResponse"=>
-                     { "Resposta"=>
-                        { "ID"=>Time.now.strftime("%y%m%d%H%M%S"),
-                          "Status"=>"Sucesso",
-                          "Token" => "T2N0L0X8E0S71217U2H3W1T4F4S4G4K731D010V0S0V0S080M010E0Q082X2"
-                        }
-                     })
+  context "make a billet checkout" do
+    it "should have status Sucesso" do
+      MoIP.stub!(:post).and_return("ns1:EnviarInstrucaoUnicaResponse"=>{ "Resposta"=>{ "ID"=>Time.now.strftime("%y%m%d%H%M%S"), "Status"=>"Sucesso", "Token" => "T2N0L0X8E0S71217U2H3W1T4F4S4G4K731D010V0S0V0S080M010E0Q082X2" }})
+      response = MoIP.checkout(@billet)
+      response["Status"].should == "Sucesso"
     end
+  end
 
-    it "with old api should be deprecated" do
-      deprecations = collect_deprecations{ MoIP.checkout(@billet) }
-
-      deprecations.should_not be_empty
-      deprecations.any? {|w| w =~ /MoIP.checkout has been deprecated/ }.should be_true
+  context "make a debit checkout" do
+    it "should have status Sucesso" do
+      MoIP.stub!(:post).and_return("ns1:EnviarInstrucaoUnicaResponse"=>{ "Resposta"=>{ "ID"=>Time.now.strftime("%y%m%d%H%M%S"), "Status"=>"Sucesso", "Token" => "T2N0L0X8E0S71217U2H3W1T4F4S4G4K731D010V0S0V0S080M010E0Q082X2" }})
+      response = MoIP.checkout(@debit)
+      response["Status"].should == "Sucesso"
     end
+  end
 
-    context "when it is a billet checkout" do
-      it "should raise an exception when razao parameter is not passed" do
-        error = "É necessário informar a razão do pagamento"
-
-        lambda { MoIP::Client.checkout(@billet_without_razao) }.should
-        raise_error(MoIP::MissingPaymentTypeError,error)
-      end
-
-      it "should have status 'Sucesso'" do
-        response = MoIP::Client.checkout(@billet)
-        response["Status"].should == "Sucesso"
-      end
+  context "make a debit checkout without pass a institution" do
+    it "should have status Falha" do
+      @incorrect_debit = { :value => "37.90", :id_proprio => id, :forma => "DebitoBancario", :pagador => @pagador }
+      MoIP.stub!(:post).and_return("ns1:EnviarInstrucaoUnicaResponse"=>{ "Resposta"=>{"Status"=>"Falha", "Erro"=>"Pagamento direto não é possível com a instituição de pagamento enviada" }})
+      lambda { MoIP.checkout(@incorrect_debit) }.should raise_error(StandardError, "Pagamento direto não é possível com a instituição de pagamento enviada")
     end
+  end
 
-    context "when it is a debit checkout" do
-      it "should have status 'Sucesso' with valid arguments" do
-        response = MoIP::Client.checkout(@debit)
-        response["Status"].should == "Sucesso"
-      end
-
-      it "should have status 'Falha' when a instituition is not passed as argument" do
-        @incorrect_debit = { :value => "37.90", :id_proprio => id,
-                             :forma => "DebitoBancario", :pagador => @pagador,
-                             :razao => "Pagamento"}
-
-        error = "Pagamento direto não é possível com a instituição de pagamento enviada"
-
-        MoIP::Client.stub!(:post)
-            .and_return("ns1:EnviarInstrucaoUnicaResponse"=>
-                            { "Resposta"=>
-                                {
-                                 "Status"=>"Falha",
-                                 "Erro"=>error
-                                }
-                            })
-        error = "Pagamento direto não é possível com a instituição de pagamento enviada"
-        lambda { MoIP::Client.checkout(@incorrect_debit) }.should
-            raise_error(MoIP::WebServerResponseError, error)
-      end
-
-      it "should raise an exception if payer informations were not passed" do
-        @incorrect_debit = { :value => "37.90", :id_proprio => id,
-                             :forma => "DebitoBancario",
-                             :instituicao => "BancoDoBrasil",
-                             :razao => "Pagamento"
-                           }
-
-        lambda { MoIP::Client.checkout(@incorrect_debit) }.should
-            raise_error(MoIP::MissingPayerError, "É obrigatório passar as informações do pagador")
-      end
+  context "make a debit checkout without pass the payer informations" do
+    it "should raise an exception" do
+      @incorrect_debit = { :value => "37.90", :id_proprio => id, :forma => "DebitoBancario", :instituicao => "BancoDoBrasil" }
+      lambda { MoIP.checkout(@incorrect_debit) }.should raise_error(StandardError, "É obrigatório passar as informações do pagador")
     end
+  end
 
-    context "when it is a credit card checkout" do
-      it "should have status 'Sucesso' with valid arguments" do
-        response = MoIP::Client.checkout(@credit)
-        response["Status"].should == "Sucesso"
-      end
-
-      it "should have status 'Falha' when the card informations were not passed as argument" do
-        @incorrect_credit = { :value => "8.90", :id_proprio => id,
-                              :forma => "CartaoCredito", :pagador => @pagador,
-                              :razao => "Pagamento"
-                            }
-
-        error = "Pagamento direto não é possível com a instituição de pagamento enviada"
-        MoIP::Client.stub!(:post)
-            .and_return("ns1:EnviarInstrucaoUnicaResponse"=>
-                        {
-                        "Resposta"=>
-                            {
-                             "Status"=>"Falha",
-                             "Erro"=>error
-                            }
-                        })
-
-        error = "Pagamento direto não é possível com a instituição de pagamento enviada"
-        lambda { MoIP::Client.checkout(@incorrect_credit) }.should
-            raise_error(MoIP::WebServerResponseError, error)
-      end
+  context "make a credit card checkout" do
+    it "should have status Sucesso" do
+      MoIP.stub!(:post).and_return("ns1:EnviarInstrucaoUnicaResponse"=>{ "Resposta"=>{ "ID"=>Time.now.strftime("%y%m%d%H%M%S"), "Status"=>"Sucesso", "Token" => "T2N0L0X8E0S71217U2H3W1T4F4S4G4K731D010V0S0V0S080M010E0Q082X2" }})
+      response = MoIP.checkout(@credit)
+      response["Status"].should == "Sucesso"
     end
+  end
 
-    context "in error scenario" do
-      it "should raise an exception if response is nil" do
-        MoIP::Client.stub!(:post).and_return(nil)
-        lambda { MoIP::Client.checkout(@billet) }.should
-            raise_error(StandardError,"Ocorreu um erro ao chamar o webservice")
-      end
-
-      it "should raise an exception if status is fail" do
-        MoIP::Client.stub!(:post)
-            .and_return("ns1:EnviarInstrucaoUnicaResponse"=>
-                        { "Resposta"=>
-                            {"Status"=>"Falha",
-                             "Erro"=>"O status da resposta é Falha"
-                            }
-                        })
-
-        lambda { MoIP::Client.checkout(@billet) }.should raise_error(StandardError, "O status da resposta é Falha")
-      end
+  context "make a credit card checkout without pass card informations" do
+    it "should have status Falha" do
+      @incorrect_credit = { :value => "8.90", :id_proprio => id, :forma => "CartaoCredito", :pagador => @pagador }
+      MoIP.stub!(:post).and_return("ns1:EnviarInstrucaoUnicaResponse"=>{ "Resposta"=>{"Status"=>"Falha", "Erro"=>"Pagamento direto não é possível com a instituição de pagamento enviada" }})
+      lambda { MoIP.checkout(@incorrect_credit) }.should raise_error(StandardError, "Pagamento direto não é possível com a instituição de pagamento enviada")
     end
   end
 
-  context "query a transaction token" do
-    before(:each) do
-      MoIP::Client.stub!(:get)
-        .and_return("ns1:ConsultarTokenResponse"=>
-                    { "RespostaConsultar"=>
-                        {"Status"=>"Sucesso",
-                         "ID"=>"201010291031001210000000046760"
-                        }
-                    })
+  context "in error scenario" do
+    it "should raise an exception if response is nil" do
+      MoIP.stub!(:post).and_return(nil)
+      lambda { MoIP.checkout(@billet) }.should raise_error(StandardError, "Ocorreu um erro ao chamar o webservice")
     end
 
-    it "with old api should be deprecated" do
-      deprecations = collect_deprecations{ MoIP.query(token) }
-
-      deprecations.should_not be_empty
-      deprecations.any? {|w| w =~ /MoIP.query has been deprecated/ }.should be_true
+    it "should raise an exception if status is fail" do
+      MoIP.stub!(:post).and_return("ns1:EnviarInstrucaoUnicaResponse"=>{ "Resposta"=>{"Status"=>"Falha", "Erro"=>"O status da resposta é Falha" }})
+      lambda { MoIP.checkout(@billet) }.should raise_error(StandardError, "O status da resposta é Falha")
     end
+  end
 
+  context "query a transaction token" do
     it "should retrieve the transaction" do
-      response = MoIP::Client.query(token)
+      MoIP.stub!(:get).and_return("ns1:ConsultarTokenResponse"=>{ "RespostaConsultar"=>{"Status"=>"Sucesso", "ID"=>"201010291031001210000000046760" }})
+      response = MoIP.query(token)
       response["Status"].should == "Sucesso"
     end
+  end
 
-    context "in a error scenario" do
-      it "should retrieve status 'Falha'" do
-        MoIP::Client.stub!(:get)
-            .and_return("ns1:ConsultarTokenResponse"=>
-                        { "RespostaConsultar"=>
-                            {"Status"=>"Falha",
-                             "Erro"=>"Instrução não encontrada",
-                             "ID"=>"201010291102522860000000046768"
-                            }
-                        })
-        query = "000000000000000000000000000000000000000000000000000000000000"
-        lambda { MoIP::Client.query(query) }.should raise_error(StandardError, "Instrução não encontrada")
-      end
+  context "query a transaction token in a error scenario" do
+    it "should retrieve status Falha" do
+      MoIP.stub!(:get).and_return("ns1:ConsultarTokenResponse"=>{ "RespostaConsultar"=>{"Status"=>"Falha", "Erro"=>"Instrução não encontrada", "ID"=>"201010291102522860000000046768"}})
+      lambda { MoIP.query("000000000000000000000000000000000000000000000000000000000000") }.should raise_error(StandardError, "Instrução não encontrada")
     end
   end
 
   context "build the MoIP URL" do
-    it "with old api should be deprecated" do
-      deprecations = collect_deprecations{ MoIP.moip_page(token) }
-
-      deprecations.should_not be_empty
-      deprecations.any? {|w| w =~ /MoIP.moip_page has been deprecated/ }.should be_true
-    end
-
     it "should build the correct URL" do
-      page = "https://desenvolvedor.moip.com.br/sandbox/Instrucao.do?token=#{token}"
-      MoIP::Client.moip_page(token).should ==  page
+      MoIP.moip_page(token).should == "https://desenvolvedor.moip.com.br/sandbox/Instrucao.do?token=#{token}"
     end
 
     it "should raise an error if the token is not informed" do
-      error = "É necessário informar um token para retornar os dados da transação"
-      lambda { MoIP::Client.moip_page("").should
-          raise_error(ArgumentError, error) }
+      lambda { MoIP.moip_page("").should raise_error(ArgumentError, "É necessário informar um token para retornar os dados da transação") }
     end
 
     it "should raise an error if nil is passed as the token" do
-      error = "É necessário informar um token para retornar os dados da transação"
-      lambda { MoIP::Client.moip_page(nil).should
-          raise_error(ArgumentError, error) }
+      lambda { MoIP.moip_page(nil).should raise_error(ArgumentError, "É necessário informar um token para retornar os dados da transação") }
     end
   end
 
   context "when receive notification" do
     before(:each) do
-      @params = { "id_transacao" => "Pag62", "valor" => "8.90",
-                  "status_pagamento" => "3", "cod_moip" => "001",
-                  "forma_pagamento" => "73", "tipo_pagamento" => "BoletoBancario",
-                  "email_consumidor" => "presidente@planalto.gov.br" }
-    end
-
-    it "with old api should be deprecated" do
-      deprecations = collect_deprecations{ MoIP.notification(@param) }
-
-      deprecations.should_not be_empty
-      deprecations.any? {|w| w =~ /MoIP.notification has been deprecated/ }.should be_true
+      @params = { "id_transacao" => "Pag62", "valor" => "8.90", "status_pagamento" => "3", "cod_moip" => "001", "forma_pagamento" => "73", "tipo_pagamento" => "BoletoBancario", "email_consumidor" => "presidente@planalto.gov.br" }
     end
 
     it "should return a hash with the params extracted from NASP" do
-      response = { :transaction_id => "Pag62", :amount => "8.90",
-                   :status => "printed", :code => "001",
-                   :payment_type => "BoletoBancario",
-                   :email => "presidente@planalto.gov.br" }
-
-      MoIP::Client.notification(@params).should == response
+      response = { :transaction_id => "Pag62", :amount => "8.90", :status => "printed", :code => "001", :payment_type => "BoletoBancario", :email => "presidente@planalto.gov.br" }
+      MoIP.notification(@params).should == response
     end
 
     it "should return valid status based on status code" do
@@ -275,15 +147,4 @@ describe "Make payments with the MoIP API" do
     "T2X0Q1N021E0B2S9U1P0V3Y0G1F570Y2P4M0P000M0Z0F0J0G0U4N6C7W5T9"
   end
 
-  def collect_deprecations
-    old_behavior = ActiveSupport::Deprecation.behavior
-    deprecations = []
-    ActiveSupport::Deprecation.behavior = Proc.new do |message, callstack|
-      deprecations << message
-    end
-    result = yield
-    deprecations
-  ensure
-    ActiveSupport::Deprecation.behavior = old_behavior
-  end
 end

metadata

@@ -3,11 +3,11 @@ name: moip
 version: !ruby/object:Gem::Version 
   prerelease: false
   segments: 
-  - 1
+  - 2
+  - 0
   - 0
-  - 4
   segments_generated: true
-  version: 1.0.4
+  version: 2.0.0
 platform: ruby
 authors: 
 - Guilherme Nascimento
@@ -91,11 +91,41 @@ dependencies:
         segments_generated: true
         version: 2.1.0
   requirement: *id005
+- !ruby/object:Gem::Dependency 
+  prerelease: false
+  type: :development
+  name: rspec
+  version_requirements: &id006 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 2
+        - 1
+        - 0
+        segments_generated: true
+        version: 2.1.0
+  requirement: *id006
+- !ruby/object:Gem::Dependency 
+  prerelease: false
+  type: :development
+  name: rspec
+  version_requirements: &id007 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 2
+        - 1
+        - 0
+        segments_generated: true
+        version: 2.1.0
+  requirement: *id007
 - !ruby/object:Gem::Dependency 
   prerelease: false
   type: :runtime
   name: nokogiri
-  version_requirements: &id006 !ruby/object:Gem::Requirement 
+  version_requirements: &id008 !ruby/object:Gem::Requirement 
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
@@ -105,27 +135,27 @@ dependencies:
         - 3
         segments_generated: true
         version: 1.4.3
-  requirement: *id006
+  requirement: *id008
 - !ruby/object:Gem::Dependency 
   prerelease: false
   type: :runtime
   name: httparty
-  version_requirements: &id007 !ruby/object:Gem::Requirement 
+  version_requirements: &id009 !ruby/object:Gem::Requirement 
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
         segments: 
         - 0
-        - 8
+        - 6
         - 1
         segments_generated: true
-        version: 0.8.1
-  requirement: *id007
+        version: 0.6.1
+  requirement: *id009
 - !ruby/object:Gem::Dependency 
   prerelease: false
   type: :runtime
   name: activesupport
-  version_requirements: &id008 !ruby/object:Gem::Requirement 
+  version_requirements: &id010 !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
@@ -135,7 +165,7 @@ dependencies:
         - 2
         segments_generated: true
         version: 2.3.2
-  requirement: *id008
+  requirement: *id010
 description: "Gem para utiliza\xC3\xA7\xC3\xA3o da API MoIP"
 email: guilherme.ruby@gmail.com
 executables: []
@@ -153,9 +183,8 @@ files:
 - README.markdown
 - Rakefile
 - VERSION
+- lib/direct_payment.rb
 - lib/moip.rb
-- lib/moip/client.rb
-- lib/moip/direct_payment.rb
 - moip.gemspec
 - spec/moip_spec.rb
 has_rdoc: true

data/lib/moip/client.rb

@@ -1,62 +0,0 @@
-# encoding: utf-8
-require 'httparty'
-
-module MoIP
-  class WebServerResponseError < StandardError ; end
-  class MissingTokenError < StandardError ; end
-
-  class Client
-    include HTTParty
-
-    base_uri "#{MoIP.uri}/ws/alpha"
-    basic_auth MoIP.token, MoIP.key
-
-    class << self
-
-      # Envia uma instrução para pagamento único
-      def checkout(attributes = {})
-        full_data = peform_action!(:post, 'EnviarInstrucao/Unica', :body => DirectPayment.body(attributes))
-
-        get_response!(full_data["ns1:EnviarInstrucaoUnicaResponse"]["Resposta"])
-      end
-
-      # Consulta dos dados das autorizações e pagamentos associados à Instrução
-      def query(token)
-        full_data = peform_action!(:get, "ConsultarInstrucao/#{token}")
-
-        get_response!(full_data["ns1:ConsultarTokenResponse"]["RespostaConsultar"])
-      end
-
-      # Retorna a URL de acesso ao MoIP
-      def moip_page(token)
-        raise(MissingTokenError, "É necessário informar um token para retornar os dados da transação") if token.nil?
-        "#{MoIP.uri}/Instrucao.do?token=#{token}"
-      end
-
-      # Monta o NASP
-      def notification(params)
-        notification = {}
-        notification[:transaction_id] = params["id_transacao"]
-        notification[:amount]         = params["valor"]
-        notification[:status]         = MoIP::STATUS[params["status_pagamento"].to_i]
-        notification[:code]           = params["cod_moip"]
-        notification[:payment_type]   = params["tipo_pagamento"]
-        notification[:email]          = params["email_consumidor"]
-        notification
-      end
-
-      private
-
-      def peform_action!(action_name, url, options = {})
-        response = self.send(action_name, url, options)
-        raise(WebServerResponseError, "Ocorreu um erro ao chamar o webservice") if response.nil?
-        response
-      end
-
-      def get_response!(data)
-        raise(WebServerResponseError, data["Erro"]) if data["Status"] == "Falha"
-        data
-      end
-    end
-  end
-end