fontcustom 1.0.1 → 1.1.0.pre

data/spec/fixtures/shared/templates/custom.css

@@ -0,0 +1 @@
+.foo { color: black; }

data/spec/fixtures/shared/templates/regular.css

@@ -0,0 +1 @@
+.foo { color: blue; }

data/spec/fixtures/util/config-is-in-dir/fontcustom.yml

@@ -0,0 +1 @@
+font_name: Config Is in Dir

data/spec/fixtures/util/fontcustom-malformed.yml

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

data/spec/fixtures/util/rails-like/config/fontcustom.yml

@@ -0,0 +1 @@
+font_name: Rails Like

data/spec/fontcustom/actions_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe Fontcustom::Actions do
+  class Generator
+    include Fontcustom::Actions
+    attr_accessor :opts
+
+    def initialize
+      @opts = { :project_root => fixture, :verbose => true }
+      @shell = Thor::Shell::Color.new
+    end
+  end
+
+  context "#say_changed" do
+    it "should strip :project_root from changed paths" do
+      changed = %w|a b c|.map { |file| fixture(file) }
+      gen = Generator.new
+      output = capture(:stdout) { gen.say_changed(:success, changed) }
+      output.should_not match(fixture)
+    end
+  end
+end

data/spec/fontcustom/generator/font_spec.rb

@@ -6,63 +6,78 @@ describe Fontcustom::Generator::Font do
     Fontcustom::Generator::Font.new([opts])
   end
 
-  context "#check_input" do
-    it "should raise error if input doesn't exist" do
-      options = { :input => fixture("does-not-exist") }
-      expect { generator(options).invoke :check_input }.to raise_error Fontcustom::Error, /doesn't exist/
-    end
-
-    it "should raise error if input isn't a directory" do
-      options = { :input => fixture("not-a-dir") }
-      expect { generator(options).invoke :check_input }.to raise_error Fontcustom::Error, /isn't a directory/
-    end
-
-    it "should raise error if input doesn't contain vectors" do
-      options = { :input => fixture("empty") }
-      expect { generator(options).invoke :check_input }.to raise_error Fontcustom::Error, /doesn't contain any vectors/
-    end
-  end
-
-  context "#check_output" do
-    it "should raise an error if output isn't a directory" do
-      options = { :output => fixture("not-a-dir") }
-      expect { generator(options).check_output }.to raise_error Fontcustom::Error, /isn't a directory/
-    end
-
-    it "should create output dir and data file if they don't exist" do
-      gen = generator :output => fixture("create-me")
-      gen.stub(:add_file)
-      gen.should_receive(:add_file).with(fixture("create-me") + "/.fontcustom-data", anything)
-      gen.check_output
+  context "#prepare_output_dirs" do
+    it "should create output dir if it doesn't exist" do
+      options = {
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "create-me"
+      }
+      gen = generator options
+      gen.stub(:empty_directory)
+      gen.should_receive(:empty_directory).with(fixture("create-me"), anything)
+      gen.prepare_output_dirs
+    end
+
+    it "should create dirs for multiple output dirs" do
+      options = {
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => {
+          :fonts => "assets/fonts",
+          :css => "assets/stylesheets"
+        }
+      }
+      gen = generator options
+      gen.stub(:empty_directory)
+      gen.should_receive(:empty_directory).twice
+      gen.prepare_output_dirs
     end
   end
 
   context "#get_data" do
-    it "should assign empty data model if no data file is empty" do
-      gen = generator :output => fixture("empty-data")
-      data = gen.get_data
+    it "should assign empty data model if data file is empty or missing" do
+      options = {
+        :project_root => fixture,
+        :input => "shared/vectors"
+      }
+      gen = generator options
+      gen.get_data
+      data = gen.instance_variable_get("@data")
       data.should == Fontcustom::DATA_MODEL
     end
 
     it "should assign @data from data file" do
-      gen = generator :output => fixture("mixed-output")
-      data = gen.get_data
+      options = {
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "mixed-output"
+      }
+      gen = generator options
+      gen.get_data
+      data = gen.instance_variable_get("@data")
       data[:fonts] =~ data_file_contents[:fonts]
     end
   end
 
   context "#reset_output" do
     subject do
