rack-less 1.5.0 → 2.0.0

data/test/app_helper.rb

@@ -0,0 +1,25 @@
+require 'rack/test'
+require 'webrat'
+
+class Test::Unit::TestCase
+  include Rack::Test::Methods
+  include Webrat::Methods
+  include Webrat::Matchers
+ 
+  Webrat.configure do |config|
+    config.mode = :rack
+  end 
+
+  class << self
+
+    def should_respond_with_compiled_css
+      should "return compiled css" do
+        assert_equal 200, @response.status, "status is not '#{Rack::Utils::HTTP_STATUS_CODES[200]}'"
+        assert @response.headers["Content-Type"].include?(Rack::Less::MIME_TYPE), "content type is not '#{Rack::Less::MIME_TYPE}'"
+        assert_equal @compiled.strip, @response.body.strip, "the compiled css is incorrect"
+      end
+    end
+
+  end
+
+end

data/test/config_test.rb

@@ -0,0 +1,215 @@
+require "test/helper"
+require 'rack/less/config'
+
+class ConfigTest < Test::Unit::TestCase
+  context 'Rack::Less::Config' do
+    setup do
+      @config = Rack::Less::Config.new
+    end
+
+    { :cache => false,
+      :compress => false,
+      :combinations => {},
+      :cache_bust => false
+    }.each do |k,v|
+      should "default #{k} correctly" do
+        assert_equal v, @config.send(k)
+      end
+
+      should "have an accessor for #{k}" do
+        assert_respond_to @config, k, "no reader for #{k}"
+        assert_respond_to @config, "#{k}=".to_sym, "no writer for #{k}"
+      end
+    end
+
+    should "provide boolean readers" do
+      assert_respond_to @config, :cache?, "no reader for :cache?"
+      assert_equal !!@config.cache, @config.cache?
+      assert_respond_to @config, :compress?, "no reader for :compress?"
+      assert_equal !!@config.compress, @config.compress?
+    end
+
+    should "allow init with setting hash" do
+      settings = {
+        :cache => true,
+        :compress => true,
+        :combinations => {
+          'all' => ['one', 'two']
+        },
+        :cache_bust => false
+      }
+      config = Rack::Less::Config.new settings
+
+      assert_equal true, config.cache
+      assert_equal true, config.compress
+      combinations = {'all' => ['one', 'two']}
+      assert_equal combinations, config.combinations
+    end
+
+    should "be accessible at Rack::Less class level" do
+      assert_respond_to Rack::Less, :configure
+      assert_respond_to Rack::Less, :config
+      assert_respond_to Rack::Less, :config=
+      assert_respond_to Rack::Less, :combinations
+      assert_respond_to Rack::Less, :cache_bust
+      assert_respond_to Rack::Less, :stylesheet
+    end
+
+    context "given a new configuration" do
+      setup do
+        @old_config = Rack::Less.config
+        @settings = {
+          :cache => true,
+          :compress => true,
+          :combinations => {
+            'all' => ['one', 'two']
+          },
+          :cache_bust => false
+        }
+        @traditional_config = Rack::Less::Config.new @settings
+      end
+      teardown do
+        Rack::Less.config = @old_config
+      end
+
+      should "allow Rack::Less to directly apply settings" do
+        Rack::Less.config = @traditional_config.dup
+
+        assert_equal @traditional_config.cache, Rack::Less.config.cache
+        assert_equal @traditional_config.compress, Rack::Less.config.compress
+        assert_equal @traditional_config.combinations, Rack::Less.config.combinations
+        assert_equal @traditional_config.cache_bust, Rack::Less.config.cache_bust
+      end
+
+      should "allow Rack::Less to apply settings using a block" do
+        Rack::Less.configure do |config|
+          config.cache    = true
+          config.compress = true
+          config.combinations = {
+            'all' => ['one', 'two']
+          }
+          config.cache_bust = false
+        end
+
+        assert_equal @traditional_config.cache, Rack::Less.config.cache
+        assert_equal @traditional_config.compress, Rack::Less.config.compress
+        assert_equal @traditional_config.combinations, Rack::Less.config.combinations
+        assert_equal @traditional_config.cache_bust, Rack::Less.config.cache_bust
+      end
+
+      context "helpers" do
+        setup do
+          @settings = {
+            :combinations => {
+              'all' => ['one', 'two']
+            },
+            :cache_bust => false
+          }
+        end
+
+        context "#combinations" do
+          should "should be able to access it's values with a parameter" do
+            config = Rack::Less::Config.new @settings
+
+            assert_equal [], config.combinations('one')
+            assert_equal [], config.combinations('wtf')
+            assert_equal ['one.css', 'two.css'], config.combinations('all')
+          end
+
+          context "if cache setting is true" do
+            setup do
+              @settings[:cache] = true
+            end
+
+            should "use the lookup parameter instead of the value" do
+              config = Rack::Less::Config.new @settings
+
+              assert_equal 'all.css', config.combinations('all')
+            end
+          end
+
+        end
+
+        context "#stylesheet" do
+          should "should be able to stylesheet references" do
+            config = Rack::Less::Config.new @settings
+
+            assert_equal 'one.css', config.stylesheet('one')
+            assert_equal 'wtf.css', config.stylesheet('wtf')
+          end
+
+          should "should be able to access combination values with a parameter" do
+            config = Rack::Less::Config.new @settings
+
+            assert_equal ['one.css', 'two.css'], config.stylesheet('all')
+          end
+
+          context "if cache setting is true" do
+            setup do
+              @settings[:cache] = true
+            end
+
+            should "use the lookup parameter instead of the value" do
+              config = Rack::Less::Config.new @settings
+
+              assert_equal 'one.css', config.stylesheet('one')
+              assert_equal 'all.css', config.stylesheet('all')
+            end
+          end
+
+          context "when cache_bust is false" do
+            setup do
+              @settings[:cache_bust] = false
+            end
+
+            should "should not put in a cache bust value" do
+              config = Rack::Less::Config.new @settings
+
+              assert_equal 'one.css', config.stylesheet('one')
+            end
+          end
+
+          context "when cache_bust is nil" do
+            setup do
+              @settings[:cache_bust] = nil
+            end
+
+            should "should not put in a cache bust value" do
+              config = Rack::Less::Config.new @settings
+
+              assert_equal 'one.css', config.stylesheet('one')
+            end
+          end
+
+          context "when cache_bust is true" do
+            setup do
+              @settings[:cache_bust] = true
+            end
+
+            should "always put a timestamp value on the end of the href" do
+              config = Rack::Less::Config.new @settings
+
+              assert_match /one.css\?[0-9]+/, config.stylesheet('one')
+            end
+          end
+
+          context "when timestamp specified" do
+            setup do
+              @stamp = Time.now.to_i - 100_000
+              @settings[:cache_bust] = @stamp
+            end
+
+            should "always use that timestamp" do
+              config = Rack::Less::Config.new @settings
+
+              assert_equal ["one.css?#{@stamp}", "two.css?#{@stamp}"], config.stylesheet('all')
+            end
+
+          end
+        end
+
+      end
+    end
+
+  end
+end

