br_nfe 1.1.0 → 2.0.0

data/test/br_nfe/service/concerns/rules/cancelamento_nfs_test.rb

@@ -0,0 +1,15 @@
+require 'test_helper'
+
+class RuleCancelamentoNfsTest < BrNfe::Service::Base
+	include BrNfe::Service::Concerns::Rules::CancelamentoNfs
+end
+
+describe BrNfe::Service::Concerns::Rules::CancelamentoNfs do
+	subject { RuleCancelamentoNfsTest.new(nfe_number: 1254, codigo_cancelamento: 1, emitente: emitente) }
+	let(:emitente) { FactoryGirl.build(:emitente, endereco: endereco) }
+	let(:endereco) { FactoryGirl.build(:endereco) } 
+	
+	it { must validate_presence_of(:nfe_number) }
+	it { must validate_presence_of(:codigo_cancelamento) }
+	
+end

data/test/br_nfe/service/concerns/rules/consulta_nfs_por_rps_test.rb

@@ -0,0 +1,36 @@
+require 'test_helper'
+require 'br_nfe/helper/have_rps_test'
+
+
+describe BrNfe::Service::Concerns::Rules::ConsultaNfsPorRps do
+	class RuleConsultaNfsPorRpsTest < BrNfe::Service::Base
+		include BrNfe::Service::Concerns::Rules::ConsultaNfsPorRps
+	end
+
+	subject { RuleConsultaNfsPorRpsTest.new(rps: rps, emitente: emitente) }
+	let(:emitente) { FactoryGirl.build(:emitente, endereco: endereco) }
+	let(:endereco) { FactoryGirl.build(:endereco) } 
+	let(:rps) { FactoryGirl.build(:br_nfe_rps) } 
+
+	it "deve ter o o helper HaveRps incluido" do
+		subject.class.included_modules.must_include BrNfe::Helper::HaveRps
+	end
+
+	describe "rps" do
+		include BrNfeTest::HelperTest::HaveRpsTest
+	end
+
+	describe "#validate_rps" do
+		it "se o RPS for inválido deve add os erros do RPS no registro" do
+			rps.stubs(:errors).returns(stub(full_messages: ["Erro rps"]))
+			rps.stubs(:valid?).returns(false)
+			subject.valid?.must_equal false
+			subject.errors.full_messages.must_include( "Rps Erro rps" )
+		end
+		it "se o RPS for válido o registro também será valido" do
+			rps.stubs(:valid?).returns(true)
+			subject.valid?.must_equal true
+			subject.errors.must_be_empty
+		end
+	end
+end

data/test/br_nfe/service/concerns/rules/consulta_nfse_test.rb

@@ -0,0 +1,42 @@
+require 'test_helper'
+
+class RuleConsultaNfseTest < BrNfe::Service::Base
+	include BrNfe::Service::Concerns::Rules::ConsultaNfse
+end
+
+describe BrNfe::Service::Concerns::Rules::ConsultaNfse do
+	subject { RuleConsultaNfseTest.new(nfe_number: '554', start_date: 1.month.ago, end_date: Time.now, emitente: emitente) }
+	let(:emitente) { FactoryGirl.build(:emitente, endereco: endereco) }
+	let(:endereco) { FactoryGirl.build(:endereco) } 
+	let(:rps) { FactoryGirl.build(:br_nfe_rps) } 
+
+	it "deve ter o o helper HaveDestinatario incluido" do
+		subject.class.included_modules.must_include BrNfe::Helper::HaveDestinatario
+	end
+
+	it "deve ter o o helper HaveIntermediario incluido" do
+		subject.class.included_modules.must_include BrNfe::Helper::HaveIntermediario
+	end
+
+	describe "#start_date" do
+		it "o get do start_date deve retornar uma data no formato YYYY-MM-DD" do
+			subject.start_date = '20/05/2010'
+			subject.start_date.must_equal '2010-05-20'
+		end
+		it "se start_date for nil o get deve retornar em branco" do
+			subject.start_date = nil
+			subject.start_date.must_equal ''
+		end
+	end
+
+	describe "#end_date" do
+		it "o get do end_date deve retornar uma data no formato YYYY-MM-DD" do
+			subject.end_date = '20/08/2015'
+			subject.end_date.must_equal '2015-08-20'
+		end
+		it "se end_date for nil o get deve retornar em branco" do
+			subject.end_date = nil
+			subject.end_date.must_equal ''
+		end		
+	end
+end

