invoice_printer 0.0.2 → 0.0.3

data/test/dates_box_test.rb

@@ -11,6 +11,7 @@ class DatesBoxTest < Minitest::Test
     invoice = InvoicePrinter::Document.new(params)
     rendered_pdf = InvoicePrinter.render(document: invoice)
     pdf_analysis = PDF::Inspector::Text.analyze(rendered_pdf)
+
     assert_equal true, pdf_analysis.strings.include?('Issue date:')
     assert_equal true, pdf_analysis.strings.include?('Due date:')
   end
@@ -23,6 +24,7 @@ class DatesBoxTest < Minitest::Test
     invoice = InvoicePrinter::Document.new(params)
     rendered_pdf = InvoicePrinter.render(document: invoice)
     pdf_analysis = PDF::Inspector::Text.analyze(rendered_pdf)
+
     assert_equal true, pdf_analysis.strings.include?('Issue date:')
     assert_equal false, pdf_analysis.strings.include?('Due date:')
   end
@@ -35,6 +37,7 @@ class DatesBoxTest < Minitest::Test
     invoice = InvoicePrinter::Document.new(params)
     rendered_pdf = InvoicePrinter.render(document: invoice)
     pdf_analysis = PDF::Inspector::Text.analyze(rendered_pdf)
+
     assert_equal false, pdf_analysis.strings.include?('Issue date:')
     assert_equal true, pdf_analysis.strings.include?('Due date:')
   end
@@ -47,6 +50,7 @@ class DatesBoxTest < Minitest::Test
     invoice = InvoicePrinter::Document.new(params)
     rendered_pdf = InvoicePrinter.render(document: invoice)
     pdf_analysis = PDF::Inspector::Text.analyze(rendered_pdf)
+
     assert_equal false, pdf_analysis.strings.include?('Issue date:')
     assert_equal false, pdf_analysis.strings.include?('Due date:')
   end

data/test/invoice_printer_test.rb

@@ -7,8 +7,8 @@ class InvoicePrinterTest < Minitest::Test
     invoice = InvoicePrinter::Document.new(default_document_params)
     rendered_pdf = InvoicePrinter.render(document: invoice)
     pdf_analysis = PDF::Inspector::Text.analyze(rendered_pdf)
-
     strings = InvoicePrinter::PDFDocument.new(document: invoice).to_a
+
     assert_equal strings, pdf_analysis.strings
   end
 end

data/test/items_table_test.rb

@@ -19,11 +19,9 @@ class ItemsTableTest < Minitest::Test
     test_ommiting_column(column: 'price', label: :price_per_item)
   end
 
-  # TODO: We need to distinguish between items' taxes and document taxes
-  #
-  #def test_omitting_tax_column
-  #  test_ommiting_column(column: 'tax', label: :tax)
-  #end
+  def test_omitting_tax_column
+    test_ommiting_column(column: 'tax', label: :tax)
+  end
 
   def test_omitting_amount_column
     test_ommiting_column(column: 'amount', label: :amount)

data/test/labels_test.rb

@@ -6,7 +6,6 @@ class LabelsTest < Minitest::Test
   def test_setting_global_labels
     labels = { provider: 'Default Provider', purchaser: 'Default Purchaser' }
     InvoicePrinter.labels = labels
-
     invoice = InvoicePrinter::Document.new(default_document_params)
     rendered_pdf = InvoicePrinter.render(document: invoice)
     pdf_analysis = PDF::Inspector::Text.analyze(rendered_pdf)
@@ -17,7 +16,6 @@ class LabelsTest < Minitest::Test
 
   def test_setting_instant_labels
     labels = { provider: 'Current Provider', purchaser: 'Current Purchaser' }
-
     invoice = InvoicePrinter::Document.new(default_document_params)
     rendered_pdf = InvoicePrinter.render(document: invoice, labels: labels)
     pdf_analysis = PDF::Inspector::Text.analyze(rendered_pdf)

data/test/test_ext.rb

@@ -1,13 +1,28 @@
+# Requiring this file extends the original PDFDocument class with +to_a+ method
+# returning the strings representation of the class.
+
 module InvoicePrinter
   class PDFDocument
     # Expose the document as an array of attributes in order as their
     # appear on PDF
     def to_a
       strings = []
-
       strings << @labels[:name]
       strings << @document.number
+      strings << provider_box
+      strings << purchaser_box
+      strings << account_box
+      strings << dates_box
+      strings << items_table
+      strings << totals_table
+      strings.flatten.reject(&:empty?)
+    end
 
+    private
+
+    # Strings representaion of provider's box
+    def provider_box
+      strings = []
       strings << @labels[:provider]
       strings << @document.provider_name
       strings << "#{@document.provider_street}    #{@document.provider_street_number}".strip
