roda 1.0.0 → 1.1.0

data/lib/roda/plugins/render_each.rb

@@ -24,8 +24,6 @@ class Roda
     # not set a local variable inside the template.
     module RenderEach
       module InstanceMethods
-        EMPTY_STRING = ''.freeze
-
         # For each value in enum, render the given template using the
         # given opts.  The template and options hash are passed to +render+.
         # Additional options supported:

data/lib/roda/plugins/streaming.rb

@@ -154,8 +154,7 @@ class Roda
             end
           end
 
-          res = response
-          request.halt [res.status || 200, res.headers, Stream.new(opts, &block)]
+          throw :halt, response.finish_with_body(Stream.new(opts, &block))
         end
       end
     end

data/lib/roda/plugins/view_subdirs.rb

@@ -6,7 +6,7 @@ class Roda
     # use, and template names that do not contain a slash will
     # automatically use that view subdirectory.  Example:
     #
-    #   plugin :render
+    #   plugin :render, :layout=>'./layout'
     #   plugin :view_subdirs
     #
     #   route do |r|
@@ -25,6 +25,12 @@ class Roda
     #
     # This plugin should be loaded after the render plugin, since
     # it works by overriding parts of the render plugin.
+    # 
+    # Note that when a view subdirectory is set, the layout will
+    # also be looked up in the subdirectory unless it contains
+    # a slash.  So if you want to use a view subdirectory for
+    # templates but have a shared layout, you should make sure your
+    # layout contains a slash, similar to the example above.
     module ViewSubdirs
       module InstanceMethods
         # Set the view subdirectory to use.  This can be set to nil

data/lib/roda/version.rb

@@ -1,3 +1,3 @@
 class Roda
-  RodaVersion = '1.0.0'.freeze
+  RodaVersion = '1.1.0'.freeze
 end

data/spec/assets/css/app.scss

@@ -0,0 +1 @@
+body { color: red; }

data/spec/assets/css/no_access.css

@@ -0,0 +1 @@
+no access

data/spec/assets/css/raw.css

@@ -0,0 +1 @@
+body { color: blue; }

data/spec/assets/js/head/app.js

@@ -0,0 +1 @@
+console.log('test')

data/spec/integration_spec.rb

@@ -18,7 +18,7 @@ describe "integration" do
 
   end
 
-  it "should setup middleware using use " do
+  it "should setup middleware using use" do
     c = @c
     app(:bare) do 
       use c, "First", "Second" do
@@ -35,7 +35,24 @@ describe "integration" do
     body('/hello').should == 'D First Second Block'
   end
 
-  it "should inherit middleware in subclass " do
+  it "should support adding middleware using use after route block setup" do
+    c = @c
+    app(:bare) do 
+      route do |r|
+        r.get "hello" do
+          "D #{r.env['m.first']} #{r.env['m.second']} #{r.env['m.block']}"
+        end
+      end
+
+      use c, "First", "Second" do
+        "Block"
+      end
+    end
+
+    body('/hello').should == 'D First Second Block'
+  end
+
+  it "should inherit middleware in subclass" do
     c = @c
     @app = Class.new(app(:bare){use(c, '1', '2'){"3"}})
     @app.route do  |r|
@@ -47,7 +64,55 @@ describe "integration" do
     body('/hello').should == 'D 1 2 3'
   end
 
-  it "should not have future middleware additions to parent class affect subclass " do
+  it "should inherit route in subclass" do
+    c = @c
+    app(:bare) do
+      use(c, '1', '2'){"3"}
+      route do |r|
+        r.get "hello" do
+          "D #{r.env['m.first']} #{r.env['m.second']} #{r.env['m.block']}"
+        end
+      end
+    end
+    @app = Class.new(app)
+
+    body('/hello').should == 'D 1 2 3'
+  end
+
+  it "should use instance of subclass when inheriting routes" do
+    c = @c
+    obj = nil
+    app(:bare) do
+      use(c, '1', '2'){"3"}
+      route do |r|
+        r.get "hello" do
+          obj = self
+          "D #{r.env['m.first']} #{r.env['m.second']} #{r.env['m.block']}"
+        end
+      end
+    end
+    @app = Class.new(app)
+
+    body('/hello').should == 'D 1 2 3'
+    obj.should be_a_kind_of(@app)
+  end
+
+  it "should handle middleware added to subclass using superclass route" do
+    c = @c
+    app(:bare) do
+      route do |r|
+        r.get "hello" do
+          "D #{r.env['m.first']} #{r.env['m.second']} #{r.env['m.block']}"
+        end
+      end
+    end
+    @app = Class.new(app)
+    @app.use(c, '1', '2'){"3"}
+
+    body('/hello').should == 'D 1 2 3'
+  end
+
+  it "should not have future middleware additions to superclass affect subclass" do
     c = @c
     a = app
     @app = Class.new(a)
