roda 0.9.0 → 1.0.0

data/spec/plugin/backtracking_array_spec.rb

@@ -0,0 +1,38 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "backtracking_array plugin" do 
+  it "backtracks to next entry in array if later matcher fails" do
+    app(:backtracking_array) do |r|
+      r.is %w'a a/b' do |id|
+        id
+      end
+
+      r.is %w'c c/d', %w'd e' do |a, b|
+        "#{a}-#{b}" 
+      end
+
+      r.is [%w'f f/g', %w'g g/h'] do |id|
+        id
+      end
+    end
+
+    status.should == 404
+
+    body("/a").should == 'a'
+    body("/a/b").should == 'a/b'
+    status("/a/b/").should == 404
+
+    body("/c/d").should == 'c-d'
+    body("/c/e").should == 'c-e'
+    body("/c/d/d").should == 'c/d-d'
+    body("/c/d/e").should == 'c/d-e'
+    status("/c/d/").should == 404
+
+    body("/f").should == 'f'
+    body("/f/g").should == 'f/g'
+    body("/g").should == 'g'
+    body("/g/h").should == 'g/h'
+    status("/f/g/").should == 404
+    status("/g/h/").should == 404
+  end
+end

data/spec/plugin/content_for_spec.rb

@@ -0,0 +1,34 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+begin
+  require 'tilt/erb'
+rescue LoadError
+  warn "tilt not installed, skipping content_for plugin test"  
+else
+describe "content_for plugin" do
+  before do
+    app(:bare) do
+      plugin :render
+      render_opts[:views] = "./spec/views"
+      plugin :content_for
+
+      route do |r|
+        r.root do
+          view(:inline=>"<% content_for :foo do %>foo<% end %>bar", :layout=>{:inline=>'<%= yield %> <%= content_for(:foo) %>'})
+        end
+        r.get 'a' do
+          view(:inline=>"bar", :layout=>{:inline=>'<%= content_for(:foo) %> <%= yield %>'})
+        end
+      end
+    end
+  end
+
+  it "should be able to set content in template and get that content in the layout" do
+    body.strip.should == "bar foo"
+  end
+
+  it "should work if content is not set by the template" do
+    body('/a').strip.should == "bar"
+  end
+end
+end

data/spec/plugin/csrf_spec.rb

@@ -0,0 +1,49 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+begin
+  require 'rack/csrf'
+rescue LoadError
+  warn "rack_csrf not installed, skipping csrf plugin test"  
+else
+describe "csrf plugin" do 
+  it "adds csrf protection and csrf helper methods" do
+    app(:bare) do
+      use Rack::Session::Cookie, :secret=>'1'
+      plugin :csrf, :skip=>['POST:/foo']
+
+      route do |r|
+        r.get do
+          response['TAG'] = csrf_tag
+          response['METATAG'] = csrf_metatag
+          response['TOKEN'] = csrf_token
+          response['FIELD'] = csrf_field
+          response['HEADER'] = csrf_header
+          'g'
+        end
+        r.post 'foo' do
+          'bar'
+        end
+        r.post do
+          'p'
+        end
+      end
+    end
+
+    io = StringIO.new
+    status('REQUEST_METHOD'=>'POST', 'rack.input'=>io).should == 403
+    body('/foo', 'REQUEST_METHOD'=>'POST', 'rack.input'=>io).should == 'bar'
+
+    env = proc{|h| h['Set-Cookie'] ? {'HTTP_COOKIE'=>h['Set-Cookie'].sub("; path=/; HttpOnly", '')} : {}}
+    s, h, b = req
+    s.should == 200
+    field = h['FIELD']
+    token = Regexp.escape(h['TOKEN'])
+    h['TAG'].should =~ /\A<input type="hidden" name="#{field}" value="#{token}" \/>\z/
+    h['METATAG'].should =~ /\A<meta name="#{field}" content="#{token}" \/>\z/
+    b.should == ['g']
+    s, _, b = req('/', env[h].merge('REQUEST_METHOD'=>'POST', 'rack.input'=>io, "HTTP_#{h['HEADER']}"=>h['TOKEN']))
+    s.should == 200
+    b.should == ['p']
+  end
+end
+end

data/spec/plugin/flash_spec.rb

@@ -1,10 +1,5 @@
 require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
 
-begin
-  require 'sinatra/flash/hash'
-rescue LoadError
-  warn "sinatra-flash not installed, skipping flash plugin test"  
-else
 describe "flash plugin" do 
   it "flash.now[] sets flash for current page" do
     app(:bare) do
