fontcustom 1.1.0.pre → 1.1.0.pre2

data/lib/fontcustom/version.rb

@@ -1,3 +1,3 @@
 module Fontcustom
-  VERSION = "1.1.0.pre"
+  VERSION = "1.1.0.pre2"
 end

data/lib/fontcustom/watcher.rb

@@ -5,36 +5,38 @@ module Fontcustom
   class Watcher
     def initialize(opts)
       @opts = opts
-      @vector_listener = Listen.to(@opts[:input][:vectors]).relative_paths(true).filter(/\.(eps|svg)$/).change(&callback)
+      @vector_listener = Listen.to(@opts.input[:vectors]).relative_paths(true).filter(/\.(eps|svg)$/).change(&callback)
 
-      templates = @opts[:templates].dup
+      templates = @opts.templates.dup
       templates.delete_if do |template|
-        template.match Util.gem_lib_path
+        template.match Fontcustom.gem_lib
       end
       unless templates.empty?
         templates = templates.map do |template|
           File.basename template
         end
-        @template_listener = Listen.to(@opts[:input][:templates]).relative_paths(true).filter(/(#{templates.join("|")})/).change(&callback)
+        @template_listener = Listen.to(@opts.input[:templates]).relative_paths(true).filter(/(#{templates.join("|")})/).change(&callback)
       end
 
-      @opts[:blocking] = @opts[:blocking] == false ? false : true
-      unless @opts[:blocking]
-        @vector_listener = @vector_listener.polling_fallback_message(false) 
-        @template_listener = @template_listener.polling_fallback_message(false) if @template_listener 
+      # Modified to allow testing
+      @is_test = @opts.instance_variable_get :@is_test
+      if @is_test
+        @vector_listener = @vector_listener.polling_fallback_message(false)
+        @template_listener = @template_listener.polling_fallback_message(false) if @template_listener
       end
     end
 
     def watch
-      puts "Font Custom is watching your icons at #{@opts[:input][:vectors]}. Press Ctrl + C to stop."
-      compile unless @opts[:skip_first]
+      puts "Font Custom is watching your icons at #{@opts.input[:vectors]}. Press Ctrl + C to stop."
+      compile unless @opts.skip_first
 
-      if @opts[:blocking]
-        @vector_listener.start!
-        @template_listener.start! if @template_listener
-      else
+      # Non-blocking if test
+      if @is_test
         @vector_listener.start
         @template_listener.start if @template_listener
+      else
+        @vector_listener.start!
+        @template_listener.start! if @template_listener
       end
 
     rescue Fontcustom::Error => e

data/spec/fixtures/generators/.fontcustom-data-corrupted

@@ -0,0 +1,18 @@
+'Totally invalid JSON.'
+{
+  'fonts": [
+    "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.woff",
+    "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.ttf",
+    "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.eot",
+    "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.svg"
+  ],
+  "templates": [
+    "fontcustom.css"
+  ],
+  "file_name": "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e",
+  "glyphs": [
+    "a_r3ally-exotic-f1le-name",
+    "c",
+    "d"
+  ]
+}

data/spec/fixtures/generators/fontcustom.yml

@@ -0,0 +1 @@
+# noop

data/spec/fixtures/options/fontcustom-empty.yml

@@ -0,0 +1 @@
+# nothing

data/spec/fixtures/options/fontcustom.yml

@@ -0,0 +1 @@
+font_name: Custom Name From Config

data/spec/fontcustom/generator/font_spec.rb

@@ -1,8 +1,13 @@
 require "spec_helper"
 
 describe Fontcustom::Generator::Font do
+  # Silence messages without passing :verbose => false to everything
+  before(:each) do
+    Fontcustom::Options.any_instance.stub :say_message
+  end
+  
   def generator(options)
-    opts = Fontcustom::Util.collect_options options
+    opts = Fontcustom::Options.new(options)
     Fontcustom::Generator::Font.new([opts])
   end
 
@@ -39,7 +44,8 @@ describe Fontcustom::Generator::Font do
     it "should assign empty data model if data file is empty or missing" do
       options = {
         :project_root => fixture,
-        :input => "shared/vectors"
+        :input => "shared/vectors",
+        :verbose => false
       }
       gen = generator options
       gen.get_data
@@ -50,13 +56,28 @@ describe Fontcustom::Generator::Font do
     it "should assign @data from data file" do
       options = {
         :project_root => fixture,
+        :config => "generators",
         :input => "shared/vectors",
-        :output => "mixed-output"
+        :output => "mixed-output",
+        :verbose => false
       }
       gen = generator options
       gen.get_data
       data = gen.instance_variable_get("@data")
-      data[:fonts] =~ data_file_contents[:fonts]
+      data.should_not == Fontcustom::DATA_MODEL
+    end
+
+    it "should throw an error if the data file is corrupted." do
+      options = {
+        :project_root => fixture,
+        :config => "generators",
+        :data_cache => "generators/.fontcustom-data-corrupted",
+        :input => "shared/vectors",
+        :output => "mixed-output",
+        :verbose => false
+      }
+      gen = generator options
+      expect { gen.get_data }.to raise_error Fontcustom::Error, /corrupted/
     end
   end
 
@@ -70,8 +91,7 @@ describe Fontcustom::Generator::Font do
       }
       gen = generator options
       gen.stub :remove_file
-      gen.stub :append_to_file
-      gen.stub :clear_file
+      gen.stub :overwrite_file
       gen.instance_variable_set(:@data, data_file_contents)
       gen
     end
@@ -95,8 +115,7 @@ describe Fontcustom::Generator::Font do
 
     it "should update the data file" do
       file = fixture(".fontcustom-data")
-      subject.should_receive(:clear_file).once.with(file)
-      subject.should_receive(:append_to_file).once.with(file, /"fonts":/, :verbose => false)
+      subject.should_receive(:overwrite_file).once.with(file, /"fonts":/)
       subject.reset_output
     end
 
@@ -175,7 +194,7 @@ describe Fontcustom::Generator::Font do
       )
       gen.instance_variable_set :@data, data_file_contents
       stdout = capture(:stdout) { gen.announce_files }
-      stdout.should =~ /created.+\.(woff|ttf|eot|svg)/
+      stdout.should =~ /create.+\.(woff|ttf|eot|svg)/
     end
 
     it "should print nothing if verbose is false" do
@@ -198,12 +217,10 @@ describe Fontcustom::Generator::Font do
         :input => "shared/vectors",
         :output => "output"
       )
