afipws 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 43b67a6ffe5b7c3c7a8673bb131bad701008aebe
-  data.tar.gz: eca42e8c7b56f99cf21d3a65c170f3298d0a5edd
+  metadata.gz: 3bb1054b4ad45099d4b950edb154f0961e2a8100
+  data.tar.gz: 3768c0065771f1b021e136a30ce4d56fbdcf0204
 SHA512:
-  metadata.gz: c54c43e24b381eef8df652b8a104c9dbed4841959493871c32cfa780979b5ace1ad2365213ab0b57252dcf5f3aa021eb5f5f33d23870eb67255d05a639b6ea68
-  data.tar.gz: 9415493baf942f65673e4a24d33101836a9b3e66fa634395683dd26b51a6f2794f457cd6ec5bc6f1d2ba16cb7c117975f2d27881ec1ec86c7eba228e2f4b96aa
+  metadata.gz: beea7ebef5946e80acfd6ae40277000e9433964b38f669a0a0d6a4cb789b52e6e0856f35b5d804de7754b150501d8e68fa5d557ac1f2b4d0eb7cb7d30bdc75c8
+  data.tar.gz: bdcc106d8fdbeb4fee5000795ecba43d5483ede917a578874d52b2c66df60c930254c3da54ae9bcc54d5d8eca47956ed3aa5b507f8443f65e12e8f4e572f0d40

data/.gitignore

