br_danfe 0.16.0 → 0.17.3

data/spec/br_danfe/danfe_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe BrDanfe::Danfe do
   let(:xml) do
-    <<-eos
+    <<~XML
       <nfeProc>
         <NFe>
           <infNFe>
@@ -17,7 +17,7 @@ describe BrDanfe::Danfe do
           </infNFe>
         </NFe>
       </nfeProc>
-    eos
+    XML
   end
 
   subject { described_class.new(xml) }

data/spec/br_danfe/helper_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe BrDanfe::Helper do
   describe '.no_fiscal_value?' do
     let(:xml_homologation) do
-      xml = <<-eos
+      xml = <<~XML
         <nfeProc>
           <NFe>
             <infNFe>
@@ -18,25 +18,25 @@ describe BrDanfe::Helper do
             </protNFe>
           </NFe>
         </nfeProc>
-      eos
+      XML
 
       Nokogiri::XML(xml)
     end
 
     let(:xml_unauthorized) do
-      xml = <<-eos
+      xml = <<~XML
         <nfeProc>
           <protNFe>
             <infProt></infProt>
           </protNFe>
         </nfeProc>
-      eos
+      XML
 
       Nokogiri::XML(xml)
     end
 
     let(:xml_authorized) do
-      xml = <<-eos
+      xml = <<~XML
         <nfeProc>
           <NFe>
             <infNFe>
@@ -51,7 +51,7 @@ describe BrDanfe::Helper do
             </infProt>
           </protNFe>
         </nfeProc>
-      eos
+      XML
 
       Nokogiri::XML(xml)
     end
@@ -78,13 +78,13 @@ describe BrDanfe::Helper do
   describe '.unauthorized?' do
     context 'when XML is unauthorized' do
       let(:xml_unauthorized) do
-        xml = <<-eos
+        xml = <<~XML
           <nfeProc>
             <protNFe>
               <infProt></infProt>
             </protNFe>
           </nfeProc>
-        eos
+        XML
 
         Nokogiri::XML(xml)
       end
@@ -96,7 +96,7 @@ describe BrDanfe::Helper do
 
     context 'when XML is authorized' do
       let(:xml_authorized) do
-        xml = <<-eos
+        xml = <<~XML
           <nfeProc>
             <protNFe>
               <infProt>
@@ -104,7 +104,7 @@ describe BrDanfe::Helper do
               </infProt>
             </protNFe>
           </nfeProc>
-        eos
+        XML
 
         Nokogiri::XML(xml)
       end
@@ -118,7 +118,7 @@ describe BrDanfe::Helper do
   describe '.homologation?' do
     context 'when tpAmb is equal to "2"' do
       let(:xml_homologation) do
-        xml = <<-eos
+        xml = <<~XML
           <nfeProc>
             <NFe>
               <infNFe>
@@ -128,7 +128,7 @@ describe BrDanfe::Helper do
               </infNFe>
             </NFe>
           </nfeProc>
-        eos
+        XML
 
         Nokogiri::XML(xml)
       end
@@ -140,7 +140,7 @@ describe BrDanfe::Helper do
 
     context 'when tpAmb is different to "2"' do
       let(:xml_production) do
-        xml = <<-eos
+        xml = <<~XML
           <nfeProc>
             <NFe>
               <infNFe>
@@ -150,7 +150,7 @@ describe BrDanfe::Helper do
               </infNFe>
             </NFe>
           </nfeProc>
-        eos
+        XML
 
         Nokogiri::XML(xml)
       end
@@ -214,7 +214,7 @@ describe BrDanfe::Helper do
     end
 
     describe 'when has timezone' do
-      it'returns the formated string with the timezone' do
+      it 'returns the formated string with the timezone' do
         expect(described_class.format_datetime('2013-10-18T13:54:04-03:00', with_time_zone: true)).to eql '18/10/2013 13:54:04-03:00'
       end
     end

data/spec/br_danfe/mdfe_lib/document_spec.rb