-      gen.stub :clear_file
-      gen.stub :append_to_file
+      gen.stub :overwrite_file
       gen.instance_variable_set(:@data, data_file_contents)
       file = File.join fixture(".fontcustom-data")
-      gen.should_receive(:clear_file).once.with(file)
-      gen.should_receive(:append_to_file).once.with do |path, content|
+      gen.should_receive(:overwrite_file).once.with do |path, content|
         path.should == file
         content.should match(/"fonts":/)
         content.should match(/"glyphs":/)
@@ -218,8 +235,7 @@ describe Fontcustom::Generator::Font do
         :output => "output",
         :verbose => false
       )
-      gen.stub :clear_file
-      gen.stub :append_to_file
+      gen.stub :overwrite_file
       gen.instance_variable_set(:@data, data_file_contents)
       stdout = capture(:stdout) { gen.save_data }
       stdout.should == ""

data/spec/fontcustom/generator/template_spec.rb

@@ -1,8 +1,13 @@
 require "spec_helper"
 
 describe Fontcustom::Generator::Template do
+  # Silence messages without passing :verbose => false to everything
+  before(:each) do
+    Fontcustom::Options.any_instance.stub :say_message
+  end
+  
   def generator(options)
-    opts = Fontcustom::Util.collect_options options
+    opts = Fontcustom::Options.new(options)
     Fontcustom::Generator::Template.new([opts])
   end
 
@@ -10,16 +15,18 @@ describe Fontcustom::Generator::Template do
     it "should raise error if data file doesn't exist" do
       gen = generator(
         :project_root => fixture,
-        :input => "shared/vectors"
+        :input => "shared/vectors",
+        :verbose => false
       )
-      expect { gen.get_data }.to raise_error Fontcustom::Error, /no \.fontcustom-data/
+      expect { gen.get_data }.to raise_error Fontcustom::Error, /\.fontcustom-data is required/
     end
 
     # TODO ensure data file is correct
     it "should assign @data from data file" do
       gen = generator(
         :project_root => fixture("generators"),
-        :input => "../shared/vectors"
+        :input => "../shared/vectors",
+        :verbose => false
       )
       gen.get_data
       gen.instance_variable_get(:@data)[:templates].should =~ data_file_contents[:templates]