data/test/br_nfe/{servico/base_test.rb → service/concerns/rules/recepcao_lote_rps_test.rb}

@@ -1,9 +1,17 @@
 require 'test_helper'
 
-describe BrNfe::Servico::Base do
-	subject { FactoryGirl.build(:br_nfe_servico_base) }
+class RuleRecepcaoLoteRpsTest < BrNfe::Service::Base
+	include BrNfe::Service::Concerns::Rules::RecepcaoLoteRps
+end
+
+describe BrNfe::Service::Concerns::Rules::RecepcaoLoteRps do
+	subject { RuleRecepcaoLoteRpsTest.new(numero_lote_rps: 545, operacao: '1', emitente: emitente) }
+	let(:emitente) { FactoryGirl.build(:emitente, endereco: endereco) }
+	let(:endereco) { FactoryGirl.build(:endereco) } 
 	let(:rps) { FactoryGirl.build(:br_nfe_rps) } 
-		
+
+	it { must validate_presence_of(:numero_lote_rps) }
+
 	describe "#lote_rps" do
 		it "deve inicializar como um vetor vazio" do
 			subject.class.new.lote_rps.must_equal []
@@ -25,7 +33,7 @@ describe BrNfe::Servico::Base do
 		it "se não houver nenhum rps, deve add um erro" do
 			subject.lote_rps = []
 			subject.errors.full_messages.must_equal( [] )
-			subject.send(:validar_lote_rps)
+			subject.valid?.must_equal false
 			subject.errors.full_messages.must_equal( ["Deve conter ao menos 1 RPS"] )
 		end
 		context "deve validar o rps setando a opcao validar_recepcao_rps" do

data/test/br_nfe/{servico → service}/intermediario_test.rb

@@ -1,6 +1,6 @@
 require 'test_helper'
 
-describe BrNfe::Servico::Intermediario do
+describe BrNfe::Service::Intermediario do
 	subject { FactoryGirl.build(:intermediario) }
 	describe "validations" do
 		it { must validate_presence_of(:cpf_cnpj) }

data/test/br_nfe/service/item_test.rb

@@ -0,0 +1,34 @@
+require 'test_helper'
+
+describe BrNfe::Service::Item do
+	subject { FactoryGirl.build(:service_item) }
+	
+	describe "#default_values" do
+		context "#quantity" do
+			it "valor padrão deve ser 1" do
+				subject.class.new.quantity.must_equal 1
+			end
+			it "posso modificar o valor" do
+				subject.class.new(quantity: 50.88).quantity.must_equal 50.88			
+			end			
+		end
+	end
+
+	describe "#total_value" do
+		it "se não for setado nenhum valor deve sempre retornar a multiplicação entre a quantidade e o valor unitário" do
+			subject.total_value = nil
+			subject.assign_attributes(quantity: 3.5, unit_value: 10.0)
+			subject.total_value.must_equal 35.0
+		end
+		it "se setar um valor em total_value não deve multiplicar a quantidade com o valor unitário" do
+			subject.total_value = 66.0
+			subject.assign_attributes(quantity: 3.5, unit_value: 10.0)
+			subject.total_value.must_equal 66.0
+		end
+		it "se quantidade ou valor unitário forem nil deve retornar zero" do
+			subject.total_value = nil
+			subject.assign_attributes(quantity: nil, unit_value: nil)
+			subject.total_value.must_equal 0.0
+		end
+	end
+end

data/test/br_nfe/{servico → service}/rps_test.rb

@@ -1,37 +1,45 @@
 require 'test_helper'
 
