squib 0.8.0 → 0.9.0

data/spec/args/save_batch_spec.rb

@@ -1,51 +1,51 @@
-require 'spec_helper'
-require 'squib/args/save_batch'
-
-describe Squib::Args::SaveBatch do
-  subject(:save_batch) {Squib::Args::SaveBatch.new}
-
-  context 'dir' do
-    it 'is created if not exists (and warns)' 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
-        save_batch.load! opts
-        expect(save_batch).to have_attributes({dir: ['tocreate']})
-        expect(Dir.exists? 'tocreate').to be true
-      end
-    end
-  end
-
-  context 'rotate' do
-    it 'does nothing by default' do
-      opts = {}
-      save_batch.load! opts
-      expect(save_batch[0]).to have_attributes({rotate: false, angle: 0})
-    end
-
-    it 'rotates by pi/2 with true' do
-      opts = {rotate: true}
-      save_batch.load! opts
-      expect(save_batch[0]).to have_attributes({rotate: true, angle: Math::PI / 2})
-    end
-
-    it 'rotates by pi/2' do
-      opts = {rotate: :clockwise}
-      save_batch.load! opts
-      expect(save_batch[0]).to have_attributes({rotate: true, angle: Math::PI / 2})
-    end
-
-    it 'rotates by pi/2 with counterclockwise' do
-      opts = {rotate: :counterclockwise}
-      save_batch.load! opts
-      expect(save_batch[0]).to have_attributes({rotate: true, angle: 3 * Math::PI / 2})
-    end
-
-    it 'raises error on a number' do
-      opts = {rotate: 5.0}
-      expect { save_batch.load!(opts) }.to raise_error('invalid option to rotate: only [true, false, :clockwise, :counterclockwise]')
-    end
-  end
-
-end
+require 'spec_helper'
+require 'squib/args/save_batch'
+
+describe Squib::Args::SaveBatch do
+  subject(:save_batch) {Squib::Args::SaveBatch.new}
+
+  context 'dir' do
+    it 'is created if not exists (and warns)' 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
+        save_batch.load! opts
+        expect(save_batch).to have_attributes({dir: ['tocreate']})
+        expect(Dir.exists? 'tocreate').to be true
+      end
+    end
+  end
+
+  context 'rotate' do
+    it 'does nothing by default' do
+      opts = {}
+      save_batch.load! opts
+      expect(save_batch[0]).to have_attributes({rotate: false, angle: 0})
+    end
+
+    it 'rotates by pi/2 with true' do
+      opts = {rotate: true}
+      save_batch.load! opts
+      expect(save_batch[0]).to have_attributes({rotate: true, angle: Math::PI / 2})
+    end
+
+    it 'rotates by pi/2' do
+      opts = {rotate: :clockwise}
+      save_batch.load! opts
+      expect(save_batch[0]).to have_attributes({rotate: true, angle: Math::PI / 2})
+    end
+
+    it 'rotates by pi/2 with counterclockwise' do
+      opts = {rotate: :counterclockwise}
+      save_batch.load! opts
+      expect(save_batch[0]).to have_attributes({rotate: true, angle: 3 * Math::PI / 2})
+    end
+
+    it 'raises error on a number' do
+      opts = {rotate: 5.0}
+      expect { save_batch.load!(opts) }.to raise_error('invalid option to rotate: only [true, false, :clockwise, :counterclockwise]')
+    end
+  end
+
+end

data/spec/args/scale_box_spec.rb

