juicer 0.2.6 → 1.0.0

data/test/unit/juicer/cache_buster_test.rb

@@ -0,0 +1,104 @@
+# -*- coding: utf-8 -*-
+
+require "test_helper"
+require "juicer/cache_buster"
+
+class CacheBusterTest < Test::Unit::TestCase
+  def setup
+    @filename = "tmp.cachebuster.txt"
+    File.open(@filename, "w") { |f| f.puts "Testing" }
+  end
+
+  def teardown
+    File.delete(@filename)
+  end
+
+  context "cleaning files with cache busters" do
+    should "remove cache buster query parameters" do
+      assert_equal @filename, Juicer::CacheBuster.clean("#@filename?jcb=1234567890")
+    end
+
+    should "remove cache buster query parameters, but preserve other query parameters" do
+      assert_equal "#@filename?some=param&others=again", Juicer::CacheBuster.clean("#@filename?some=param&jcb=1234567890&others=again")
+    end
+
+    should "remove cache buster query parameters with no name" do
+      assert_equal @filename, Juicer::CacheBuster.clean("#@filename?1234567890", nil)
+    end
+
+    should "remove cache buster query parameters with custom name" do
+      assert_equal @filename, Juicer::CacheBuster.clean("#@filename?cb=1234567890", :cb)
+    end
+
+    should "remove hard cache buster" do
+      assert_equal @filename, Juicer::CacheBuster.clean(@filename.sub(/(\.txt)/, '-jcb1234567890\1'))
+    end
+
+    should "remove hard cache buster and preserve query params" do
+      assert_equal "#@filename?hey=there", Juicer::CacheBuster.clean("#@filename?hey=there".sub(/(\.txt)/, '-jcb1234567890\1'))
+    end
+  end
+
+  context "creating soft cache busters" do
+    should "clean file before adding new cache buster" do
+      cache_buster = "jcb=1234567890"
+      assert_no_match /#{cache_buster}/, Juicer::CacheBuster.soft("#@filename?#{cache_buster}")
+    end
+
+    should "preserve query parameters" do
+      parameters = "id=1"
+      assert_match /#{parameters}/, Juicer::CacheBuster.soft("#@filename?#{parameters}")
+    end
+
+    should "raise error if file is not found" do
+      assert_raise ArgumentError do
+        Juicer::CacheBuster.path("#@filename.ico")
+      end
+    end
+
+    should "include mtime as query parameter" do
+      mtime = File.mtime(@filename).to_i
+      assert_equal "#@filename?jcb=#{mtime}", Juicer::CacheBuster.soft(@filename)
+    end
+
+    should "include only mtime when parameter name is nil" do
+      mtime = File.mtime(@filename).to_i
+      assert_equal "#@filename?#{mtime}", Juicer::CacheBuster.soft(@filename, nil)
+    end
+
+    should "include custom parameter name" do
+      mtime = File.mtime(@filename).to_i
+      assert_equal "#@filename?juicer=#{mtime}", Juicer::CacheBuster.soft(@filename, :juicer)
+    end
+  end
+
+  context "hard cache busters" do
+    setup { @filename.sub!(/\.txt/, '') }
+    teardown { @filename = "#@filename.txt" }
+
+    should "clean file before adding new cache buster" do
+      cache_buster = "-jcb1234567890"
+      assert_no_match /#{cache_buster}/, Juicer::CacheBuster.hard("#@filename#{cache_buster}.txt")
+    end
+
+    should "preserve query parameters" do
+      parameters = "id=1"
+      assert_match /#{parameters}/, Juicer::CacheBuster.hard("#@filename.txt?#{parameters}")
+    end
+
+    should "include mtime in filename" do
+      mtime = File.mtime("#@filename.txt").to_i
+      assert_equal "#@filename-jcb#{mtime}.txt", Juicer::CacheBuster.hard("#@filename.txt")
+    end
+
+    should "include only mtime when parameter name is nil" do
+      mtime = File.mtime("#@filename.txt").to_i
+      assert_equal "#@filename-#{mtime}.txt", Juicer::CacheBuster.hard("#@filename.txt", nil)
+    end
+
+    should "include custom parameter name" do
+      mtime = File.mtime("#@filename.txt").to_i
+      assert_equal "#@filename-juicer#{mtime}.txt", Juicer::CacheBuster.hard("#@filename.txt", :juicer)
+    end
+  end
+end

