bollettino 1.0.0 → 1.0.1

data/lib/bollettino/renderer/address.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Renderer
+    class Address < Base
+      class << self
+        def render(image, address)
+          render_street(image, address)
+          render_zip(image, address)
+          render_location(image, address)
+        end
+
+        private
+
+        def render_street(image, address)
+          write_text(image, [85, 267], address.street[0..29], KERNING_NORMAL, FONT_SIZE_SMALL)
+          write_text(image, [1508, 267], address.street[0..22], KERNING_BOX_SMALLEST)
+        end
+
+        def render_zip(image, address)
+          write_text(image, [85, 223], address.zip[0..29], KERNING_NORMAL, FONT_SIZE_SMALL)
+          write_text(image, [1508, 200], address.zip[0..4], KERNING_BOX_SMALLEST)
+        end
+
+        def render_location(image, address)
+          write_text(image, [85, 180], address.location[0..29], KERNING_NORMAL, FONT_SIZE_SMALL)
+          write_text(image, [1713, 200], address.location[0..16], KERNING_BOX_SMALLEST)
+        end
+      end
+    end
+  end
+end

data/lib/bollettino/renderer/base.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Renderer
+    # Base renderer
+    #
+    # @abstract Subclass and override {.render} to create a renderer
+    #
+    # @author Alessandro Desantis <desa.alessandro@gmail.com>
+    class Base
+      class << self
+        KERNING_NORMAL = 1
+
+        KERNING_BOX = 17.5
+        KERNING_BOX_SMALLER = 17
+        KERNING_BOX_SMALLEST = 16
+
+        FONT_SIZE_NORMAL = 30
+        FONT_SIZE_SMALL = 25
+
+        # Renders the given model on the image.
+        #
+        # @param image [MiniMagick::Image]
+        # @param model
+        #
+        # @abstract This method must be overridden by the renderers
+        def render(_image, _model)
+          fail NotImplementedError
+        end
+
+        protected
+
+        def write_text(image, coords, text, kerning = KERNING_NORMAL, font_size = FONT_SIZE_NORMAL)
+          image.combine_options do |c|
+            c.font 'courier'
+            c.fill 'black'
+            c.pointsize font_size
+            c.gravity 'southwest'
+            c.kerning kerning
+            c.draw %(text #{coords.join(',')} "#{text.to_s.upcase.gsub('"', '\"')}")
+          end
+        end
+      end
+    end
+  end
+end

data/lib/bollettino/renderer/errors.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Renderer
+    # Rendering error
+    #
+    # This error is usually raised when some data can't be renderered because
+    # it's malformed.
+    #
+    # @author Alessandro Desantis <desa.alessandro@gmail.com>
+    class RenderingError < StandardError
+    end
+  end
+end

data/lib/bollettino/renderer/payee.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Renderer
+    class Payee < Base
+      class << self
+        def render(image, payee)
+          render_account_number(image, payee)
+          render_name(image, payee)
+        end
+
+        private
+
+        def render_account_number(image, payee)
+          if payee.account_number.length > 10
+            fail RenderingError, "Account number can't be longer than 10 characters"
+          end
+
+          [[265, 695], [1310, 695]].each do |coords|
+            write_text(image, coords, payee.account_number, KERNING_BOX)
+          end
+        end
+
+        def render_name(image, payee)
+          write_text(image, [90, 585], payee.name[0..46])
+          write_text(image, [90, 545], payee.name[47..93])
+
+          write_text(image, [1105, 590], payee.name[0..33], KERNING_BOX_SMALLER)
+          write_text(image, [1105, 545], payee.name[34..67], KERNING_BOX_SMALLER)
+        end
+      end
+    end
+  end
+end

data/lib/bollettino/renderer/payer.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Renderer
+    class Payer < Base
+      class << self
+        def render(image, payer)
+          render_name(image, payer)
+          render_address(image, payer)
+        end
+
+        private
+
+        def render_name(image, payer)
+          write_text(image, [85, 360], payer.name[0..24])
+          write_text(image, [85, 315], payer.name[25..49])
+
+          write_text(image, [1508, 375], payer.name[0..22], KERNING_BOX_SMALLEST)
+          write_text(image, [1508, 330], payer.name[23..45], KERNING_BOX_SMALLEST)
+        end
+
+        def render_address(image, payer)
+          Address.render(image, payer.address)
+        end
+      end
+    end
+  end
+end

data/lib/bollettino/renderer/payment_order.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Renderer
+    class PaymentOrder < Base
+      class << self
+        def render(image, payment_order)
+          render_numeric_amount(image, payment_order)
+          render_text_amount(image, payment_order)
+          render_reason(image, payment_order)
+        end
+
+        private
+
+        def render_numeric_amount(image, payment_order)
+          numeric_amount = payment_order.numeric_amount
+          numeric_amount = format('%.2f', numeric_amount).delete('.')
+
+          if numeric_amount.length > 7
+            fail RenderingError, "Numeric amount can't be longer than 7 total digits"
+          end
+
+          numeric_amount_x = 1000 - 35 * numeric_amount.length.to_i
+
+          [[numeric_amount_x, 690], [numeric_amount_x + 1315, 690]].each do |coords|
+            write_text(image, coords, numeric_amount, KERNING_BOX)
+          end
+        end
+
+        def render_reason(image, payment_order)
+          write_text(image, [90, 475], payment_order.reason[0..46])
+          write_text(image, [90, 435], payment_order.reason[47..93])
+
+          write_text(image, [1110, 475], payment_order.reason[0..60])
+          write_text(image, [1110, 435], payment_order.reason[61..121])
+        end
+
+        def render_text_amount(image, payment_order)
+          write_text(image, [245, 650], payment_order.text_amount[0..38])
+          write_text(image, [1435, 650], payment_order.text_amount[0..44])
+        end
+      end
+    end
+  end
+end

data/lib/bollettino/renderer/slip.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Renderer
+    class Slip < Base
+      class << self
+        def render(image, slip)
+          render_payment_order(image, slip)
+          render_payee(image, slip)
+          render_payer(image, slip)
+        end
+
+        private
+
+        def render_payment_order(image, slip)
+          PaymentOrder.render(image, slip.payment_order)
+        end
+
+        def render_payee(image, slip)
+          Payee.render(image, slip.payee)
+        end
+
+        def render_payer(image, slip)
+          Payer.render(image, slip.payer)
+        end
+      end
+    end
+  end
+end

data/lib/bollettino/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Bollettino
-  VERSION = '1.0.0'
+  VERSION = '1.0.1'
 end

data/spec/integration/generation_spec.rb

@@ -1,17 +1,17 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe 'The generation' do
   let(:payee) do
-    Bollettino::Payee.new(
+    Bollettino::Model::Payee.new(
       account_number: '0123456789',
       name: 'Acme Inc.'
     )
   end
 
   let(:payer) do
-    Bollettino::Payer.new(
+    Bollettino::Model::Payer.new(
       name: 'John Doe',
-      address: Bollettino::Address.new(
+      address: Bollettino::Model::Address.new(
         street: '3681 Foggy Moor',
         zip: '19147-0834',
         location: 'Grayson'
@@ -20,16 +20,15 @@ RSpec.describe 'The generation' do
   end
 
   let(:payment_order) do
-    Bollettino::PaymentOrder.new(
+    Bollettino::Model::PaymentOrder.new(
       numeric_amount: 54.31,
       text_amount: 'Cinquantaquattro/31',
       reason: 'Invoice INV-1391'
     )
-
   end
 
   let(:slip) do
-    Bollettino::Slip.new(
+    Bollettino::Model::Slip.new(
       payee: payee,
       payer: payer,
       payment_order: payment_order
@@ -38,13 +37,13 @@ RSpec.describe 'The generation' do
 
   let(:slip_path) { File.expand_path('../../../tmp/slip.png', __FILE__) }
 
-  before(:each) do
+  before do
     FileUtils.rm(slip_path) if File.exist?(slip_path)
   end
 
   it 'creates the slip' do
     expect {
       Bollettino::Generator.new.generate!(slip, slip_path)
-    }.to change{File.exist?(slip_path)}.from(false).to(true)
+    }.to change { File.exist?(slip_path) }.from(false).to(true)
   end
 end

data/spec/spec_helper.rb

@@ -1,3 +1,6 @@
+require 'coveralls'
+Coveralls.wear!
+
 require 'bollettino'
 
 # This file was generated by the `rspec --init` command. Conventionally, all
@@ -33,7 +36,12 @@ RSpec.configure do |config|
 
   # rspec-mocks config goes here. You can use an alternate test double
   # library (such as bogus or mocha) by changing the `mock_with` option here.
-  config.mock_with :mocha
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
 
   # These two settings work together to allow you to limit a spec run
   # to individual examples or groups you care about by tagging them with

data/spec/unit/bollettino/generator_spec.rb

@@ -1,49 +1,22 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Bollettino::Generator do
   subject { described_class.new }
 
   describe '#generate!' do
-    let(:slip) do
-      rs = (('A'..'Z').to_a * 10).join('')
+    let(:slip) { instance_double('Bollettino::Model::Slip') }
+    let(:image) { instance_double('MiniMagick::Image') }
 
-      stub(
-        payee: stub(
-          account_number: '0123456789',
-          name: rs
-        ),
-        payment_order: stub(
-          numeric_amount: 1,
-          text_amount: rs,
-          reason: rs
-        ),
-        payer: stub(
-          name: rs,
-          address: stub(
-            street: rs,
-            location: rs,
-            zip: rs
-          )
-        )
-      )
+    before do
+      allow(MiniMagick::Image).to receive(:open).and_return(image)
+      allow(Bollettino::Renderer::Slip).to receive(:render)
     end
 
-    it 'writes the image' do
-      image = mock()
-      image
-        .expects(:write)
+    it 'creates the image' do
+      expect(image).to receive(:write)
         .with('slip.png')
         .once
 
-      MiniMagick::Image
-        .expects(:open)
-        .once
-        .returns(image)
-
-      Bollettino::Renderer::SlipRenderer
-        .expects(:render)
-        .once
-
       subject.generate!(slip, 'slip.png')
     end
   end

data/spec/unit/bollettino/model/address_spec.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Model
+    RSpec.describe Address do
+      subject do
+        described_class.new(
+          street: '3681 Foggy Moor',
+          zip: '19147-0834',
+          city: 'Grayson',
+          state: 'Pennsylvania'
+        )
+      end
+
+      it 'is instantiated correctly' do
+        expect { subject }.not_to raise_error
+      end
+    end
+  end
+end

data/spec/unit/bollettino/model/payee_spec.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Model
+    RSpec.describe Payee do
+      subject do
+        described_class.new(
+          account_number: '0123456789',
+          name: 'Acme Inc.'
+        )
+      end
+
+      it 'is instantiated correctly' do
+        expect { subject }.not_to raise_error
+      end
+    end
+  end
+end

data/spec/unit/bollettino/model/payer_spec.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module Bollettino
+  module Model
+    RSpec.describe Payer do
+      subject do
+        described_class.new(
+          name: 'John Doe',
+          address: address
+        )
+      end
+
+      let(:address) { instance_double('Bollettino::Model::Address') }
+
+      before do
+        allow(address).to receive(:to_hash).and_return({})
+      end
+
+      it 'is instantiated correctly' do
+        expect { subject }.not_to raise_error
+      end
+    end
+  end
+end