@@ -15,10 +30,16 @@ module InvoicePrinter
       strings << @document.provider_city
       strings << @document.provider_city_part
       strings << @document.provider_extra_address_line
+      strings << "#{@labels[:ic]}:    #{@document.provider_ic}" \
+        unless @document.provider_ic.empty?
+      strings << "#{@labels[:dic]}:    #{@document.provider_dic}" \
+        unless @document.provider_dic.empty?
+      strings
+    end
 
-      strings << "#{@labels[:ic]}:    #{@document.provider_ic}" unless @document.provider_ic.empty?
-      strings << "#{@labels[:dic]}:    #{@document.provider_dic}" unless @document.provider_dic.empty?
-
+    # Strings representaion of purchaser's box
+    def purchaser_box
+      strings = []
       strings << @labels[:purchaser]
       strings << @document.purchaser_name
       strings << "#{@document.purchaser_street}    #{@document.purchaser_street_number}".strip
@@ -26,31 +47,43 @@ module InvoicePrinter
       strings << @document.purchaser_city
       strings << @document.purchaser_city_part
       strings << @document.purchaser_extra_address_line
+      strings << "#{@labels[:ic]}:    #{@document.purchaser_ic}" \
+        unless @document.purchaser_ic.empty?
+      strings << "#{@labels[:dic]}:    #{@document.purchaser_dic}" \
+        unless @document.purchaser_dic.empty?
+      strings
+    end
 
-      strings << "#{@labels[:ic]}:    #{@document.purchaser_ic}" unless @document.purchaser_ic.empty?
-      strings << "#{@labels[:dic]}:    #{@document.purchaser_dic}" unless @document.purchaser_dic.empty?
-
-      # Account
+    # Strings representaion of account's box
+    def account_box
+      strings = []
       if @document.bank_account_number.nil?
         strings << @labels[:payment_in_cash]
       else
         strings << @labels[:payment_by_transfer]
       end
-
       strings << "#{@labels[:account_number]}:"
       strings << @document.bank_account_number
       strings << "#{@labels[:swift]}:"
       strings << @document.account_swift
       strings << "#{@labels[:iban]}:"
       strings << @document.account_iban
+      strings
+    end
 
-      # Dates
+    # Strings representaion of dates box
+    def dates_box
+      strings = []
       strings << "#{@labels[:issue_date]}:"
       strings << @document.issue_date
       strings << "#{@labels[:due_date]}:"
       strings << @document.due_date
+      strings
+    end
 
-      # Items table
+    # Strings representaion of items table
+    def items_table
+      strings = []
       strings << @labels[:item] if determine_items_structure[:names]
       strings << @labels[:quantity] if determine_items_structure[:quantities]
       strings << @labels[:unit] if determine_items_structure[:units]
@@ -58,8 +91,12 @@ module InvoicePrinter
       strings << @labels[:tax] if determine_items_structure[:taxes]
       strings << @labels[:amount] if determine_items_structure[:amounts]
       strings << items_to_a(@document.items)
+      strings
+    end
 
-      # Total table
+    # Strings representaion of totals table
+    def totals_table
+      strings = []
       strings << "#{@labels[:subtotal]}:"
       strings << @document.subtotal
       strings << "#{@labels[:tax]}:"
@@ -68,17 +105,11 @@ module InvoicePrinter
       strings << @document.tax2
       strings << "#{@labels[:tax3]}:"
       strings << @document.tax3
-
       strings << "#{@labels[:total]}:   #{@document.total}"
-
-      # TODO: dynamically test page numbers
-      strings << '1 / 1'
-
-      strings.flatten.reject(&:empty?)
+      strings
     end
 
-    private
-
+    # Convert document +items+ to a single string array
     def items_to_a(items)
       ary = []
       items.each do |item|
@@ -87,6 +118,7 @@ module InvoicePrinter
       ary.flatten
     end
 
+    # Convert document +item+ to a single string array
     def item_to_a(item)
       ary = []
       ary << item.name

data/test/test_helper.rb

@@ -3,7 +3,7 @@ require 'invoice_printer'
 require 'test_ext'
 require 'minitest/autorun'
 
-# Helpers for easy-to-build documents
+# Test helpers
 module InvoicePrinterHelpers
   def default_document_params
     {

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: invoice_printer
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Josef Strzibny
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-12 00:00:00.000000000 Z
+date: 2016-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: prawn
@@ -78,6 +78,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- examples/czech_invoice.jpg
 - examples/czech_invoice.pdf
 - examples/czech_invoice.rb
 - examples/example.jpg