br_nfe 1.0.2 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 95a7593fa277c48b625c3acf17b6f8feb0bf5989
-  data.tar.gz: d96d3e26d74c21df7d0195f1091d6b80b6dfa1bb
+  metadata.gz: 11fb152b3151c62d8608888d5ba95174e4d2a435
+  data.tar.gz: 23472fba54d77618bd59b9793295971597513638
 SHA512:
-  metadata.gz: 3b6cd55de324a626cff4dbe8dc4afd9a0498e31263bd366fecbd43b7a3378986effa5a029267fe157bed1b5c39bcb1978ca0e9871bc889826faf54a4f34e4968
-  data.tar.gz: d7db379de7593dff9b50fcb6d63ee158daf540b542ed180149f13ad96d56707ba24d03f0b07fad0dc7516189e3ce7cfdc317c44fb58ec1e3d80501cefefa98ca
+  metadata.gz: 732f08b5b62348717188127aca18718c09063a5a56014512825e12fa4528f0a0a37578a0ebb07febeac1d491e212e8ff3bcc689eb0ba73d5ced80d40efe0fe6b
+  data.tar.gz: 9cdb3eda197abddafaf1625dff5b4d275cae4cbf6ca7cab83e4dd66e34f286483e8943b415c213cf69c698ae53c38684a18e45cc6c024b2b7157aff4cbc49f2f

data/README.markdown

@@ -8,8 +8,8 @@
 # **BrNfe**
 Gem para emissão de notas fiscais eletrônicas.
 
-## Versão 1.0.1
 **Motivo e Objetivo**
+
 Devido a falta de padronização dos parâmetros e a forma de envio e resposta na transmissão de Notas Fiscais de Serviços (NFS), esta gem vem com o objetivo de obter uma forma padronizada dessa tarefa, e assim, facilitando a vida de muitos desenvolvedores.
 
 **O que essa gem faz?**

data/lib/br_nfe/base.rb

@@ -3,22 +3,22 @@ module BrNfe
 		
 		include BrNfe::Helper::HaveEmitente
 
-		attr_accessor :certificado_password
-		attr_accessor :certificado_path
-		attr_accessor :certificado_value
+		attr_accessor :certificate_pkcs12_password
+		attr_accessor :certificate_pkcs12_path
+		attr_accessor :certificate_pkcs12_value
 
 		validate :validar_emitente
-		validates :certificado, presence: true, if: :certificado_obrigatorio?
+		validates :certificate, :certificate_key, presence: true, if: :certificado_obrigatorio?
 
-		# Método que deve ser sobrescrito para as ações que necessitam do certificado para assinatura
+		# Método que deve ser sobrescrito para as ações que necessitam do certificate_pkcs12 para assinatura
 		def certificado_obrigatorio?
 			false
 		end
 
-		# Caso não tenha o certificado salvo em arquivo, pode setar a string do certificado direto pelo atributo certificado_value
-		# Caso tenha o certificado em arquivo, basta setar o atributo certificado_path e deixar o atributo certificado_value em branco
-		def certificado_value
-			@certificado_value ||= File.read(certificado_path)
+		# Caso não tenha o certificate_pkcs12 salvo em arquivo, pode setar a string do certificate_pkcs12 direto pelo atributo certificate_pkcs12_value
+		# Caso tenha o certificate_pkcs12 em arquivo, basta setar o atributo certificate_pkcs12_path e deixar o atributo certificate_pkcs12_value em branco
+		def certificate_pkcs12_value
+			@certificate_pkcs12_value ||= File.read(certificate_pkcs12_path)
 		end
 
 		attr_accessor :env
@@ -60,13 +60,13 @@ module BrNfe
 			{}
 		end
 
-		def certificado
-			@certificado ||= OpenSSL::PKCS12.new(certificado_value, certificado_password)
+		def certificate_pkcs12
+			@certificate_pkcs12 ||= OpenSSL::PKCS12.new(certificate_pkcs12_value, certificate_pkcs12_password)
 		rescue
 		end
 
-		def certificado=(value)
-			@certificado = value
+		def certificate_pkcs12=(value)
+			@certificate_pkcs12 = value
 		end
 
 		def wsdl_encoding
@@ -89,6 +89,22 @@ module BrNfe
 			})
 		end
 