@@ -1,71 +1,71 @@
-require 'spec_helper'
-require 'squib/args/scale_box'
-
-describe Squib::Args::ScaleBox do
-  subject(:box) { Squib::Args::ScaleBox.new({}) }
-
-  context 'unit conversion' do
-    it 'converts units on all args' do
-      args = {x: ['1in', '2in'], y: 300, width: '1in', height: '1in'}
-      box.load!(args, expand_by: 2)
-      expect(box).to have_attributes(
-        x: [300, 600],
-        y: [300, 300],
-        width: [300, 300],
-        height: [300, 300],
-      )
-    end
-  end
-
-  context 'validation' do
-    it 'replaces with deck width and height' do
-      args = {width: :deck, height: :deck}
-      deck = OpenStruct.new(width: 123, height: 456)
-      box = Squib::Args::Box.new(deck)
-      box.load!(args, expand_by: 1)
-      expect(box).to have_attributes(width: [123], height: [456])
-    end
-
-    it 'allows :native' do
-      args = {width: :native, height: :native}
-      box.load!(args, expand_by: 1)
-      expect(box).to have_attributes(width: [:native], height: [:native])
-    end
-
-    it 'allows native to be a string' do
-      args = {width: 'native'}
-      box.load!(args, expand_by: 1)
-      expect(box).to have_attributes(width: [:native], height: [:native])
-    end
-
-    it 'allows :scale on width if height has to_f' do
-      args = {width: :scale, height: 75}
-      box.load!(args, expand_by: 1)
-      expect(box).to have_attributes(width: [:scale], height: [75])
-    end
-
-    it 'allows :scale on width if height has to_f' do
-      args = {width: 75, height: :scale}
-      box.load!(args, expand_by: 1)
-      expect(box).to have_attributes(width: [75], height: [:scale])
-    end
-
-    it 'disallows both :scale' do
-      args = {width: :scale, height: :scale}
-      expect { box.load!(args, expand_by: 1) }.to raise_error('if width is :scale, height must be a number')
-    end
-
-    it 'disallows non-to_f on width' do
-      args = {width: :foo}
-      expect { box.load!(args, expand_by: 1) }.to raise_error('width must be a number, :scale, :native, or :deck')
-    end
-
-    it 'disallows non-to_f on height' do
-      args = {height: :foo}
-      expect { box.load!(args, expand_by: 1) }.to raise_error('height must be a number, :scale, :native, or :deck')
-    end
-
-  end
-
-
-end
+require 'spec_helper'
+require 'squib/args/scale_box'
+
+describe Squib::Args::ScaleBox do
+  subject(:box) { Squib::Args::ScaleBox.new({}) }
+
+  context 'unit conversion' do
+    it 'converts units on all args' do
+      args = {x: ['1in', '2in'], y: 300, width: '1in', height: '1in'}
+      box.load!(args, expand_by: 2)
+      expect(box).to have_attributes(
+        x: [300, 600],
+        y: [300, 300],
+        width: [300, 300],
+        height: [300, 300],
+      )
+    end
+  end
+
+  context 'validation' do
+    it 'replaces with deck width and height' do
+      args = {width: :deck, height: :deck}
+      deck = OpenStruct.new(width: 123, height: 456)
+      box = Squib::Args::Box.new(deck)
+      box.load!(args, expand_by: 1)
+      expect(box).to have_attributes(width: [123], height: [456])
+    end
+
+    it 'allows :native' do
+      args = {width: :native, height: :native}
+      box.load!(args, expand_by: 1)
+      expect(box).to have_attributes(width: [:native], height: [:native])
+    end
+
+    it 'allows native to be a string' do
+      args = {width: 'native'}
+      box.load!(args, expand_by: 1)
+      expect(box).to have_attributes(width: [:native], height: [:native])
+    end
+
+    it 'allows :scale on width if height has to_f' do
+      args = {width: :scale, height: 75}
+      box.load!(args, expand_by: 1)
+      expect(box).to have_attributes(width: [:scale], height: [75])
+    end
+
+    it 'allows :scale on width if height has to_f' do
+      args = {width: 75, height: :scale}
+      box.load!(args, expand_by: 1)
+      expect(box).to have_attributes(width: [75], height: [:scale])
+    end
+
+    it 'disallows both :scale' do
+      args = {width: :scale, height: :scale}
+      expect { box.load!(args, expand_by: 1) }.to raise_error('if width is :scale, height must be a number')
+    end
+
+    it 'disallows non-to_f on width' do
+      args = {width: :foo}
+      expect { box.load!(args, expand_by: 1) }.to raise_error('width must be a number, :scale, :native, or :deck')
+    end
+
+    it 'disallows non-to_f on height' do
+      args = {height: :foo}
+      expect { box.load!(args, expand_by: 1) }.to raise_error('height must be a number, :scale, :native, or :deck')
+    end
+
+  end
+
+
+end