@@ -56,4 +51,73 @@ describe "flash plugin" do
     b.join.should == 'bb'
   end
 end
+
+describe "FlashHash" do 
+  before do
+    @h = Roda::RodaPlugins::Flash::FlashHash.new
+  end
+
+  it ".new should accept nil for empty hash" do
+    @h = Roda::RodaPlugins::Flash::FlashHash.new(nil)
+    @h.now.should == {}
+    @h.next.should == {}
+  end
+
+  it ".new should accept a hash" do
+    @h = Roda::RodaPlugins::Flash::FlashHash.new(1=>2)
+    @h.now.should == {1=>2}
+    @h.next.should == {}
+  end
+
+  it "#[]= assigns to next flash" do
+    @h[1] = 2
+    @h.now.should == {}
+    @h.next.should == {1=>2}
+  end
+
+  it "#discard removes given key from next hash" do
+    @h[1] = 2
+    @h[nil] = 3
+    @h.next.should == {1=>2, nil=>3}
+    @h.discard(nil)
+    @h.next.should == {1=>2}
+    @h.discard(1)
+    @h.next.should == {}
+  end
+
+  it "#discard removes all entries from next hash with no arguments" do
+    @h[1] = 2
+    @h[nil] = 3
+    @h.next.should == {1=>2, nil=>3}
+    @h.discard
+    @h.next.should == {}
+  end
+
+  it "#keep copies entry for key from current hash to next hash" do
+    @h.now[1] = 2
+    @h.now[nil] = 3
+    @h.next.should == {}
+    @h.keep(nil)
+    @h.next.should == {nil=>3}
+    @h.keep(1)
+    @h.next.should == {1=>2, nil=>3}
+  end
+
+  it "#keep copies all entries from current hash to next hash" do
+    @h.now[1] = 2
+    @h.now[nil] = 3
+    @h.next.should == {}
+    @h.keep
+    @h.next.should == {1=>2, nil=>3}
+  end
+
+  it "#sweep replaces current hash with next hash" do
+    @h[1] = 2
+    @h[nil] = 3
+    @h.next.should == {1=>2, nil=>3}
+    @h.now.should == {}
+    @h.sweep.should == {1=>2, nil=>3}
+    @h.next.should == {}
+    @h.now.should == {1=>2, nil=>3}
+  end
 end

data/spec/plugin/head_spec.rb

@@ -0,0 +1,35 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "head plugin" do 
+  it "considers HEAD requests as GET requests which return no body" do
+    app(:head) do |r|
+      r.root do
+        'root'
+      end
+
+      r.get 'a' do
+        'a'
+      end
+
+      r.is 'b', :method=>[:get, :post] do
+        'b'
+      end
+    end
+
+    s, h, b = req
+    s.should == 200
+    h['Content-Length'].should == '4'
+    b.should == ['root']
+
+    s, h, b = req('REQUEST_METHOD' => 'HEAD')
+    s.should == 200
+    h['Content-Length'].should == '4'
+    b.should == []
+
+    body('/a').should == 'a'
+    status('/a', 'REQUEST_METHOD' => 'HEAD').should == 200
+
+    body('/b').should == 'b'
+    status('/b', 'REQUEST_METHOD' => 'HEAD').should == 200
+  end
+end

data/spec/plugin/json_spec.rb

@@ -0,0 +1,50 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "json plugin" do
+  before do
+    c = Class.new do
+      def to_json
+        '[1]'
+      end
+    end
+
+    app(:bare) do
+      plugin :json
+      json_result_classes << c
+
+      route do |r|
+        r.is 'array' do
+          [1, 2, 3]
+        end
+
+        r.is "hash" do
+          {'a'=>'b'}
+        end
+
+        r.is 'c' do
+          c.new
+        end
+      end
+    end
+  end
+
+  it "should use a json content type for a json response" do
+    header('Content-Type', "/array").should == 'application/json'
+    header('Content-Type', "/hash").should == 'application/json'
+    header('Content-Type', "/c").should == 'application/json'
+    header('Content-Type').should == 'text/html'
+  end
+
+  it "should convert objects to json" do
+    body('/array').gsub(/\s/, '').should == '[1,2,3]'
+    body('/hash').gsub(/\s/, '').should == '{"a":"b"}'
+    body('/c').should == '[1]'
+    body.should == ''
+  end
+
+  it "should work when subclassing" do
+    @app = Class.new(app)
+    app.route{[1]}
+    body.should == '[1]'
+  end
+end