+		def certificate=(value)
+			@certificate = value
+		end
+
+		def certificate
+			@certificate ||= certificate_pkcs12.try :certificate
+		end
+
+		def certificate_key
+			@certificate_key ||= certificate_pkcs12.try :key
+		end
+
+		def certificate_key=(value)
+			@certificate_key = value
+		end
+
 	private
 
 		def tag_cpf_cnpj(xml, cpf_cnpj)
@@ -113,7 +129,7 @@ module BrNfe
 					
 					signature.KeyInfo {
 						signature.X509Data {
-							signature.X509Certificate certificado.certificate.to_s.gsub(/\-\-\-\-\-[A-Z]+ CERTIFICATE\-\-\-\-\-/, "").gsub(/\n/,"")
+							signature.X509Certificate certificate.to_s.gsub(/\-\-\-\-\-[A-Z]+ CERTIFICATE\-\-\-\-\-/, "").gsub(/\n/,"")
 						}
 					}
 				end
@@ -140,7 +156,7 @@ module BrNfe
 
 		def xml_signature_value(xml)
 			sign_canon = canonicalize(xml)
-			signature_hash  = certificado.key.sign(OpenSSL::Digest::SHA1.new, sign_canon)
+			signature_hash  = certificate_key.sign(OpenSSL::Digest::SHA1.new, sign_canon)
 			remove_quebras Base64.encode64( signature_hash )
 		end
 

data/lib/br_nfe/servico/betha/v2/gateway.rb

@@ -121,12 +121,21 @@ module BrNfe
 								xml.IssRetido                 value_true_false(rps.iss_retido?)
 								xml.ResponsavelRetencao       "#{rps.responsavel_retencao}".max_size(1) unless rps.responsavel_retencao.blank?
 								xml.ItemListaServico          BrNfe::Helper.only_number(rps.item_lista_servico).rjust(4,'0').max_size(5) unless rps.item_lista_servico.blank?
-								xml.CodigoCnae                BrNfe::Helper.only_number(rps.codigo_cnae).max_size(7) unless rps.codigo_cnae.blank?
+								
+								# Não devo enviar o CNAE na base de testes pois o municipio enviado será 0 (zero)
+								if env != :test
+									xml.CodigoCnae                BrNfe::Helper.only_number(rps.codigo_cnae).max_size(7) unless rps.codigo_cnae.blank?
+								end
+								
 								xml.CodigoTributacaoMunicipio "#{rps.codigo_tributacao_municipio}".max_size(20)      unless rps.codigo_tributacao_municipio.blank?
 								xml.Discriminacao             "#{rps.discriminacao}".max_size(2_000).remove_accents
-								xml.CodigoMunicipio           BrNfe::Helper.only_number(rps.codigo_municipio).max_size(7)
+								
+								# Na base de testes devo enviar o código do município como 0 (zero)
+								xml.CodigoMunicipio           (env == :test ? 0 : BrNfe::Helper.only_number(rps.codigo_municipio).max_size(7))
+								
 								# Não posso enviar o CodigoPais se a exigibilidade_iss não for 4 (exportação) - by bethagambis.com
 								xml.CodigoPais                BrNfe::Helper.only_number(rps.codigo_pais).max_size(4) if !rps.codigo_pais.blank? && rps.exigibilidade_iss == '4'
+								
 								xml.ExigibilidadeISS          "#{rps.exigibilidade_iss}".max_size(2)
 								xml.MunicipioIncidencia       BrNfe::Helper.only_number(rps.municipio_incidencia).max_size(7) unless rps.municipio_incidencia.blank?
 								xml.NumeroProcesso            rps.numero_processo.max_size(30)     unless rps.numero_processo.blank?

data/lib/br_nfe/servico/rps.rb

@@ -14,6 +14,7 @@ module BrNfe
 				record.validates :data_emissao, :item_lista_servico, :discriminacao, :codigo_municipio, 
 				                 :valor_servicos, :base_calculo, presence: true
 				record.validates :valor_iss, :aliquota, presence: true, unless: :iss_retido?
+				record.validates :municipio_incidencia, presence: true, if: :municipio_incidencia_obrigatorio?
 				
 				record.validates :valor_servicos, :valor_deducoes, :valor_pis, :valor_cofins, :valor_inss, :valor_ir, 
 				          :valor_csll, :outras_retencoes, :valor_iss, :aliquota, :base_calculo, 