data/spec/args/sheet_spec.rb

@@ -1,58 +1,58 @@
-require 'spec_helper'
-require 'squib/args/sheet'
-
-describe Squib::Args::Sheet do
-
-  context 'dsl overrides' do
-    subject(:sheet) { Squib::Args::Sheet.new({}, {file: 'foo'}) }
-
-    it 'works when specified' do
-      sheet.load!({}) # go right to defaults
-      expect(sheet.file).to eq( 'foo' ) # dsl method default override
-    end
-
-  end
-
-  context 'rows and colums' do
-    subject(:sheet) { Squib::Args::Sheet.new({}, {}, 4) }
-
-    it 'does nothing on a perfect fit' do
-      opts = { columns: 2, rows: 2 }
-      sheet.load! opts
-      expect(sheet).to have_attributes(columns: 2, rows: 2)
-    end
-
-    it 'keeps both if specified' do
-      opts = { columns: 1, rows: 1 }
-      sheet.load! opts
-      expect(sheet).to have_attributes(columns: 1, rows: 1)
-    end
-
-    it 'computes properly on non-integer' do
-      opts = {columns: 1, rows: :infinite}
-      sheet.load! opts
-      expect(sheet).to have_attributes( columns: 1, rows: 4 )
-    end
-
-    it 'computes properly on unspecified rows' do
-      opts = {columns: 1}
-      sheet.load! opts
-      expect(sheet).to have_attributes( columns: 1, rows: 4 )
-    end
-
-    it 'computes properly on unspecified, too-big column' do
-      opts = {}
-      sheet.load! opts
-      expect(sheet).to have_attributes( columns: 5, rows: 1 )
-    end
-
-    it 'fails on a non-integer column' do
-      opts = {columns: :infinite}
-      expect { sheet.load!(opts) }.to raise_error('columns must be an integer')
-    end
-
-
-
-  end
-
-end
+require 'spec_helper'
+require 'squib/args/sheet'
+
+describe Squib::Args::Sheet do
+
+  context 'dsl overrides' do
+    subject(:sheet) { Squib::Args::Sheet.new({}, {file: 'foo'}) }
+
+    it 'works when specified' do
+      sheet.load!({}) # go right to defaults
+      expect(sheet.file).to eq( 'foo' ) # dsl method default override
+    end
+
+  end
+
+  context 'rows and colums' do
+    subject(:sheet) { Squib::Args::Sheet.new({}, {}, 4) }
+
+    it 'does nothing on a perfect fit' do
+      opts = { columns: 2, rows: 2 }
+      sheet.load! opts
+      expect(sheet).to have_attributes(columns: 2, rows: 2)
+    end
+
+    it 'keeps both if specified' do
+      opts = { columns: 1, rows: 1 }
+      sheet.load! opts
+      expect(sheet).to have_attributes(columns: 1, rows: 1)
+    end
+
+    it 'computes properly on non-integer' do
+      opts = {columns: 1, rows: :infinite}
+      sheet.load! opts
+      expect(sheet).to have_attributes( columns: 1, rows: 4 )
+    end
+
+    it 'computes properly on unspecified rows' do
+      opts = {columns: 1}
+      sheet.load! opts
+      expect(sheet).to have_attributes( columns: 1, rows: 4 )
+    end
+
+    it 'computes properly on unspecified, too-big column' do
+      opts = {}
+      sheet.load! opts
+      expect(sheet).to have_attributes( columns: 5, rows: 1 )
+    end
+
+    it 'fails on a non-integer column' do
+      opts = {columns: :infinite}
+      expect { sheet.load!(opts) }.to raise_error('columns must be an integer')
+    end
+
+
+
+  end
+
+end

data/spec/args/showcase_special_spec.rb

