From 5042ee8edcdc665501ce38b294196f25c1840cc5 Mon Sep 17 00:00:00 2001
From: Andy Meneely <andy@se.rit.edu>
Date: Sat, 15 Aug 2015 14:59:08 -0400
Subject: [PATCH] Ripping out the notorious input_helpers.rb

---
 lib/squib/api/save.rb      |   5 -
 lib/squib/card.rb          |   1 -
 lib/squib/deck.rb          |  16 ++-
 lib/squib/input_helpers.rb | 242 -----------------------------------
 spec/input_helpers_spec.rb | 249 -------------------------------------
 5 files changed, 7 insertions(+), 506 deletions(-)
 delete mode 100644 lib/squib/input_helpers.rb
 delete mode 100644 spec/input_helpers_spec.rb

diff --git a/lib/squib/api/save.rb b/lib/squib/api/save.rb
index f3acd8f..faef24c 100644
--- a/lib/squib/api/save.rb
+++ b/lib/squib/api/save.rb
@@ -142,11 +142,6 @@ module Squib
       range = Args::CardRange.new(opts[:range], deck_size: size)
       hand  = Args::HandSpecial.new(height).load!(opts, expand_by: size, layout: layout, dpi: dpi)
       sheet = Args::Sheet.new(custom_colors, {file: 'hand.png', trim_radius: 0}).load!(opts, expand_by: size, layout: layout, dpi: dpi)
-
-      opts = {file: 'hand.png', fill_color: :white, radius: :auto, trim_radius: 0}
-             .merge(opts)
-      opts = needs(opts,[:range, :margin, :trim, :trim_radius, :creatable_dir, :file_to_save])
-      opts[:radius] = 0.3 * height if opts[:radius] == :auto
       render_hand(range, sheet, hand)
     end
 
diff --git a/lib/squib/card.rb b/lib/squib/card.rb
index 25afa1c..d92867b 100644
--- a/lib/squib/card.rb
+++ b/lib/squib/card.rb
@@ -1,5 +1,4 @@
 require 'cairo'
-require 'squib/input_helpers'
 require 'squib/graphics/cairo_context_wrapper'
 
 module Squib
diff --git a/lib/squib/deck.rb b/lib/squib/deck.rb
index 4211b9b..3f2f619 100644
--- a/lib/squib/deck.rb
+++ b/lib/squib/deck.rb
@@ -1,16 +1,15 @@
-require 'yaml'
-require 'pp'
 require 'forwardable'
+require 'pp'
 require 'squib'
-require 'squib/card'
-require 'squib/progress'
-require 'squib/input_helpers'
-require 'squib/constants'
-require 'squib/layout_parser'
 require 'squib/args/unit_conversion'
+require 'squib/card'
 require 'squib/conf'
-require 'squib/graphics/showcase'
+require 'squib/constants'
 require 'squib/graphics/hand'
+require 'squib/graphics/showcase'
+require 'squib/layout_parser'
+require 'squib/progress'
+
 
 # The project module
 #
@@ -22,7 +21,6 @@ module Squib
   # @api public
   class Deck
     include Enumerable
-    include Squib::InputHelpers
     extend Forwardable
 
     # Attributes for the width, height (in pixels) and number of cards
