assets_booster 0.0.1 → 0.0.2

data/lib/assets_booster/tasks/tasks.rake

@@ -1,29 +1,27 @@
-require "assets_booster/configuration"
 require "assets_booster/packager"
-
 namespace :assets_booster do
   desc 'Merge assets'
   task :merge do
-    AssetsBooster::Packager.init
-    AssetsBooster::Packager.merge_all
+    packager = AssetsBooster::Packager.new
+    packager.merge_all
   end
 
   desc 'Merge and compile assets'
   task :compile do
-    AssetsBooster::Packager.init
-    AssetsBooster::Packager.compile_all
+    packager = AssetsBooster::Packager.new
+    packager.compile_all
   end
 
   desc 'Delete all asset builds'
   task :delete do
-    AssetsBooster::Packager.init
-    AssetsBooster::Packager.delete_all
+    packager = AssetsBooster::Packager.new
+    packager.delete_all
   end
 
-  desc 'Generate assets_booster.yml from existing assets'
+  desc 'Generate a configuration file for existing assets'
   task :setup do
-    AssetsBooster::Packager.init
-    AssetsBooster::Configuration.create
+    packager = AssetsBooster::Packager.new
+    packager.create_configuration
   end
 end
 

data/lib/assets_booster/version.rb

@@ -1,3 +1,3 @@
 module AssetsBooster
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/assets_booster/view_helper.rb

@@ -1,14 +1,11 @@
 module AssetsBooster
   module ViewHelper
-    def should_merge?
-      AssetPackage.merge_environments.include?(Rails.env)
-    end
-
     def assets_booster_tag(type, *names)
       options = names.extract_options!
-      packages = AssetsBooster::Packager.packages[type]
+      packager = AssetsBooster::Railtie.packager
+      packages = packager.packages[type]
       html = names.map do |name|
-        methode, sources = packages[name].view_helper
+        methode, sources = packages[name].view_helper(packager)
         send(methode, sources, options)
       end*"\n"
       html.html_safe

data/spec/compiler/base.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+module AssetsBooster
+  module Compiler
+    shared_examples_for "a compiler" do
+      subject{ described_class.new }
+
+      describe "name" do
+        it "should return a string" do
+          subject.name.should_not be_empty
+        end
+      end
+    end
+  end
+end
+

data/spec/compiler/closure_spec.rb

@@ -0,0 +1,16 @@
+require 'compiler/base'
+require 'assets_booster/compiler/closure'
+module AssetsBooster
+  module Compiler
+    describe Closure do
+      it_behaves_like "a compiler" do
+        describe "compile" do
+          it "should compile the source file" do
+            subject.compile("var a = 'test'; alert(a);").should == 'var a="test";alert(a);'
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/compiler/dummy_spec.rb

@@ -0,0 +1,16 @@
+require 'compiler/base'
+require 'assets_booster/compiler/dummy'
+module AssetsBooster
+  module Compiler
+    describe Dummy do
+      it_behaves_like "a compiler" do
+        describe "compile" do
+          it "should compile the source file" do
+            subject.compile("var a = 'test'; alert(a);").should == "var a = 'test'; alert(a);"
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/compiler/jsmin_spec.rb

@@ -0,0 +1,16 @@
+require 'compiler/base'
+require 'assets_booster/compiler/jsmin'
+module AssetsBooster
+  module Compiler
+    describe JSMin do
+      it_behaves_like "a compiler" do
+        describe "compile" do
+          it "should compile the source file" do
+            subject.compile("var a = 'test'; alert(a);").should == "var a='test';alert(a);"
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/compiler/rainpress_spec.rb

@@ -0,0 +1,16 @@
+require 'compiler/base'
+require 'assets_booster/compiler/rainpress'
+module AssetsBooster
+  module Compiler
+    describe Rainpress do
+      it_behaves_like "a compiler" do
+        describe "compile" do
+          it "should compile the source file" do
+            subject.compile("{ color: #ff0000; }").should == "{color:red}"
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/compiler/uglify_spec.rb

@@ -0,0 +1,16 @@
+require 'compiler/base'
+require 'assets_booster/compiler/uglify'
+module AssetsBooster
+  module Compiler
+    describe Uglify do
+      it_behaves_like "a compiler" do
+        describe "compile" do
+          it "should compile the source file" do
+            subject.compile("var a = 'test'; alert(a);").should == 'var a="test";alert(a)'
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/merger/base.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+require 'assets_booster/merger/base'
+module AssetsBooster
+  module Merger
+    shared_examples_for "a merger" do
+      subject{ described_class.new([]) }
+
+      describe "name" do
+        it "should return a string" do
+          subject.name.should_not be_empty
+        end
+      end
+
+      describe "mtime" do
+        subject{ described_class.new(['a', 'b', 'c']) }
+        it "should return the last modification time of all sources" do
+          File.stub(:read){ "" }
+          File.stub(:mtime) do |asset|
+            case asset
+            when 'a' then 1
+            when 'b' then 3
+            when 'c' then 2
+            end
+          end
+          subject.mtime.should == 3
+        end
+      end
+    end
+  end
+end
+

