roda 0.9.0

data/spec/module_spec.rb

@@ -0,0 +1,29 @@
+require File.expand_path("spec_helper", File.dirname(__FILE__))
+
+describe "Roda.request_module and .response_module" do
+  it "should include given module in request or response class" do
+    app(:bare) do
+      request_module(Module.new{def h; halt response.finish end})
+      response_module(Module.new{def finish; [1, {}, []] end})
+
+      route do |r|
+        r.h
+      end
+    end
+
+    req.should == [1, {}, []]
+  end
+
+  it "should accept blocks and turn them into modules" do
+    app(:bare) do
+      request_module{def h; halt response.finish end}
+      response_module{def finish; [1, {}, []] end}
+
+      route do |r|
+        r.h
+      end
+    end
+
+    req.should == [1, {}, []]
+  end
+end

data/spec/opts_spec.rb

@@ -0,0 +1,42 @@
+require File.expand_path("spec_helper", File.dirname(__FILE__))
+
+describe "opts" do
+  it "is inheritable and allows overriding" do
+    c = Class.new(Roda)
+    c.opts[:foo] = "bar"
+    c.opts[:foo].should == "bar"
+
+    sc = Class.new(c)
+    sc.opts[:foo].should == "bar"
+
+    sc.opts[:foo] = "baz"
+    sc.opts[:foo].should == "baz"
+    c.opts[:foo].should == "bar"
+  end
+
+  it "should be available as an instance methods" do
+    app(:bare) do
+      opts[:hello] = "Hello World"
+
+      route do |r|
+        r.on do
+          opts[:hello]
+        end
+      end
+    end
+
+    body.should == "Hello World"
+  end
+
+  it "should only shallow clone by default" do
+    c = Class.new(Roda)
+    c.opts[:foo] = "bar"
+    c.opts[:foo].should == "bar"
+
+    sc = Class.new(c)
+    sc.opts[:foo].replace("baz")
+
+    sc.opts[:foo].should == "baz"
+    c.opts[:foo].should == "baz"
+  end
+end

data/spec/plugin/all_verbs_spec.rb

@@ -0,0 +1,29 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "all_verbs plugin" do 
+  it "adds method for each http verb" do
+    app(:all_verbs) do |r|
+      r.delete{'d'}
+      r.head{'h'}
+      r.options{'o'}
+      r.patch{'pa'}
+      r.put{'pu'}
+      r.trace{'t'}
+      if Rack::Request.method_defined?(:link?)
+        r.link{'l'}
+        r.unlink{'u'}
+      end
+    end
+
+    body('REQUEST_METHOD'=>'DELETE').should == 'd'
+    body('REQUEST_METHOD'=>'HEAD').should == 'h'
+    body('REQUEST_METHOD'=>'OPTIONS').should == 'o'
+    body('REQUEST_METHOD'=>'PATCH').should == 'pa'
+    body('REQUEST_METHOD'=>'PUT').should == 'pu'
+    body('REQUEST_METHOD'=>'TRACE').should == 't'
+    if Rack::Request.method_defined?(:link?)
+      body('REQUEST_METHOD'=>'LINK').should == 'l'
+      body('REQUEST_METHOD'=>'UNLINK').should == 'u'
+    end
+  end
+end

data/spec/plugin/default_headers_spec.rb

@@ -0,0 +1,63 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "default_headers plugin" do 
+  it "sets the default headers to use for the response" do
+    h = {'Content-Type'=>'text/json', 'Foo'=>'bar'}
+
+    app(:bare) do
+      plugin :default_headers, h
+      route do |r|
+      end
+    end
+
+    req[1].should == h
+    req[1].should_not equal(h)
+  end
+
+  it "should not override existing default headers" do
+    h = {'Content-Type'=>'text/json', 'Foo'=>'bar'}
+
+    app(:bare) do
+      plugin :default_headers, h
+      plugin :default_headers
+
+      route do |r|
+      end
+    end
+
+    req[1].should == h
+  end
+
+  it "should allow modifying the default headers at a later point" do
+    h = {'Content-Type'=>'text/json', 'Foo'=>'bar'}
+
+    app(:bare) do
+      plugin :default_headers
+      default_headers['Content-Type'] = 'text/json'
+      default_headers['Foo'] = 'bar'
+
+      route do |r|
+      end
+    end
+
+    req[1].should == h
+  end
+
+  it "should work correctly in subclasses" do
+    h = {'Content-Type'=>'text/json', 'Foo'=>'bar'}
+
+    app(:bare) do
+      plugin :default_headers
+      default_headers['Content-Type'] = 'text/json'
+      default_headers['Foo'] = 'bar'
+
+      route do |r|
+      end
+    end
+
+    @app = Class.new(@app)
+    @app.route{}
+
+    req[1].should == h
+  end
+end

