roda 1.1.0 → 1.2.0

data/spec/plugin/empty_root_spec.rb

@@ -0,0 +1,14 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "empty_root plugin" do 
+  it "makes root match on emtpy path" do
+    app(:empty_root) do |r|
+      r.root{"root"}
+      "notroot"
+    end
+
+    body.should == 'root'
+    body("").should == 'root'
+    body("a").should == 'notroot'
+  end
+end

data/spec/plugin/environments_spec.rb

@@ -0,0 +1,31 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "environments plugin" do 
+  before do
+    app
+    app.plugin :environments, :development
+  end
+
+  it "adds environment accessor for getting/setting the environment" do
+    app.environment.should == :development
+    app.environment = :test
+    app.environment.should == :test
+    
+    app.plugin :environments, :production
+    app.environment.should == :production
+  end
+
+  it "adds predicates for testing the environment" do
+    app.development?.should == true
+    app.test?.should == false
+    app.production?.should == false
+  end
+
+  it "adds configure method which yields if no arguments are given or an environment matches" do
+    a = []
+    app.configure{a << 1}
+    app.configure(:development){|ap| a << ap}
+    app.configure(:test, :production){a << 2}
+    a.should == [1, app]
+  end
+end

data/spec/plugin/h_spec.rb

@@ -3,9 +3,7 @@ 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
+      h("<form>") + h(:form)
     end
 
     body.should == '&lt;form&gt;form'

data/spec/plugin/header_matchers_spec.rb

@@ -59,3 +59,17 @@ describe "host matcher" do
     body("HTTP_HOST" => "example.com").should == '0'
   end
 end
+
+describe "user_agent matcher" do
+  it "should accept pattern and match against user-agent" do
+    app(:header_matchers) do |r|
+      r.on :user_agent=>/(chrome)(\d+)/ do |agent, num|
+        "a-#{agent}-#{num}"
+      end
+    end
+
+    body("HTTP_USER_AGENT" => "chrome31").should ==  "a-chrome-31"
+    status.should == 404
+  end
+end
+

data/spec/plugin/hooks_spec.rb

@@ -20,11 +20,9 @@ describe "hooks plugin" do
       end
 
       route do |r|
-        r.is "" do
-          f = response['foo']
-          response['foo'] = 'baz'
-          f
-        end
+        f = response['foo']
+        response['foo'] = 'baz'
+        f
       end
     end
   end

data/spec/plugin/mailer_spec.rb

@@ -0,0 +1,191 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+begin
+  require 'mail'
+rescue LoadError
+  warn "mail not installed, skipping mail plugin test"  
+else
+Mail.defaults do
+  delivery_method :test
+end
+
+describe "mailer plugin" do 
+  def deliveries
+    Mail::TestMailer.deliveries
+  end
+
+  after do
+    deliveries.clear
+  end
+
+  setup_email = lambda do 
+    from "f@example.com"
+    to "t@example.com"
+    subject 's'
+  end
+
+  it "supports sending emails via the routing tree" do
+    app(:mailer) do |r|
+      r.mail do
+        instance_exec(&setup_email)
+        cc "c@example.com"
+        bcc "b@example.com"
+        response['X-Foo'] = 'Bar'
+        "b"
+      end
+    end
+
+    m = app.mail('/foo')
+    deliveries.should == []
+    m.from.should == ['f@example.com']
+    m.to.should == ['t@example.com']
+    m.cc.should == ['c@example.com']
+    m.bcc.should == ['b@example.com']
+    m.subject.should == 's'
+    m.body.should == 'b'
+    m.header['X-Foo'].to_s.should == 'Bar'
+
+    m.deliver!
+    deliveries.should == [m]
+
+    deliveries.clear
+    m = app.sendmail('/foo')
+    deliveries.should == [m]
+    m.from.should == ['f@example.com']
+    m.to.should == ['t@example.com']
+    m.cc.should == ['c@example.com']
+    m.bcc.should == ['b@example.com']
+    m.subject.should == 's'
+    m.body.should == 'b'
+    m.header['X-Foo'].to_s.should == 'Bar'
+  end
+
+  it "supports arguments to mail/sendmail methods, yielding them to the route blocks" do
+    app(:mailer) do |r|
+      instance_exec(&setup_email)
+      r.mail "foo" do |*args|
+        "foo#{args.inspect}"
+      end
+      r.mail :d do |*args|
+        args.inspect
+      end
+    end
+
+    app.mail('/foo', 1, 2).body.should == 'foo[1, 2]'
+    app.sendmail('/bar', 1, 2).body.should == '["bar", 1, 2]'
+  end
+
+  it "supports attachments" do
+    app(:mailer) do |r|
+      r.mail do
+        instance_exec(&setup_email)
+        add_file __FILE__
+      end
+    end
+
+    m = app.mail('foo')
+    m.attachments.length.should == 1
+    m.attachments.first.content_type.should =~ /mailer_spec\.rb/
+    m.content_type.should =~ /\Amultipart\/mixed/
+  end
+
+  it "supports regular web requests in same application" do
+    app(:mailer) do |r|
+      r.get "foo", :param=>'bar' do |bar|
+        "foo#{bar}"
+      end
+      r.mail "bar" do
+        instance_exec(&setup_email)
+        "b"
+      end
+    end
+
+    body("/foo", 'QUERY_STRING'=>'bar=baz', 'rack.input'=>StringIO.new).should == 'foobaz'
+    app.mail('/bar').body.should == 'b'
+  end
+
+  it "supports multipart email using text_part/html_pat" do
+    app(:mailer) do |r|
+      r.mail do
+        instance_exec(&setup_email)
+        text_part "t"
+        html_part "h"
+      end
+    end
+
+    m = app.mail('/foo')
+    m.text_part.body.should == 't'
+    m.html_part.body.should == 'h'
+    m.content_type.should =~ /\Amultipart\/alternative/
+  end
+
+  it "supports setting arbitrary email headers for multipart emails" do
+    app(:mailer) do |r|
+      r.mail do
+        instance_exec(&setup_email)
+        text_part "t", "X-Text"=>'T'
+        html_part "h", "X-HTML"=>'H'
+      end
+    end
+
+    m = app.mail('/foo')
+    m.text_part.body.should == 't'
+    m.text_part.header['X-Text'].to_s.should == 'T'
+    m.html_part.body.should == 'h'
+    m.html_part.header['X-HTML'].to_s.should == 'H'
+    m.content_type.should =~ /\Amultipart\/alternative/
+  end
+
+  it "raises error if mail object is not returned" do
+    app(:mailer){}
+    proc{app.mail('/')}.should raise_error(Roda::RodaPlugins::Mailer::Error)
+  end
+
+  it "does not raise an error when using an explicitly empty body" do
+    app(:mailer){""}
+    proc{app.mail('/')}.should_not raise_error
+  end
+
+  it "supports setting the default content-type for emails when loading the plugin" do
+    app(:bare) do
+      plugin :mailer, :content_type=>'text/html'
+      route{""}
+    end
+    app.mail('/').content_type.should =~ /\Atext\/html/
+  end
+
+  it "supports loading the plugin multiple times" do
+    app(:bare) do
+      plugin :mailer, :content_type=>'text/html'
+      plugin :mailer
+      route{""}
+    end
+    app.mail('/').content_type.should =~ /\Atext\/html/
+  end
+
+  it "supports manually overridding the default content-type for emails" do
+    app(:bare) do
+      plugin :mailer, :content_type=>'text/html'
+      route do
+        response['Content-Type'] = 'text/foo'
+        ""
+      end
+    end
+    app.mail('/').content_type.should =~ /\Atext\/foo/
+  end
+
+  it "supports handle setting the default content type when attachments are used" do
+    app(:bare) do
+      plugin :mailer, :content_type=>'text/html'
+      route do
+        add_file 'spec/assets/css/raw.css'
+        "a"
+      end
+    end
+    m = app.mail('/')
+    m.content_type.should =~ /\Amultipart\/mixed/
+    m.parts.first.content_type.should =~ /\Atext\/css/
+    m.parts.last.content_type.should =~ /\Atext\/html/
+  end
+end
+end

data/spec/plugin/match_affix_spec.rb

@@ -0,0 +1,22 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "match_affix plugin" do 
+  it "allows changing the match prefix/suffix" do
+    app(:bare) do
+      plugin :match_affix, "", /(\/|\z)/
+
+      route do |r|
+        r.on "/albums" do |b|
+          r.on "b/:id" do |id, s|
+            "b-#{b}-#{id}-#{s.inspect}"
+          end
+
+          "albums-#{b}"
+        end
+      end
+    end
+
+    body("/albums/a/1").should == 'albums-/'
+    body("/albums/b/1").should == 'b-/-1-""'
+  end
+end

data/spec/plugin/multi_run_spec.rb

@@ -0,0 +1,31 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "multi_run plugin" do 
+  it "adds Roda.run method for setting up prefix delegations to other rack apps" do
+    app(:multi_run) do |r|
+      r.multi_run
+      "c"
+    end
+
+    app.run "a", Class.new(Roda).class_eval{route{"a1"}; app}
+
+    body("/a").should == 'a1'
+    body("/b").should == 'c'
+    body("/b/a").should == 'c'
+    body.should == 'c'
+
+    app.run "b", Class.new(Roda).class_eval{route{"b1"}; app}
+
+    body("/a").should == 'a1'
+    body("/b").should == 'b1'
+    body("/b/a").should == 'b1'
+    body.should == 'c'
+
+    app.run "b/a", Class.new(Roda).class_eval{route{"b2"}; app}
+
+    body("/a").should == 'a1'
+    body("/b").should == 'b1'
+    body("/b/a").should == 'b2'
+    body.should == 'c'
+  end
+end

data/spec/plugin/named_templates_spec.rb

@@ -0,0 +1,65 @@
+require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
+
+describe "named_templates plugin" do 
+  it "adds template method method for naming templates, and have render recognize it" do
+    app(:bare) do
+      plugin :named_templates
+
+      template :foo do
+        @b = 2
+        "foo<%= @a %><%= @b %>"
+      end
+      template :layout, :engine=>:str do
+        @c = 3
+        'bar#{@a}#{@c}-#{yield}-baz'
+      end
+
+      route do |r|
+        @a = 1
+        view(:foo)
+      end
+    end
+
+    body.should == 'bar13-foo12-baz'
+    @app = Class.new(@app)
+    body.should == 'bar13-foo12-baz'
+  end
+
+  it "works with the view_subdirs plugin" do
+    app(:bare) do
+      plugin :render
+      plugin :view_subdirs
+      plugin :named_templates
+
+      template "foo/bar" do
+        @b = 2
+        "foobar<%= @a %><%= @b %>"
+      end
+      template "foo/layout", :engine=>:str do
+        @c = 3
+        'foo#{@a}#{@c}-#{yield}-baz'
+      end
+      template "bar/layout", :engine=>:str do
+        @c = 3
+        'bar#{@a}#{@c}-#{yield}-baz'
+      end
+
+      route do |r|
+        r.is 'foo' do
+          set_view_subdir :foo
+          @a = 1
+          view(:bar)
+        end
+        r.is 'bar' do
+          set_view_subdir :bar
+          @a = 4
+          @b = 2
+          view(:inline=>"barfoo<%= @a %><%= @b %>")
+        end
+      end
+    end
+
+    body('/foo').should == 'foo13-foobar12-baz'
+    body('/bar').should == 'bar43-barfoo42-baz'
+  end
+end

data/spec/plugin/path_spec.rb

@@ -1,6 +1,19 @@
 require File.expand_path("spec_helper", File.dirname(File.dirname(__FILE__)))
 
 describe "path plugin" do 