data/test/{juicer/test_chainable.rb → unit/juicer/chainable_test.rb}

@@ -1,4 +1,4 @@
-require File.expand_path(File.join(File.dirname(__FILE__), %w[.. test_helper])) unless defined?(Juicer)
+require "test_helper"
 
 class Host
   include Juicer::Chainable

data/test/unit/juicer/command/install_test.rb

@@ -0,0 +1,58 @@
+require "test_helper"
+
+class TestInstallCommand < Test::Unit::TestCase
+  def setup
+    @io = StringIO.new
+    @command = Juicer::Command::Install.new(Logger.new(@io))
+    FileUtils.rm_rf(path(".juicer/lib")) if File.exists?(path(".juicer/lib"))
+  end
+
+  context "checking version" do
+    should "bring up latest from installer" do
+      assert_equal "1.0", @command.version(Juicer::Install::JSLintInstaller.new)
+    end
+
+    should "use explicit version" do
+      @command.instance_eval { @version = "1.0.1" }
+      assert_equal "1.0.1", @command.version(Juicer::Install::JSLintInstaller.new)
+    end
+  end
+
+  context "executing command" do
+    should "require atleast one argument" do
+      assert_raise ArgumentError do
+        @command.execute
+      end
+    end
+
+    should "install single library" do
+      installer = Juicer::Install::JSLintInstaller.new(path(".juicer"))
+      assert !installer.installed?
+
+      @command.instance_eval { @path = path(".juicer") }
+      @command.execute("jslint")
+      
+      assert installer.installed?
+    end
+
+    should "not install already installed library" do
+      installer = Juicer::Install::JSLintInstaller.new(path(".juicer"))
+      installer.install
+      assert installer.installed?
+
+      @command.execute("jslint")
+      assert_match(/is already installed in/, @io.string)
+    end
+
+    should "install specific version" do
+      installer = Juicer::Install::JSLintInstaller.new(path(".juicer"))
+      assert !installer.installed?("0.9")
+
+      @command.instance_eval { @path = path(".juicer") }
+      @command.instance_eval { @version = "0.9" }
+      @command.execute("jslint")
+      
+      assert installer.installed?("0.9")
+    end
+  end
+end

data/test/{juicer/command/test_list.rb → unit/juicer/command/list_test.rb}

@@ -1,10 +1,10 @@
-require File.expand_path(File.join(File.dirname(__FILE__), %w[.. .. test_helper])) unless defined?(Juicer)
+require "test_helper"
 
 class TestListCommand < Test::Unit::TestCase
 
   def setup
-    @io = StringIO.new
-    @command = Juicer::Command::List.new(@io)
+    @log = Juicer::LOGGER
+    @command = Juicer::Command::List.new(@log)
     Juicer::Test::FileSetup.new.create
   end
 
@@ -19,33 +19,31 @@ class TestListCommand < Test::Unit::TestCase
   end
 
   def test_list_css_file
-    @command.execute "test/data/a.css"
+    result = @command.execute "test/data/a.css"
 
     msg = <<-STDOUT
 Dependency chain for test/data/a.css:
   test/data/b.css
   test/data/a.css
+STDOUT
 
-    STDOUT
-
-    assert_equal msg, @io.string
+    assert_equal msg, result
   end
 
   def test_list_js_file
-    @command.execute "test/data/a.js"
+    result = @command.execute "test/data/a.js"
 
     msg = <<-STDOUT
 Dependency chain for test/data/a.js:
   test/data/b.js
   test/data/a.js
+STDOUT
 
-    STDOUT
-
-    assert_equal msg, @io.string
+    assert_equal msg, result
   end
 
   def test_list_several_files
-    @command.execute ["test/data/a.js", "test/data/b.js"]
+    result = @command.execute ["test/data/a.js", "test/data/b.js"]
 
     msg = <<-STDOUT
 Dependency chain for test/data/a.js:
@@ -55,10 +53,24 @@ Dependency chain for test/data/a.js:
 Dependency chain for test/data/b.js:
   test/data/a.js
   test/data/b.js
+STDOUT
+
+    assert_equal msg, result
+  end
+
+  def test_list_several_files_with_empties
+    result = @command.execute ["test/data/a.css", "test/data/b.css"]
 
-    STDOUT
+    msg = <<-STDOUT
+Dependency chain for test/data/a.css:
+  test/data/b.css
+  test/data/a.css
+
+Dependency chain for test/data/b.css:
+  test/data/b.css
+STDOUT
 