-describe BrNfe::Servico::Rps do
+describe BrNfe::Service::Rps do
 	subject { FactoryGirl.build(:br_nfe_rps) }
 	let(:destinatario)  { FactoryGirl.build(:destinatario) }
 	let(:intermediario) { FactoryGirl.build(:intermediario) }
 	let(:condicao_pagamento) { FactoryGirl.build(:condicao_pagamento) }
+	let(:item_1) { FactoryGirl.build(:service_item) } 
+	let(:item_2) { FactoryGirl.build(:service_item) } 
 
 	describe "validations" do
 		it { must validate_presence_of(:numero) }
 		it { must validate_presence_of(:serie) }
 		it { must validate_presence_of(:tipo) }
+		context "Quando #validate_rps? for false os valores de RPS não devem ser obrigatórios" do
+			before { subject.stubs(:validate_rps?).returns(false) }
+			it { wont validate_presence_of(:numero) }
+			it { wont validate_presence_of(:serie) }
+			it { wont validate_presence_of(:tipo) }
+		end
 
 		describe "option validar_recepcao_rps" do
 			context "quando for true" do
 				before { subject.validar_recepcao_rps = true }
 				it { must validate_presence_of(:data_emissao) }
 				it { must validate_presence_of(:item_lista_servico) }
-				it { must validate_presence_of(:discriminacao) }
+				it { must validate_presence_of(:description) }
 				it { must validate_presence_of(:codigo_municipio) }
-				it { must validate_presence_of(:valor_servicos) }
-				it { must validate_presence_of(:base_calculo) }
+				it { must validate_numericality_of(:total_services).is_greater_than(0) }
+				it { must validate_numericality_of(:base_calculation).is_greater_than(0) }
 				
-				it { must validate_numericality_of(:valor_servicos) }
-				it { must validate_numericality_of(:valor_deducoes) }
+				it { must validate_numericality_of(:total_services) }
+				it { must validate_numericality_of(:deductions) }
 				it { must validate_numericality_of(:valor_pis) }
 				it { must validate_numericality_of(:valor_cofins) }
 				it { must validate_numericality_of(:valor_inss) }
 				it { must validate_numericality_of(:valor_ir) }
 				it { must validate_numericality_of(:valor_csll) }
 				it { must validate_numericality_of(:outras_retencoes) }
-				it { must validate_numericality_of(:valor_iss) }
-				it { must validate_numericality_of(:aliquota) }
-				it { must validate_numericality_of(:base_calculo) }
+				it { must validate_numericality_of(:total_iss) }
+				it { must validate_numericality_of(:iss_tax_rate) }
+				it { must validate_numericality_of(:base_calculation) }
 				it { must validate_numericality_of(:desconto_incondicionado) }
 				it { must validate_numericality_of(:desconto_condicionado) }
 
@@ -45,16 +53,16 @@ describe BrNfe::Servico::Rps do
 					subject.valid?
 				end
 
-				describe "option iss_retido?" do
+				describe "option iss_retained?" do
 					context "quando for true" do
-						before { subject.stubs(:iss_retido?).returns(true) }
-						it { wont validate_presence_of(:valor_iss) }
-						it { wont validate_presence_of(:aliquota) }
+						before { subject.stubs(:iss_retained?).returns(true) }
+						it { wont validate_presence_of(:total_iss) }
+						it { wont validate_presence_of(:iss_tax_rate) }
 					end
 					context "quando for false" do
-						before { subject.stubs(:iss_retido?).returns(false) }
-						it { must validate_presence_of(:valor_iss) }
-						it { must validate_presence_of(:aliquota) }
+						before { subject.stubs(:iss_retained?).returns(false) }
+						it { must validate_presence_of(:total_iss) }
+						it { must validate_presence_of(:iss_tax_rate) }
 					end
 				end
 			end
@@ -63,22 +71,22 @@ describe BrNfe::Servico::Rps do
 				before { subject.validar_recepcao_rps = false }
 				it { wont validate_presence_of(:data_emissao) }
 				it { wont validate_presence_of(:item_lista_servico) }