-      gen = generator :output => fixture("mixed-output")
+      options = {
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "mixed-output",
+        :verbose => false
+      }
+      gen = generator options
       gen.stub :remove_file
-      Fontcustom::Util.stub :clear_file
       gen.stub :append_to_file
+      gen.stub :clear_file
       gen.instance_variable_set(:@data, data_file_contents)
       gen
     end
 
     it "should delete fonts from @data[:fonts]" do
-      subject.should_receive(:remove_file).exactly(4).times.with(/fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e/, :verbose => true)
+      subject.should_receive(:remove_file).exactly(4).times.with(/fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e/, anything)
       subject.reset_output
     end
 
@@ -77,10 +92,10 @@ describe Fontcustom::Generator::Font do
       subject.reset_output
       subject.instance_variable_get(:@data)[:fonts].should be_empty
     end
-    
+
     it "should update the data file" do
-      file = File.join(fixture("mixed-output"), ".fontcustom-data")
-      Fontcustom::Util.should_receive(:clear_file).once.with(file)
+      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.reset_output
     end
@@ -93,18 +108,23 @@ describe Fontcustom::Generator::Font do
 
   context "#generate" do
     subject do
-      gen = generator(:input => fixture("vectors"), :output => fixture("mixed-output"))
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "mixed-output",
+        :verbose => false
+      )
       gen.stub(:"`").and_return fontforge_output
       gen
     end
-    
+
     it "should call fontforge" do
       subject.should_receive(:"`").with(/fontforge -script/)
       subject.generate
     end
 
     it "should pass options to fontforge" do
-      subject.should_receive(:"`").with(/#{fixture("vectors")}.+#{fixture("mixed-output")}/)
+      subject.should_receive(:"`").with(/#{fixture("shared/vectors")}.+#{fixture("mixed-output")}/)
       subject.generate
     end
 
@@ -115,65 +135,90 @@ describe Fontcustom::Generator::Font do
     end
 
     it "should raise error if fontforge fails" do
-      gen = generator(:input => fixture("vectors"), :output => fixture("fake-dir-should-cause-failure"), :debug => true)
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "fake-dir-should-cause-failure",
+        :debug => true
+      )
       expect { capture(:stdout) { gen.generate } }.to raise_error Fontcustom::Error, /failed unexpectedly/
     end
   end
 
   context "#collect_data" do
     it "should assign @data from @json" do
-      gen = generator(:input => fixture("vectors"), :output => fixture("mixed-output"))
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "output"
+      )
       gen.instance_variable_set(:@data, Fontcustom::DATA_MODEL)
       gen.instance_variable_set(:@json, data_file_contents.to_json)
       gen.collect_data
       data = gen.instance_variable_get(:@data)
       data[:glyphs].should =~ ["c", "d", "a_r3ally-exotic-f1le-name"]
-      data[:file_name].should == "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e" 
       data[:fonts].should =~ [
-        "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.eot", 
-        "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.svg", 
-        "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.ttf", 
-        "fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.woff"
+        fixture("output/fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.eot"),
+        fixture("output/fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.svg"),
+        fixture("output/fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.ttf"),
+        fixture("output/fontcustom_cc5ce52f2ae4f9ce2e7ee8131bbfee1e.woff")
       ]
     end
   end
 
   context "#announce_files" do
     it "should print generated files to console" do
-      gen = generator(:input => fixture("vectors"), :output => fixture("mixed-output"))
-      gen.instance_variable_set :@data, data_file_contents 
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "output"
+      )
+      gen.instance_variable_set :@data, data_file_contents
       stdout = capture(:stdout) { gen.announce_files }
-      stdout.should =~ /create.+\.(woff|ttf|eot|svg)/
+      stdout.should =~ /created.+\.(woff|ttf|eot|svg)/
     end
 
     it "should print nothing if verbose is false" do