data/spec/merger/css_spec.rb

@@ -0,0 +1,138 @@
+require 'merger/base'
+require 'assets_booster/merger/css'
+module AssetsBooster
+  module Merger
+    describe CSS do
+      it_behaves_like "a merger" do
+        describe "merge" do
+          subject{ described_class.new(['a.css', 'nested/b.css', 'c.css']) }
+
+          it "should merge sources and rewrite urls" do
+            File.stub(:read) do |source| 
+              case source
+              when 'a.css'
+                "{background:url(gucki.png)}"
+              when 'nested/b.css'
+                "{background:url(gucki.png)}"
+              when 'c.css'
+                "{background:url(http://www.example.com/gucki.png)}"
+              end
+            end
+            subject.merge("target.css").should == "{background:url(gucki.png)}\n{background:url(nested/gucki.png)}\n{background:url(http://www.example.com/gucki.png)}"
+          end
+
+          it "should merge sources and respect import directives" do
+            File.stub(:read) do |source| 
+              case source
+              when 'a.css'
+                "@import d.css"
+              when 'nested/b.css'
+                "@import url(http://www.example.com/test.css)"
+              when 'c.css'
+                "{color:#fff;}"
+              when 'd.css'
+                "{color:#f00;}"
+              end
+            end
+            subject.merge("target.css").should == "{color:#f00;}\n\n@import url(http://www.example.com/test.css)\n{color:#fff;}"
+          end
+        end
+
+        describe "absolute_url?" do
+          it "should detect absolute urls" do
+            [
+              ["http://www.example.com", true],
+              ["HTTP://www.example.com", true],
+              ["https://www.example.com", true],
+              ["/absolute.css", true],
+              ["relative.css", false],
+              ["another/relative.css", false],
+            ].each do |url, result| 
+              subject.absolute_url?(url).should == result
+            end
+          end
+        end
+        
+        describe "path_difference" do
+          it "should return the difference" do
+            [
+              ["", "", ""],
+              ["test", "", "test"],
+              ["test", "test", ""],
+              ["home/test", "", "home/test"],
+              ["home/test", "home", "test"],
+              ["/home/test", "/home", "test"],
+            ].each do |source, target, result| 
+              subject.path_difference(source, target).should == result
+            end
+          end
+          it "should raise if source and target dont share a common base path" do
+            lambda{ subject.path_difference("/home/gucki/test", "/home/peter") }.should raise_error(ArgumentError)
+          end
+        end
+  
+        describe "rewrite_urls" do
+          it "should rewrite url properties" do
+            subject.rewrite_urls("{color:#f00}", "/home/test", "/home/test").should == "{color:#f00}"
+            subject.rewrite_urls("{color:#f00; background:url(test.png)}", "nested", "").should == "{color:#f00; background:url(nested/test.png)}"
+            source_folder = "/home/test/nested"
+            target_folder = "/home/test"
+            [
+              [
+                "{color:#f00}",
+                "{color:#f00}",
+              ],
+              [
+                "{color:#f00; background:url(test.png)}",
+                "{color:#f00; background:url(nested/test.png)}",
+              ],
+              [
+                "{color:#f00; background:url(\"test.png\")}",
+                "{color:#f00; background:url(\"nested/test.png\")}", 
+              ],
+              [
+                "{color:#f00; background:url('test.png')}",
+                "{color:#f00; background:url('nested/test.png')}", 
+              ],
+              [
+                "{color:#f00; background:url('test file.png')}",
+                "{color:#f00; background:url('nested/test file.png')}", 
+              ],
+              [
+                "{color:#f00; background:url(../test.png)}",
+                "{color:#f00; background:url(nested/../test.png)}", 
+              ],
+            ].each do |input, output| 
+              subject.rewrite_urls(input, source_folder, target_folder).should == output
+            end
+          end
+        end
+        
+        describe "extract_url" do
+          it "should extract urls" do
+            [
+              ["'test.png'", ["test.png", "'"]],
+              ['"test.png"', ["test.png", '"']],
+              ["test.png", ["test.png", ""]], 
+            ].each do |input, output| 
+              subject.extract_url(input).should == output
+            end
+          end
+        end
+        
+        describe "dirname" do
+          it "should return the folder of the given path" do
+            [
+              ["test.png", ""], 
+              ["test/test.png", "test"], 
+              ["/test/nested/test.png", "/test/nested"], 
+            ].each do |input, output| 
+              subject.dirname(input).should == output
+            end
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/merger/simple_spec.rb

@@ -0,0 +1,18 @@
+require 'merger/base'
+require 'assets_booster/merger/simple'
+module AssetsBooster
+  module Merger
+    describe Simple do
+      it_behaves_like "a merger" do
+        describe "merge" do
+          subject{ described_class.new(['a', 'b', 'c']) }
+          it "should merge source files" do
+            File.stub(:read){ |source| source }
+            subject.merge("target.css").should == "a\nb\nc"
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/package/base.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require 'assets_booster/package/base'
+module AssetsBooster
+  module Package
+    shared_examples_for "a package" do
+      subject{ described_class.new("test", []) }
+      
+      before do
+        Rails.stub(:root){ "/rails" }
+      end
+      
+      describe "name" do
+        it "should return a string" do
+          subject.name.should_not be_empty
+        end
+      end
+      
+      describe "view_helper_method" do
+        it "should return a symbol" do
+          subject.view_helper_method.should be_an_instance_of(Symbol)
+        end
+      end
+    end
+  end
+end
+

data/spec/package/javascript_spec.rb

@@ -0,0 +1,16 @@
+require 'package/base'
+require 'assets_booster/package/javascript'
+module AssetsBooster
+  module Package
+    describe Javascript do
+      it_behaves_like "a package" do
+        describe "asset_path" do
+          it "should return a javascript filename" do
+            subject.asset_path("lala").should match(/javascripts\/lala\.js/) 
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/package/stylesheet_spec.rb

@@ -0,0 +1,16 @@
+require 'package/base'
+require 'assets_booster/package/stylesheet'
+module AssetsBooster
+  module Package
+    describe Stylesheet do
+      it_behaves_like "a package" do
+        describe "asset_path" do
+          it "should return a stylesheet filename" do
+            subject.asset_path("lala").should match(/stylesheets\/lala\.css/) 
+          end
+        end
+      end
+    end
+  end
+end
+

data/spec/packager_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+require 'assets_booster/packager'
+module AssetsBooster
+  describe Packager do
+    before do
+      Rails.stub(:root){ "/rails" }
+      @messages = []
+      AssetsBooster.stub(:log){ |message| @messages << message }
+    end
+
+    describe "create_configuration" do
+      it "should abort if a configuration file already exists" do
+        File.should_receive(:exists?).with(subject.configuration_filename).and_return(true)
+        File.should_not_receive(:open).with(subject.configuration_filename, "w")
+        subject.create_configuration
+        @messages[0].should match(/already exists/)
+      end
+
+      it "should create a configration file from current assets" do
+        Dir.should_receive(:[]).with("/rails/public/javascripts/*.js").and_return(%w(/rails/public/javascripts/jquery.js /rails/public/javascripts/rails.js))
+        Dir.should_receive(:[]).with("/rails/public/stylesheets/*.css").and_return(%w(/rails/public/stylesheets/screen.css))
+        File.should_receive(:open).with(subject.configuration_filename, "w").and_yield(nil)
+        YAML.should_receive(:dump) do |config, file|
+          config['packages']['javascript']['base'].size.should == 2
+          config['packages']['stylesheet']['base'].size.should == 1
+        end
+        subject.create_configuration 
+        @messages[0].should match(/created/)
+      end
+    end
+    
+    describe "compile_all" do
+      it "should compile all assets of all packages" do
+        YAML.should_receive(:load_file).with(subject.configuration_filename).and_return(
+          'packages' => {
+            'javascript' => {
+              'base' => %w(jquery rails),
+            },
+            'stylesheet' => {
+              'base' => %w(screen),
+            },
+          },
+          'options' => {
+            'javascript' => {
+              'compiler' => "uglify",
+            },
+            'stylesheet' => {
+              'compiler' => "rainpress",
+            },
+            'environments' => %w(test),
+          }
+        )
+
+        # merging css
+        File.should_receive(:read).with("/rails/public/stylesheets/screen.css").once.and_return("html { color: #f00; }")
+        file = double("File")
+        file.should_receive(:write).with("html { color: #f00; }").once
+        File.should_receive(:open).with("/rails/public/stylesheets/base_packaged.css", "w").twice.and_yield(file)
+        File.should_receive(:mtime).with("/rails/public/stylesheets/screen.css").once.and_return(13)
+        File.should_receive(:utime).with(13, 13, "/rails/public/stylesheets/base_packaged.css").twice
+
+        # compiling css
+        file.should_receive(:write).with("html{color:red}").once
+
+        # merging js
+        File.should_receive(:read).with("/rails/public/javascripts/jquery.js").once.and_return("var action = 'jquery'; alert('jquery')")
+        File.should_receive(:read).with("/rails/public/javascripts/rails.js").once.and_return("action = 4*3; alert('rails')")
+        file = double("File")
+        file.should_receive(:write).with("var action = 'jquery'; alert('jquery')\naction = 4*3; alert('rails')").once
+        File.should_receive(:open).with("/rails/public/javascripts/base_packaged.js", "w").twice.and_yield(file)
+        File.should_receive(:mtime).with("/rails/public/javascripts/jquery.js").once.and_return(13)
+        File.should_receive(:mtime).with("/rails/public/javascripts/rails.js").once.and_return(14)
+        File.should_receive(:utime).with(14, 14, "/rails/public/javascripts/base_packaged.js").twice
+
+        # compiling js
+        file.should_receive(:write).with("var action=\"jquery\";alert(\"jquery\"),action=12,alert(\"rails\")").once
+
+        subject.compile_all
+        @messages[0].should match(/Merging assets.*CSS Merger/)
+        @messages[1].should match(/Compiling.*Rainpress/)
+        @messages[3].should match(/Merging assets.*Simple Merger/)
+        @messages[4].should match(/Compiling.*UglifyJS/)
+      end
+    end
+  end
+end
+