-				it { wont validate_presence_of(:discriminacao) }
+				it { wont validate_presence_of(:description) }
 				it { wont validate_presence_of(:codigo_municipio) }
-				it { wont validate_presence_of(:valor_servicos) }
-				it { wont validate_presence_of(:base_calculo) }
+				it { wont validate_numericality_of(:total_services).is_greater_than(0) }
+				it { wont validate_numericality_of(:base_calculation).is_greater_than(0) }
 				
-				it { wont validate_numericality_of(:valor_servicos) }
-				it { wont validate_numericality_of(:valor_deducoes) }
+				it { wont validate_numericality_of(:total_services) }
+				it { wont validate_numericality_of(:deductions) }
 				it { wont validate_numericality_of(:valor_pis) }
 				it { wont validate_numericality_of(:valor_cofins) }
 				it { wont validate_numericality_of(:valor_inss) }
 				it { wont validate_numericality_of(:valor_ir) }
 				it { wont validate_numericality_of(:valor_csll) }
 				it { wont validate_numericality_of(:outras_retencoes) }
-				it { wont validate_numericality_of(:valor_iss) }
-				it { wont validate_numericality_of(:aliquota) }
-				it { wont validate_numericality_of(:base_calculo) }
+				it { wont validate_numericality_of(:total_iss) }
+				it { wont validate_numericality_of(:iss_tax_rate) }
+				it { wont validate_numericality_of(:base_calculation) }
 				it { wont validate_numericality_of(:desconto_incondicionado) }
 				it { wont validate_numericality_of(:desconto_condicionado) }
 
@@ -92,16 +100,16 @@ describe BrNfe::Servico::Rps do
 					subject.valid?
 				end
 
-				describe "option iss_retido?" do
+				describe "option iss_retained?" do
 					context "quando for true" do
-						before { subject.stubs(:iss_retido?).returns(true) }
-						it { wont validate_presence_of(:valor_iss) }
-						it { wont validate_presence_of(:aliquota) }
+						before { subject.stubs(:iss_retained?).returns(true) }
+						it { wont validate_presence_of(:total_iss) }
+						it { wont validate_presence_of(:iss_tax_rate) }
 					end
 					context "quando for false" do
-						before { subject.stubs(:iss_retido?).returns(false) }
-						it { wont validate_presence_of(:valor_iss) }
-						it { wont validate_presence_of(:aliquota) }
+						before { subject.stubs(:iss_retained?).returns(false) }
+						it { wont validate_presence_of(:total_iss) }
+						it { wont validate_presence_of(:iss_tax_rate) }
 					end
 				end
 			end
@@ -109,31 +117,31 @@ describe BrNfe::Servico::Rps do
 		
 	end
 
-	describe "#contem_substituicao?" do
+	describe "#replace_invoice?" do
 		before do
 			subject.assign_attributes(numero_substituicao: '1', serie_substituicao: '2', tipo_substituicao: '1')
 		end
 		it "true se numero_substituicao, serie_substituicao e tipo_substituicao tiver algum valor" do
-			subject.contem_substituicao?.must_equal true
+			subject.replace_invoice?.must_equal true
 		end
 		it "false se numero_substituicao não tiver valor" do
 			subject.numero_substituicao = ''
-			subject.contem_substituicao?.must_equal false
+			subject.replace_invoice?.must_equal false
 		end
 		it "false se serie_substituicao não tiver valor" do
 			subject.serie_substituicao = ''
-			subject.contem_substituicao?.must_equal false
+			subject.replace_invoice?.must_equal false
 		end
 		it "false se tipo_substituicao não tiver valor" do
 			subject.tipo_substituicao = ''
-			subject.contem_substituicao?.must_equal false
+			subject.replace_invoice?.must_equal false
 		end
 	end
 
 
 	describe "#destinatario" do
 		it "sempre deve retornar um objeto BrNfe::Destinatario" do