data/spec/plugin/multi_route_spec.rb

@@ -5,7 +5,7 @@ describe "multi_route plugin" do
     app(:bare) do
       plugin :multi_route
 
-      route(:get) do |r|
+      route("get") do |r|
         r.is "" do
           "get"
         end
@@ -15,7 +15,7 @@ describe "multi_route plugin" do
         end
       end
 
-      route(:post) do |r|
+      route("post") do |r|
         r.is "" do
           "post"
         end
@@ -26,15 +26,22 @@ describe "multi_route plugin" do
       end
 
       route do |r|
+        r.on 'foo' do
+          r.multi_route do
+            "foo"
+          end
+        end
+
         r.get do
-          route(:get)
+          r.route("get")
 
           r.is "b" do
             "getb"
           end
         end
+
         r.post do
-          route(:post)
+          r.route("post")
 
           r.is "b" do
             "postb"
@@ -55,6 +62,15 @@ describe "multi_route plugin" do
     status('/c', 'REQUEST_METHOD'=>'POST').should == 404
   end
 
+  it "uses multi_route to dispatch to any named route" do
+    status('/foo').should == 404
+    body('/foo/get/').should == 'get'
+    body('/foo/get/a').should == 'geta'
+    body('/foo/post/').should == 'post'
+    body('/foo/post/a').should == 'posta'
+    body('/foo/post/b').should == 'foo'
+  end
+
   it "handles loading the plugin multiple times correctly" do
     app.plugin :multi_route
     body.should == 'get'
@@ -71,14 +87,14 @@ describe "multi_route plugin" do
     @app = Class.new(@app)
     @app.route do |r|
       r.get do
-        route(:post)
+        r.route("post")
 
         r.is "b" do
           "1b"
         end
       end
       r.post do
-        route(:get)
+        r.route("get")
 
         r.is "b" do
           "2b"

data/spec/plugin/not_allowed_spec.rb

@@ -0,0 +1,55 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "not_allowed plugin" do 
+  it "skips the current block if pass is called" do
+    app(:not_allowed) do |r|
+      r.get '' do
+        'a'
+      end
+
+      r.is "c" do
+        r.get do
+          "cg"
+        end
+
+        r.post do
+          "cp"
+        end
+
+        "c"
+      end
+
+      r.get do
+        r.is 'b' do
+          'b'
+        end
+        r.is /(d)/ do |s|
+          s
+        end
+        r.get /(e)/ do |s|
+          s
+        end
+      end
+    end
+
+    body.should == 'a'
+    status('REQUEST_METHOD'=>'POST').should == 405
+    header('Allow', 'REQUEST_METHOD'=>'POST').should == 'GET'
+
+    body('/b').should == 'b'
+    status('/b', 'REQUEST_METHOD'=>'POST').should == 404
+
+    body('/d').should == 'd'
+    status('/d', 'REQUEST_METHOD'=>'POST').should == 404
+
+    body('/e').should == 'e'
+    status('/d', 'REQUEST_METHOD'=>'POST').should == 404
+
+    body('/c').should == 'cg'
+    body('/c').should == 'cg'
+    body('/c', 'REQUEST_METHOD'=>'POST').should == 'cp'
+    body('/c', 'REQUEST_METHOD'=>'PATCH').should == 'c'
+    status('/c', 'REQUEST_METHOD'=>'PATCH').should == 405
+    header('Allow', '/c', 'REQUEST_METHOD'=>'PATCH').should == 'GET, POST'
+  end
+end

data/spec/plugin/pass_spec.rb

@@ -1,8 +1,13 @@
 require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
 
 describe "pass plugin" do 
-  it "executes on no arguments" do
+  it "skips the current block if pass is called" do
     app(:pass) do |r|
+      r.root do
+        r.pass if env['FOO']
+        'root'
+      end
+
       r.on :id do |id|
         r.pass if id == 'foo'
         id
@@ -13,11 +18,12 @@ describe "pass plugin" do
       end
     end
 
+    body.should == 'root'
+    status('FOO'=>true).should == 404
     body("/a").should == 'a'
     body("/a/b").should == 'a'
     body("/foo/a").should == 'fooa'
     body("/foo/a/b").should == 'fooa'
     status("/foo").should == 404
-    status.should == 404
   end
 end