data/test/env.rb

@@ -0,0 +1,9 @@
+# Add test and lib paths to the $LOAD_PATH
+[ File.dirname(__FILE__),
+  File.join(File.dirname(__FILE__), '..', 'lib')
+].each do |path|
+  full_path = File.expand_path(path)
+  $LOAD_PATH.unshift(full_path) unless $LOAD_PATH.include?(full_path)
+end
+
+require 'rack/less'

data/test/fixtures/mock_options.rb

@@ -0,0 +1,9 @@
+class MockOptions
+  include Rack::Less::Options
+  
+  def initialize
+    @env = nil
+    initialize_options
+  end
+end
+

data/test/fixtures/sinatra/app.rb

@@ -0,0 +1,9 @@
+require 'sinatra/base'
+
+class SinatraApp < Sinatra::Base
+  
+  configure do
+    set :root, File.expand_path(File.dirname(__FILE__))
+  end
+  
+end

data/test/fixtures/sinatra/app/stylesheets/all_compiled.css

@@ -0,0 +1,7 @@
+#header { color: #6c94be; }
+div { width: 2; }
+
+div {
+  width: 20px;
+  height: 30px;
+}

data/test/fixtures/sinatra/app/stylesheets/all_one.less

@@ -0,0 +1,10 @@
+@nice-blue: #5B83AD;
+@light-blue: @nice-blue + #111;
+
+#header {
+  color: @light-blue;
+}
+
+div {
+  width: 1 + 1
+}

data/test/fixtures/sinatra/app/stylesheets/all_two.less

@@ -0,0 +1,4 @@
+div {
+  width: 20px;
+  height: 30px;
+}

data/test/fixtures/sinatra/app/stylesheets/css.css

@@ -0,0 +1,4 @@
+SPAN {
+  width: 20px;
+  height: 30px;
+}

data/test/fixtures/sinatra/app/stylesheets/css_compiled.css

@@ -0,0 +1,4 @@
+SPAN {
+  width: 20px;
+  height: 30px;
+}

data/test/fixtures/sinatra/app/stylesheets/nested/file.css

@@ -0,0 +1,10 @@
+@nice-blue: #5B83AD;
+@light-blue: @nice-blue + #111;
+
+#header {
+  color: @light-blue;
+}
+
+div {
+  width: 1 + 1
+}

data/test/fixtures/sinatra/app/stylesheets/nested/file_compiled.css

@@ -0,0 +1,2 @@
+#header { color: #6c94be; }
+div { width: 2; }

data/test/fixtures/sinatra/app/stylesheets/nested/really/really.less

@@ -0,0 +1,10 @@
+@bad-blue: #5B83FF;
+@bad-light-blue: @bad-blue + #111;
+
+#header {
+  color: @bad-light-blue;
+}
+
+div {
+  width: 2 + 2
+}

data/test/fixtures/sinatra/app/stylesheets/nested/really/really_compiled.css

@@ -0,0 +1,2 @@
+#header { color: #6c94ff; }
+div { width: 4; }

data/test/fixtures/sinatra/app/stylesheets/normal.less

@@ -0,0 +1,10 @@
+@nice-blue: #5B83AD;
+@light-blue: @nice-blue + #111;
+
+#header {
+  color: @light-blue;
+}
+
+div {
+  width: 1 + 1
+}

data/test/fixtures/sinatra/app/stylesheets/normal_compiled.css

@@ -0,0 +1,2 @@
+#header { color: #6c94be; }
+div { width: 2; }

data/test/fixtures/sinatra/app/stylesheets/some-styles.less

@@ -0,0 +1,8 @@
+SPAN {
+  width: 20px;
+  height: 30px;
+}
+
+DIV {
+  color: #fff;
+}

data/test/fixtures/sinatra/app/stylesheets/some_styles.less

@@ -0,0 +1,8 @@
+SPAN {
+  width: 20px;
+  height: 30px;
+}
+
+DIV {
+  color: #fff;
+}

data/test/fixtures/sinatra/app/stylesheets/styles1.less

@@ -0,0 +1,8 @@
+SPAN {
+  width: 20px;
+  height: 30px;
+}
+
+DIV {
+  color: #fff;
+}

data/test/helper.rb

@@ -0,0 +1,77 @@
+require 'rubygems'
+require 'bundler'
+Bundler.setup
+
+require 'test_belt'
+require 'test/env'
+
+class Test::Unit::TestCase
+
+  def file_path(*segments)
+    segs = segments.unshift([File.dirname(__FILE__), '..']).flatten
+    File.expand_path(segs.join(File::SEPARATOR))
+  end
+
+  def self.should_compile_source(name, desc)
+    context desc do
+      setup do
+        @compiled = File.read(File.join(@source_folder, "#{name}_compiled.css"))
+        @source = Rack::Less::Source.new(name, :folder => @source_folder)
+      end
+
+      should "compile LESS" do
+        assert_equal @compiled.strip, @source.compiled.strip, '.compiled is incorrect'
+        assert_equal @compiled.strip, @source.to_css.strip, '.to_css is incorrect'
+        assert_equal @compiled.strip, @source.css.strip, '.css is incorrect'
+      end
+    end
+  end
+
+  def env_defaults
+    Rack::Less::Base.defaults.merge({
+      Rack::Less::Base.option_name(:root) => file_path('test','fixtures','sinatra')
+    })
+  end
+
+  def less_request(method, path_info)
+    Rack::Less::Request.new(@defaults.merge({
+      'REQUEST_METHOD' => method,
+      'PATH_INFO' => path_info
+    }))
+  end
+
+  def less_response(css)
+    Rack::Less::Response.new(@defaults, css)
+  end
+
+  def self.should_not_be_a_valid_rack_less_request(args)
+    context "to #{args[:method].upcase} #{args[:resource]} (#{args[:description]})" do
+      setup do
+        @request = less_request(args[:method], args[:resource])
+      end
+
+      should "not be a valid endpoint for Rack::Less" do
+        not_valid = !@request.get?
+        not_valid ||= !@request.for_css?
+        not_valid ||= @request.source.files.empty?
+        assert not_valid, 'request is a GET for .css format and has source'
+        assert !@request.for_less?, 'the request is for less'
+      end
+    end
+  end
+  def self.should_be_a_valid_rack_less_request(args)
+    context "to #{args[:method].upcase} #{args[:resource]} (#{args[:description]})" do
+      setup do
+        @request = less_request(args[:method], args[:resource])
+      end
+
+      should "be a valid endpoint for Rack::Less" do
+        assert @request.get?, 'the request is not a GET'
+        assert @request.for_css?, 'the request is not for css'
+        assert !@request.source.files.empty?, 'the request resource has no source'
+        assert @request.for_less?, 'the request is not for less'
+      end
+    end
+  end
+
+end