-    assert_equal msg, @io.string
+    assert_equal msg, result
   end
 
   def test_list_files_unable_to_guess_type

data/test/unit/juicer/command/merge_test.rb

@@ -0,0 +1,162 @@
+require "test_helper"
+
+class TestMergeCommand < Test::Unit::TestCase
+  def setup
+    @io = StringIO.new
+    @merge = Juicer::Command::Merge.new(Logger.new(@io))
+
+    Juicer::Test::FileSetup.new.create
+
+    ["a.min.css", "not-ok.min.js"].each { |f| File.delete(path(f)) if File.exists?(path(f)) }
+  end
+
+  context "getting minifyer" do
+    should "return nil when minifyer is not specified" do
+      @merge.instance_eval { @minifyer = nil }
+
+      Juicer::Command::Merge.publicize_methods do
+        assert_nil @merge.minifyer
+      end
+    end
+
+    should "return nil when minifyer is empty string" do
+      @merge.instance_eval { @minifyer = "" }
+
+      Juicer::Command::Merge.publicize_methods do
+        assert_nil @merge.minifyer
+      end
+    end
+
+    should "return nil when minifyer is 'none'" do
+      Juicer::Command::Merge.publicize_methods do
+        @merge.instance_eval { @minifyer = "none" }
+        assert_nil @merge.minifyer
+
+        @merge.instance_eval { @minifyer = "None" }
+        assert_nil @merge.minifyer
+
+        @merge.instance_eval { @minifyer = "NONE" }
+        assert_nil @merge.minifyer
+      end
+    end
+
+    should "get specified minifyer" do
+      Juicer::Command::Merge.publicize_methods do
+        assert @merge.minifyer.class == Juicer::Minifyer::YuiCompressor
+      end
+    end
+  end
+
+  context "output name" do
+    should "have suffix prepended with min when input is a file" do
+      Juicer::Command::Merge.publicize_methods do
+        assert_equal File.expand_path("test.min.js"), @merge.output("test.js")
+      end
+    end
+
+    should "be timestamp when input is not provided" do
+      Juicer::Command::Merge.publicize_methods do
+        assert_match(/\d{10}\.min\.tmp/, @merge.output)
+      end
+    end
+
+    should "be instance variable output" do
+      Juicer::Command::Merge.publicize_methods do
+        @merge.instance_eval { @output = "output.css" }
+        assert_equal File.expand_path("output.css"), @merge.output
+        assert_equal File.expand_path("output.css"), @merge.output("bleh.css")
+      end
+    end
+
+    should "should be generated when output is directory" do
+      Juicer::Command::Merge.publicize_methods do
+        @merge.instance_eval { @output = path("css") }
+        assert_equal File.join(path("css"), "file.min.css"), @merge.output("file.css")
+      end
+    end
+  end
+
+  context "get merger" do
+    should "return object from valid type" do
+      Juicer::Command::Merge.publicize_methods do
+        assert_equal Juicer::Merger::JavaScriptMerger, @merge.merger("bleh.js")
+      end
+    end
+
+    should "default to js for invalid type" do
+      Juicer::Command::Merge.publicize_methods do
+        assert_equal Juicer::Merger::JavaScriptMerger, @merge.merger("bleh.txt")
+        assert_match(/Unknown type 'txt', defaulting to 'js'/, @io.string)
+      end
+    end
+
+    should "use preset type" do
+      Juicer::Command::Merge.publicize_methods do
+        @merge.instance_eval { @type = "css" }
+        assert_equal Juicer::Merger::StylesheetMerger, @merge.merger
+        assert_equal Juicer::Merger::StylesheetMerger, @merge.merger("bleh.txt")
+      end
+    end
+  end
+
+  context "merging" do
+    should "raise error without input" do
+      assert_raise SystemExit do
+        @merge.execute([])
+      end
+    end
+
+    should "raise error with bogus input" do
+      assert_raise SystemExit do
+        @merge.execute(["*.css", "bleh/*.js"])
+      end
+    end
+
+    should "fail if output exists" do
+      assert_raise SystemExit do
+        @merge.instance_eval { @output = path("a.css") }
+        @merge.execute(path("a.css"))
+        assert_match(/Run again with --force to overwrite/, @io.string)
+      end
+    end
+
+    should "update output when force option is set" do
+      assert_nothing_raised do
+        @merge.instance_eval { @force = true }
+        @merge.execute(path("a.css"))
+      end
+    end
+
+    should "merge files" do
+      begin
+        @merge.instance_eval { @output = path("a.min.css") }
+        assert @merge.execute(path("a1.css"))
+        assert_match "h2{font-size:10px;}html{background:red;}h1{font-size:12px;}body{width:800px;}", IO.read(path("a.min.css"))
+      rescue Test::Unit::AssertionFailedError => err
+        raise err
+      rescue Exception => err
+        puts err.message
+      end
+    end
+
+    should "raise error when jslint does not pass" do
+      assert_raise SystemExit do
+        @merge.execute(path("not-ok.js"))
+        assert_match(/Problems were detected during verification/, @io.string)
+        assert_no_match(/Ignoring detected problems/, @io.string)
+      end
+    end
+
+    should "ignore jslint problems" do
+      @merge.instance_eval { @ignore = true }
+      
+      assert_nothing_raised do
+        @merge.execute(path("not-ok.js"))
+        assert_match(/Problems were detected during verification/, @io.string)
+        assert_match(/Ignoring detected problems/, @io.string)
+      end
+    end
+  
+    should_eventually "warn about duplicated image urls for embedding"
+  end
+end