@@ -98,6 +99,10 @@ module BrNfe
 					destinatario.errors.full_messages.map{|msg| errors.add(:base, "Destinatário: #{msg}") }
 				end
 			end
+
+			def municipio_incidencia_obrigatorio?
+				"#{exigibilidade_iss}".in?(['1','01','6','06','7','07'])
+			end
 		end
 	end
 end

data/lib/br_nfe/version.rb

@@ -1,8 +1,8 @@
 module BrNfe
 	module Version
 		MAJOR = 1 #inclui alterações de API e pode quebrar compatibilidade com versões anteriores
-		MINOR = 0 #inclui novas funcionalidades, sem quebrar APIs existentes
-		PATCH = 2 #corrige bugs ou traz melhorias em implementações já existentes
+		MINOR = 1 #inclui novas funcionalidades, sem quebrar APIs existentes
+		PATCH = 0 #corrige bugs ou traz melhorias em implementações já existentes
 		CURRENT = "#{MAJOR}.#{MINOR}.#{PATCH}"
 	end
 end

data/lib/br_nfe.rb

@@ -41,7 +41,6 @@ module BrNfe
 	autoload :Destinatario
 	autoload :Response
 	autoload :Base
-	autoload :Seed
 	autoload :CondicaoPagamento
 
 	module Servico

data/test/br_nfe/base_test.rb

@@ -7,19 +7,23 @@ describe BrNfe::Base do
 	let(:emitente) { FactoryGirl.build(:emitente) } 
 
 	before do
-		subject.stubs(:certificado).returns(certificado)
+		subject.stubs(:certificate_pkcs12).returns(certificado)
 	end
 
 	describe "validations" do
 		context "obrigatoriedade do certificado" do
-			before { subject.unstub(:certificado) }
+			before { subject.unstub(:certificate_pkcs12) }
 			it "deve ser obrigatorio se certificado_obrigatorio? for true" do
-				subject.expects(:certificado_obrigatorio?).returns(true)
-				must validate_presence_of(:certificado)
+				subject.stubs(:certificado_obrigatorio?).returns(true)
+				subject.certificate_pkcs12 = nil
+				must validate_presence_of(:certificate)
+				must validate_presence_of(:certificate_key)
 			end
 			it "não deve ser obrigatorio se certificado_obrigatorio? for false" do
-				subject.expects(:certificado_obrigatorio?).returns(false)
-				wont validate_presence_of(:certificado)
+				subject.stubs(:certificado_obrigatorio?).returns(false)
+				subject.certificate_pkcs12 = nil
+				wont validate_presence_of(:certificate)
+				wont validate_presence_of(:certificate_key)
 			end
 		end
 		context "validação do emitente" do
@@ -61,15 +65,15 @@ describe BrNfe::Base do
 		subject.certificado_obrigatorio?.must_equal false
 	end
 
-	describe "#certificado_value" do
+	describe "#certificate_pkcs12_value" do
 		it "se tiver algum valor setado deve retornar esse valor" do
-			subject.certificado_value = "algum valor"
-			subject.certificado_value.must_equal "algum valor"
+			subject.certificate_pkcs12_value = "algum valor"
+			subject.certificate_pkcs12_value.must_equal "algum valor"
 		end
 		it "se não tiver um valor deve carregar o arquivo setado no atributo certificado_path" do
-			subject.certificado_path = "algum/lugar.pfx"
+			subject.certificate_pkcs12_path = "algum/lugar.pfx"
 			File.expects(:read).with("algum/lugar.pfx").returns("valor do arquivo")
-			subject.certificado_value.must_equal "valor do arquivo"
+			subject.certificate_pkcs12_value.must_equal "valor do arquivo"
 		end
 	end
 
@@ -203,7 +207,7 @@ describe BrNfe::Base do
 			it "deve assinar um xml com o certificado" do
 				sh1 = OpenSSL::Digest::SHA1.new
 				OpenSSL::Digest::SHA1.stubs(:new).returns(sh1)