@@ -7,13 +7,13 @@ describe BrDanfe::MdfeLib::Document do
   describe '#method_missing' do
     context 'when prawn has the method' do
       it 'does not throws an error' do
-        expect { subject.text }.not_to raise_error NameError
+        expect(subject).to respond_to :text
       end
     end
 
     context 'when prawn does not have the method' do
       it 'throws an error' do
-        expect { subject.any_method }.to raise_error NameError
+        expect { subject.non_existent_method }.to raise_error NameError
       end
     end
   end

data/spec/br_danfe/mdfe_lib/fisco_control_spec.rb

@@ -27,13 +27,11 @@ describe BrDanfe::MdfeLib::FiscoControl do
   end
 
   describe '#generate' do
-    after { File.delete(output_pdf) if File.exist?(output_pdf) }
+    before { File.delete(output_pdf) if File.exist?(output_pdf) }
 
     it 'generates the title' do
-      title = 'CONTROLE DO FISCO'
-
       subject.generate
-      expect(pdf_text).to include title
+      expect(pdf_text).to include 'CONTROLE DO FISCO'
     end
 
     it 'generates the bar code' do
@@ -46,10 +44,26 @@ describe BrDanfe::MdfeLib::FiscoControl do
     end
 
     it 'generates the nfe key' do
-      nfe_key = "Chave de Acesso\n32210717781119000141580010000001211000000003"
-
       subject.generate
-      expect(pdf_text).to include nfe_key
+      expect(pdf_text).to include "Chave de Acesso\n32210717781119000141580010000001211000000003"
+    end
+
+    context 'when the XML does not have protMDFe tag' do
+      let(:xml_as_string) do
+        <<~XML
+          <?xml version="1.0" encoding="UTF-8"?>
+          <mdfeProc xmlns="http://www.portalfiscal.inf.br/mdfe" versao="3.00"></mdfeProc>
+        XML
+      end
+
+      it 'does not generate the bar code and the nfe key' do
+        expect(File.exist?(output_pdf)).to be false
+
+        subject.generate
+        pdf.render_file output_pdf
+
+        expect("#{base_dir}fisco_control#without-barcode.pdf").to have_same_content_of file: output_pdf
+      end
     end
   end
 end

data/spec/br_danfe/mdfe_lib/header_spec.rb

@@ -1,6 +1,10 @@
 require 'spec_helper'
 
 describe BrDanfe::MdfeLib::Header do
+  let(:qr_code_mdfe) do
+    url = 'https://dfe-portal.svrs.rs.gov.br/mdfe/QRCode?chMDFe=32210717781119000141580010000001211000000003&tpAmb=1'
+    "<qrCodMDFe>#{url}</qrCodMDFe>"
+  end
   let(:xml_as_string) do
     <<~XML
       <?xml version="1.0" encoding="UTF-8"?>
@@ -32,19 +36,14 @@ describe BrDanfe::MdfeLib::Header do
             </emit>
           </infMDFe>
           <infMDFeSupl>
-            <qrCodMDFe>https://dfe-portal.svrs.rs.gov.br/mdfe/QRCode?chMDFe=32210717781119000141580010000001211000000003&amp;tpAmb=1</qrCodMDFe>
+            #{qr_code_mdfe}
           </infMDFeSupl>
         </MDFe>
       </mdfeProc>
     XML
   end
-
   let(:pdf) { BrDanfe::MdfeLib::Document.new }
   let(:xml) { BrDanfe::XML.new(xml_as_string) }
-
-  let(:base_dir) { './spec/fixtures/mdfe/lib/' }
-  let(:output_pdf) { "#{base_dir}output.pdf" }
-
   let(:logo_dimensions) { { width: 100, height: 100 } }
   let(:logo) { 'spec/fixtures/logo.png' }
 
@@ -54,9 +53,12 @@ describe BrDanfe::MdfeLib::Header do
     PDF::Inspector::Text.analyze(pdf.render).strings.join("\n")
   end
 
-  after { File.delete(output_pdf) if File.exist?(output_pdf) }
-
   describe '#generate' do