@@ -35,8 +42,7 @@ describe Fontcustom::Generator::Template do
         :verbose => false
       )
       gen.stub :remove_file
-      gen.stub :clear_file
-      gen.stub :append_to_file
+      gen.stub :overwrite_file
       gen.instance_variable_set(:@data, data_file_contents)
       gen
     end
@@ -60,8 +66,7 @@ describe Fontcustom::Generator::Template do
 
     it "should update the data file" do
       file = fixture("generators/.fontcustom-data")
-      subject.should_receive(:clear_file).once.with(file)
-      subject.should_receive(:append_to_file).once.with(file, /"templates":/, :verbose => false)
+      subject.should_receive(:overwrite_file).once.with(file, /"templates":/)
       subject.reset_output
     end
 
@@ -72,17 +77,17 @@ describe Fontcustom::Generator::Template do
   end
 
   context "#make_relative_paths" do
-    it "should assign :css_to_fonts and :preview_to_css" do
+    it "should assign :css_to_fonts, :preview_to_fonts, and :preprocessor_to_fonts" do
       gen = generator(
         :project_root => fixture,
         :input => "shared/vectors",
-        :output => {:fonts => "foo/fonts", :css => "output/css", :preview => ""}
+        :output => {:fonts => "foo/fonts", :css => "output/css", :preview => "views/"}
       )
       gen.instance_variable_set "@data", data_file_contents
       gen.make_relative_paths
       data = gen.instance_variable_get("@data")
       data[:paths][:css_to_fonts].should match("../../foo/fonts")
-      data[:paths][:preview_to_css].should match("output/css/")
+      data[:paths][:preview_to_fonts].should match("../foo/fonts")
       data[:paths][:preprocessor_to_fonts].should eq(data[:paths][:css_to_fonts])
     end
 
@@ -109,7 +114,6 @@ describe Fontcustom::Generator::Template do
       gen.make_relative_paths
       data = gen.instance_variable_get("@data")
       data[:paths][:css_to_fonts].should match("./")
-      data[:paths][:preview_to_css].should match("./")
     end
   end
 
@@ -124,8 +128,7 @@ describe Fontcustom::Generator::Template do
       )
       gen.instance_variable_set :@data, data_file_contents
       gen.stub :template
-      gen.stub :clear_file
-      gen.stub :append_to_file
+      gen.stub :overwrite_file
       gen
     end
 
@@ -138,8 +141,7 @@ describe Fontcustom::Generator::Template do
 
     it "should update data file with generated templates" do
       file = fixture("generators/.fontcustom-data")
-      subject.should_receive(:clear_file).once.with(file)
-      subject.should_receive(:append_to_file).once.with do |path, content|
+      subject.should_receive(:overwrite_file).once.with do |path, content|
         path.should == file
         content.should match(/fontcustom\.css/)
         content.should match(/_fontcustom\.scss/)
@@ -164,11 +166,11 @@ describe Fontcustom::Generator::Template do
         )
         gen.instance_variable_set :@data, data_file_contents
         gen.stub :template
-        gen.stub :clear_file
-        gen.stub :append_to_file
+        gen.stub :overwrite_file
         gen
       end
 
+      # 6 times because preview also includes preview-css
       it "should output custom templates to their matching :output paths" do
         subject.should_receive(:template).exactly(5).times do |*args|
           if File.basename(args[0]) == "custom.css"

data/spec/fontcustom/options_spec.rb