@@ -60,4 +125,31 @@ describe "integration" do
 
     body('/hello').should == 'D   '
   end
+
+  it "should not have future middleware additions to subclass affect superclass" do
+    c = @c
+    a = app do |r|
+      r.get "hello" do
+        "D #{r.env['m.first']} #{r.env['m.second']} #{r.env['m.block']}"
+      end
+    end
+    @app = Class.new(a)
+    @app.use(c, '1', '2'){"3"}
+    @app = a
+
+    body('/hello').should == 'D   '
+  end
+
+  it "should have app return the rack application to call" do
+    app(:bare){}.app.should == nil
+    app.route{|r|}
+    app.app.should be_a_kind_of(Proc)
+    c = Class.new{def initialize(app) @app = app end; def call(env) @app.call(env) end} 
+    app.use c
+    app.app.should be_a_kind_of(c)
+  end
+
+  it "should have route_block return the route block" do
+    app{|r| 1}.route_block.call(nil).should == 1
+  end
 end

data/spec/matchers_spec.rb

@@ -104,7 +104,7 @@ describe "r.is" do
 
   it "matches regexps" do
     app do |r|
-      r.is /(\w+)/ do |id|
+      r.is(/(\w+)/) do |id|
         id
       end
     end
@@ -458,7 +458,7 @@ describe "path matchers" do
 
   it "a path with some regex captures" do
     app do |r|
-      r.on /user(\d+)/ do |uid|
+      r.on(/user(\d+)/) do |uid|
         uid
       end
     end

data/spec/plugin/assets_spec.rb