diff --git a/lib/squib/input_helpers.rb b/lib/squib/input_helpers.rb
deleted file mode 100644
index 7b0d461..0000000
--- a/lib/squib/input_helpers.rb
+++ /dev/null
@@ -1,242 +0,0 @@
-require 'squib/constants'
-require 'squib/args/unit_conversion'
-
-module Squib
-  # :nodoc:
-  # @api private
-  module InputHelpers
-
-    # :nodoc:
-    # @api private
-    def needs(opts, params)
-      Squib.logger.debug {"Method #{caller(1,1)} was given the following opts: #{opts}"}
-      opts = layoutify(opts) if params.include? :layout
-      opts = Squib::SYSTEM_DEFAULTS.merge(opts)
-      opts = expand_singletons(opts, params)
-      opts = rangeify(opts) if params.include? :range
-      opts = fileify(opts) if params.include? :file
-      opts = fileify(opts, false) if params.include? :file_to_save
-      opts = colorify(opts, true) if params.include? :nillable_color
-      opts = dirify(opts, :dir, true) if params.include? :creatable_dir
-      opts = dirify(opts, :img_dir, false) if params.include? :img_dir
-      opts = fileify(opts, false) if params.include? :files
-      opts = colorify(opts) if params.include? :color
-      opts = colorify(opts, false, :fill_color) if params.include? :fill_color
-      opts = colorify(opts, false, :stroke_color) if params.include? :stroke_color
-      opts = fontify(opts) if params.include? :font
-      opts = radiusify(opts) if params.include? :rect_radius
-      opts = svgidify(opts) if params.include? :svgid
-      opts = formatify(opts) if params.include? :formats
-      opts = rotateify(opts) if params.include? :rotate
-      opts = rowify(opts) if params.include? :rows
-      opts = faceify(opts) if params.include? :face
-      opts = convert_units(opts, params)
-      opts
-    end
-    module_function :needs
-
-    # :nodoc:
-    # @api private
-    def expand_singletons(opts, needed_params)
-      Squib::EXPANDING_PARAMS.each_pair do |param_name, api_param|
-        if needed_params.include? param_name
-          unless opts[api_param].respond_to?(:each)
-            opts[api_param] = [opts[api_param]] * @cards.size
-          end
-        end
-      end
-      Squib.logger.debug {"After expand_singletons: #{opts}"}
-      opts
-    end
-    module_function :expand_singletons
-
-    # Layouts have to come before, so we repeat expand_singletons here
-    # :nodoc:
-    # @api private
-    def layoutify(opts)
-      unless opts[:layout].respond_to?(:each)
-        opts[:layout] = [opts[:layout]] * @cards.size
-      end
-      opts[:layout].each_with_index do |layout, i|
-        unless layout.nil?
-          entry = @layout[layout.to_s]
-          unless entry.nil?
-            entry.each do |key, value|
-              opts[key.to_sym] = [] if opts[key.to_sym].nil?
-              opts[key.to_sym][i] ||= entry[key] #don't override if it's already there
-            end
-          else
-            Squib.logger.warn ("Layout entry '#{layout}' does not exist." )
-          end
-        end
-      end
-      Squib.logger.debug {"After layoutify: #{opts}"}
-      opts
-    end
-    module_function :layoutify
-
-    # :nodoc:
-    # @api private
-    def formatify(opts)
-      opts[:format] = [opts[:format]].flatten
-      opts
-    end
-    module_function :formatify
-
-    # :nodoc:
-    # @api private
-    def rangeify (opts)
-      range = opts[:range]
-      raise 'Range cannot be nil' if range.nil?
-      range = 0..(@cards.size-1) if range == :all
-      range = range..range if range.is_a? Integer
-      if range.max > (@cards.size-1)
-        raise ArgumentError.new("#{range} is outside of deck range of 0..#{@cards.size-1}")
-      end
-      opts[:range] = range
-      Squib.logger.debug {"After rangeify: #{opts}"}
-      opts
-    end
-    module_function :rangeify
-
-    # :nodoc:
-    # @api private
-    def fileify(opts, file_must_exist=true)
-      [opts[:file]].flatten.each do |file|
-        if file_must_exist and !File.exists?(file)
-          raise "File #{File.expand_path(file)} does not exist!"
-        end
-      end
-      opts
-    end
-    module_function :fileify
-
-    # :nodoc:
-    # @api private
-    def dirify(opts, key, allow_create=false)
-      return opts if Dir.exists?(opts[key])
-      if allow_create
-        Squib.logger.warn("Dir '#{opts[key]}' does not exist, creating it.")
-        Dir.mkdir opts[key]
-        return opts
-      else
-        raise "'#{opts[key]}' does not exist!"
-      end
-    end
-    module_function :dirify
-
-    # :nodoc:
-    # @api private
-    def colorify(opts, nillable=false, key=:color)
-      opts[key].each_with_index do |color, i|
-        unless nillable && color.nil?
-          if custom_colors.key? color.to_s
-            color = custom_colors[color.to_s]
-          end
-          opts[key][i] = color
-        end
-      end
-      Squib.logger.debug {"After colorify: #{opts}"}
-      opts
-    end
-    module_function :colorify
-
-    # :nodoc:
-    # @api private
-    def fontify (opts)
-      opts[:font].each_with_index do |font, i|
-        opts[:font][i] = @font if font==:use_set
-        opts[:font][i] = Squib::SYSTEM_DEFAULTS[:default_font] if font == :default
-      end
-      Squib.logger.debug {"After fontify: #{opts}"}
-      opts
-    end
-    module_function :fontify
-
-    # :nodoc:
-    # @api private
-    def radiusify(opts)
-      opts[:radius].each_with_index do |radius, i|
-        unless radius.nil?
-          opts[:x_radius][i] = radius
-          opts[:y_radius][i] = radius
-        end
-      end
-      Squib.logger.debug {"After radiusify: #{opts}"}
-      opts
-    end
-    module_function :radiusify
-
-    # :nodoc:
-    # @api private
-    def svgidify(opts)
-      opts[:id].each_with_index do |id, i|
-        unless id.nil?
-          opts[:id][i] = '#' << id unless id.start_with? '#'
-        end
-      end
-      Squib.logger.debug {"After svgidify: #{opts}"}
-      opts
-    end
-    module_function :svgidify
-
-    # :nodoc:
-    # @api private
-    def rotateify(opts)
-      case opts[:rotate]
-      when true, :clockwise
-        opts[:angle] = 0.5 * Math::PI
-      when :counterclockwise
-        opts[:angle] = 1.5 * Math::PI
-      end
-      Squib.logger.debug {"After rotateify: #{opts}"}
-      opts
-    end
-    module_function :rotateify
-
-    # Convert units
-    # :nodoc:
-    # @api private
-    def convert_units(opts, needed_params)
-      UNIT_CONVERSION_PARAMS.each_pair do |param_name, api_param|
-        if needed_params.include? param_name
-          if EXPANDING_PARAMS.include? param_name
-            opts[api_param].each_with_index do |arg, i|
-              opts[api_param][i] = Args::UnitConversion.parse(arg, @dpi)
-            end
-          else #not an expanding param
-            opts[api_param] = Args::UnitConversion.parse(opts[api_param], @dpi)
-          end
-        end
-      end
-      Squib.logger.debug {"After convert_units: #{opts}"}
-      return opts
-    end
-    module_function :convert_units
-
-     # Handles expanding rows. If the "rows" does not respond to to_i (e.g. :infinite),
-    # then compute what we need based on number of cards and number of columns.
-    # :nodoc:
-    # @api private
-    def rowify(opts)
-      unless opts[:rows].respond_to? :to_i
-        raise "Columns must be an integer" unless opts[:columns].respond_to? :to_i
-        if @cards.size < opts[:columns].to_i
-          opts[:rows] = 1
-        else
-          opts[:rows] = (@cards.size / opts[:columns].to_i).ceil
-        end
-      end
-      opts
-    end
-
-    # Used for showcase - face right if it's :right
-    # :nodoc:
-    # @api private
-    def faceify(opts)
-      opts[:face] = (opts[:face].to_s.downcase == 'right')
-      opts
-    end
-
-  end
-end
diff --git a/spec/input_helpers_spec.rb b/spec/input_helpers_spec.rb
deleted file mode 100644
index b682404..0000000
--- a/spec/input_helpers_spec.rb
+++ /dev/null
@@ -1,249 +0,0 @@
-require 'spec_helper'
-require 'squib'
-require 'squib/input_helpers'
-
-class DummyDeck
-  include  Squib::InputHelpers
-  attr_accessor :layout, :cards, :custom_colors, :width, :height, :dpi
-end
-
-describe Squib::InputHelpers do
-
-  before(:each) do
-    @deck = DummyDeck.new
-    @deck.layout = {
-      'blah' => {x: 25},
-      'apples' => {x: 35},
-      'oranges' => {y: 45},
-    }
-    @deck.cards = %w(a b)
-    @deck.custom_colors = {}
-    @deck.width = 100
-    @deck.height = 200
-    @deck.dpi = 300
-  end
-
-  context '#layoutify' do
-    it 'warns on the logger when the layout does not exist' do
-      expect(Squib.logger).to receive(:warn).with("Layout entry 'foo' does not exist.").twice
-      expect(Squib.logger).to receive(:debug)
-      expect(@deck.send(:layoutify, {layout: :foo})).to eq({layout: [:foo,:foo]})
-    end
-
-    it 'applies the layout in a normal situation' do
-      expect(@deck.send(:layoutify, {layout: :blah})).to \
-        eq({layout: [:blah, :blah], x: [25, 25]})
-    end
-
-    it 'applies two different layouts for two different situations' do
-      expect(@deck.send(:layoutify, {layout: ['blah', 'apples']})).to \
-        eq({layout: ['blah','apples'], x: [25, 35]})
-    end
-
-    it 'still has nils when not applied two different layouts differ in structure' do
-      expect(@deck.send(:layoutify, {layout: ['apples', 'oranges']})).to \
-        eq({layout: ['apples','oranges'], x: [35], y: [nil, 45]})
-      #...this might behavior that is hard to debug for users. Trying to come up with a warning or something...
-    end
-
-    it 'also looks up based on strings' do
-      expect(@deck.send(:layoutify, {layout: 'blah'})).to \
-        eq({layout: ['blah','blah'], x: [25, 25]})
-    end
-
-  end
-
-  context '#rangeify' do
-    it 'must be within the card size range' do
-      expect{@deck.send(:rangeify, {range: 2..3})}.to \
-        raise_error(ArgumentError, '2..3 is outside of deck range of 0..1')
-    end
-
-    it 'cannot be nil' do
-      expect{@deck.send(:rangeify, {range: nil})}.to \
-        raise_error(RuntimeError, 'Range cannot be nil')
-    end
-
-    it 'defaults to a range of all cards if :all' do
-      expect(@deck.send(:rangeify, {range: :all})).to eq({range: 0..1})
-    end
-  end
-
-  context '#fileify' do
-    it 'should throw an error if the file does not exist' do
-      expect{@deck.send(:fileify, {file: 'nonexist.txt'}, true)}.to \
-        raise_error(RuntimeError,"File #{File.expand_path('nonexist.txt')} does not exist!")
-    end
-  end
-
-  context '#dirify' do
-    it 'should raise an error if the directory does not exist' do
-      expect{@deck.send(:dirify, {dir: 'nonexist'}, :dir, false)}.to \
-        raise_error(RuntimeError,"'nonexist' does not exist!")
-    end
-
-    it 'should warn and make a directory creation is allowed' do
-      opts = {dir: 'tocreate'}
-      Dir.chdir(output_dir) do
-        FileUtils.rm_rf('tocreate', secure: true)
-        expect(Squib.logger).to receive(:warn).with("Dir 'tocreate' does not exist, creating it.").once
-        expect(@deck.send(:dirify, opts, :dir, true)).to eq(opts)
-        expect(Dir.exists? 'tocreate').to be true
-      end
-    end
-
-  end
-
-  context '#colorify' do
-    it 'should pass through if nillable' do
-      color = @deck.send(:colorify, {color: ['#fff']}, true)[:color]
-      expect(color).to eq(['#fff'])
-    end
-
-    it 'pulls from custom colors in the config' do
-      @deck.custom_colors['foo'] = '#abc'
-      expect(@deck.send(:colorify, {color: [:foo]}, false)[:color][0].to_s).to \
-        eq('#abc')
-    end
-
-    it 'pulls custom colors even when a string' do
-      @deck.custom_colors['foo'] = '#abc'
-      expect(@deck.send(:colorify, {color: ['foo']}, false)[:color][0].to_s).to \
-        eq('#abc')
-    end
-  end
-
-  context '#rotateify' do
-    it 'computes a clockwise rotate properly' do
-      opts = @deck.send(:rotateify, {rotate: :clockwise})
-      expect(opts).to eq({ :angle => 0.5 * Math::PI,
-                           :rotate => :clockwise
-                         })
-    end
-
-    it 'computes a counter-clockwise rotate properly' do
-      opts = @deck.send(:rotateify, {rotate: :counterclockwise})
-      expect(opts).to eq({ :angle => 1.5 * Math::PI,
-                           :rotate => :counterclockwise
-                         })
-    end
-  end
-
-  context '#convert_units' do
-    it 'does not touch arrays integers' do
-      args = {x: [156]}
-      needed_params = [:x]
-      opts = @deck.send(:convert_units, args, needed_params)
-      expect(opts).to eq({ :x => [156] })
-    end
-
-    it 'does not touch arrays floats' do
-      args = {x: [156.2]}
-      needed_params = [:x]
-      opts = @deck.send(:convert_units, args, needed_params)
-      expect(opts).to eq({ :x => [156.2] })
-    end
-
-    it 'converts array of all inches' do
-      args = {x: ['1in', '2in']}
-      needed_params = [:x]
-      opts = @deck.send(:convert_units, args, needed_params)
-      expect(opts).to eq({:x => [300.0, 600.0] }) #assume 300dpi default
-    end
-
-    it 'converts array of some inches' do
-      args = {x: [156, '2in']}
-      needed_params = [:x]
-      opts = @deck.send(:convert_units, args, needed_params)
-      expect(opts).to eq({:x => [156.0, 600.0]}) #assume 300dpi default
-    end
-
-    it 'handles whitespace' do
-      args = {x: ['1in   ']}
-      needed_params = [:x]
-      opts = @deck.send(:convert_units, args, needed_params)
-      expect(opts).to eq({:x => [300.0] }) #assume 300dpi default
-    end
-
-     it 'converts centimeters' do
-      args = {x: ['2cm']}
-      needed_params = [:x]
-      opts = @deck.send(:convert_units, args, needed_params)
-      expect(opts).to eq({:x => [236.2204722] }) #assume 300dpi default
-    end
-
-    it 'handles non-expading singletons' do
-      args = {margin: '1in', trim: '1in', gap: '1in'}
-      needed_params = [:margin, :trim, :gap]
-      opts = @deck.send(:convert_units, args, needed_params)
-      expect(opts).to eq({margin: 300, trim: 300, gap: 300}) #assume 300dpi default
-    end
-
-  end
-
-  context '#rowify' do
-    before(:each) do
-      @default_opts = { rows: :infinite, columns: 5 }
-    end
-    
-    it 'does nothing on an integer' do
-      opts = @deck.send(:rowify, @default_opts.merge({columns: 2, rows: 2}))
-      expect(opts).to eq({ columns: 2,
-                           rows: 2
-                        })
-    end
-
-    it 'computes properly on non-integer' do
-      opts = @deck.send(:rowify, @default_opts.merge({columns: 1, rows: :infinite}))
-      expect(opts).to eq({ columns: 1,
-                           rows: 2
-                        })
-    end
-    
-    it 'computes properly on unspecified rows' do
-      opts = @deck.send(:rowify, @default_opts.merge({columns: 3}))
-      expect(opts).to eq({ columns: 3,
-                           rows: 1
-                        })
-    end
-  end
-
-  context '#faceify' do
-    it 'is false on left' do
-      opts = @deck.send(:faceify, {face: :left})
-      expect(opts).to eq({ face: false })
-    end
-
-    it 'is true on right' do
-      opts = @deck.send(:faceify, {face: 'Right'})
-      expect(opts).to eq({ face: true })
-    end
-
-    it 'is false on anything else' do
-      opts = @deck.send(:faceify, {face: 'flugelhorn'})
-      expect(opts).to eq({ face: false })
-    end
-  end
-
-  context '#formatify' do
-    it 'sets format to nil when format is not set' do
-      opts = @deck.send(:formatify, {foo: true})
-      expect(opts).to eq({
-        foo: true,
-        format: [nil]
-        })
-    end
-
-    it 'updates the format to array' do
-      opts = @deck.send(:formatify, {format: :png})
-      expect(opts).to eq({format: [:png]})
-    end
-
-    it 'updates the format to flattened array' do
-      opts = @deck.send(:formatify, {format: [[:png]]})
-      expect(opts).to eq({format: [:png]})
-    end
-
-  end
-
-end