-			novo = BrNfe::Servico::Rps.new
+			novo = BrNfe::Service::Rps.new
 			novo.destinatario.class.must_equal BrNfe::Destinatario
 			novo.destinatario = nil
 			novo.destinatario.class.must_equal BrNfe::Destinatario
@@ -144,7 +152,7 @@ describe BrNfe::Servico::Rps do
 		it "a classe do destinatario deve segir a configuração de BrNfe.destinatario_class" do
 			BrNfe.destinatario_class = BrNfe::Emitente
 			
-			novo = BrNfe::Servico::Rps.new
+			novo = BrNfe::Service::Rps.new
 			novo.destinatario.class.must_equal BrNfe::Emitente
 			novo.destinatario = nil
 			novo.destinatario.class.must_equal BrNfe::Emitente
@@ -189,7 +197,7 @@ describe BrNfe::Servico::Rps do
 
 	describe "#intermediario" do
 		it "valor padrão é nil" do
-			novo = BrNfe::Servico::Rps.new
+			novo = BrNfe::Service::Rps.new
 			novo.intermediario.must_be_nil
 			novo.intermediario = 'outro valor qualquer'
 			novo.intermediario.must_be_nil
@@ -198,7 +206,7 @@ describe BrNfe::Servico::Rps do
 		it "a classe do intermediario deve segir a configuração de BrNfe.intermediario_class" do
 			BrNfe.intermediario_class = BrNfe::Emitente
 			
-			novo = BrNfe::Servico::Rps.new(intermediario: BrNfe::Emitente.new)
+			novo = BrNfe::Service::Rps.new(intermediario: BrNfe::Emitente.new)
 			novo.intermediario.class.must_equal BrNfe::Emitente
 			
 			novo.intermediario = nil
@@ -207,7 +215,7 @@ describe BrNfe::Servico::Rps do
 			novo.intermediario = 'outro valor qualquer'
 			novo.intermediario.must_be_nil
 
-			BrNfe.intermediario_class = BrNfe::Servico::Intermediario
+			BrNfe.intermediario_class = BrNfe::Service::Intermediario
 		end
 
 		it "atributos pode ser atribuidos em forma de bloco" do
@@ -259,7 +267,7 @@ describe BrNfe::Servico::Rps do
 
 	describe "#condicao_pagamento" do
 		it "valor padrão é nil" do
-			novo = BrNfe::Servico::Rps.new
+			novo = BrNfe::Service::Rps.new
 			novo.condicao_pagamento.must_be_nil
 			novo.condicao_pagamento = 'outro valor qualquer'
 			novo.condicao_pagamento.must_be_nil
@@ -268,7 +276,7 @@ describe BrNfe::Servico::Rps do
 		it "a classe do condicao_pagamento deve segir a configuração de BrNfe.condicao_pagamento_class" do
 			BrNfe.condicao_pagamento_class = BrNfe::Emitente
 			
-			novo = BrNfe::Servico::Rps.new(condicao_pagamento: BrNfe::Emitente.new)
+			novo = BrNfe::Service::Rps.new(condicao_pagamento: BrNfe::Emitente.new)
 			novo.condicao_pagamento.class.must_equal BrNfe::Emitente
 			
 			novo.condicao_pagamento = nil
@@ -352,4 +360,144 @@ describe BrNfe::Servico::Rps do
 		end
 	end
 