-				subject.certificado.key.expects(:sign).with(sh1, "<Um><Xml> Com valor</Xml></Um>").returns('mv\xBFH\xE3\xF5Z\x0F\xE1*0D')
+				subject.certificate_pkcs12.key.expects(:sign).with(sh1, "<Um><Xml> Com valor</Xml></Um>").returns('mv\xBFH\xE3\xF5Z\x0F\xE1*0D')
 				
 				subject.send(:xml_signature_value, xml).must_equal 'bXZceEJGSFx4RTNceEY1Wlx4MEZceEUxKjBE'
 			end
@@ -260,23 +264,23 @@ describe BrNfe::Base do
 			end
 		end
 
-		describe "#certificado" do
+		describe "#certificate_pkcs12" do
 			before do
-				subject.unstub(:certificado)
+				subject.unstub(:certificate_pkcs12)
 			end
-			it "deve ler o certificado PKCS12 do atributo certificado_value e com a senha do certificado_password" do
-				subject.assign_attributes(certificado_value: "CERTIFICADO", certificado_password: 'pWd123')
-				OpenSSL::PKCS12.expects(:new).with("CERTIFICADO", 'pWd123').returns('certificado')
-				subject.certificado.must_equal 'certificado'
+			it "deve ler o certificate_pkcs12 PKCS12 do atributo certificate_pkcs12_value e com a senha do certificate_pkcs12_password" do
+				subject.assign_attributes(certificate_pkcs12: nil, certificate_pkcs12_value: "CERTIFICADO", certificate_pkcs12_password: 'pWd123')
+				OpenSSL::PKCS12.expects(:new).with("CERTIFICADO", 'pWd123').returns('certificate_pkcs12')
+				subject.certificate_pkcs12.must_equal 'certificate_pkcs12'
 			end
-			it "se já tem um certificado na variavel @certificado não deve ler novamente do PKCS12" do
-				subject.instance_variable_set(:@certificado, certificado)
+			it "se já tem um certificate_pkcs12 na variavel @certificate_pkcs12 não deve ler novamente do PKCS12" do
+				subject.instance_variable_set(:@certificate_pkcs12, certificado)
 				OpenSSL::PKCS12.expects(:new).never
-				subject.certificado.must_equal certificado
+				subject.certificate_pkcs12.must_equal certificado
 			end
-			it "posso setar o certificado" do
-				subject.certificado = 'certificado 123'
-				subject.certificado.must_equal 'certificado 123'
+			it "posso setar o certificate_pkcs12" do
+				subject.certificate_pkcs12 = 'certificate_pkcs12 123'
+				subject.certificate_pkcs12.must_equal 'certificate_pkcs12 123'
 			end
 		end
 

data/test/br_nfe/servico/betha/v1/cancelamento_nfs_test.rb

@@ -11,7 +11,11 @@ describe BrNfe::Servico::Betha::V1::CancelamentoNfs do
 	describe "validations" do
 		it { must validate_presence_of(:numero_nfse) }
 		it { must validate_presence_of(:codigo_cancelamento) }
-		it { must validate_presence_of(:certificado) }
+		context "validações do certificado" do
+			before { subject.certificate_pkcs12 = nil }
+			it { must validate_presence_of(:certificate) }
+			it { must validate_presence_of(:certificate_key) }
+		end
 	end
 
 	describe "#wsdl" do

data/test/br_nfe/servico/betha/v1/recepcao_lote_rps_test.rb

@@ -12,7 +12,11 @@ describe BrNfe::Servico::Betha::V1::RecepcaoLoteRps do
 
 	describe "validations" do
 		it { must validate_presence_of(:numero_lote_rps) }
-		it { must validate_presence_of(:certificado) }
+		context "validações do certificado" do
+			before { subject.certificate_pkcs12 = nil }
+			it { must validate_presence_of(:certificate) }
+			it { must validate_presence_of(:certificate_key) }
+		end
 		
 		it "deve chamar o metodo validar_lote_rps" do
 			subject.expects(:validar_lote_rps)

data/test/br_nfe/servico/betha/v2/cancelamento_nfs_test.rb

@@ -13,7 +13,11 @@ describe BrNfe::Servico::Betha::V2::CancelamentoNfs do
 	describe "validations" do
 		it { must validate_presence_of(:numero_nfse) }
 		it { must validate_presence_of(:codigo_cancelamento) }
-		it { must validate_presence_of(:certificado) }
+		context "validações do certificado" do
+			before { subject.certificate_pkcs12 = nil }
+			it { must validate_presence_of(:certificate) }
+			it { must validate_presence_of(:certificate_key) }
+		end
 	end
 
 	describe "#method_wsdl" do