+    let(:base_dir) { './spec/fixtures/mdfe/lib/' }
+    let(:output_pdf) { "#{base_dir}output.pdf" }
+
+    before { File.delete(output_pdf) if File.exist?(output_pdf) }
+
     it 'generates the emitter informations' do
       emitter_name = 'VENTURIM AGROCRIATIVA LTDA EPP'
       emitter_address = "RODOVIA ES 473 KM 13, nº 0\nVENDA NOVA DO IMIGRANTE - ES   CEP 29.375-000"
@@ -72,15 +74,12 @@ describe BrDanfe::MdfeLib::Header do
       expect(pdf_text).to include emitter_ie
     end
 
-    it'generates the DAMDFE title' do
-      title = "DAMDFE\n - Documento Auxiliar de Manifesto Eletrônico de Documentos Fiscais"
-
+    it 'generates the DAMDFE title' do
       subject.generate
-
-      expect(pdf_text).to include title
+      expect(pdf_text).to include "DAMDFE\n - Documento Auxiliar de Manifesto Eletrônico de Documentos Fiscais"
     end
 
-    it 'generates the qr code' do
+    it 'generates the QR-code' do
       expect(File.exist?(output_pdf)).to be false
 
       subject.generate
@@ -89,12 +88,22 @@ describe BrDanfe::MdfeLib::Header do
       expect("#{base_dir}header#render-qr-code.pdf").to have_same_content_of file: output_pdf
     end
 
-    it 'generates the company CNPJ' do
-      cnpj = "CNPJ: \n17781119000141"
+    context 'if the QR-code URL is not present' do
+      let(:qr_code_mdfe) { '<qrCodMDFe></qrCodMDFe>' }
 
-      subject.generate
+      it 'does not generates the QR-code' do
+        expect(File.exist?(output_pdf)).to be false
+
+        subject.generate
+        pdf.render_file output_pdf
+
+        expect("#{base_dir}header#render-without-qr-code.pdf").to have_same_content_of file: output_pdf
+      end
+    end
 
-      expect(pdf_text).to include cnpj
+    it 'generates the company CNPJ' do
+      subject.generate
+      expect(pdf_text).to include "CNPJ: \n17781119000141"
     end
 
     describe 'logo' do

data/spec/br_danfe/mdfe_lib/notes_spec.rb

@@ -22,9 +22,6 @@ describe BrDanfe::MdfeLib::Notes do
     XML
   end
 
-  let(:base_dir) { './spec/fixtures/mdfe/lib/' }
-  let(:output_pdf) { "#{base_dir}output.pdf" }
-
   let(:pdf) { BrDanfe::MdfeLib::Document.new }
   let(:xml) { BrDanfe::XML.new(xml_as_string) }
 
@@ -37,6 +34,11 @@ describe BrDanfe::MdfeLib::Notes do
   before { pdf.move_cursor_to 345 }
 
   describe '#generate' do
+    let(:base_dir) { './spec/fixtures/mdfe/lib/' }
+    let(:output_pdf) { "#{base_dir}output.pdf" }
+
+    before { File.delete(output_pdf) if File.exist?(output_pdf) }
+
     it 'generates the title' do
       title = 'Observações'
 
@@ -76,8 +78,6 @@ describe BrDanfe::MdfeLib::Notes do
       expect(pdf_text).not_to include taxpayer_information
     end
 
-    after { File.delete(output_pdf) if File.exist?(output_pdf) }
-
     it 'creates a new page if aditional information do not fit on first page' do
       expect(File.exist?(output_pdf)).to be false
 

data/spec/br_danfe/mdfe_lib/totalizer_spec.rb

@@ -53,7 +53,7 @@ describe BrDanfe::MdfeLib::Totalizer do
       expect(pdf_text).to include total_weight
     end
 
-    it'generates the totalizer title' do
+    it 'generates the totalizer title' do
       title = 'Modelo Rodoviário de Carga'
 
       subject.generate

data/spec/br_danfe/mdfe_spec.rb