data/spec/plugin/error_handler_spec.rb

@@ -0,0 +1,67 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "error_handler plugin" do 
+  it "executes only if error raised" do
+    app(:bare) do
+      plugin :error_handler
+
+      error do |e|
+        e.message
+      end
+
+      route do |r|
+        r.on "a" do
+          "found"
+        end
+
+        raise ArgumentError, "bad idea"
+      end
+    end
+
+    body("/a").should == 'found'
+    status("/a").should == 200
+    body.should == 'bad idea'
+    status.should == 500
+  end
+
+  it "can override status inside error block" do
+    app(:bare) do
+      plugin :error_handler do |e|
+        response.status = 501
+        e.message
+      end
+
+      route do |r|
+        raise ArgumentError, "bad idea"
+      end
+    end
+
+    status.should == 501
+  end
+
+  it "can set error via the plugin block" do
+    app(:bare) do
+      plugin :error_handler do |e|
+        e.message
+      end
+
+      route do |r|
+        raise ArgumentError, "bad idea"
+      end
+    end
+
+    body.should == 'bad idea'
+  end
+
+  it "has default error handler also raise" do
+    app(:bare) do
+      plugin :error_handler
+
+      route do |r|
+        raise ArgumentError, "bad idea"
+      end
+    end
+
+    proc{req}.should raise_error(ArgumentError)
+  end
+end

data/spec/plugin/flash_spec.rb

@@ -0,0 +1,59 @@
+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
+      use Rack::Session::Cookie, :secret => "1"
+      plugin :flash
+
+      route do |r|
+        r.on do
+          flash.now['a'] = 'b'
+          flash['a']
+        end
+      end
+    end
+
+    body.should == 'b'
+  end
+
+  it "flash[] sets flash for next page" do
+    app(:bare) do
+      use Rack::Session::Cookie, :secret => "1"
+      plugin :flash
+
+      route do |r|
+        r.on 'a' do
+          "c#{flash['a']}"
+        end
+
+        r.on do
+          flash['a'] = "b#{flash['a']}"
+          flash['a'] || ''
+        end
+      end
+    end
+
+    env = proc{|h| h['Set-Cookie'] ? {'HTTP_COOKIE'=>h['Set-Cookie'].sub("; path=/; HttpOnly", '')} : {}}
+    _, h, b = req
+    b.join.should == ''
+    _, h, b = req(env[h])
+    b.join.should == 'b'
+    _, h, b = req(env[h])
+    b.join.should == 'bb'
+    _, h, b = req('/a', env[h])
+    b.join.should == 'cbbb'
+    _, h, b = req(env[h])
+    b.join.should == ''
+    _, h, b = req(env[h])
+    b.join.should == 'b'
+    _, h, b = req(env[h])
+    b.join.should == 'bb'
+  end
+end
+end

data/spec/plugin/h_spec.rb

@@ -0,0 +1,13 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "h plugin" do 
+  it "adds h method for html escaping" do
+    app(:h) do |r|
+      r.on do
+        h("<form>") + h(:form)
+      end
+    end
+
+    body.should == '&lt;form&gt;form'
+  end
+end

data/spec/plugin/halt_spec.rb

@@ -0,0 +1,62 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "halt plugin" do
+  it "should still have halt return rack response as argument given it as argument" do
+    app(:halt) do |r|
+      r.halt [200, {}, ['foo']]
+    end
+
+    body.should ==  "foo"
+  end
+
+  it "should consider string argument as response body" do
+    app(:halt) do |r|
+      r.halt "foo"
+    end
+
+    body.should ==  "foo"
+  end
+
+  it "should consider integer argument as response status" do
+    app(:halt) do |r|
+      r.halt 300
+    end
+
+    status.should == 300 
+  end
+
+  it "should consider 2 arguments as response status and body" do
+    app(:halt) do |r|
+      r.halt 300, "foo"
+    end
+
+    status.should == 300 
+    body.should == "foo"
+  end
+
+  it "should consider 3 arguments as response" do
+    app(:halt) do |r|
+      r.halt 300, {'a'=>'b'}, "foo"
+    end
+
+    status.should == 300 
+    header('a').should == 'b'
+    body.should == "foo"
+  end
+
+  it "should raise an error for too many arguments" do
+    app(:halt) do |r|
+      r.halt 300, {'a'=>'b'}, "foo", 1
+    end
+
+    proc{req}.should raise_error(Roda::RodaError)
+  end
+
+  it "should raise an error for single argument not integer, String, or Array" do
+    app(:halt) do |r|
+      r.halt('a'=>'b')
+    end
+
+    proc{req}.should raise_error(Roda::RodaError)
+  end
+end