data/test/{juicer/command/test_util.rb → unit/juicer/command/util_test.rb}

@@ -1,4 +1,4 @@
-require File.expand_path(File.join(File.dirname(__FILE__), %w[.. .. test_helper])) unless defined?(Juicer)
+require "test_helper"
 
 class Dummy
   include Juicer::Command::Util
@@ -6,6 +6,10 @@ end
 
 class TestCommandUtil < Test::Unit::TestCase
 
+  CSS_FILES = %w{a.css a1.css b.css b1.css c1.css d1.css path_test.css path_test2.css}
+  JS_FILES = %w{a.js}
+  ALL_FILES = (CSS_FILES + JS_FILES).sort
+
   def setup
     @impl = Dummy.new
     Juicer::Test::FileSetup.new.create
@@ -21,19 +25,19 @@ class TestCommandUtil < Test::Unit::TestCase
   def test_files_from_single_glob_pattern
     files = @impl.files("test/data/*.css")
     assert files.is_a?(Array)
-    assert_equal %w{a.css a1.css b.css b1.css c1.css d1.css path_test.css path_test2.css}.collect { |f| "test/data/#{f}" }.join, files.sort.join
+    assert_equal CSS_FILES.collect { |f| "test/data/#{f}" }.join, files.sort.join
   end
 
   def test_files_from_mixed_arguments
     files = @impl.files("test/data/*.css", "test/data/a.js")
     assert files.is_a?(Array)
-    assert_equal %w{a.css a.js a1.css b.css b1.css c1.css d1.css path_test.css path_test2.css}.collect { |f| "test/data/#{f}" }.join, files.sort.join
+    assert_equal ALL_FILES.collect { |f| "test/data/#{f}" }.join, files.sort.join
   end
 
   def test_files_from_array
     files = @impl.files(["test/data/*.css", "test/data/a.js"])
     assert files.is_a?(Array)
-    assert_equal %w{a.css a.js a1.css b.css b1.css c1.css d1.css path_test.css path_test2.css}.collect { |f| "test/data/#{f}" }.join, files.sort.join
+    assert_equal ALL_FILES.collect { |f| "test/data/#{f}" }.join, files.sort.join
   end
 
   def test_relative_path_single_file
@@ -43,12 +47,12 @@ class TestCommandUtil < Test::Unit::TestCase
   def test_relative_path_many_files
     files = @impl.relative(Dir.glob("test/data/*.css"))
     assert files.is_a?(Array)
-    assert_equal %w{a.css a1.css b.css b1.css c1.css d1.css path_test.css path_test2.css}.collect { |f| "test/data/#{f}" }.join, files.sort.join
+    assert_equal CSS_FILES.collect { |f| "test/data/#{f}" }.join, files.sort.join
   end
 
   def test_relative_path_many_files_explicit_reference
     files = @impl.relative(Dir.glob("test/data/*.css"), "lib")
     assert files.is_a?(Array)
-    assert_equal %w{a.css a1.css b.css b1.css c1.css d1.css path_test.css path_test2.css}.collect { |f| "../test/data/#{f}" }.join, files.sort.join
+    assert_equal CSS_FILES.collect { |f| "../test/data/#{f}" }.join, files.sort.join
   end
 end