juicer 0.2.6 → 1.0.0

data/test/unit/juicer/install/installer_base_test.rb

@@ -0,0 +1,214 @@
+require "test_helper"
+
+module Juicer
+  module Install
+    class SomeMagicInstaller < Juicer::Install::Base
+      def install(version = nil)
+        version ||= "1.0.0"
+        ver = super(version)
+        File.open(File.join(@install_dir, path, "bin", ver + ".app"), "w") do |file|
+          file.puts version
+        end
+
+        version
+      end
+
+      def uninstall(version = "1.0.0")
+        super(version) do |path, version|
+          File.delete(File.join(path, "bin", version + ".app"))
+        end
+      end
+    end
+
+    class SomeOtherInstaller < Juicer::Install::SomeMagicInstaller
+      def install(version = nil)
+        version ||= "1.0.0"
+        super(version)
+      end
+
+      def latest
+        "1.0.0"
+      end
+    end
+  end
+end
+
+class TestInstallerBase < Test::Unit::TestCase
+  def initialize(*args)
+    super
+    @juicer_home = File.expand_path(File.join("test", "data", ".juicer"))
+  end
+
+  def setup
+    FileUtils.rm_rf @juicer_home
+    @installer = Juicer::Install::SomeMagicInstaller.new(@juicer_home)
+  end
+
+  context "installation directory" do
+    should "should be Juicer.home by default" do
+      installer = Juicer::Install::SomeMagicInstaller.new
+      assert_equal Juicer.home, installer.install_dir
+    end
+
+    should "override installation directory" do
+      installer = Juicer::Install::SomeMagicInstaller.new("/home/baluba")
+      assert_equal "/home/baluba", installer.install_dir
+    end
+  end
+
+  context "latest" do
+    should "raise NotImplementedError" do
+      assert_raise NotImplementedError do
+        @installer.latest
+      end
+    end
+  end
+
+  context "paths and name" do
+    should "reflect installer class name" do
+      assert_equal "lib/some_magic", @installer.path
+    end
+
+    should "reflect installer class name for bin path" do
+      assert_equal "lib/some_magic/bin", @installer.bin_path
+    end
+
+    should "reflect class name in human name" do
+      assert_equal "Some Magic", @installer.name
+    end
+  end
+
+  context "already installed module" do
+    should "report as installed" do
+      assert !@installer.installed?("x.y.z")
+      @installer.install("x.y.z")
+      assert @installer.installed?("x.y.z")
+    end
+
+    should "fail re-installation" do
+      @installer.install("1.0.0")
+
+      assert_raise RuntimeError do
+        @installer.install("1.0.0")
+      end
+    end
+  end
+
+  context "installation" do
+    should "create bin and release folders" do
+      assert_equal "1.0.0", @installer.install("1.0.0")
+      assert File.exists?(File.join(@juicer_home, "lib/some_magic/bin"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_magic/1.0.0"))
+    end
+  end
+
+  context "uninstall" do
+    should "remove library path when only version is uninstalled" do
+      @installer.install("1.0.0")
+      @installer.uninstall("1.0.0")
+      assert !File.exists?(File.join(@juicer_home, "lib/some_magic"))
+    end
+
+    should "keep other versions" do
+      @installer.install("1.0.0")
+      @installer.install("1.0.1")
+      @installer.uninstall("1.0.0")
+      assert !File.exists?(File.join(@juicer_home, "lib/some_magic/1.0.0"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_magic"))
+    end
+  end
+
+  context "download" do
+    # TODO: Don't download
+    should "cache files" do
+      File.expects(:delete).at_most(0)
+      @installer.download("http://www.julienlecomte.net/yuicompressor/")
+      filename = File.join(@juicer_home, "download/some_magic/yuicompressor")
+      assert File.exists?(filename)
+      @installer.download("http://www.julienlecomte.net/yuicompressor/")
+    end
+
+    should "redownload cached files when forced" do
+      File.expects(:delete).at_most(1)
+      @installer.download("http://www.julienlecomte.net/yuicompressor/")
+      filename = File.join(@juicer_home, "download/some_magic/yuicompressor")
+      assert File.exists?(filename)
+      File.expects(:open).at_most(1)
+      @installer.download("http://www.julienlecomte.net/yuicompressor/", true)
+    end
+  end
+
+  context "installer dependencies" do
+    should "not return true from installer when missing dependencies" do
+      @installer.install
+
+      installer = Juicer::Install::SomeMagicInstaller.new(@juicer_home)
+      assert !installer.installed?("1.0.1"), "Installer should be installed"
+
+      installer.dependency Juicer::Install::SomeOtherInstaller
+      assert !installer.installed?("1.0.1"), "Installer should not report as being installed when missing dependencies"
+    end
+
+    should "install single dependency" do
+      installer = Juicer::Install::SomeMagicInstaller.new(@juicer_home)
+      installer.dependency Juicer::Install::SomeOtherInstaller
+      installer.install "1.0.0"
+      assert File.exists?(File.join(@juicer_home, "lib/some_magic/1.0.0"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_other/1.0.0"))
+    end
+
+    should "not raise error when they exist" do
+      installer = Juicer::Install::SomeMagicInstaller.new(@juicer_home)
+      installer.dependency Juicer::Install::SomeOtherInstaller
+
+      dep = Juicer::Install::SomeOtherInstaller.new
+      dep.install unless dep.installed?
+
+      assert_nothing_raised do
+        installer.install "1.0.0"
+      end
+    end
+
+    should "get dependency from single symbol" do
+      installer = Juicer::Install::SomeMagicInstaller.new(@juicer_home)
+      installer.dependency :some_other
+      installer.install "1.0.0"
+      assert File.exists?(File.join(@juicer_home, "lib/some_magic/1.0.0"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_other/1.0.0"))
+    end
+
+    should "get dependency from single string" do
+      installer = Juicer::Install::SomeMagicInstaller.new(@juicer_home)
+      installer.dependency "some_other"
+      installer.install "1.0.0"
+      assert File.exists?(File.join(@juicer_home, "lib/some_magic/1.0.0"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_other/1.0.0"))
+    end
+
+    should "support multiple dependencies" do
+      installer = Juicer::Install::SomeMagicInstaller.new(@juicer_home)
+      installer.dependency :some_other
+      installer.dependency :some_other, "2.0.4"
+      installer.dependency :some_other, "3.0.5"
+      installer.install "1.0.0"
+      assert File.exists?(File.join(@juicer_home, "lib/some_magic/1.0.0"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_other/1.0.0"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_other/2.0.4"))
+      assert File.exists?(File.join(@juicer_home, "lib/some_other/3.0.5"))
+    end
+  end
+
+  context "resolving class" do
+    should "accept class input" do
+      assert_equal Juicer::Install::SomeMagicInstaller, Juicer::Install.get(Juicer::Install::SomeMagicInstaller)
+    end
+
+    should "accept symbol input" do
+      assert_equal Juicer::Install::SomeMagicInstaller, Juicer::Install.get(:some_magic)
+    end
+
+    should "accept string input" do
+      assert_equal Juicer::Install::SomeMagicInstaller, Juicer::Install.get("some_magic")
+    end
+  end
+end

data/test/{juicer/install/test_jslint_installer.rb → unit/juicer/install/jslint_installer_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 TestJsLintInstaller < Test::Unit::TestCase
   def setup

data/test/{juicer/install/test_rhino_installer.rb → unit/juicer/install/rhino_installer_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 TestRhinoInstaller < Test::Unit::TestCase
   def setup

data/test/{juicer/install/test_yui_compressor_installer.rb → unit/juicer/install/yui_compressor_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 TestYuiCompressorInstall < Test::Unit::TestCase
   def setup
@@ -35,22 +35,22 @@ class TestYuiCompressorInstall < Test::Unit::TestCase
     assert !File.exists?(path(".juicer/lib/yui_compressor"))
   end
 
-  def test_install_specific_version
-    @installer.install("2.3.5")
+  # def test_install_specific_version
+  #   @installer.install("2.3.5")
 
-    assert File.exists?(path(".juicer/lib/yui_compressor/bin"))
-    assert File.exists?(path(".juicer/lib/yui_compressor/2.3.5"))
-    assert_equal "yuicompressor-2.3.5.jar", File.basename(Dir.glob(path(".juicer/lib/yui_compressor/bin/*"))[0])
-  end
+  #   assert File.exists?(path(".juicer/lib/yui_compressor/bin"))
+  #   assert File.exists?(path(".juicer/lib/yui_compressor/2.3.5"))
+  #   assert_equal "yuicompressor-2.3.5.jar", File.basename(Dir.glob(path(".juicer/lib/yui_compressor/bin/*"))[0])
+  # end
 
-  def test_uninstall_should_leave_directories_when_other_versions_are_installed
-    @installer.install
-    @installer.install("2.3.5")
-    assert File.exists?(path(".juicer/lib/yui_compressor/bin"))
-    assert_equal 6, Dir.glob(path(".juicer/lib/yui_compressor/**/*")).find_all { |f| File.file?(f) }.length
+  # def test_uninstall_should_leave_directories_when_other_versions_are_installed
+  #   @installer.install
+  #   @installer.install("2.3.5")
+  #   assert File.exists?(path(".juicer/lib/yui_compressor/bin"))
+  #   assert_equal 6, Dir.glob(path(".juicer/lib/yui_compressor/**/*")).find_all { |f| File.file?(f) }.length
 
-    @installer.uninstall("2.3.5")
-    assert File.exists?(path(".juicer/lib/yui_compressor"))
-    assert_equal 3, Dir.glob(path(".juicer/lib/yui_compressor/**/*")).find_all { |f| File.file?(f) }.length
-  end
+  #   @installer.uninstall("2.3.5")
+  #   assert File.exists?(path(".juicer/lib/yui_compressor"))
+  #   assert_equal 3, Dir.glob(path(".juicer/lib/yui_compressor/**/*")).find_all { |f| File.file?(f) }.length
+  # end
 end

data/test/unit/juicer/jslint_test.rb

@@ -0,0 +1,60 @@
+require "test_helper"
+
+class TestJsLint < Test::Unit::TestCase
+  def initialize(*args)
+    @path = File.expand_path(path("../bin"))
+    @file = path("jsltest.js")
+    super
+  end
+  
+  def setup
+    FileUtils.mkdir(path("")) unless File.exists?(path(""))
+    File.open(@file, "w") { |f| f.puts "" }
+  end
+
+  def teardown
+    File.delete(@file)
+  end
+
+  context "verifying file with jslint" do
+    should "shell out to rhino/jslint" do
+      jslint = Juicer::JsLint.new(:bin_path => @path)
+      jslint.expects(:execute).with("-jar \"#{@path}/rhino1_7R2-RC1.jar\" \"#{@path}/jslint-1.0.js\" \"#{@file}\"").returns("jslint: No problems")
+      
+      assert jslint.check(@file).ok?
+    end
+  end
+
+  context "jslint report returns" do
+    should "be a report object for valid files" do
+      jslint = Juicer::JsLint.new(:bin_path => @path)
+      jslint.expects(:execute).returns("jslint: No problems")
+
+      assert_equal Juicer::JsLint::Report, jslint.check(@file).class
+    end
+
+    should "be a report object for invalid files" do
+      jslint = Juicer::JsLint.new(:bin_path => @path)
+      jslint.expects(:execute).returns("Wrong use of semicolon\nWrong blabla")
+      
+      assert_equal Juicer::JsLint::Report, jslint.check(path("not-ok.js")).class
+    end
+  end
+
+  context "errors" do
+    should "be available on report" do
+      jslint = Juicer::JsLint.new(:bin_path => @path)
+      jslint.expects(:execute).returns("Wrong use of semicolon\nWrong blabla\nWrong use of semicolon\nWrong blabla")
+
+      assert_equal 2, jslint.check(@file).errors.length
+    end
+
+    should "be error objects" do
+      jslint = Juicer::JsLint.new(:bin_path => @path)
+      jslint.expects(:execute).returns("Wrong use of semicolon\nWrong blabla")
+
+      error = jslint.check(@file).errors.first
+      assert_equal Juicer::JsLint::Error, error.class
+    end
+  end
+end

data/test/{juicer/merger/test_base.rb → unit/juicer/merger/base_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 TestMergerBase < Test::Unit::TestCase
 

data/test/{juicer/merger/test_javascript_merger.rb → unit/juicer/merger/javascript_merger_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 TestJavaScriptMerger < Test::Unit::TestCase
 
@@ -14,7 +14,7 @@ class TestJavaScriptMerger < Test::Unit::TestCase
 
   def test_init
     Juicer::Merger::JavaScriptMerger.publicize_methods do
-      assert_equal Juicer::Merger::JavaScriptDependencyResolver, @file_merger.dependency_resolver.class
+      assert_equal Juicer::JavaScriptDependencyResolver, @file_merger.dependency_resolver.class
     end
   end
 

data/test/{juicer/merger/test_stylesheet_merger.rb → unit/juicer/merger/stylesheet_merger_test.rb}

@@ -1,23 +1,25 @@
-require File.expand_path(File.join(File.dirname(__FILE__), %w[.. .. test_helper])) unless defined?(Juicer)
+require "test_helper"
 
 class TestStylesheetMerger < Test::Unit::TestCase
 
   def setup
     @file_merger = Juicer::Merger::StylesheetMerger.new
     Juicer::Test::FileSetup.new.create
+    @cwd = Dir.pwd
     Dir.chdir path("")
   end
 
   def teardown
     file = path('test_out.css')
     File.delete(file) if File.exists?(file)
+    Dir.chdir(@cwd)
   end
 
-  def test_init
-    Juicer::Merger::StylesheetMerger.publicize_methods do
-      assert_equal Juicer::Merger::CssDependencyResolver, @file_merger.dependency_resolver.class
-    end
-  end
+ def test_init
+   Juicer::Merger::StylesheetMerger.publicize_methods do
+     assert_equal Juicer::CssDependencyResolver, @file_merger.dependency_resolver.class
+   end
+ end
 
   def test_merge
     Juicer::Merger::StylesheetMerger.publicize_methods do
@@ -78,7 +80,7 @@ EOF
     end
   end
 
-  def test_resolve_path_error_when_relative_missing_web_root
+  def test_resolve_path_error_when_relative_missing_document_root
     merger = Juicer::Merger::StylesheetMerger.new [], :relative_urls => true
 
     Juicer::Merger::StylesheetMerger.publicize_methods do
@@ -89,7 +91,7 @@ EOF
   end
 
   def test_resolve_path_should_make_absolute_urls_relative
-    merger = Juicer::Merger::StylesheetMerger.new [], :relative_urls => true, :web_root => "/home/usr"
+    merger = Juicer::Merger::StylesheetMerger.new [], :relative_urls => true, :document_root => "/home/usr"
 
     Juicer::Merger::StylesheetMerger.publicize_methods do
       merger.instance_eval { @root = Pathname.new "/home/usr/design" }
@@ -107,7 +109,7 @@ EOF
     end
   end
 
-  def test_resolve_path_error_when_missing_absolute_web_root
+  def test_resolve_path_error_when_missing_absolute_document_root
     merger = Juicer::Merger::StylesheetMerger.new [], :absolute_urls => true
 
     Juicer::Merger::StylesheetMerger.publicize_methods do
@@ -118,7 +120,7 @@ EOF
   end
 
   def test_resolve_path_should_make_relative_urls_absolute
-    merger = Juicer::Merger::StylesheetMerger.new [], :absolute_urls => true, :web_root => "/home/usr"
+    merger = Juicer::Merger::StylesheetMerger.new [], :absolute_urls => true, :document_root => "/home/usr"
 
     Juicer::Merger::StylesheetMerger.publicize_methods do
       merger.instance_eval { @root = Pathname.new "/home/usr/design" }
@@ -136,7 +138,7 @@ EOF
   end
 
   def test_resolve_path_should_redefine_absolute_urls
-    merger = Juicer::Merger::StylesheetMerger.new [], :relative_urls => true, :web_root => "/home/usr"
+    merger = Juicer::Merger::StylesheetMerger.new [], :relative_urls => true, :document_root => "/home/usr"
 
     Juicer::Merger::StylesheetMerger.publicize_methods do
       merger.instance_eval { @root = Pathname.new "/home/usr/design2/css" }
@@ -156,8 +158,8 @@ EOF
     end
   end
 
-  def test_resolve_paths_should_handle_relative_web_roots
-    merger = Juicer::Merger::StylesheetMerger.new [], :web_root => "test/data", :relative_urls => true
+  def test_resolve_paths_should_handle_relative_document_roots
+    merger = Juicer::Merger::StylesheetMerger.new [], :document_root => "test/data", :relative_urls => true
     merger << File.expand_path("css/test2.css")
 
     Juicer::Merger::StylesheetMerger.publicize_methods do

data/test/unit/juicer/minifyer/closure_compressor_test.rb

@@ -0,0 +1,107 @@
+require "test_helper"
+require "juicer/minifyer/closure_compiler"
+
+class ClosureCompilerTest < Test::Unit::TestCase
+  def setup
+    @jar = "compiler.jar"
+    @input = "in-file.css"
+    @output = "out-file.css"
+    @cmd = %Q{-jar "#@jar"}
+    @closure = Juicer::Minifyer::ClosureCompiler.new
+    @closure.stubs(:locate_jar).returns(@jar)
+  end
+
+  context "#save" do
+    should "overwrite existing file" do
+      @closure.expects(:execute).with(%Q{#@cmd -js_output_file "#@output" -js "#@output"})
+      @closure.save(@output, @output)
+    end
+
+    should "write compressed input to output" do
+      @closure.expects(:execute).with(%Q{#@cmd -js_output_file "#@output" -js "#@input"})
+      @closure.save(@input, @output)
+    end
+
+    should "create non-existant path" do
+      output = "some/nested/directory"
+      @closure.expects(:execute).with(%Q{#@cmd -js_output_file "#{output}/file.css" -js "#@input"})
+      FileUtils.expects(:mkdir_p).with(output)
+      @closure.save(@input, "#{output}/file.css")
+    end
+  end
+
+  context "locating jar" do
+    setup do
+      # Avoid developer env settings
+      @closure_home = ENV['CLOSUREC_HOME']
+      ENV.delete('CLOSUREC_HOME')
+    end
+
+    teardown do
+      ENV['CLOSUREC_HOME'] = @closure_home
+      File.delete('compiler-2.3.4.jar') if File.exists?('compiler-2.3.4.jar')
+      File.delete('compiler-2.3.5.jar') if File.exists?('compiler-2.3.5.jar')
+      File.delete('compiler.jar') if File.exists?('compiler.jar')
+      FileUtils.rm_rf("another") if File.exists?("another")
+    end
+    
+    should "not find jar when no jars on path" do
+      Juicer::Minifyer::ClosureCompiler.publicize_methods do
+        closure = Juicer::Minifyer::ClosureCompiler.new
+
+        assert_nil closure.locate_jar
+      end
+    end
+
+    should "find only jar in path" do
+      Juicer::Minifyer::ClosureCompiler.publicize_methods do
+        File.open('compiler-2.3.4.jar', 'w') { |f| f.puts '' }
+        closure = Juicer::Minifyer::ClosureCompiler.new
+
+        assert_equal File.expand_path('compiler-2.3.4.jar'), closure.locate_jar
+      end
+    end
+
+    should "find most recent of two jars on path" do
+      Juicer::Minifyer::ClosureCompiler.publicize_methods do
+        # Create files
+        File.open('compiler-2.3.4.jar', 'w') { |f| f.puts '' }
+        File.open('compiler-2.3.5.jar', 'w') { |f| f.puts '' }
+
+        closure = Juicer::Minifyer::ClosureCompiler.new
+        
+        # Test
+        assert_equal File.expand_path('compiler-2.3.5.jar'), closure.locate_jar
+      end
+    end
+
+    should "find most recent of three jar files on path" do
+      Juicer::Minifyer::ClosureCompiler.publicize_methods do
+        # Create files
+        File.open('compiler-2.3.4.jar', 'w') { |f| f.puts '' }
+        File.open('compiler-2.3.5.jar', 'w') { |f| f.puts '' }
+        File.open('compiler.jar', 'w') { |f| f.puts '' }
+
+        closure = Juicer::Minifyer::ClosureCompiler.new
+        
+        # Test
+        assert_equal File.expand_path('compiler.jar'), closure.locate_jar
+      end
+    end
+
+    should "find jar in custom directory" do
+      Juicer::Minifyer::ClosureCompiler.publicize_methods do
+        # Prepare
+        Dir.mkdir('another')
+        File.open('another/compiler-2.3.4.jar', 'w') { |f| f.puts "" }
+
+        closure = Juicer::Minifyer::ClosureCompiler.new
+        
+        # Test
+        assert_nil closure.locate_jar
+        closure = Juicer::Minifyer::ClosureCompiler.new({ :bin_path => 'another' })
+        assert_equal File.expand_path('another/compiler-2.3.4.jar'), closure.locate_jar
+      end
+    end
+  end
+end