@@ -1,15 +1,15 @@
-require 'spec_helper'
-require 'squib/args/showcase_special'
-
-describe Squib::Args::ShowcaseSpecial do
-  subject(:showcase_special) { Squib::Args::ShowcaseSpecial.new }
-
-  context '#face_right?' do
-    it 'compares face to right' do
-      opts = { face: 'LEFT ' }
-      showcase_special.load! opts
-      expect(showcase_special.face_right?).to be false
-    end
-  end
-
-end
+require 'spec_helper'
+require 'squib/args/showcase_special'
+
+describe Squib::Args::ShowcaseSpecial do
+  subject(:showcase_special) { Squib::Args::ShowcaseSpecial.new }
+
+  context '#face_right?' do
+    it 'compares face to right' do
+      opts = { face: 'LEFT ' }
+      showcase_special.load! opts
+      expect(showcase_special.face_right?).to be false
+    end
+  end
+
+end

data/spec/args/transform_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+require 'squib/args/transform'
+
+describe Squib::Args::Box do
+  subject(:trans) { Squib::Args::Transform.new }
+  let(:expected_defaults) { {x: [0], y: [0], crop_width: [:native], crop_height: [:native] } }
+
+  context 'validation' do
+    it 'replaces with deck width and height' do
+      args  = {crop_width: :deck, crop_height: :deck}
+      deck  = OpenStruct.new(width: 123, height: 456)
+      trans = Squib::Args::Transform.new(deck)
+      trans.load!(args, expand_by: 1)
+      expect(trans).to have_attributes(crop_width: [123], crop_height: [456])
+    end
+
+    it 'has radius override x_radius and y_radius' do
+      args = {crop_corner_x_radius: 1, crop_corner_y_radius: 2, crop_corner_radius: 3}
+      trans.load!(args, expand_by: 2)
+      expect(trans).to have_attributes(crop_corner_x_radius: [3, 3], crop_corner_y_radius: [3, 3])
+    end
+
+  end
+
+end

data/spec/card_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+require 'squib/card'
+
+describe Squib::Card do
+  it 'logs fatal for unknown backend' do
+    deck = OpenStruct.new(dir: '.', prefix: '', count_format: '%02d', backend: 'broken robots')
+
+    expect(Squib::logger).to receive(:fatal).with("Back end not recognized: 'broken robots'")
+    expect { Squib::Card.new(deck, 100, 100, 0) }.to raise_error SystemExit
+  end
+end

data/spec/conf_spec.rb

@@ -4,17 +4,17 @@ require 'spec_helper'
 describe Squib::Conf do
 
   it 'parses the project template file just fine' do
-    conf = Squib::Conf.load(project_template('config.yml'))
+    conf = Squib::Conf.load project_template('config.yml')
     expect(conf.backend).to eq(Squib::Conf::DEFAULTS['backend'])
   end
 
   it 'parses an empty file' do
-    conf = Squib::Conf.load(conf('empty.yml'))
+    conf = Squib::Conf.load conf('empty.yml')
     expect(conf.backend).to eq(Squib::Conf::DEFAULTS['backend'])
   end
 
   it 'parses the sample custom config' do
-    conf = Squib::Conf.load(sample_file('custom-config.yml'))
+    conf = Squib::Conf.load sample_file('custom-config.yml')
     expect(conf.progress_bars).to  be true
     expect(conf.text_hint).to      eq '#FF0000'
     expect(conf.custom_colors).to  eq({ 'foo' => '#ccc' })
@@ -26,4 +26,14 @@ describe Squib::Conf do
     expect(Squib::Conf.new.antialias).to          eq 'subpixel'
   end
 
+  it 'warns when the yml has an unrecognized option' do
+    expect(Squib::logger).to receive(:warn).with('Unrecognized configuration option(s): unicorns')
+    Squib::Conf.load conf('unrecognized.yml')
+  end
+
+  it 'helps Andy get full coverage with a test on to_s' do
+    conf = Squib::Conf.load conf('empty.yml')
+    expect(conf.to_s).to start_with 'Conf: '
+  end
+
 end