@@ -4,12 +4,11 @@ describe BrDanfe::Mdfe do
   let(:base_dir) { './spec/fixtures/mdfe/lib/' }
   let(:output_pdf) { "#{base_dir}output.pdf" }
 
-  let(:xml_as_string) { File.read('./spec/fixtures/mdfe/lib/xml-mdfe.xml') }
-
-  let(:xml) { BrDanfe::XML.new(xml_as_string) }
+  let(:xml) { File.read('./spec/fixtures/mdfe/lib/xml-mdfe.xml') }
 
   subject { described_class.new(xml) }
-  after { File.delete(output_pdf) if File.exist?(output_pdf) }
+
+  before { File.delete(output_pdf) if File.exist?(output_pdf) }
 
   before do
     subject.logo_options.logo_dimensions = { width: 100, height: 100 }
@@ -34,7 +33,7 @@ describe BrDanfe::Mdfe do
   end
 
   context 'when MDF-e has multiple pages' do
-    let(:xml_as_string) { File.read('./spec/fixtures/mdfe/lib/xml-mdfe-multiples-pages.xml') }
+    let(:xml) { File.read('./spec/fixtures/mdfe/lib/xml-mdfe-multiples-pages.xml') }
 
     it 'renders the header and the identification on each page' do
       expect(File.exist?(output_pdf)).to be false

data/spec/br_danfe/qr_code_spec.rb

@@ -11,23 +11,14 @@ describe BrDanfe::QrCode do
     )
   end
 