data/test/br_nfe/servico/betha/v2/envio_lote_rps_sincrono_test.rb

@@ -10,7 +10,12 @@ describe BrNfe::Servico::Betha::V2::EnvioLoteRpsSincrono do
 
 	describe "validations" do
 		it { must validate_presence_of(:numero_lote_rps) }
-		it { must validate_presence_of(:certificado) }
+		
+		context "validações do certificado" do
+			before { subject.certificate_pkcs12 = nil }
+			it { must validate_presence_of(:certificate) }
+			it { must validate_presence_of(:certificate_key) }
+		end
 		
 		it "deve chamar o metodo validar_lote_rps" do
 			subject.expects(:validar_lote_rps)

data/test/br_nfe/servico/betha/v2/gera_nfse_test.rb

@@ -1,7 +1,7 @@
 require 'test_helper'
 
 describe BrNfe::Servico::Betha::V2::GeraNfse do
-	subject        { FactoryGirl.build(:br_nfe_servico_betha_v2_gera_nfse, emitente: emitente, rps: rps, certificado: certificado) }
+	subject        { FactoryGirl.build(:br_nfe_servico_betha_v2_gera_nfse, emitente: emitente, rps: rps, certificate_pkcs12: certificado) }
 	let(:emitente) { FactoryGirl.build(:emitente) }
 	let(:rps)      { FactoryGirl.build(:br_nfe_rps) }
 	let(:certificado) { Certificado.new } 
@@ -11,8 +11,12 @@ describe BrNfe::Servico::Betha::V2::GeraNfse do
 	end
 
 	describe "validations" do
-		it { must validate_presence_of(:certificado) }
-		
+
+		context "validações do certificado" do
+			before { subject.certificate_pkcs12 = nil }
+			it { must validate_presence_of(:certificate) }
+			it { must validate_presence_of(:certificate_key) }
+		end
 		context "deve validar o rps" do
 			it "quando o rps for válido não deve setar nenhuma mensagem no objeto" do
 				rps.stubs(:errors).returns(stub(full_messages: ["Erro rps"]))

data/test/br_nfe/servico/betha/v2/recepcao_lote_rps_test.rb

@@ -10,7 +10,12 @@ describe BrNfe::Servico::Betha::V2::RecepcaoLoteRps do
 
 	describe "validations" do
 		it { must validate_presence_of(:numero_lote_rps) }
-		it { must validate_presence_of(:certificado) }
+		
+		context "validações do certificado" do
+			before { subject.certificate_pkcs12 = nil }
+			it { must validate_presence_of(:certificate) }
+			it { must validate_presence_of(:certificate_key) }
+		end
 		
 		it "deve chamar o metodo validar_lote_rps" do
 			subject.expects(:validar_lote_rps)

data/test/br_nfe/servico/betha/v2/substituicao_nfse_test.rb

@@ -1,7 +1,7 @@
 require 'test_helper'
 
 describe BrNfe::Servico::Betha::V2::SubstituicaoNfse do
-	subject        { FactoryGirl.build(:br_nfe_servico_betha_v2_substituicao_nfse, emitente: emitente, rps: rps, certificado: certificado) }
+	subject        { FactoryGirl.build(:br_nfe_servico_betha_v2_substituicao_nfse, emitente: emitente, rps: rps, certificate_pkcs12: certificado) }
 	let(:emitente) { FactoryGirl.build(:emitente) }
 	let(:rps)      { FactoryGirl.build(:br_nfe_rps) }
 	let(:certificado) { Certificado.new } 
@@ -11,7 +11,11 @@ describe BrNfe::Servico::Betha::V2::SubstituicaoNfse do
 	end
 
 	describe "validations" do
-		it { must validate_presence_of(:certificado) }
+		context "validações do certificado" do
+			before { subject.certificate_pkcs12 = nil }
+			it { must validate_presence_of(:certificate) }
+			it { must validate_presence_of(:certificate_key) }
+		end
 		it { must validate_presence_of(:numero_nfse) }
 		it { must validate_presence_of(:codigo_cancelamento) }
 		

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: br_nfe
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.1.0
 platform: ruby
 authors:
 - Bruno M. Mergen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-29 00:00:00.000000000 Z
+date: 2015-10-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake