lash 0.1.1 → 1.0.1

data/lib/lash/version.rb

@@ -1,3 +1,3 @@
 module Lash
-  VERSION = "0.1.1"
+  VERSION = "1.0.1"
 end

data/spec/lib/assets_host_spec.rb

@@ -0,0 +1,56 @@
+require File.expand_path( '../../spec_helper.rb', __FILE__ )
+require 'lash/assets_host'
+
+describe Lash::AssetsHost do
+  
+  
+  context "asset_id" do
+    it "should use the git repo hash for the cachebusting id" do 
+      Rails.root = File.expand_path( "../../../", __FILE__ )
+      Lash::AssetsHost.use_git_asset_id
+      ENV['RAILS_ASSET_ID'].should match /\A[a-f0-9]{40}\Z/
+    end
+  end
+  
+  context "static assets server" do
+    before :each do
+      Lash.lash_options[:use_asset_servers] = true
+      @request = mock()
+      @request.stub("ssl?").and_return(false)
+      @request.stub("host_with_port").and_return("lvh.me")
+    end
+    
+    
+    it "should prepend and assets subdomain" do
+      Lash::AssetsHost.resolve_static_asset_server_for_source( "pickles", @request ) \
+        .should match %r{http://assets\d\.lvh\.me}
+    end
+    
+    it "should return nil when disabled" do
+      Lash.lash_options[:use_asset_servers] = false
+      Lash::AssetsHost.resolve_static_asset_server_for_source( "poodles", @request ) \
+        .should be_nil
+    end
+    
+    it "should use https protocol in ssl" do
+      @request.stub("ssl?").and_return(true)
+      Lash::AssetsHost.resolve_static_asset_server_for_source( "aligators", @request ) \
+      .should match %r{\Ahttps://}
+    end
+
+    it "should return nil in ssl when disabled " do
+      @request.stub("ssl?").and_return(true)
+      Lash.lash_options[:use_asset_servers_in_ssl] = false
+      Lash::AssetsHost.resolve_static_asset_server_for_source( "aligators", @request ) \
+      .should be_nil
+    end
+    
+    it "should modify subdomain when more than 2 parts" do
+      @request.stub("host_with_port").and_return("mickey.mouse.com")
+      Lash::AssetsHost.resolve_static_asset_server_for_source( "pickles", @request ) \
+        .should match %r{http://mickey-assets\d\.mouse\.com}
+    end    
+    
+  end
+  
+end

data/spec/lib/bundle_helper_spec.rb

@@ -0,0 +1,99 @@
+require File.expand_path( '../../spec_helper', __FILE__ )
+require 'lash/bundle_helper'
+
+
+class BH 
+  include Lash::BundleHelper
+  
+  attr_accessor :params
+  attr_accessor :cookies
+  def initialize 
+    @params = {}
+    @cookies = {}
+  end
+end
+
+class String
+  def html_safe
+    self
+  end
+end
+
+
+describe Lash::BundleHelper do
+
+  before :each do
+    @fake = BH.new
+    Rails.root = File.expand_path( "../../test_app", __FILE__ )
+  end
+
+  context "bundle_files?" do
+    context "in development" do
+      before :each do
+        Rails.env = :development
+      end
+      
+      it "should be false" do
+        @fake.bundle_files?.should == false
+      end
+
+      it "should be true if params includes bundle=yes" do 
+        @fake.params[:bundle] = "yes"
+        @fake.bundle_files?.should == true
+      end
+
+    end
+
+    context "in production" do
+      before :each do
+        Rails.env = :production
+      end
+      
+      it "should be true" do
+        @fake.bundle_files?.should == true
+      end
+
+      it "should be false if params includes bundle=no in production" do
+        @fake.params[:bundle] = "no"
+        @fake.bundle_files?.should == false
+      end
+      
+    end
+    
+  end
+  
+  
+  context "javascript_bundle" do 
+    
+    before :each do
+      @fake.stub( :javascript_src_tag ).and_return {|f| "%#{f}%" }
+    end
+    
+    
+    context "when bundle_files? is true" do
+      before :each do
+        Rails.env = :production
+      end
+
+      it "should return bundle_application.js" do
+        script = @fake.javascript_bundle( "application" )
+        script.should =~ /%bundle_application.js%/
+      end
+    end
+    
+    context "when bundle_files? is false" do
+      before :each do
+        Rails.env = :development
+      end
+      
+      it "should return application, application2 and validate scripts" do
+        script = @fake.javascript_bundle( "application" )
+        %w{ application application2 tools/validate }.each do |s|
+          script.should match %r{application/#{s}.js}
+        end
+      end
+      
+    end
+  end
+
+end

data/spec/lib/closure_minifier_spec.rb

@@ -0,0 +1,67 @@
+require File.expand_path( '../../spec_helper', __FILE__ )
+require 'lash/files'
+require 'lash/closure_minifier'
+require 'fileutils'
+
+describe Lash::ClosureMinifier, :slow => true do
+  
+  before :each do
+    @minifier = Lash::ClosureMinifier.new
+    @target = File.join( TMP_ROOT, 'tmp.js' ) 
+    @application_scripts = Lash::Files.recursive_file_list( File.join( JAVASCRIPTS_ROOT, 'application' ), 'js' )
+    
+    File.delete @target if File.exist? @target
+    File.delete "#{@target}.gz" if File.exist? "#{@target}.gz"
+
+    @result = @minifier.minify( @application_scripts, @target )
+  end
+  
+  it "should create output directory when it doesn't exist" do
+    outdir = File.dirname( @target )
+    FileUtils.remove_dir outdir, true
+    @minifier.minify( @application_scripts, @target )
+    File.exist?( outdir ).should be_true
+  end
+  
+  it "should not fail" do
+    @result.should be_true
+  end
+  
+  it "should minify to target folder" do
+    File.exist?( @target ).should be_true
+  end
+  
+  it "should be smaller than the original files" do
+    size = @application_scripts.inject(0) { |n,s| n = n + File.size( s ) }
+    File.size( @target ).should < size
+  end
+
+  it "should not be zero length file" do
+    File.size( @target ).should > 0
+  end
+  
+  it "should create a gzippped companion file" do
+    File.exist?(@target + '.gz').should be_true
+  end
+
+  it "should join compiler options" do
+    @minifier.options = {
+      :one => "1",
+      "--open" => "two",
+      "--on" => nil
+    }
+    
+    @minifier.send( :compiler_options ).should == "one 1 --open two --on"
+  end
+
+  it "should fail when it contains invalid javascript" do
+    invalid = File.join( TMP_ROOT, "invalid.js" ) 
+    File.open( invalid, "w+" ) do |f|
+      f.write( "I don't make any sense")
+    end
+    
+    @application_scripts << invalid
+    @minifier.minify( @application_scripts, @target ).should be_false
+  end
+  
+end

data/spec/lib/java_script_bundler_spec.rb

@@ -0,0 +1,46 @@
+require File.expand_path( '../../spec_helper.rb', __FILE__ )
+require 'lash/files'
+require 'lash/java_script_minifier'
+require 'lash/java_script_bundler'
+
+
+class SimpleMinifier < Lash::JavaScriptMinifier
+end
+
+describe Lash::JavaScriptBundler do
+  
+  before :each do
+    Rails.root = RAILS_ROOT
+    @bundler = Lash::JavaScriptBundler.new( :minifiers => SimpleMinifier.new )
+  end
+  
+  it "should bundle into a single file :style => :single" do
+    @bundler.stub(:migrate_best_target)
+    @bundler.should_receive( :bundle_scripts_with_minifier ).once \
+        .and_return { |f| File.open( f.first + '.tmp', "w+" ) { |s| s.write( f ) }; "#{f.first}.tmp" }
+    @bundler.bundle 'application', :style => :single
+  end
+  
+  it "should bundle individual files when :style => :individual" do
+    @bundler.stub(:migrate_best_target)
+    @bundler.should_receive( :bundle_scripts_with_minifier ).at_least(3).times \
+        .and_return { |f| File.open( f.first + '.tmp', "w+" ) { |s| s.write( f ) }; "#{f.first}.tmp" }
+    @bundler.bundle 'application', :style => :individual
+  end
+  
+  it "should use single style for directories that are not included in cdn_dirs and demand_dirs" do
+    @bundler.should_receive(:bundle_into_single_script)
+    @bundler.bundle "application"    
+  end
+
+  it "should use individual style for directories that are not included in cdn_dirs" do
+    @bundler.should_receive(:bundle_individual_scripts)
+    @bundler.bundle "cdn"
+  end
+
+  it "should use individual style for directories that are not included in demand_dirs" do
+    @bundler.should_receive(:bundle_individual_scripts)
+    @bundler.bundle "demand"
+  end
+  
+end

data/spec/lib/lash_files_spec.rb

@@ -0,0 +1,68 @@
+require File.expand_path( '../../spec_helper.rb', __FILE__ )
+require 'lash/files'
+
+describe Lash::Files do
+
+  context "recursive_file_list" do 
+  
+    it "should find files with extension including a leading period" do
+      Lash::Files.recursive_file_list( File.join( JAVASCRIPTS_ROOT, 'application' ), '.js' ) \
+        .count.should > 0
+    end
+  
+    it "should find files with extension without a leading period" do
+      Lash::Files.recursive_file_list( File.join( JAVASCRIPTS_ROOT, 'application' ), 'js' ) \
+        .count.should > 0
+    end
+  
+    it "should find files without an extension" do
+      Lash::Files.recursive_file_list( File.join( JAVASCRIPTS_ROOT, 'application' ), nil ) \
+        .count.should > 0
+    end
+  
+    context "javascripts/application" do
+    
+      before :each do
+        @files = Lash::Files.recursive_file_list File.join( JAVASCRIPTS_ROOT, 'application' ), '.js'
+      end
+    
+      it "should find at least 3 files" do
+        @files.count.should >= 3
+      end
+    
+      it "should include all files in application folder" do
+        %w{ application.js application2.js tools/validate.js }.each do |f|
+          @files.should include( File.join( JAVASCRIPTS_ROOT, 'application', f ) )
+        end
+      end    
+    
+      it "should not include README" do
+        %w{ README }.each do |f|
+          @files.should_not include( File.join( JAVASCRIPTS_ROOT, 'application', f ) )
+        end
+      end
+    end
+  end
+  
+  context "get_top_level_directories" do
+    
+    before :each do
+      @dirs = Lash::Files.get_top_level_directories( JAVASCRIPTS_ROOT )
+    end
+    
+    it "should find application, cdn, demand" do
+      %w{ application cdn demand }.each do |f|
+        @dirs.should include( File.join( JAVASCRIPTS_ROOT, f ) )
+      end
+    end
+    
+    it "should not find application/tools" do
+      %w{ application/tools }.each do |f|
+        @dirs.should_not include( File.join( JAVASCRIPTS_ROOT, f ) )
+      end
+      
+    end
+    
+  end
+
+ end

data/spec/spec_helper.rb

@@ -1,29 +1,19 @@
-$:.unshift File.expand_path( File.join( File.dirname( __FILE__ ), '..' ) )
+$:.unshift File.join( File.dirname( __FILE__ ), 'support' )
 
-require 'rubygems'
+require 'lash'
 
+RAILS_ROOT = File.expand_path( '../test_app', __FILE__ )
+PUBLIC_ROOT = File.join( RAILS_ROOT, 'public' )
+JAVASCRIPTS_ROOT = File.join( PUBLIC_ROOT, 'javascripts' )
+STYLESHEETS_ROOT = File.join( PUBLIC_ROOT, 'stylesheets' )
+IMAGES_ROOT = File.join( PUBLIC_ROOT, 'images' )
+TMP_ROOT = File.expand_path( "../../tmp", __FILE__ )
 
-ENV["RAILS_ENV"] ||= 'test'
-require 'ffaker'
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f} 
 
-Dir[Rails.root.join("spec/support/**/*.rb")].each {|f| require f}
-Dir[Rails.root.join("spec/factories/**/*.rb")].each { |f| require f }
-
-RSpec.configure do |config|
-  # == Mock Framework
-  #
-  # If you prefer to use mocha, flexmock or RR, uncomment the appropriate line:
-  #
-  # config.mock_with :mocha
-  # config.mock_with :flexmock
-  # config.mock_with :rr
-  config.mock_with :rspec
-
-  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
-  # config.fixture_path = "#{::Rails.root}/spec/fixtures"
-
-  # If you're not using ActiveRecord, or you'd prefer not to run each of your
-  # examples within a transaction, remove the following line or assign false
-  # instead of true.
-  config.use_transactional_fixtures = true
-end
+RSpec.configure do |c|
+  c.filter_run :focus => true
+  c.run_all_when_everything_filtered = true
+#  c.fail_fast = true
+  c.filter_run_excluding :slow => true
+end

data/spec/support/rails_fake.rb

@@ -0,0 +1,31 @@
+require 'rspec'
+require 'rspec/mocks'
+
+class RailsFake
+  attr_accessor :env
+  attr_accessor :root
+  
+  def initialize
+    @env = RSpec::Mocks::Mock.new
+  end  
+  
+  def env=(s)
+    %w{ production development test }.each do |e|
+      @env.stub("#{e}?").and_return( e == s.to_s )    
+    end
+  end
+  
+end
+
+class String
+  def html_safe
+    self
+  end
+  
+  def parameterize
+    self
+  end
+end
+
+
+::Rails = ::RailsFake.new

data/spec/test_app/.gitignore

@@ -0,0 +1,4 @@
+.bundle
+db/*.sqlite3
+log/*.log
+tmp/

data/spec/test_app/Rakefile

@@ -0,0 +1,8 @@
+require 'rubygems'
+require File.expand_path( "../../support/rails_fake.rb", __FILE__ )
+Rails.root = File.expand_path( '..', __FILE__ )
+require 'lash'
+
+import File.expand_path( "../../../lib/lash/tasks/lash.rake", __FILE__ )
+
+Lash.lash_options[:use_git_asset_id] = false

data/spec/test_app/public/images/ui-sprite.png.sprite

@@ -0,0 +1,3 @@
+16 x 32
+I public/sprites/ui/accept.png 0 0 16 16
+I public/sprites/ui/add.png 0 16 16 16