-      gen = generator(:input => fixture("vectors"), :output => fixture("mixed-output"), :verbose => false)
-      gen.instance_variable_set :@data, data_file_contents 
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "output",
+        :verbose => false
+      )
+      gen.instance_variable_set :@data, data_file_contents
       stdout = capture(:stdout) { gen.announce_files }
       stdout.should == ""
     end
   end
 
   context "#save_data" do
-    it "should update data file (TEMP)" do
-      gen = generator(:output => fixture("mixed-output"))
-      Fontcustom::Util.stub :clear_file
+    it "should update data file" do
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "output"
+      )
+      gen.stub :clear_file
       gen.stub :append_to_file
       gen.instance_variable_set(:@data, data_file_contents)
-      file = File.join(fixture("mixed-output"), ".fontcustom-data")
-      Fontcustom::Util.should_receive(:clear_file).once.with(file)
+      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|
         path.should == file
         content.should match(/"fonts":/)
         content.should match(/"glyphs":/)
-        content.should match(/"file_name":/)
       end
       gen.save_data
     end
 
     it "should be silent if verbose is false" do
-      gen = generator(:output => fixture("mixed-output"), :verbose => false)
-      Fontcustom::Util.stub :clear_file
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "output",
+        :verbose => false
+      )
+      gen.stub :clear_file
       gen.stub :append_to_file
       gen.instance_variable_set(:@data, data_file_contents)
       stdout = capture(:stdout) { gen.save_data }

data/spec/fontcustom/generator/template_spec.rb

@@ -7,37 +7,42 @@ describe Fontcustom::Generator::Template do
   end
 
   context "#get_data" do
-    it "should raise error if data file doesn't exist" do 
-      gen = generator :output => fixture("empty")
+    it "should raise error if data file doesn't exist" do
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors"
+      )
       expect { gen.get_data }.to raise_error Fontcustom::Error, /no \.fontcustom-data/
     end
 
+    # TODO ensure data file is correct
     it "should assign @data from data file" do
-      gen = generator :output => fixture("mixed-output")
+      gen = generator(
+        :project_root => fixture("generators"),
+        :input => "../shared/vectors"
+      )
       gen.get_data
       gen.instance_variable_get(:@data)[:templates].should =~ data_file_contents[:templates]
     end
   end
 
-  context "#check_templates" do
-    it "should raise an error if no templates are given" do 
-      gen = generator :templates => []
-      expect { gen.check_templates }.to raise_error Fontcustom::Error, /No templates were specified/
-    end
-  end
-
   context "#reset_output" do
     subject do
-      gen = generator :output => fixture("mixed-output")
+      gen = generator(
+        :project_root => fixture("generators"),
+        :input => "../shared/vectors",
+        :output => "mixed-output",
+        :verbose => false
+      )
       gen.stub :remove_file
-      Fontcustom::Util.stub :clear_file
+      gen.stub :clear_file
       gen.stub :append_to_file
       gen.instance_variable_set(:@data, data_file_contents)
       gen
     end
 
     it "should delete files from @data[:templates]" do
-      subject.should_receive(:remove_file).once.with(/fontcustom\.css/, :verbose => true)
+      subject.should_receive(:remove_file).once.with(/fontcustom\.css/, anything)
       subject.reset_output
     end
 
@@ -52,10 +57,10 @@ describe Fontcustom::Generator::Template do
       subject.reset_output
       subject.instance_variable_get(:@data)[:templates].should be_empty
     end
-    
+
     it "should update the data file" do
-      file = File.join(fixture("mixed-output"), ".fontcustom-data")
-      Fontcustom::Util.should_receive(:clear_file).once.with(file)
+      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.reset_output
     end
@@ -66,41 +71,131 @@ describe Fontcustom::Generator::Template do
     end
   end
 