@@ -0,0 +1,428 @@
+require 'spec_helper'
+
+describe Fontcustom::Options do
+  def options(args = {})
+    Fontcustom::Options.new(args)
+  end
+
+  def silent
+    Fontcustom::Options.any_instance.stub :say_message
+  end
+
+  before(:each) { Fontcustom::Options.any_instance.stub(:set_options) }
+
+  context "#initialize" do
+    it "should overwite example defaults with real defaults" do
+      # imitates the default hash passed by thor
+      options = Fontcustom::Options.new(Fontcustom::EXAMPLE_OPTIONS.dup)
+      options = options.instance_variable_get(:@cli_options)
+      Fontcustom::EXAMPLE_OPTIONS.keys.each do |key|
+        options[key].should == Fontcustom::DEFAULT_OPTIONS[key]
+      end
+    end
+  end
+
+  context ".set_config_path" do
+    context "when :config is set" do
+      # TODO abstract this into Util?
+      it "should follow ../../ relative paths" do
+        args = {
+          :project_root => fixture("shared/vectors"),
+          :config => "../../options"
+        }
+        o = options args
+        o.send :set_config_path
+        o.instance_variable_get(:@config).should == fixture("options/fontcustom.yml")
+      end
+
+      it "should use options[:config] if it's a file" do
+        args = {
+          :project_root => fixture,
+          :config => "options/any-file-name.yml"
+        }
+        o = options args
+        o.send :set_config_path
+        o.instance_variable_get(:@config).should == fixture("options/any-file-name.yml")
+      end
+
+      it "should search for fontcustom.yml if options[:config] is a dir" do
+        args = {
+          :project_root => fixture,
+          :config => "options/config-is-in-dir"
+        }
+        o = options args
+        o.send :set_config_path
+        o.instance_variable_get(:@config).should == fixture("options/config-is-in-dir/fontcustom.yml")
+      end
+
+      it "should raise error if :config doesn't exist" do
+        args = {
+          :project_root => fixture,
+          :config => "does-not-exist"
+        }
+        o = options args
+        expect { o.send :set_config_path }.to raise_error Fontcustom::Error, /configuration file was not found/
+      end
+    end
+
+    context "when :config is not set" do
+      it "should find fontcustom.yml at :project_root/fontcustom.yml" do
+        args = { :project_root => fixture("options") }
+        o = options args
+        o.send :set_config_path
+        o.instance_variable_get(:@config).should == fixture("options/fontcustom.yml")
+      end
+
+      it "should find fontcustom.yml at :project_root/config/fontcustom.yml" do
+        args = { :project_root => fixture("options/rails-like") }
+        o = options args
+        o.send :set_config_path
+        o.instance_variable_get(:@config).should == fixture("options/rails-like/config/fontcustom.yml")
+      end
+
+      it "should be false if nothing is found" do
+        args = { :project_root => fixture("options/no-config-here") }
+        o = options args
+        o.send :set_config_path
+        o.instance_variable_get(:@config).should == false
+      end
+    end
+  end
+
+  context ".load_config" do
+    def args
+      { :project_root => fixture, :verbose => false }
+    end
+
+    it "should raise error if fontcustom.yml isn't valid" do
+      o = options args
+      o.instance_variable_set :@config, fixture("options/fontcustom-malformed.yml")
+      expect { o.send :load_config }.to raise_error Fontcustom::Error, /failed to load/
+    end
+
+    it "should assign empty hash :config is false" do
+      o = options args
+      o.instance_variable_set :@config, false
+      o.send :load_config
+      o.instance_variable_get(:@config_options).should == {}
+    end
+
+    it "should assign empty hash if fontcustom.yml is blank" do
+      o = options args
+      o.instance_variable_set :@config, fixture("options/fontcustom-empty.yml")
+      o.send :load_config
+      o.instance_variable_get(:@config_options).should == {}
+    end
+  end
+
+  context ".merge_options" do
+    before(:each) { silent }
+
+    it "should set instance variables for each option key" do
+      o = options
+      o.instance_variable_set :@config_options, {}
+      o.send :merge_options
+      o.instance_variables.length.should == Fontcustom::DEFAULT_OPTIONS.length + 2 # @shell, @mock_proxy (rspec)
+    end
+
+    it "should overwrite defaults with config file" do
+      o = options
+      o.instance_variable_set :@config_options, { :input => "config" }
+      o.send :merge_options
+      o.instance_variable_get(:@input).should == "config"
+    end
+
+    it "should overwrite config file and defaults with CLI options" do
+      o = options
+      cli = o.instance_variable_get :@cli_options
+      o.instance_variable_set :@config_options, { :input => "config", :output => "output" }
+      o.instance_variable_set :@cli_options, cli.merge( :input => "cli" )
+      o.send :merge_options
+      o.instance_variable_get(:@input).should == "cli"
+      o.instance_variable_get(:@output).should == "output"
+    end
+
+    it "should normalize the font name" do
+      o = options
+      o.instance_variable_set :@config_options, { :input => "config", :font_name => " A_stR4nG3  nAm3 Ø&  " }
+      o.send :merge_options
+      o.instance_variable_get(:@font_name).should == "A_stR4nG3--nAm3---"
+    end
+  end
+
+  context ".set_data_path" do
+    it "should set :data_cache in the config dir by default" do
+      silent
+      o = options
+      o.instance_variable_set :@config, "path/to/config/fontcustom.yml"
+      o.instance_variable_set :@data_cache, nil
+      o.send :set_data_path
+      o.instance_variable_get(:@data_cache).should == "path/to/config/.fontcustom-data"
+    end
+  end
+
+  context "#send :set_input_paths" do
+    it "should raise error if input[:vectors] doesn't contain vectors" do
+      o = options
+      o.instance_variable_set :@project_root, fixture
+      o.instance_variable_set :@input, "shared/vectors-empty"
+      expect { o.send :set_input_paths }.to raise_error Fontcustom::Error, /doesn't contain any vectors/
+    end
+
+    context "when @input is a hash" do
+      it "should set :templates as :vectors if :templates isn't set" do
+        hash = { :vectors => "shared/vectors" }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@input, hash
+        o.send :set_input_paths
+        input = o.instance_variable_get :@input
+        input[:templates].should == fixture("shared/vectors")
+      end
+
+      it "should preserve :templates if it's set" do
+        hash = { :vectors => "shared/vectors", :templates => "shared/templates" }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@input, hash
+        o.send :set_input_paths
+        input = o.instance_variable_get :@input
+        input[:templates].should == fixture("shared/templates")
+      end
+
+      it "should raise an error if :vectors isn't set" do
+        hash = { :templates => "shared/templates" }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@input, hash
+        expect { o.send :set_input_paths }.to raise_error Fontcustom::Error, /contain a "vectors" key/
+      end
+
+      it "should raise an error if :vectors doesn't point to an existing directory" do
+        hash = { :vectors => "shared/not-a-dir" }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@input, hash
+        expect { o.send :set_input_paths }.to raise_error Fontcustom::Error, /should be a directory/
+      end
+
+      it "should follow ../../ relative paths" do
+        hash = { :vectors => "../../shared/vectors", :templates => "../../shared/templates" }
+        o = options
+        o.instance_variable_set :@project_root, fixture("generators/mixed-output")
+        o.instance_variable_set :@input, hash
+        o.send :set_input_paths
+        input = o.instance_variable_get(:@input)
+        input[:vectors].should == fixture("shared/vectors")
+        input[:templates].should == fixture("shared/templates")
+      end
+    end
+
+    context "when @input is a string" do
+      it "should return a hash of locations" do
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@input, "shared/vectors"
+        o.send :set_input_paths
+        input = o.instance_variable_get :@input
+        input.should have_key("vectors")
+        input.should have_key("templates")
+      end
+
+      it "should set :templates to match :vectors" do
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@input, "shared/vectors"
+        o.send :set_input_paths
+        input = o.instance_variable_get :@input
+        input[:templates].should == fixture("shared/vectors")
+      end
+
+      it "should raise an error if :vectors doesn't point to a directory" do
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@input, "shared/not-a-dir"
+        expect { o.send :set_input_paths }.to raise_error Fontcustom::Error, /should be a directory/
+      end
+
+      it "should follow ../../ relative paths" do
+        o = options
+        o.instance_variable_set :@project_root, fixture("generators/mixed-output")
+        o.instance_variable_set :@input, "../../shared/vectors"
+        o.send :set_input_paths
+        input = o.instance_variable_get(:@input)
+        input[:vectors].should == fixture("shared/vectors")
+        input[:templates].should == fixture("shared/vectors")
+      end
+    end
+  end
+
+  context ".set_output_paths" do
+    context "when @output is nil" do
+      it "should default to :project_root/:font_name" do
+        silent
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@font_name, "Test-Font"
+        o.instance_variable_set :@output, nil
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output[:fonts].should == fixture("Test-Font")
+      end
+
+      it "should print a warning" do
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@font_name, "Test-Font"
+        o.instance_variable_set :@output, nil
+        stdout = capture(:stdout) { o.send :set_output_paths }
+        stdout.should match("Test-Font")
+      end
+    end
+
+    context "when @output is a hash" do
+      it "should set :css and :preview to match :fonts if either aren't set" do
+        hash = { :fonts => "output/fonts" }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@output, hash
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output[:css].should == fixture("output/fonts")
+        output[:preview].should == fixture("output/fonts")
+      end
+
+      it "should preserve :css and :preview if they do exist" do
+        hash = {
+          :fonts => "output/fonts",
+          :css => "output/styles",
+          :preview => "output/preview"
+        }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@output, hash
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output[:css].should == fixture("output/styles")
+        output[:preview].should == fixture("output/preview")
+      end
+
+      it "should create additional paths if they are given" do
+        hash = {
+          :fonts => "output/fonts",
+          "special.js" => "assets/javascripts"
+        }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@output, hash
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output["special.js"].should == fixture("assets/javascripts")
+      end
+
+      it "should raise an error if :fonts isn't set" do
+        hash = { :css => "output/styles" }
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@output, hash
+        expect { o.send :set_output_paths }.to raise_error Fontcustom::Error, /contain a "fonts" key/
+      end
+
+      it "should follow ../../ relative paths" do
+        hash = {
+          :fonts => "../../output/fonts",
+          :css => "../../output/css",
+          :preview => "../../output/views"
+        }
+        o = options
+        o.instance_variable_set :@project_root, fixture("generators/mixed-output")
+        o.instance_variable_set :@output, hash
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output[:fonts].should == fixture("output/fonts")
+        output[:css].should == fixture("output/css")
+        output[:preview].should == fixture("output/views")
+      end
+    end
+
+    context "when @output is a string" do
+      it "should return a hash of output locations" do
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@output, "output/fonts"
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output.should be_a(Hash)
+        output.should have_key("fonts")
+        output.should have_key("css")
+        output.should have_key("preview")
+      end
+
+      it "should set :css and :preview to match :fonts" do
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@output, "output/fonts"
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output[:css].should == fixture("output/fonts")
+        output[:preview].should == fixture("output/fonts")
+      end
+
+      it "should raise an error if :fonts exists but isn't a directory" do
+        o = options
+        o.instance_variable_set :@project_root, fixture
+        o.instance_variable_set :@output, "shared/not-a-dir"
+        expect { o.send :set_output_paths }.to raise_error Fontcustom::Error, /directory, not a file/
+      end
+
+      it "should follow ../../ relative paths" do
+        o = options
+        o.instance_variable_set :@project_root, fixture("generators/mixed-output")
+        o.instance_variable_set :@output, "../../something/else"
+        o.send :set_output_paths
+        output = o.instance_variable_get :@output
+        output[:fonts].should == fixture("something/else")
+        output[:css].should == fixture("something/else")
+        output[:preview].should == fixture("something/else")
+      end
+    end
+  end
+
+  context ".set_template_paths" do
+    it "should expand shorthand for packaged templates" do
+      o = options
+      o.instance_variable_set :@project_root, fixture
+      o.instance_variable_set :@input, { :templates => "shared/templates" }
+      o.instance_variable_set :@templates, %w|preview css scss bootstrap bootstrap-scss bootstrap-ie7 bootstrap-ie7-scss|
+      o.send :set_template_paths
+      templates = o.instance_variable_get :@templates
+      templates.should =~ [
+        File.join(Fontcustom.gem_lib, "templates", "fontcustom-preview.html"),
+        File.join(Fontcustom.gem_lib, "templates", "fontcustom.css"),
+        File.join(Fontcustom.gem_lib, "templates", "_fontcustom.scss"),
+        File.join(Fontcustom.gem_lib, "templates", "fontcustom-bootstrap.css"),
+        File.join(Fontcustom.gem_lib, "templates", "_fontcustom-bootstrap.scss"),
+        File.join(Fontcustom.gem_lib, "templates", "fontcustom-bootstrap-ie7.css"),
+        File.join(Fontcustom.gem_lib, "templates", "_fontcustom-bootstrap-ie7.scss")
+      ]
+    end
+
+    it "should find custom templates in :template_path" do
+      o = options
+      o.instance_variable_set :@project_root, fixture
+      o.instance_variable_set :@input, { :templates => fixture("shared/templates") }
+      o.instance_variable_set :@templates, %w|custom.css|
+      o.send :set_template_paths
+      templates = o.instance_variable_get :@templates
+      templates.should =~ [fixture("shared/templates/custom.css")]
+    end
+
+    it "should raise an error if a template does not exist" do
+      o = options
+      o.instance_variable_set :@project_root, fixture
+      o.instance_variable_set :@input, { :templates => "shared/templates" }
+      o.instance_variable_set :@templates, %w|fake-template.txt|
+      expect { o.send :set_template_paths }.to raise_error Fontcustom::Error, /does not exist/
+    end
+  end
+end