@@ -3,4 +3,5 @@ pkg/*
 .bundle
 spec/manual/*.key
 spec/manual/*.crt
-.DS_Store
+.DS_Store
+tmp

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    afipws (1.0.2)
+    afipws (1.0.3)
       activesupport
       builder
       nokogiri

data/lib/afipws/version.rb

@@ -1,3 +1,3 @@
 module Afipws
-  VERSION = "1.0.2"
+  VERSION = "1.0.3"
 end

data/lib/afipws/wsaa.rb

@@ -16,6 +16,7 @@ module Afipws
       @ttl = options[:ttl] || 2400
       @cuit = options[:cuit]
       @client = Client.new Hash(options[:savon]).reverse_merge(wsdl: WSDL[@env])
+      @ta_path = File.join(Dir.pwd, 'tmp', "#{@cuit}-#{@env}-ta.dump")
     end
 
     def generar_tra service, ttl
@@ -25,7 +26,6 @@ module Afipws
         xml.header do
           xml.uniqueId Time.now.to_i
           xml.generationTime xsd_datetime Time.now - ttl
-          # TODO me parece que no le da mucha bola el WS al expirationTime
           xml.expirationTime xsd_datetime Time.now + ttl
         end
         xml.service service
@@ -49,23 +49,36 @@ module Afipws
     def login
       response = @client.raw_request :login_cms, in0: tra(@key, @cert, @service, @ttl)
       ta = Nokogiri::XML(Nokogiri::XML(response.to_xml).text)
-      { token: ta.css('token').text, sign: ta.css('sign').text,
+      {
+        token: ta.css('token').text,
+        sign: ta.css('sign').text,
         generation_time: from_xsd_datetime(ta.css('generationTime').text),
-        expiration_time: from_xsd_datetime(ta.css('expirationTime').text) }
+        expiration_time: from_xsd_datetime(ta.css('expirationTime').text)
+      }
     rescue Savon::SOAPFault => f
       raise WSError, f.message
     end
 
-    # Obtiene un TA, lo cachea hasta que expire, y devuelve el hash Auth listo para pasarle al Client
-    # en los otros WS.
+    # Obtiene un TA, lo cachea hasta que expire, y devuelve el hash Auth listo para pasarle al Client en los otros WS
     def auth
-      @ta = login if ta_expirado?
-      { auth: { token: @ta[:token], sign: @ta[:sign], cuit: @cuit } }
+      ta = obtener_y_cachear_ta
+      {auth: {token: ta[:token], sign: ta[:sign], cuit: @cuit}}
     end
 
     private
-    def ta_expirado?
-      @ta.nil? or @ta[:expiration_time] <= Time.now
+
+    # Previene el error 'El CEE ya posee un TA valido para el acceso al WSN solicitado' que se genera cuando se pide el token varias veces en poco tiempo
+    def obtener_y_cachear_ta
+      @ta ||= restore_ta
+      if ta_expirado? @ta
+        @ta = login
+        persist_ta @ta
+      end
+      @ta
+    end
+
+    def ta_expirado? ta
+      ta.nil? || ta[:expiration_time] <= Time.now
     end
 
     def xsd_datetime time
@@ -75,5 +88,17 @@ module Afipws
     def from_xsd_datetime str
       Time.parse(str) rescue nil
     end
+
+    def restore_ta
+      Marshal.load(File.read(@ta_path)) if File.exists?(@ta_path)
+    end
+
+    def persist_ta ta
+      dirname = File.dirname(@ta_path)
+      unless File.directory?(dirname)
+        FileUtils.mkdir_p(dirname)
+      end
+      File.open(@ta_path, "wb") { |f| f.write(Marshal.dump(ta)) }
+    end
   end
 end

data/lib/afipws/wsfe.rb

@@ -15,8 +15,7 @@ module Afipws
     def initialize options = {}
       @env = (options[:env] || :test).to_sym
       @wsaa = options[:wsaa] || WSAA.new(options.merge(service: 'wsfe'))
-      ssl_version = :TLSv1
-      @client = Client.new Hash(options[:savon]).reverse_merge(wsdl: WSDL[@env], ssl_version: ssl_version, convert_request_keys_to: :camelcase)
+      @client = Client.new Hash(options[:savon]).reverse_merge(wsdl: WSDL[@env], ssl_version: :TLSv1, convert_request_keys_to: :camelcase)
     end
 
     def dummy

data/spec/afipws/wsaa_spec.rb

@@ -47,13 +47,21 @@ describe Afipws::WSAA do
   end
 
   context "auth" do
-    before { Time.stubs(:now).returns(now = Time.local(2010,1,1)) }
+    before do
+      FileUtils.rm_rf Dir.glob('tmp/*ta.dump')
+      Time.stubs(:now).returns(Time.local(2010, 1, 1))
+    end
 
-    it "debería cachear TA" do
-      subject.expects(:login).once.returns(ta = {token: 'token', sign: 'sign', expiration_time: Time.now + 60})
-      subject.auth
-      subject.auth
-      subject.ta.should equal ta
+    it "debería cachear TA en la instancia y disco" do
+      ws = Afipws::WSAA.new
+      ws.expects(:login).once.returns(ta = {token: 'token', sign: 'sign', expiration_time: Time.now + 60})
+      ws.auth
+      ws.auth
+      ws.ta.should equal ta
+
+      ws = Afipws::WSAA.new
+      ws.auth
+      ws.ta.should == ta
     end
 
     it "si el TA expiró debería ejecutar solicitar uno nuevo" do

data/spec/afipws/wsfe_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Afipws::WSFE do
   let(:auth) { {auth: {token: 't', sign: 's', expiration_time: 12.hours.from_now}} }
   let(:ws) { Afipws::WSFE.new cuit: '1', wsaa: Afipws::WSAA.new.tap { |wsaa| wsaa.stubs auth: auth } }
-  
+
   context "Métodos de negocio" do
     it "dummy" do
       savon.expects(:fe_dummy).returns(fixture('fe_dummy/success'))
@@ -13,56 +13,56 @@ describe Afipws::WSFE do
     it "tipos_comprobantes" do
       savon.expects(:fe_param_get_tipos_cbte).with(message: auth).returns(fixture('fe_param_get_tipos_cbte/success'))
       ws.tipos_comprobantes.should == [
-        { id: 1, desc: "Factura A", fch_desde: Date.new(2010,9,17), fch_hasta: nil }, 
+        { id: 1, desc: "Factura A", fch_desde: Date.new(2010,9,17), fch_hasta: nil },
         { id: 2, desc: "Nota de Débito A", fch_desde: Date.new(2010,9,18), fch_hasta: Date.new(2011,9,18) }]
     end
-    
+
     it "tipos_documentos" do
       savon.expects(:fe_param_get_tipos_doc).with(message: auth).returns(fixture('fe_param_get_tipos_doc/success'))
       ws.tipos_documentos.should == [{ id: 80, desc: "CUIT", fch_desde: Date.new(2008,7,25), fch_hasta: nil }]
     end
-    
+
     it "tipos_monedas" do
       savon.expects(:fe_param_get_tipos_monedas).with(message: auth).returns(fixture('fe_param_get_tipos_monedas/success'))
       ws.tipos_monedas.should == [
-        { id: 'PES', desc: "Pesos Argentinos", fch_desde: Date.new(2009,4,3), fch_hasta: nil }, 
+        { id: 'PES', desc: "Pesos Argentinos", fch_desde: Date.new(2009,4,3), fch_hasta: nil },
         { id: '002', desc: "Dólar Libre EEUU", fch_desde: Date.new(2009,4,16), fch_hasta: nil }]
     end
-    
+
     it "tipos_iva" do
       savon.expects(:fe_param_get_tipos_iva).with(message: auth).returns(fixture('fe_param_get_tipos_iva/success'))
-      ws.tipos_iva.should == [{ id: 5, desc: "21%", fch_desde: Date.new(2009,2,20), fch_hasta: nil }] 
+      ws.tipos_iva.should == [{ id: 5, desc: "21%", fch_desde: Date.new(2009,2,20), fch_hasta: nil }]
     end
-    
+
     it "tipos_tributos" do
       savon.expects(:fe_param_get_tipos_tributos).with(message: auth).returns(fixture('fe_param_get_tipos_tributos/success'))
       ws.tipos_tributos.should == [{ id: 2, desc: "Impuestos provinciales", fch_desde: Date.new(2010,9,17), fch_hasta: nil }]
     end
-    
+
     it "puntos_venta" do
       savon.expects(:fe_param_get_ptos_venta).with(message: auth).returns(fixture('fe_param_get_ptos_venta/success'))
       ws.puntos_venta.should == [
         { nro: 1, emision_tipo: "CAE", bloqueado: false, fch_baja: nil },
         { nro: 2, emision_tipo: "CAEA", bloqueado: true, fch_baja: Date.new(2011,1,31) }]
     end
-    
+
     context "cotizacion" do
       it "cuando la moneda solicitada existe" do
         savon.expects(:fe_param_get_cotizacion).with(message: auth.merge(mon_id: 'DOL')).returns(fixture('fe_param_get_cotizacion/dolar'))
         ws.cotizacion('DOL').should == 3.976
       end
-      
+
       it "cuando la moneda no existe" do
         savon.expects(:fe_param_get_cotizacion).with(message: auth.merge(mon_id: 'PES')).returns(fixture('fe_param_get_cotizacion/inexistente'))
         lambda { ws.cotizacion('PES') }.should raise_error Afipws::WSError, /602: Sin Resultados/
       end
     end
-    
+
     it "cant_max_registros_x_lote" do
       savon.expects(:fe_comp_tot_x_request).with(message: auth).returns(fixture('fe_comp_tot_x_request/success'))
       ws.cant_max_registros_x_lote.should == 250
     end
-    
+
     context "autorizar_comprobante" do
       it "debería devolver un hash con el CAE y su fecha de vencimiento" do
         savon.expects(:fecae_solicitar).with(message: has_path(
@@ -82,13 +82,13 @@ describe Afipws::WSFE do
           '//FeCAEReq/FeDetReq/FECAEDetRequest[1]/Tributos/Tributo[1]/Alic' => 5.2,
           '//FeCAEReq/FeDetReq/FECAEDetRequest[1]/Tributos/Tributo[1]/Importe' => 7.8
         )).returns(fixture('fecae_solicitar/autorizacion_1_cbte'))
-        rta = ws.autorizar_comprobantes(cbte_tipo: 1, pto_vta: 2, comprobantes: [{cbte_nro: 1, concepto: 1, 
-          doc_nro: 30521189203, doc_tipo: 80, cbte_fch: Date.new(2011,01,13), 
+        rta = ws.autorizar_comprobantes(cbte_tipo: 1, pto_vta: 2, comprobantes: [{cbte_nro: 1, concepto: 1,
+          doc_nro: 30521189203, doc_tipo: 80, cbte_fch: Date.new(2011,01,13),
           imp_total: 1270.48, imp_neto: 1049.98, imp_iva: 220.50, mon_id: 'PES', mon_cotiz: 1,
           iva: { alic_iva: [{ id: 5, base_imp: 1049.98, importe: 220.50 }]},
           tributos: { tributo: [{ id: 0, base_imp: 150, alic: 5.2, importe: 7.8 }] }
         }])
-        rta[0].should have_entries cae: '61023008595705', cae_fch_vto: Date.new(2011,01,23), cbte_nro: 1, 
+        rta[0].should have_entries cae: '61023008595705', cae_fch_vto: Date.new(2011,01,23), cbte_nro: 1,
           resultado: 'A', observaciones: []
         rta.size.should == 1
       end
@@ -105,7 +105,7 @@ describe Afipws::WSFE do
             { id: 4, base_imp: 50, importe: 5.25 }
         ]}}])
       end
-      
+
       it "con varios comprobantes aprobados" do
         savon.expects(:fecae_solicitar).with(message: has_path(
           '//FeCAEReq/FeCabReq/CantReg' => 2,
@@ -118,7 +118,7 @@ describe Afipws::WSFE do
         rta[0].should have_entries cbte_nro: 5, cae: '61033008894096'
         rta[1].should have_entries cbte_nro: 6, cae: '61033008894101'
       end
-      
+
       it "con 2 observaciones" do
         savon.expects(:fecae_solicitar).with(message: :any).returns(fixture 'fecae_solicitar/dos_observaciones')
         rta = ws.autorizar_comprobantes comprobantes: []
@@ -132,15 +132,15 @@ describe Afipws::WSFE do
         rta[0].should have_entries observaciones: [{code: 10048, msg: 'Msg 1'}]
       end
     end
-    
+
     context "solicitar_caea" do
       it "debería mandar automáticamente el período y orden" do
         Date.stubs today: Date.new(2011,1,27)
         savon.expects(:fecaea_solicitar).with(message: has_path('//Periodo' => '201102', '//Orden' => 1)).returns(fixture 'fecaea_solicitar/success')
-        ws.solicitar_caea.should have_entries caea: '21043476341977', fch_tope_inf: Date.new(2011,03,17), 
+        ws.solicitar_caea.should have_entries caea: '21043476341977', fch_tope_inf: Date.new(2011,03,17),
           fch_vig_desde: Date.new(2011,02,01), fch_vig_hasta: Date.new(2011,02,15)
       end
-      
+
       context "periodo_para_solicitud_caea" do
         it "cuando estoy en la primer quincena" do
           Date.stubs today: Date.new(2011,1,12)
@@ -148,28 +148,28 @@ describe Afipws::WSFE do
           Date.stubs today: Date.new(2011,1,15)
           ws.periodo_para_solicitud_caea.should == { periodo: '201101', orden: 2 }
         end
-        
+
         it "cuando estoy en la segunda quincena" do
           Date.stubs today: Date.new(2011,1,16)
           ws.periodo_para_solicitud_caea.should == { periodo: '201102', orden: 1 }
           Date.stubs today: Date.new(2011,1,31)
           ws.periodo_para_solicitud_caea.should == { periodo: '201102', orden: 1 }
-        end        
+        end
       end
-      
+
       it "cuando el caea ya fue otorgado debería consultarlo y devolverlo" do
         Date.stubs today: Date.new(2011,1,27)
         savon.expects(:fecaea_solicitar).with(message: has_path('//Periodo' => '201102', '//Orden' => 1)).returns(fixture 'fecaea_solicitar/caea_ya_otorgado')
         savon.expects(:fecaea_consultar).with(message: has_path('//Periodo' => '201102', '//Orden' => 1)).returns(fixture 'fecaea_consultar/success')
         ws.solicitar_caea.should have_entries caea: '21043476341977', fch_vig_desde: Date.new(2011,02,01)
-      end    
-      
+      end
+
       it "cuando hay otro error debería burbujearlo" do
         savon.expects(:fecaea_solicitar).with(message: :any).returns(fixture 'fecaea_solicitar/error_distinto')
         lambda { ws.solicitar_caea }.should raise_error Afipws::WSError, /15007/
-      end  
+      end
     end
-    
+
     it "informar_comprobantes_caea" do
       savon.expects(:fecaea_reg_informativo).with(message: has_path(
         '//Auth/Token' => 't',
@@ -189,29 +189,29 @@ describe Afipws::WSFE do
       rta[0].should have_entries cbte_nro: 1, caea: '21043476341977', resultado: 'A', observaciones: []
       rta[1].should have_entries cbte_nro: 2, caea: '21043476341977', resultado: 'R', observaciones: [{code: 724, msg: 'Msg'}]
     end
-    
+
     it "informar_caea_sin_movimientos" do
       savon.expects(:fecaea_sin_movimiento_informar).with(message: has_path(
         '//Auth/Token' => 't',
-        '//PtoVta' => 4, 
+        '//PtoVta' => 4,
         '//CAEA' => '21043476341977'
       )).returns(fixture 'fecaea_sin_movimiento_informar/success')
       rta = ws.informar_caea_sin_movimientos('21043476341977', 4)
       rta.should have_entries caea: '21043476341977', resultado: 'A'
     end
-    
+
     context "consultar_caea" do
       it "consultar_caea" do
         savon.expects(:fecaea_consultar).with(message: has_path('//Periodo' => '201101', '//Orden' => 1)).returns(fixture 'fecaea_consultar/success')
         ws.consultar_caea(Date.new(2011,1,1)).should have_entries caea: '21043476341977', fch_tope_inf: Date.new(2011,03,17)
       end
     end
-    
+
     it "ultimo_comprobante_autorizado" do
       savon.expects(:fe_comp_ultimo_autorizado).with(message: has_path('//PtoVta' => 1, '//CbteTipo' => 1)).returns(fixture 'fe_comp_ultimo_autorizado/success')
       ws.ultimo_comprobante_autorizado(pto_vta: 1, cbte_tipo: 1).should == 20
     end
-    
+
     it "consultar_comprobante" do
       savon.expects(:fe_comp_consultar).with(message: has_path(
         '//Auth/Token' => 't', '//FeCompConsReq/PtoVta' => 1, '//FeCompConsReq/CbteTipo' => 2, '//FeCompConsReq/CbteNro' => 3
@@ -221,21 +221,23 @@ describe Afipws::WSFE do
       rta[:emision_tipo].should == 'CAE'
     end
   end
-  
+
   context "autenticacion" do
+    before { FileUtils.rm_rf Dir.glob('tmp/*ta.dump') }
+
     it "debería autenticarse usando el WSAA" do
       wsfe = Afipws::WSFE.new cuit: '1', cert: 'cert', key: 'key'
       wsfe.wsaa.cert.should == 'cert'
       wsfe.wsaa.key.should == 'key'
       wsfe.wsaa.service.should == 'wsfe'
-      wsfe.wsaa.expects(:login).returns({ token: 't', sign: 's' })
+      wsfe.wsaa.expects(:login).returns({token: 't', sign: 's'})
       savon.expects(:fe_param_get_tipos_cbte).with(message: has_path(
         '//Auth/Token' => 't', '//Auth/Sign' => 's', '//Auth/Cuit' => '1'
       )).returns(fixture 'fe_param_get_tipos_cbte/success')
       wsfe.tipos_comprobantes
     end
   end
-  
+
   context "entorno" do
     it "debería usar las url para development cuando el env es development" do
       Afipws::Client.expects(:new).with(wsdl: 'https://wsaahomo.afip.gov.ar/ws/services/LoginCms?wsdl')
@@ -251,27 +253,27 @@ describe Afipws::WSFE do
       wsfe.env.should == :production
     end
   end
-  
+
   context "manejo de errores" do
     it "cuando hay un error" do
       savon.expects(:fe_param_get_tipos_cbte).with(message: :any).returns(fixture 'fe_param_get_tipos_cbte/failure_1_error')
-      lambda { ws.tipos_comprobantes }.should raise_error { |e| 
+      lambda { ws.tipos_comprobantes }.should raise_error { |e|
         e.should be_a Afipws::WSError
-        e.errors.should == [{ code: "600", msg: "No se corresponden token con firma" }] 
-        e.message.should == "600: No se corresponden token con firma" 
+        e.errors.should == [{ code: "600", msg: "No se corresponden token con firma" }]
+        e.message.should == "600: No se corresponden token con firma"
       }
     end
 
     it "cuando hay varios errores" do
       savon.expects(:fe_param_get_tipos_cbte).with(message: :any).returns(fixture 'fe_param_get_tipos_cbte/failure_2_errors')
-      lambda { ws.tipos_comprobantes }.should raise_error { |e| 
+      lambda { ws.tipos_comprobantes }.should raise_error { |e|
         e.should be_a Afipws::WSError
-        e.errors.should == [{ code: "600", msg: "No se corresponden token con firma" }, { code: "601", msg: "CUIT representada no incluida en token" }] 
-        e.message.should == "600: No se corresponden token con firma; 601: CUIT representada no incluida en token" 
+        e.errors.should == [{ code: "600", msg: "No se corresponden token con firma" }, { code: "601", msg: "CUIT representada no incluida en token" }]
+        e.message.should == "600: No se corresponden token con firma; 601: CUIT representada no incluida en token"
       }
     end
   end
-  
+
   context "cálculo de fechas y períodos" do
     it "periodo_para_consulta_caea" do
       ws.periodo_para_consulta_caea(Date.new(2011,1,1)).should == { periodo: '201101', orden: 1 }
@@ -280,29 +282,29 @@ describe Afipws::WSFE do
       ws.periodo_para_consulta_caea(Date.new(2011,1,31)).should == { periodo: '201101', orden: 2 }
       ws.periodo_para_consulta_caea(Date.new(2011,2,2)).should == { periodo: '201102', orden: 1 }
     end
-    
+
     it "fecha_inicio_quincena_siguiente" do
       fecha_inicio_quincena_siguiente(Date.new(2010,1,1)).should == Date.new(2010,1,16)
       fecha_inicio_quincena_siguiente(Date.new(2010,1,10)).should == Date.new(2010,1,16)
       fecha_inicio_quincena_siguiente(Date.new(2010,1,15)).should == Date.new(2010,1,16)
-      
+
       fecha_inicio_quincena_siguiente(Date.new(2010,1,16)).should == Date.new(2010,2,1)
       fecha_inicio_quincena_siguiente(Date.new(2010,1,20)).should == Date.new(2010,2,1)
       fecha_inicio_quincena_siguiente(Date.new(2010,1,31)).should == Date.new(2010,2,1)
       fecha_inicio_quincena_siguiente(Date.new(2010,12,31)).should == Date.new(2011,1,1)
     end
-    
+
     def fecha_inicio_quincena_siguiente fecha
       Date.stubs(today: fecha)
       subject.fecha_inicio_quincena_siguiente
     end
   end
-  
+
   context "comprobante_to_request" do
     def c2r comprobante
       subject.comprobante_to_request comprobante
     end
-    
+
     it "no debería enviar tag tributos si el impTrib es 0" do
       c2r(imp_trib: 0.0, tributos: { tributo: [] }).should_not have_key :tributos
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: afipws
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Emmanuel Nicolau
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-08-17 00:00:00.000000000 Z
+date: 2018-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec