roda-cj 0.9.1

data/spec/plugin/multi_route_spec.rb

@@ -0,0 +1,98 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "multi_route plugin" do 
+  before do
+    app(:bare) do
+      plugin :multi_route
+
+      route(:get) do |r|
+        r.is "" do
+          "get"
+        end
+        
+        r.is "a" do
+          "geta"
+        end
+      end
+
+      route(:post) do |r|
+        r.is "" do
+          "post"
+        end
+        
+        r.is "a" do
+          "posta"
+        end
+      end
+
+      route do |r|
+        r.get do
+          route(:get)
+
+          r.is "b" do
+            "getb"
+          end
+        end
+        r.post do
+          route(:post)
+
+          r.is "b" do
+            "postb"
+          end
+        end
+      end
+    end
+  end
+
+  it "adds named routing support" do
+    body.should == 'get'
+    body('REQUEST_METHOD'=>'POST').should == 'post'
+    body('/a').should == 'geta'
+    body('/a', 'REQUEST_METHOD'=>'POST').should == 'posta'
+    body('/b').should == 'getb'
+    body('/b', 'REQUEST_METHOD'=>'POST').should == 'postb'
+    status('/c').should == 404
+    status('/c', 'REQUEST_METHOD'=>'POST').should == 404
+  end
+
+  it "handles loading the plugin multiple times correctly" do
+    app.plugin :multi_route
+    body.should == 'get'
+    body('REQUEST_METHOD'=>'POST').should == 'post'
+    body('/a').should == 'geta'
+    body('/a', 'REQUEST_METHOD'=>'POST').should == 'posta'
+    body('/b').should == 'getb'
+    body('/b', 'REQUEST_METHOD'=>'POST').should == 'postb'
+    status('/c').should == 404
+    status('/c', 'REQUEST_METHOD'=>'POST').should == 404
+  end
+
+  it "handles subclassing correctly" do
+    @app = Class.new(@app)
+    @app.route do |r|
+      r.get do
+        route(:post)
+
+        r.is "b" do
+          "1b"
+        end
+      end
+      r.post do
+        route(:get)
+
+        r.is "b" do
+          "2b"
+        end
+      end
+    end
+
+    body.should == 'post'
+    body('REQUEST_METHOD'=>'POST').should == 'get'
+    body('/a').should == 'posta'
+    body('/a', 'REQUEST_METHOD'=>'POST').should == 'geta'
+    body('/b').should == '1b'
+    body('/b', 'REQUEST_METHOD'=>'POST').should == '2b'
+    status('/c').should == 404
+    status('/c', 'REQUEST_METHOD'=>'POST').should == 404
+  end
+end

data/spec/plugin/not_found_spec.rb

@@ -0,0 +1,99 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "not_found plugin" do 
+  it "executes on no arguments" do
+    app(:bare) do
+      plugin :not_found
+
+      not_found do
+        "not found"
+      end
+
+      route do |r|
+        r.on "a" do
+          "found"
+        end
+      end
+    end
+
+    body.should == 'not found'
+    status.should == 404
+    body("/a").should == 'found'
+    status("/a").should == 200
+  end
+
+  it "allows overriding status inside not_found" do
+    app(:bare) do
+      plugin :not_found
+
+      not_found do
+        response.status = 403
+        "not found"
+      end
+
+      route do |r|
+      end
+    end
+
+    status.should == 403
+  end
+
+  it "does not modify behavior if not_found is not called" do
+    app(:not_found) do |r|
+      r.on "a" do
+        "found"
+      end
+    end
+
+    body.should == ''
+    body("/a").should == 'found'
+  end
+
+  it "can set not_found via the plugin block" do
+    app(:bare) do
+      plugin :not_found do
+        "not found"
+      end
+
+      route do |r|
+        r.on "a" do
+          "found"
+        end
+      end
+    end
+
+    body.should == 'not found'
+    body("/a").should == 'found'
+  end
+
+  it "does not modify behavior if body is not an array" do
+    app(:bare) do
+      plugin :not_found do
+        "not found"
+      end
+
+      o = Object.new
+      def o.join() '' end
+      route do |r|
+        r.halt [404, {}, o]
+      end
+    end
+
+    body.should == ''
+  end
+
+  it "does not modify behavior if body is not an empty array" do
+    app(:bare) do
+      plugin :not_found do
+        "not found"
+      end
+
+      route do |r|
+        response.status = 404
+        response.write 'a'
+      end
+    end
+
+    body.should == 'a'
+  end
+end

data/spec/plugin/pass_spec.rb

@@ -0,0 +1,23 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "pass plugin" do 
+  it "executes on no arguments" do
+    app(:pass) do |r|
+      r.on :id do |id|
+        r.pass if id == 'foo'
+        id
+      end
+
+      r.on ":x/:y" do |x, y|
+        x + y
+      end
+    end
+
+    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

data/spec/plugin/render_spec.rb

@@ -0,0 +1,148 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+begin
+  require 'tilt'
+rescue LoadError
+  warn "tilt not installed, skipping render plugin test"  
+else
+describe "render plugin" do
+  before do
+    app(:bare) do
+      plugin :render
+      render_opts[:views] = "./spec/views"
+
+      route do |r|
+        r.on "home" do
+          view("home", :locals=>{:name => "Agent Smith", :title => "Home"}, :layout_opts=>{:locals=>{:title=>"Home"}})
+        end
+
+        r.on "about" do
+          render("about", :locals=>{:title => "About Roda"})
+        end
+
+        r.on "inline" do
+          view(:inline=>"Hello <%= name %>", :locals=>{:name => "Agent Smith"}, :layout=>nil)
+        end
+
+        r.on "path" do
+          render(:path=>"./spec/views/about.erb", :locals=>{:title => "Path"}, :layout_opts=>{:locals=>{:title=>"Home"}})
+        end
+      end
+    end
+  end
+
+  it "default actions" do
+    body("/about").strip.should == "<h1>About Roda</h1>"
+    body("/home").strip.should == "<title>Roda: Home</title>\n<h1>Home</h1>\n<p>Hello Agent Smith</p>"
+    body("/inline").strip.should == "Hello Agent Smith"
+    body("/path").strip.should == "<h1>Path</h1>"
+  end
+
+  it "with str as engine" do
+    app.render_opts[:engine] = "str"
+    body("/about").strip.should == "<h1>About Roda</h1>"
+    body("/home").strip.should == "<title>Roda: Home</title>\n<h1>Home</h1>\n<p>Hello Agent Smith</p>"
+    body("/inline").strip.should == "Hello <%= name %>"
+  end
+
+  it "with str as ext" do
+    app.render_opts[:ext] = "str"
+    body("/about").strip.should == "<h1>About Roda</h1>"
+    body("/home").strip.should == "<title>Roda: Home</title>\n<h1>Home</h1>\n<p>Hello Agent Smith</p>"
+    body("/inline").strip.should == "Hello Agent Smith"
+  end
+
+  it "custom default layout support" do
+    app.render_opts[:layout] = "layout-alternative"
+    body("/home").strip.should == "<title>Alternative Layout: Home</title>\n<h1>Home</h1>\n<p>Hello Agent Smith</p>"
+  end
+end
+
+describe "render plugin" do
+  it "simple layout support" do
+    app(:bare) do
+      plugin :render
+      
+      route do |r|
+        render(:path=>"spec/views/layout-yield.erb") do
+          render(:path=>"spec/views/content-yield.erb")
+        end
+      end
+    end
+
+    body.gsub(/\n+/, "\n").should == "Header\nThis is the actual content.\nFooter\n"
+  end
+
+  it "views without default layouts" do
+    app(:bare) do
+      plugin :render, :views=>"./spec/views", :layout=>false
+      
+      route do |r|
+        view("home", :locals=>{:name=>"Agent Smith", :title=>"Home"})
+      end
+    end
+
+    body.strip.should == "<h1>Home</h1>\n<p>Hello Agent Smith</p>"
+  end
+
+  it "layout overrides" do
+    app(:bare) do
+      plugin :render, :views=>"./spec/views"
+      
+      route do |r|
+        view("home", :locals=>{:name=>"Agent Smith", :title=>"Home" }, :layout=>"layout-alternative", :layout_opts=>{:locals=>{:title=>"Home"}})
+      end
+    end
+
+    body.strip.should == "<title>Alternative Layout: Home</title>\n<h1>Home</h1>\n<p>Hello Agent Smith</p>"
+  end
+
+  it "inline layouts and inline views" do
+    app(:render) do
+      view({:inline=>'bar'}, :layout=>{:inline=>'Foo: <%= yield %>'})
+    end
+
+    body.strip.should == "Foo: bar"
+  end
+
+  it "inline renders with opts" do
+    app(:render) do
+      render({:inline=>'<%= bar %>'}, {:engine=>'str'})
+    end
+
+    body.strip.should == '<%= bar %>'
+  end
+
+  it "render_opts inheritance" do
+    c = Class.new(Roda)
+    c.plugin :render
+    sc = Class.new(c)
+
+    c.render_opts.should_not equal(sc.render_opts)
+    c.render_opts[:layout_opts].should_not equal(sc.render_opts[:layout_opts])
+    c.render_opts[:opts].should_not equal(sc.render_opts[:opts])
+    c.render_opts[:cache].should_not equal(sc.render_opts[:cache])
+  end
+
+  it "render plugin call should not override options" do
+    c = Class.new(Roda)
+    c.plugin :render, :layout=>:foo
+    c.plugin :render
+    c.render_opts[:layout].should == :foo
+  end
+
+  it "with caching disabled" do
+    app(:bare) do
+      plugin :render, :views=>"./spec/views", :cache=>false
+      
+      route do |r|
+        view(:inline=>"Hello <%= name %>: <%= render_opts[:cache] %>", :locals=>{:name => "Agent Smith"}, :layout=>nil)
+      end
+    end
+
+    body("/inline").strip.should == "Hello Agent Smith: false"
+
+    Class.new(app).render_opts[:cache].should == false
+  end
+end
+end

data/spec/plugin/streaming_spec.rb

@@ -0,0 +1,52 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "streaming plugin" do 
+  it "adds stream method for streaming responses" do
+    app(:streaming) do |r|
+      stream do |out|
+        %w'a b c'.each{|v| out << v}
+      end
+    end
+
+    s, h, b = req
+    s.should == 200
+    h.should == {'Content-Type'=>'text/html'}
+    b.to_a.should == %w'a b c'
+  end
+
+  it "should handle errors when streaming, and run callbacks" do
+    a = []
+    app(:streaming) do |r|
+      stream(:callback=>proc{a << 'e'}) do |out|
+        %w'a b'.each{|v| out << v}
+        raise Roda::RodaError, 'foo'
+        out << 'c'
+      end
+    end
+
+    s, h, b = req
+    s.should == 200
+    h.should == {'Content-Type'=>'text/html'}
+    b.callback{a << 'd'}
+    proc{b.each{|v| a << v}}.should raise_error(Roda::RodaError)
+    a.should == %w'a b e d'
+  end
+
+  it "should handle :loop option to loop" do
+    a = []
+    app(:streaming) do |r|
+      b = %w'a b c'
+      stream(:loop=>true, :callback=>proc{a << 'e'}) do |out|
+        out << b.shift
+        raise Roda::RodaError, 'foo' if b.length == 1
+      end
+    end
+
+    s, h, b = req
+    s.should == 200
+    h.should == {'Content-Type'=>'text/html'}
+    b.callback{a << 'd'}
+    proc{b.each{|v| a << v}}.should raise_error(Roda::RodaError)
+    a.should == %w'a b e d'
+  end
+end

data/spec/plugin_spec.rb

@@ -0,0 +1,61 @@
+require File.expand_path("spec_helper", File.dirname(__FILE__))
+
+describe "plugins" do
+  it "should be able to override class, instance, response, and request methods, and execute configure method" do
+    c = Module.new do
+      self::ClassMethods = Module.new do
+        def fix(str)
+          opts[:prefix] + str.strip
+        end
+      end
+      self::InstanceMethods = Module.new do
+        def fix(str)
+          super("a" + str)
+        end
+      end
+      self::RequestMethods = Module.new do
+        def hello(&block)
+          on 'hello', &block
+        end
+      end
+      self::ResponseMethods = Module.new do
+        def foobar
+          "Default   "
+        end
+      end
+
+      def self.load_dependencies(mod, prefix)
+        mod.send(:include, Module.new do
+          def fix(str)
+            self.class.fix(str)
+          end
+        end)
+      end
+
+      def self.configure(mod, prefix)
+        mod.opts[:prefix] = prefix
+      end
+    end
+
+    app(:bare) do
+      plugin c, "Foo "
+
+      route do |r|
+        r.hello do
+          fix(response.foobar)
+        end
+      end
+    end
+
+    body('/hello').should == 'Foo aDefault'
+  end
+
+  it "should support registering plugins and loading them by symbol" do
+    Roda::RodaPlugins.register_plugin(:foo, Module.new{module self::InstanceMethods; def a; '1' end end})
+    app(:foo) do
+      a
+    end
+
+    body.should == '1'
+  end
+end

data/spec/redirect_spec.rb

@@ -0,0 +1,24 @@
+require File.expand_path("spec_helper", File.dirname(__FILE__))
+
+describe "redirects" do
+  it "should be immediately processed" do
+    app do |r|
+      r.on "about" do
+        r.redirect "/hello", 301
+        "Foo"
+      end
+      r.on do
+        r.redirect "/hello"
+        "Foo"
+      end
+    end
+
+    status.should == 302
+    header('Location').should == '/hello'
+    body.should == ''
+
+    status("/about").should == 301
+    header('Location', "/about").should == '/hello'
+    body("/about").should == ''
+  end
+end

data/spec/request_spec.rb

@@ -0,0 +1,55 @@
+require File.expand_path("spec_helper", File.dirname(__FILE__))
+
+describe "request.full_path_info" do
+  it "should return the script name and path_info as a string" do
+    app do |r|
+      r.on "foo" do
+        "#{r.full_path_info}:#{r.script_name}:#{r.path_info}"
+      end
+    end
+
+    body("/foo/bar").should ==  "/foo/bar:/foo:/bar"
+  end
+end
+
+describe "request.halt" do
+  it "should return rack response as argument given it as argument" do
+    app do |r|
+      r.halt [200, {}, ['foo']]
+    end
+
+    body.should ==  "foo"
+  end
+end
+
+describe "request.scope" do
+  it "should return roda instance" do
+    app(:bare) do
+      attr_accessor :b
+
+      route do |r|
+        self.b = 'a'
+        request.scope.b
+      end
+    end
+
+    body.should ==  "a"
+  end
+end
+
+describe "request.inspect" do
+  it "should return information about request" do
+    app(:bare) do
+      def self.inspect
+        'Foo'
+      end
+
+      route do |r|
+        request.inspect
+      end
+    end
+
+    body('/a/b').should ==  "#<Foo::RodaRequest GET /a/b>"
+    body('REQUEST_METHOD'=>'POST').should ==  "#<Foo::RodaRequest POST />"
+  end
+end

data/spec/response_spec.rb

@@ -0,0 +1,131 @@
+require File.expand_path("spec_helper", File.dirname(__FILE__))
+
+describe "cookie handling" do
+  it "should set cookies on response" do
+    app do |r|
+      response.set_cookie("foo", "bar")
+      response.set_cookie("bar", "baz")
+      "Hello"
+    end
+
+    header('Set-Cookie').should == "foo=bar\nbar=baz"
+    body.should == 'Hello'
+  end
+
+  it "should delete cookies on response" do
+    app do |r|
+      response.set_cookie("foo", "bar")
+      response.delete_cookie("foo")
+      "Hello"
+    end
+
+    header('Set-Cookie').should =~ /foo=; (max-age=0; )?expires=Thu, 01[ -]Jan[ -]1970 00:00:00 (-0000|GMT)/
+    body.should == 'Hello'
+  end
+end
+
+describe "response #[] and #[]=" do
+  it "should get/set headers" do
+    app do |r|
+      response['foo'] = 'bar'
+      response['foo'] + response.headers['foo']
+    end
+
+    header('foo').should == "bar"
+    body.should == 'barbar'
+  end
+end
+
+describe "response #write" do
+  it "should add to body" do
+    app do |r|
+      response.write 'a'
+      response.write 'b'
+    end
+
+    body.should == 'ab'
+  end
+end
+
+describe "response #finish" do
+  it "should set status to 404 if body has not been written to" do
+    app do |r|
+      s, h, b = response.finish
+      "#{s}#{h['Content-Type']}#{b.length}"
+    end
+
+    body.should == '404text/html0'
+  end
+
+  it "should set status to 200 if body has been written to" do
+    app do |r|
+      response.write 'a'
+      s, h, b = response.finish
+      response.write "#{s}#{h['Content-Type']}#{b.length}"
+    end
+
+    body.should == 'a200text/html1'
+  end
+
+  it "should not overwrite existing status" do
+    app do |r|
+      response.status = 500
+      s, h, b = response.finish
+      "#{s}#{h['Content-Type']}#{b.length}"
+    end
+
+    body.should == '500text/html0'
+  end
+end
+
+describe "response #redirect" do
+  it "should set location and status" do
+    app do |r|
+      r.on 'a' do
+        response.redirect '/foo', 303
+      end
+      r.on do
+        response.redirect '/bar'
+      end
+    end
+
+    status('/a').should == 303
+    status.should == 302
+    header('Location', '/a').should == '/foo'
+    header('Location').should == '/bar'
+  end
+end
+
+describe "response #empty?" do
+  it "should return whether the body is empty" do
+    app do |r|
+      r.on 'a' do
+        response['foo'] = response.empty?.to_s
+      end
+      r.on do
+        response.write 'a'
+        response['foo'] = response.empty?.to_s
+      end
+    end
+
+    header('foo', '/a').should == 'true'
+    header('foo').should == 'false'
+  end
+end
+
+describe "response #inspect" do
+  it "should return information about response" do
+    app(:bare) do
+      def self.inspect
+        'Foo'
+      end
+
+      route do |r|
+        response.status = 200
+        response.inspect
+      end
+    end
+
+    body.should == '#<Foo::RodaResponse 200 {"Content-Type"=>"text/html"} []>'
+  end
+end