data/spec/plugin/header_matchers_spec.rb

@@ -0,0 +1,61 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "accept matcher" do
+  it "should accept mimetypes and set response Content-Type" do
+    app(:header_matchers) do |r|
+      r.on :accept=>"application/xml" do
+        response["Content-Type"]
+      end
+    end
+
+    body("HTTP_ACCEPT" => "application/xml").should ==  "application/xml"
+    status.should == 404
+  end
+end
+
+describe "header matcher" do
+  it "should match if header present" do
+    app(:header_matchers) do |r|
+      r.on :header=>"http-accept" do
+        "bar"
+      end
+    end
+
+    body("HTTP_ACCEPT" => "application/xml").should ==  "bar"
+    status.should == 404
+  end
+end
+
+describe "host matcher" do
+  it "should match a host" do
+    app(:header_matchers) do |r|
+      r.on :host=>"example.com" do
+        "worked"
+      end
+    end
+
+    body("HTTP_HOST" => "example.com").should == 'worked'
+    status("HTTP_HOST" => "foo.com").should == 404
+  end
+
+  it "should match a host with a regexp" do
+    app(:header_matchers) do |r|
+      r.on :host=>/example/ do
+        "worked"
+      end
+    end
+
+    body("HTTP_HOST" => "example.com").should == 'worked'
+    status("HTTP_HOST" => "foo.com").should == 404
+  end
+
+  it "doesn't yield HOST" do
+    app(:header_matchers) do |r|
+      r.on :host=>"example.com" do |*args|
+        args.size.to_s
+      end
+    end
+
+    body("HTTP_HOST" => "example.com").should == '0'
+  end
+end

data/spec/plugin/hooks_spec.rb

@@ -0,0 +1,97 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "hooks plugin" do 
+  before do
+    a = @a = []
+    app(:bare) do
+      plugin :hooks
+
+      before do
+        response['foo'] = 'bar'
+      end
+
+      after do |r|
+        if r
+          a << [r[0], r[1]['foo'], r[2]]
+          r[0] += 1
+        else
+          a << r
+        end
+      end
+
+      route do |r|
+        r.is "" do
+          f = response['foo']
+          response['foo'] = 'baz'
+          f
+        end
+      end
+    end
+  end
+
+  it "adds before and after hooks for running code before and after requests" do
+    s, h, b = req
+    s.should == 201
+    h['foo'].should == 'baz'
+    b.join.should == 'bar'
+    @a.should == [[200, 'baz', ['bar']]]
+  end
+
+  it "multiple plugin calls do not override existing hooks" do
+    app.plugin :hooks
+    s, h, b = req
+    s.should == 201
+    h['foo'].should == 'baz'
+    b.join.should == 'bar'
+    @a.should == [[200, 'baz', ['bar']]]
+  end
+
+  it "after hooks are still called if an exception is raised" do
+    a = @a
+    @app.before do
+      raise Roda::RodaError, "foo"
+    end
+
+    @app.after do |r|
+      a << r
+      a << $!
+    end
+
+    proc{req}.should raise_error(Roda::RodaError)
+    a.pop.should be_a_kind_of(Roda::RodaError)
+    a.pop.should == nil
+  end
+
+  it "handles multiple before and after blocks correctly" do
+    a = @a
+    @app.before do
+      response['bar'] = "foo#{response['foo']}"
+    end
+
+    @app.after do |r|
+      a << r[1]['bar']
+      r[0] *= 2
+    end
+
+    s, h, b = req
+    s.should == 402
+    h['foo'].should == 'baz'
+    h['bar'].should == 'foo'
+    b.join.should == 'bar'
+    a.should == [[200, 'baz', ['bar']], 'foo']
+  end
+
+  it "copies before and after blocks when subclassing" do
+    @app = Class.new(@app)
+    @app.route do |r|
+      r.on do
+        "foo"
+      end
+    end
+    s, h, b = req
+    s.should == 201
+    h['foo'].should == 'bar'
+    b.join.should == 'foo'
+    @a.should == [[200, 'bar', ['foo']]]
+  end
+end