+	describe "#items" do
+		it "deve inicializar como um vetor vazio" do
+			subject.class.new.items.must_equal []
+		end
+
+		it "posso utilizar o << para adicionar valores" do
+			subject.items << item_1
+			subject.items << item_2
+			subject.items.must_equal [item_1,item_2]
+		end
+
+		it "sempre retorna um array" do
+			subject.items = item_2
+			subject.items.must_equal [item_2]
+		end
+
+		it "não permite adicionar outro objeto a não ser o definido pela class BrNfe.service_item_class" do
+			class CustomClass
+			end
+			c_class_1 = CustomClass.new
+			c_class_2 = CustomClass.new
+			BrNfe.service_item_class = CustomClass
+
+			subject.items << c_class_1
+			subject.items << c_class_2
+			subject.items << item_1
+			subject.items.must_equal [c_class_1, c_class_2]
+
+			subject.items = 'item_2'
+			subject.items.must_equal []
+
+			BrNfe.service_item_class = BrNfe::Service::Item
+		end
+	end
+
+	describe "#total_services" do
+		it "se não setar valor em total_services deve somar o total_value de todos os items" do
+			subject.total_services = nil
+			item_1.total_value = 55.70
+			item_2.total_value = 40.35
+			subject.items = [item_1, item_2]
+
+			subject.total_services.must_equal 96.05
+		end
+		it "se não houver itens e nem um valor setado deve retornar zero" do
+			subject.total_services = nil
+			subject.items = []
+
+			subject.total_services.must_equal 0.0
+		end
+		it "se setar um valor deve retornar esse valor e não pode somar dos itens" do
+			subject.total_services = 47.0
+			item_1.total_value = 55.70
+			item_2.total_value = 40.35
+			subject.items = [item_1, item_2]
+
+			subject.total_services.must_equal 47.0
+		end
+	end
+
+	describe "#base_calculation" do
+		it "se não setar valor em base_calculation deve subtratir o total_services com deductions" do
+			subject.assign_attributes({
+				base_calculation: nil,
+				total_services: 100.0,
+				deductions: 4.500000555
+			})
+
+			subject.base_calculation.must_equal 95.5
+		end
+		it "deve retornar zero se não houver valor em base_calculation e nem em total_services ou deductions" do
+			subject.assign_attributes({
+				base_calculation: nil,
+				total_services:   nil,
+				deductions:       nil
+			})
+
+			subject.base_calculation.must_equal 0.0
+		end
+		it "se setar um valor deve retornar esse valor e não pode celcular o total_services subtraindo as deductions" do
+			subject.assign_attributes({
+				base_calculation: 200,
+				total_services:   100,
+				deductions:       50
+			})
+
+			subject.base_calculation.must_equal 200
+		end
+	end
+
+	describe "#iss_tax_rate" do
+		it "deve pegar o valor de iss_tax_rate do primeiro item se não houver valor setado em iss_tax_rate do RPS" do
+			subject.iss_tax_rate = nil
+			item_1.iss_tax_rate = 0.3
+			item_2.iss_tax_rate = 0.4
+			subject.items = [item_2, item_1]
+
+			subject.iss_tax_rate.must_equal 0.4
+		end
+		it "Deve retornar nil se não houver itens e nem um valor setado" do
+			subject.iss_tax_rate = nil
+			subject.items = []
+
+			subject.iss_tax_rate.must_be_nil
+		end
+		it "se setar um valor deve retornar esse valor e não pode pegar dos itens" do
+			subject.iss_tax_rate = 0.4
+			item_1.total_value = 0.1
+			item_2.total_value = 0.2
+			subject.items = [item_1, item_2]
+
+			subject.iss_tax_rate.must_equal 0.4
+		end
+	end
+
+	describe "#cnae_code" do
+		it "deve pegar o valor de cnae_code do primeiro item se não houver valor setado em cnae_code do RPS" do
+			subject.cnae_code = nil
+			item_1.cnae_code = '11111'
+			item_2.cnae_code = '22222'
+			subject.items = [item_2, item_1]
+
+			subject.cnae_code.must_equal '22222'
+		end
+		it "Deve retornar nil se não houver itens e nem um valor setado" do
+			subject.cnae_code = nil
+			subject.items = []
+
+			subject.cnae_code.must_be_nil
+		end
+		it "se setar um valor deve retornar esse valor e não pode pegar dos itens" do
+			subject.cnae_code  = '2'
+			item_1.total_value = '3'
+			item_2.total_value = '4'
+			subject.items = [item_1, item_2]
+
+			subject.cnae_code.must_equal '2'
+		end
+	end
+
 end