-  let(:xml) do
-    xml = <<-eos
-      <infNFeSupl>
-        <qrCode><![CDATA[http://sistemas.sefaz.am.gov.br/nfceweb/consultarNFCe.jsp?p=13130901144012000152651231234567891123456786|2|2|1|A604FD51E40ED8465B787B3E9CF8C1D90E61DB13]]></qrCode>
-        <urlChave>www.sefaz.am.gov.br/nfce/consulta</urlChave>
-      </infNFeSupl>
-    eos
-
-    BrDanfe::XML.new(xml)
+  let(:qr_code_url) do
+    'http://sistemas.sefaz.am.gov.br/nfceweb/consultarNFCe.jsp?p=13130901144012000152651231234567891123456786|2|2|1|A604FD51E40ED8465B787B3E9CF8C1D90E61DB13'
   end
 
-  subject { described_class.new(pdf: pdf, xml: xml, qr_code_tag: xml['qrCode'], box_size: 3.cm) }
+  subject { described_class.new(pdf: pdf, qr_code_tag: qr_code_url, box_size: 3.cm) }
 
   describe '#render' do
-    before do
-      File.delete(output_pdf) if File.exist?(output_pdf)
-    end
+    before { File.delete(output_pdf) if File.exist?(output_pdf) }
 
     it 'renders qr-code to the pdf' do
       expect(File.exist?(output_pdf)).to be_falsey

data/spec/br_danfe/xml_spec.rb

@@ -6,12 +6,12 @@ describe BrDanfe::XML do
   describe '#version_is_310_or_newer?' do
     describe "when xml's version is equal 3.10" do
       let(:xml_as_string) do
-        <<-eos
+        <<~XML
           <NFe xmlns="http://www.portalfiscal.inf.br/nfe">
             <infNFe versao="3.10" Id="NFe35150162013294000143550010000000011000000017">
             </infNFe>
           </NFe>
-        eos
+        XML
       end
 
       it 'returns true' do
@@ -21,12 +21,12 @@ describe BrDanfe::XML do
 
     describe "when xml's version is greather 3.10" do
       let(:xml_as_string) do
-        <<-eos
+        <<~XML
           <NFe xmlns="http://www.portalfiscal.inf.br/nfe">
             <infNFe versao="4.00" Id="NFe35150162013294000143550010000000011000000017">
             </infNFe>
           </NFe>
-        eos
+        XML
       end
 
       it 'returns true' do
@@ -36,12 +36,12 @@ describe BrDanfe::XML do
 
     describe "when xml's version is minor that 3.10" do
       let(:xml_as_string) do
-        <<-eos
+        <<~XML
           <NFe xmlns="http://www.portalfiscal.inf.br/nfe">
             <infNFe versao="2.00" Id="NFe35131260891033000109550010000000011000000016">
             </infNFe>
           </NFe>
-        eos
+        XML
       end
 
       it 'returns false' do

data/spec/fixtures/nfce/lib/document#render.pdf

@@ -13,20 +13,18 @@ endobj
 3 0 obj
 << /Type /Pages
 /Count 2
-/Kids [5 0 R 8 0 R]
+/Kids [5 0 R 7 0 R]
 >>
 endobj
 4 0 obj
-<< /Length 55
+<< /Length 38
 >>
 stream
 q
 0.3 w
 
 BT
-8.50394 38.74569 Td
-/F1.0 6 Tf
-<> Tj
+8.50394 38.36169 Td
 ET
 
 Q
@@ -43,19 +41,11 @@ endobj
 /ArtBox [0 0 226.77165 51.02362]
 /Contents 4 0 R
 /Resources << /ProcSet [/PDF /Text /ImageB /ImageC /ImageI]
-/Font << /F1.0 6 0 R
->>
+/Font << >>
 >>
 >>
 endobj
 6 0 obj
-<< /Type /Font
-/Subtype /Type1
-/BaseFont /Courier
-/Encoding /WinAnsiEncoding
->>
-endobj
-7 0 obj
 << /Length 10
 >>
 stream
@@ -65,7 +55,7 @@ Q
 
 endstream
 endobj
-8 0 obj
+7 0 obj
 << /Type /Page
 /Parent 3 0 R
 /MediaBox [0 0 226.77165 51.02362]
@@ -73,27 +63,26 @@ endobj
 /BleedBox [0 0 226.77165 51.02362]
 /TrimBox [0 0 226.77165 51.02362]
 /ArtBox [0 0 226.77165 51.02362]
-/Contents 7 0 R
+/Contents 6 0 R
 /Resources << /ProcSet [/PDF /Text /ImageB /ImageC /ImageI]
 >>
 >>
 endobj
 xref
-0 9
+0 8
 0000000000 65535 f 
 0000000015 00000 n 
 0000000109 00000 n 
 0000000158 00000 n 
 0000000221 00000 n 
-0000000326 00000 n 
-0000000647 00000 n 
-0000000742 00000 n 
-0000000802 00000 n 
+0000000309 00000 n 
+0000000618 00000 n 
+0000000678 00000 n 
 trailer
-<< /Size 9
+<< /Size 8
 /Root 2 0 R
 /Info 1 0 R
 >>
 startxref
-1099
+975
 %%EOF

data/spec/fixtures/nfce/lib/product_list_with_many_products_and_long_names.xml

@@ -64,7 +64,7 @@
             <prod>
                <cProd>1173</cProd>
                <cEAN>568754456745</cEAN>
-               <xProd>Abraçadeira with a larger name with line breaks</xProd>
+               <xProd>Abraçadeira de metal with a larger name truncated</xProd>
                <NCM>01022990</NCM>
                <CFOP>5101</CFOP>
                <uCom>UN</uCom>
@@ -150,7 +150,7 @@
             <prod>
                <cProd>1173</cProd>
                <cEAN>568754456745</cEAN>
-               <xProd>Abraçadeira with a larger name with line breaks</xProd>
+               <xProd>Abraçadeira de metal with a larger name truncated</xProd>
                <NCM>01022990</NCM>
                <CFOP>5101</CFOP>
                <uCom>UN</uCom>

data/spec/fixtures/nfce/lib/product_list_with_single_product_and_long_name.xml

@@ -64,7 +64,7 @@
             <prod>
                <cProd>1173</cProd>
                <cEAN>568754456745</cEAN>
-               <xProd>Abraçadeira with a larger name with line breaks</xProd>
+               <xProd>Abraçadeira de metal with a larger name truncated</xProd>
                <NCM>01022990</NCM>
                <CFOP>5101</CFOP>
                <uCom>UN</uCom>