+  def path_app(*args, &block)
+    app(:bare) do
+      plugin :path
+      path *args, &block
+      route{|r| send(r.path_info)}
+    end
+  end
+
+  def path_block_app(b, *args, &block)
+    path_app(*args, &block)
+    app.route{|r| send(r.path_info, &b)}
+  end
+
   it "adds path method for defining named paths" do
     app(:bare) do
       plugin :path
@@ -8,13 +21,16 @@ describe "path plugin" do
       path :bar do |o|
         "/bar/#{o}"
       end
+      path :baz do |&block|
+        "/baz/#{block.call}"
+      end
 
       route do |r|
-        "#{foo_path}#{bar_path('a')}"
+        "#{foo_path}#{bar_path('a')}#{baz_path{'b'}}"
       end
     end
 
-    body.should == '/foo/bar/a'
+    body.should == '/foo/bar/a/baz/b'
   end
 
   it "raises if both path and block are given" do
@@ -26,4 +42,52 @@ describe "path plugin" do
     app.plugin :path
     proc{app.path(:foo)}.should raise_error(Roda::RodaError)
   end
+
+  it "raises if two options hashes are given" do
+    app.plugin :path
+    proc{app.path(:foo, {:name=>'a'}, :add_script_name=>true)}.should raise_error(Roda::RodaError)
+  end
+
+  it "supports :name option for naming the method" do
+    path_app(:foo, :name=>'foobar_route'){"/bar/foo"}
+    body("foobar_route").should == "/bar/foo"
+  end
+
+  it "supports :add_script_name option for automatically adding the script name" do
+    path_app(:foo, :add_script_name=>true){"/bar/foo"}
+    body("foo_path", 'SCRIPT_NAME'=>'/baz').should == "/baz/bar/foo"
+  end
+
+  it "supports path method accepting a block when using :add_script_name" do
+    path_block_app(lambda{"c"}, :foo, :add_script_name=>true){|&block| "/bar/foo/#{block.call}"}
+    body("foo_path", 'SCRIPT_NAME'=>'/baz').should == "/baz/bar/foo/c"
+  end
+
+  it "supports :url option for also creating a *_url method" do
+    path_app(:foo, :url=>true){"/bar/foo"}
+    body("foo_path", 'HTTP_HOST'=>'example.org', "rack.url_scheme"=>'http', 'SERVER_PORT'=>80).should == "/bar/foo"
+    body("foo_url", 'HTTP_HOST'=>'example.org', "rack.url_scheme"=>'http', 'SERVER_PORT'=>80).should == "http://example.org/bar/foo"
+  end
+
+  it "supports url method accepting a block when using :url" do
+    path_block_app(lambda{"c"}, :foo, :url=>true){|&block| "/bar/foo/#{block.call}"}
+    body("foo_url", 'HTTP_HOST'=>'example.org', "rack.url_scheme"=>'http', 'SERVER_PORT'=>80).should == "http://example.org/bar/foo/c"
+  end
+
+  it "supports url method name specified in :url option" do
+    path_app(:foo, :url=>:foobar_uri){"/bar/foo"}
+    body("foo_path", 'HTTP_HOST'=>'example.org', "rack.url_scheme"=>'http', 'SERVER_PORT'=>80).should == "/bar/foo"
+    body("foobar_uri", 'HTTP_HOST'=>'example.org', "rack.url_scheme"=>'http', 'SERVER_PORT'=>80).should == "http://example.org/bar/foo"
+  end
+
+  it "supports :url_only option for not creating a path method" do
+    path_app(:foo, :url_only=>true){"/bar/foo"}
+    proc{body("foo_path")}.should raise_error(NoMethodError)
+    body("foo_url", 'HTTP_HOST'=>'example.org', "rack.url_scheme"=>'http', 'SERVER_PORT'=>80).should == "http://example.org/bar/foo"
+  end
+
+  it "handles non-default ports in url methods" do
+    path_app(:foo, :url=>true){"/bar/foo"}
+    body("foo_url", 'HTTP_HOST'=>'example.org', "rack.url_scheme"=>'http', 'SERVER_PORT'=>81).should == "http://example.org:81/bar/foo"
+  end
 end