+  context "#make_relative_paths" do
+    it "should assign :css_to_fonts and :preview_to_css" do
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => {:fonts => "foo/fonts", :css => "output/css", :preview => ""}
+      )
+      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][:preprocessor_to_fonts].should eq(data[:paths][:css_to_fonts])
+    end
+
+    it "should assign :preprocessor_to_css if :preprocessor_font_path is set" do
+      gen = generator(
+        :project_root => fixture,
+        :preprocessor_font_path => "fonts/fontcustom",
+        :input => "shared/vectors",
+        :output => {:fonts => "foo/bar/fonts", :css => "output/css", :preview => ""}
+      )
+      gen.instance_variable_set "@data", data_file_contents
+      gen.make_relative_paths
+      data = gen.instance_variable_get("@data")
+      data[:paths][:preprocessor_to_fonts].should match("fonts/fontcustom")
+    end
+
+    it "should assign '.' when paths are the same" do
+      gen = generator(
+        :project_root => fixture,
+        :input => "shared/vectors",
+        :output => "output"
+      )
+      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("./")
+      data[:paths][:preview_to_css].should match("./")
+    end
+  end
+
   context "#generate" do
     subject do
-      gen = generator :output => fixture("mixed-output"), :templates => %W|scss css #{fixture("not-a-dir")}|
+      gen = generator(
+        :project_root => fixture("generators"),
+        :input => {:vectors => "../shared/vectors", :templates => "../shared/templates"},
+        :output => "mixed-output",
+        :templates => %W|scss css custom.css|,
+        :verbose => false
+      )
       gen.instance_variable_set :@data, data_file_contents
       gen.stub :template
-      Fontcustom::Util.stub :clear_file
+      gen.stub :clear_file
       gen.stub :append_to_file
       gen
     end
 
     it "should call #template for each template" do
-      subject.should_receive(:template).exactly(3).times
+      subject.should_receive(:template).exactly(3).times do |*args|
+        args[1].should match(/(fontcustom\.css|_fontcustom\.scss|custom\.css)/)
+      end
       subject.generate
     end
 
-    it "should update data file" do 
-      file = File.join fixture("mixed-output"), ".fontcustom-data"
-      Fontcustom::Util.should_receive(:clear_file).once.with(file)
+    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|
         path.should == file
         content.should match(/fontcustom\.css/)
         content.should match(/_fontcustom\.scss/)
-        content.should match(/not-a-dir/)
+        content.should match(/custom\.css/)
       end
       subject.generate
     end
 
     it "should be silent if verbose is false" do
-      gen = generator :output => fixture("mixed-output"), :templates => %W|scss css #{fixture("not-a-dir")}|, :verbose => false
-      gen.instance_variable_set :@data, data_file_contents
-      gen.stub :template
-      Fontcustom::Util.stub :clear_file
-      gen.stub :append_to_file
-      stdout = capture(:stdout) { gen.generate }
+      stdout = capture(:stdout) { subject.generate }
       stdout.should == ""
     end
+
+    context "when various output locations are given" do
+      subject do
+        gen = generator(
+          :project_root => fixture,
+          :input => {:vectors => "shared/vectors", :templates => "shared/templates"},
+          :output => {:fonts => "output/fonts", :css => "output/css", :preview => "output/views", "custom.css" => "output/custom"},
+          :templates => %W|scss preview css custom.css regular.css|,
+          :verbose => false
+        )
+        gen.instance_variable_set :@data, data_file_contents
+        gen.stub :template
+        gen.stub :clear_file
+        gen.stub :append_to_file
+        gen
+      end
+
+      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"
+            args[1].should == fixture("output/custom/custom.css")
+          end
+        end
+        subject.generate
+      end
+
+      it "should output css templates into :css" do
+        subject.should_receive(:template).exactly(5).times do |*args|
+          name = File.basename(args[0])
+          if %w|_fontcustom.scss fontcustom.css regular.css|.include? name
+            args[1].should match(/output\/css\/#{name}/)
+          end
+        end
+        subject.generate
+      end
+
+      it "should output fontcustom-preview.html into :preview" do
+        subject.should_receive(:template).exactly(5).times do |*args|
+          if File.basename(args[0]) == "fontcustom-preview.html"
+            args[1].should == fixture("output/views/fontcustom-preview.html")
+          end
+        end
+        subject.generate
+      end
+    end
   end
 end