@@ -0,0 +1,413 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+require 'fileutils'
+
+begin
+  for lib in %w'tilt sass'
+    require lib
+  end
+  run_tests = true
+rescue LoadError
+  warn "#{lib} not installed, skipping assets plugin test"
+end
+
+if run_tests
+  metadata_file = 'spec/assets/tmp/precompiled.json'
+  js_file = 'spec/assets/js/head/app.js'
+  css_file = 'spec/assets/css/no_access.css'
+  js_mtime = File.mtime(js_file)
+  js_atime = File.atime(js_file)
+  css_mtime = File.mtime(css_file)
+  css_atime = File.atime(css_file)
+  describe 'assets plugin' do
+    before do
+      app(:bare) do
+        plugin :assets,
+          :css => ['app.scss', 'raw.css'],
+          :js => { :head => ['app.js'] },
+          :path => 'spec/assets',
+          :public => 'spec',
+          :css_opts => {:cache=>false}
+
+        route do |r|
+          r.assets
+
+          r.is 'test' do
+            "#{assets(:css)}\n#{assets([:js, :head])}"
+          end
+        end
+      end
+    end
+    after do
+      File.utime(js_atime, js_mtime, js_file)
+      File.utime(css_atime, css_mtime, css_file)
+      File.delete(metadata_file) if File.file?(metadata_file)
+    end
+    after(:all) do
+      FileUtils.rm_r('spec/assets/tmp') if File.directory?('spec/assets/tmp')
+      FileUtils.rm_r('spec/public') if File.directory?('spec/public')
+    end
+
+    it 'assets_opts should use correct paths given options' do
+      keys = [:js_path, :css_path, :compiled_js_path, :compiled_css_path, :js_prefix, :css_prefix, :compiled_js_prefix, :compiled_css_prefix]
+      app.assets_opts.values_at(*keys).should == %w"spec/assets/js/ spec/assets/css/ spec/assets/app spec/assets/app assets/js/ assets/css/ assets/app assets/app"
+
+      app.plugin :assets, :path=>'bar/', :public=>'foo/', :prefix=>'as/', :js_dir=>'j/', :css_dir=>'c/', :compiled_name=>'a'
+      app.assets_opts.values_at(*keys).should == %w"bar/j/ bar/c/ foo/as/a foo/as/a as/j/ as/c/ as/a as/a"
+
+      app.plugin :assets, :path=>'bar', :public=>'foo', :prefix=>'as', :js_dir=>'j', :css_dir=>'c', :compiled_name=>'a'
+      app.assets_opts.values_at(*keys).should == %w"bar/j/ bar/c/ foo/as/a foo/as/a as/j/ as/c/ as/a as/a"
+
+      app.plugin :assets, :compiled_js_dir=>'cj', :compiled_css_dir=>'cs', :compiled_path=>'cp'
+      app.assets_opts.values_at(*keys).should == %w"bar/j/ bar/c/ foo/cp/cj/a foo/cp/cs/a as/j/ as/c/ as/cj/a as/cs/a"
+
+      app.plugin :assets, :compiled_js_route=>'cjr', :compiled_css_route=>'ccr', :js_route=>'jr', :css_route=>'cr'
+      app.assets_opts.values_at(*keys).should == %w"bar/j/ bar/c/ foo/cp/cj/a foo/cp/cs/a as/jr/ as/cr/ as/cjr/a as/ccr/a"
+
+      app.plugin :assets, :compiled_js_route=>'cj', :compiled_css_route=>'cs', :js_route=>'j', :css_route=>'c'
+      app.assets_opts.values_at(*keys).should == %w"bar/j/ bar/c/ foo/cp/cj/a foo/cp/cs/a as/j/ as/c/ as/cj/a as/cs/a"
+
+      app.plugin :assets
+      app.assets_opts.values_at(*keys).should == %w"bar/j/ bar/c/ foo/cp/cj/a foo/cp/cs/a as/j/ as/c/ as/cj/a as/cs/a"
+
+      app.plugin :assets, :compiled_js_dir=>'', :compiled_css_dir=>nil, :compiled_js_route=>nil, :compiled_css_route=>nil
+      app.assets_opts.values_at(*keys).should == %w"bar/j/ bar/c/ foo/cp/a foo/cp/a as/j/ as/c/ as/a as/a"
+
+      app.plugin :assets, :js_dir=>'', :css_dir=>nil, :js_route=>nil, :css_route=>nil
+      app.assets_opts.values_at(*keys).should == %w"bar/ bar/ foo/cp/a foo/cp/a as/ as/ as/a as/a"
+
+      app.plugin :assets, :public=>''
+      app.assets_opts.values_at(*keys).should == %w"bar/ bar/ cp/a cp/a as/ as/ as/a as/a"
+
+      app.plugin :assets, :path=>'', :compiled_path=>nil
+      app.assets_opts.values_at(*keys).should == ['', '', 'a', 'a', 'as/', 'as/', 'as/a', 'as/a']
+
+      app.plugin :assets, :prefix=>''
+      app.assets_opts.values_at(*keys).should == ['', '', 'a', 'a', '', '', 'a', 'a']
+
+      app.plugin :assets, :compiled_name=>nil
+      app.assets_opts.values_at(*keys).should == ['', '', '', '', '', '', '', '']
+    end
+
+    it 'assets_opts should use headers and dependencies given options' do
+      keys = [:css_headers, :js_headers, :dependencies]
+      app.assets_opts.values_at(*keys).should == [{'Content-Type'=>"text/css; charset=UTF-8"}, {'Content-Type'=>"application/javascript; charset=UTF-8"}, {}]
+
+      app.plugin :assets, :headers=>{'A'=>'B'}, :dependencies=>{'a'=>'b'}
+      app.assets_opts.values_at(*keys).should == [{'Content-Type'=>"text/css; charset=UTF-8", 'A'=>'B'}, {'Content-Type'=>"application/javascript; charset=UTF-8", 'A'=>'B'}, {'a'=>'b'}]
+
+      app.plugin :assets, :css_headers=>{'C'=>'D'}, :js_headers=>{'E'=>'F'}, :dependencies=>{'c'=>'d'}
+      app.assets_opts.values_at(*keys).should == [{'Content-Type'=>"text/css; charset=UTF-8", 'A'=>'B', 'C'=>'D'}, {'Content-Type'=>"application/javascript; charset=UTF-8", 'A'=>'B', 'E'=>'F'}, {'a'=>'b', 'c'=>'d'}]
+
+      app.plugin :assets
+      app.assets_opts.values_at(*keys).should == [{'Content-Type'=>"text/css; charset=UTF-8", 'A'=>'B', 'C'=>'D'}, {'Content-Type'=>"application/javascript; charset=UTF-8", 'A'=>'B', 'E'=>'F'}, {'a'=>'b', 'c'=>'d'}]
+
+      app.plugin :assets
+      app.assets_opts.values_at(*keys).should == [{'Content-Type'=>"text/css; charset=UTF-8", 'A'=>'B', 'C'=>'D'}, {'Content-Type'=>"application/javascript; charset=UTF-8", 'A'=>'B', 'E'=>'F'}, {'a'=>'b', 'c'=>'d'}]
+
+      app.plugin :assets, :headers=>{'Content-Type'=>'C', 'E'=>'G'}
+      app.assets_opts.values_at(*keys).should == [{'Content-Type'=>"C", 'A'=>'B', 'C'=>'D', 'E'=>'G'}, {'Content-Type'=>"C", 'A'=>'B', 'E'=>'F'}, {'a'=>'b', 'c'=>'d'}]
+
+      app.plugin :assets, :css_headers=>{'A'=>'B1'}, :js_headers=>{'E'=>'F1'}, :dependencies=>{'c'=>'d1'}
+      app.assets_opts.values_at(*keys).should == [{'Content-Type'=>"C", 'A'=>'B1', 'C'=>'D', 'E'=>'G'}, {'Content-Type'=>"C", 'A'=>'B', 'E'=>'F1'}, {'a'=>'b', 'c'=>'d1'}]
+    end
+
+    it 'should handle rendering assets, linking to them, and accepting requests for them when not compiling' do
+      html = body('/test')
+      html.scan(/<link/).length.should == 2
+      html =~ %r{href="(/assets/css/app\.scss)"}
+      css = body($1)
+      html =~ %r{href="(/assets/css/raw\.css)"}
+      css2 = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/js/head/app\.js)"}
+      js = body($1)
+      css.should =~ /color: red;/
+      css2.should =~ /color: blue;/
+      js.should include('console.log')
+    end
+
+    it 'should handle rendering assets, linking to them, and accepting requests for them when not compiling, with different options' do
+      app.plugin :assets, :path=>'spec/', :js_dir=>'assets/js', :css_dir=>'assets/css', :prefix=>'a',
+        :js_route=>'foo', :css_route=>'bar', :add_suffix=>true, :css_opts=>{:style=>:compressed}
+      html = body('/test')
+      html.scan(/<link/).length.should == 2
+      html =~ %r{href="(/a/bar/app\.scss.css)"}
+      css = body($1)
+      html =~ %r{href="(/a/bar/raw\.css.css)"}
+      css2 = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/a/foo/head/app\.js.js)"}
+      js = body($1)
+      css.should =~ /color:red\}/
+      css2.should =~ /color: blue;/
+      js.should include('console.log')
+    end
+
+    it 'should handle rendering assets, linking to them, and accepting requests for them when not compiling with a multi-level hash' do
+      app.plugin :assets, :path=>'spec', :js_dir=>nil, :css_dir=>nil, :prefix=>nil,
+        :css=>{:assets=>{:css=>%w'app.scss raw.css'}}, :js=>{:assets=>{:js=>{:head=>'app.js'}}}
+      app.route do |r|
+        r.assets
+        r.is 'test' do
+          "#{assets([:css, :assets, :css])}\n#{assets([:js, :assets, :js, :head])}"
+        end
+      end
+      html = body('/test')
+      html.scan(/<link/).length.should == 2
+      html =~ %r{href="(/assets/css/app\.scss)"}
+      css = body($1)
+      html =~ %r{href="(/assets/css/raw\.css)"}
+      css2 = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/js/head/app\.js)"}
+      js = body($1)
+      css.should =~ /color: red;/
+      css2.should =~ /color: blue;/
+      js.should include('console.log')
+    end
+
+    it 'should handle :group_subdirs => false' do
+      app.plugin :assets, :path=>'spec', :js_dir=>nil, :css_dir=>nil, :prefix=>nil, :group_subdirs=>false,
+        :css=>{:assets=>{:css=>%w'assets/css/app.scss assets/css/raw.css'}}, :js=>{:assets=>{:js=>{:head=>'assets/js/head/app.js'}}}
+      app.route do |r|
+        r.assets
+        r.is 'test' do
+          "#{assets([:css, :assets, :css])}\n#{assets([:js, :assets, :js, :head])}"
+        end
+      end
+      html = body('/test')
+      html.scan(/<link/).length.should == 2
+      html =~ %r{href="(/assets/css/app\.scss)"}
+      css = body($1)
+      html =~ %r{href="(/assets/css/raw\.css)"}
+      css2 = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/js/head/app\.js)"}
+      js = body($1)
+      css.should =~ /color: red;/
+      css2.should =~ /color: blue;/
+      js.should include('console.log')
+    end
+
+    it 'should handle compiling assets, linking to them, and accepting requests for them' do
+      app.compile_assets
+      html = body('/test')
+      html.scan(/<link/).length.should == 1
+      html =~ %r{href="(/assets/app\.[a-f0-9]{40}\.css)"}
+      css = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      css.should =~ /color: ?red/
+      css.should =~ /color: ?blue/
+      js.should include('console.log')
+    end
+
+    it 'should handle compiling assets, linking to them, and accepting requests for them, with different options' do
+      app.plugin :assets, :compiled_path=>nil, :js_dir=>'assets/js', :css_dir=>'assets/css', :prefix=>'a',
+        :public=>'spec/assets', :path=>'spec', :compiled_js_route=>'foo', :compiled_css_route=>'bar'
+      app.compile_assets
+      html = body('/test')
+      html.scan(/<link/).length.should == 1
+      html =~ %r{href="(/a/bar/app\.[a-f0-9]{40}\.css)"}
+      css = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/a/foo/app\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      css.should =~ /color: ?red/
+      css.should =~ /color: ?blue/
+      js.should include('console.log')
+    end
+
+    it 'should handle rendering assets, linking to them, and accepting requests for them when not compiling with a multi-level hash' do
+      app.plugin :assets, :path=>'spec', :js_dir=>nil, :css_dir=>nil, :compiled_js_dir=>nil, :compiled_css_dir=>nil,
+        :css=>{:assets=>{:css=>%w'app.scss raw.css'}}, :js=>{:assets=>{:js=>{:head=>'app.js'}}}
+      app.compile_assets
+      app.route do |r|
+        r.assets
+        r.is 'test' do
+          "#{assets([:css, :assets, :css])}\n#{assets([:js, :assets, :js, :head])}"
+        end
+      end
+      html = body('/test')
+      html.scan(/<link/).length.should == 1
+      html =~ %r{href="(/assets/app\.assets\.css\.[a-f0-9]{40}\.css)"}
+      css = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.assets\.js\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      css.should =~ /color: ?red/
+      css.should =~ /color: ?blue/
+      js.should include('console.log')
+    end
+
+    it 'should handle :group_subdirs => false when compiling' do
+      app.plugin :assets, :path=>'spec', :js_dir=>nil, :css_dir=>nil, :compiled_js_dir=>nil, :compiled_css_dir=>nil, :group_subdirs=>false,
+        :css=>{:assets=>{:css=>%w'assets/css/app.scss assets/css/raw.css'}}, :js=>{:assets=>{:js=>{:head=>'assets/js/head/app.js'}}}
+      app.compile_assets
+      app.route do |r|
+        r.assets
+        r.is 'test' do
+          "#{assets([:css, :assets, :css])}\n#{assets([:js, :assets, :js, :head])}"
+        end
+      end
+      html = body('/test')
+      html.scan(/<link/).length.should == 1
+      html =~ %r{href="(/assets/app\.assets\.css\.[a-f0-9]{40}\.css)"}
+      css = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.assets\.js\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      css.should =~ /color: ?red/
+      css.should =~ /color: ?blue/
+      js.should include('console.log')
+    end
+
+    it 'should handle automatically creating directories when compiling assets' do
+      app.plugin :assets, :public=>'spec/public'
+      app.compile_assets
+      html = body('/test')
+      html.scan(/<link/).length.should == 1
+      html =~ %r{href="(/assets/app\.[a-f0-9]{40}\.css)"}
+      css = body($1)
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      css.should =~ /color: ?red/
+      css.should =~ /color: ?blue/
+      js.should include('console.log')
+    end
+
+    it 'should handle compiling only css assets' do
+      app.compile_assets(:css)
+      html = body('/test')
+      html.scan(/<link/).length.should == 1
+      html =~ %r{href="(/assets/app\.[a-f0-9]{40}\.css)"}
+      css = body($1)
+      html.scan(/<script/).length.should == 0
+      css.should =~ /color: ?red/
+      css.should =~ /color: ?blue/
+    end
+
+    it 'should handle compiling only js assets' do
+      app.compile_assets(:js)
+      html = body('/test')
+      html.scan(/<link/).length.should == 0
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      js.should include('console.log')
+    end
+
+    it 'should handle compiling asset subfolders' do
+      app.compile_assets([:js, :head])
+      html = body('/test')
+      html.scan(/<link/).length.should == 0
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      js.should include('console.log')
+    end
+
+    it 'should handle compiling assets when only a single asset type is present' do
+      app.plugin :assets, :css=>nil
+      app.compile_assets
+      html = body('/test')
+      html.scan(/<link/).length.should == 0
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      js.should include('console.log')
+    end
+
+    it 'should handle compiling assets when an empty array is used' do
+      app.plugin :assets, :css=>[]
+      app.compile_assets
+      html = body('/test')
+      html.scan(/<link/).length.should == 0
+      html.scan(/<script/).length.should == 1
+      html =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+      js = body($1)
+      js.should include('console.log')
+    end
+
+    it '#assets should include attributes given' do
+      app.new.assets([:js, :head], 'a'=>'b').should == '<script type="text/javascript" a="b" src="/assets/js/head/app.js"></script>'
+    end
+
+    it '#assets should escape attribute values given' do
+      app.new.assets([:js, :head], 'a'=>'b"e').should == '<script type="text/javascript" a="b&quot;e" src="/assets/js/head/app.js"></script>'
+    end
+
+    it 'requests for assets should return 304 if the asset has not been modified' do
+      loc = '/assets/js/head/app.js'
+      lm = header('Last-Modified', loc)
+      status(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should == 304
+      body(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should == ''
+    end
+
+    it 'requests for assets should not return 304 if the asset has been modified' do
+      loc = '/assets/js/head/app.js'
+      lm = header('Last-Modified', loc)
+      File.utime(js_atime, js_mtime+1, js_file)
+      status(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should == 200
+      body(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should include('console.log')
+    end
+
+    it 'requests for assets should return 304 if the dependency of an asset has not been modified' do
+      app.plugin :assets, :dependencies=>{js_file=>css_file}
+      loc = '/assets/js/head/app.js'
+      lm = header('Last-Modified', loc)
+      status(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should == 304
+      body(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should == ''
+    end
+
+    it 'requests for assets should return 200 if the dependency of an asset has been modified' do
+      app.plugin :assets, :dependencies=>{js_file=>css_file}
+      loc = '/assets/js/head/app.js'
+      lm = header('Last-Modified', loc)
+      File.utime(css_atime, [css_mtime+1, js_mtime+1].max, css_file)
+      status(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should == 200
+      body(loc, 'HTTP_IF_MODIFIED_SINCE'=>lm).should include('console.log')
+    end
+
+    it 'should do a terminal match for assets' do
+      status('/assets/css/app.scss/foo').should == 404
+    end
+
+    it 'should only allow files that you specify' do
+      status('/assets/css/no_access.css').should == 404
+    end
+
+    it 'should not add routes for empty asset types' do
+      app.plugin :assets, :css=>nil
+      a = app::RodaRequest.assets_matchers
+      a.length.should == 1
+      a.first.length.should == 2
+      a.first.first.should == 'js'
+      'assets/js/head/app.js'.should =~ a.first.last
+      'assets/js/head/app2.js'.should_not =~ a.first.last
+    end
+
+    it 'should not add routes if no asset types' do
+      app.plugin :assets, :js=>nil, :css=>nil
+      app::RodaRequest.assets_matchers.should == []
+    end
+
+    it 'should support :precompiled option' do
+      app.plugin :assets, :precompiled=>metadata_file
+      File.exist?(metadata_file).should == false
+      app.new.assets([:js, :head]).should == '<script type="text/javascript"  src="/assets/js/head/app.js"></script>'
+
+      app.compile_assets
+      File.exist?(metadata_file).should == true
+      app.new.assets([:js, :head]).should =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+
+      app.plugin :assets, :compiled=>false, :precompiled=>false
+      app.new.assets([:js, :head]).should == '<script type="text/javascript"  src="/assets/js/head/app.js"></script>'
+
+      app.plugin :assets, :precompiled=>metadata_file
+      app.new.assets([:js, :head]).should =~ %r{src="(/assets/app\.head\.[a-f0-9]{40}\.js)"}
+    end
+  end
+end