rtomayko-sinatra 0.3.3 → 0.8.9

data/{test → compat}/app_test.rb

@@ -108,7 +108,9 @@ context "Sinatra" do
   end
 
   specify "renders a body with a redirect" do
-    Sinatra::EventContext.any_instance.expects(:foo).returns('blah')
+    helpers do
+      def foo ; 'blah' ; end
+    end
     get "/" do
       redirect 'foo', :foo
     end
@@ -130,13 +132,10 @@ context "Sinatra" do
   end
 
   specify "stop sets content and ends event" do
-
-    Sinatra::EventContext.any_instance.expects(:foo).never
-
     get '/set_body' do
       stop 'Hello!'
       stop 'World!'
-      foo
+      fail 'stop should have halted'
     end
 
     get_it '/set_body'
@@ -146,8 +145,13 @@ context "Sinatra" do
 
   end
 
-  specify "should set status then call helper with a var" do
-    Sinatra::EventContext.any_instance.expects(:foo).once.with(1).returns('bah!')
+  # Deprecated. WTF was going on here? What's the 1 in [:foo, 1] do?
+  xspecify "should set status then call helper with a var" do
+    helpers do
+      def foo
+        'bah!'
+      end
+    end
 
     get '/set_body' do
       stop [404, [:foo, 1]]
@@ -252,8 +256,6 @@ context "Sinatra" do
     assert_equal 'puted', body
   end
 
-  # Some Ajax libraries downcase the _method parameter value. Make
-  # sure we can handle that.
   specify "rewrites POSTs with lowercase _method param to PUT" do
     put '/' do
       'puted'
@@ -262,7 +264,6 @@ context "Sinatra" do
     body.should.equal 'puted'
   end
 
-  # Ignore any _method parameters specified in GET requests or on the query string in POST requests.
   specify "does not rewrite GETs with _method param to PUT" do
     get '/' do
       'getted'

data/{test → compat}/application_test.rb

@@ -16,7 +16,8 @@ context "Looking up a request" do
     Sinatra.application = nil
   end
 
-  specify "returns what's at the end" do
+  # Deprecated. The lookup method is no longer used.
+  xspecify "returns what's at the end" do
     block = Proc.new { 'Hello' }
     get '/', &block
 
@@ -31,7 +32,8 @@ context "Looking up a request" do
     result.block.should.be block
   end
 
-  specify "takes params in path" do
+  # Deprecated. The lookup method is no longer used.
+  xspecify "takes params in path" do
     block = Proc.new { 'Hello' }
     get '/:foo', &block
 
@@ -83,7 +85,8 @@ context "An app returns" do
 
   end
 
-  specify "the body set if set before the last" do
+  # Deprecated. The body method no longer halts.
+  xspecify "the body set if set before the last" do
 
     get '/' do
       body 'Blake'
@@ -142,14 +145,16 @@ end
 
 context "Default Application Configuration" do
 
-  specify "includes 404 and 500 error handlers" do
+  # Sinatra::ServerError is no longer used
+  xspecify "includes 404 and 500 error handlers" do
     Sinatra.application.errors.should.include(Sinatra::ServerError)
     Sinatra.application.errors[Sinatra::ServerError].should.not.be.nil
     Sinatra.application.errors.should.include(Sinatra::NotFound)
     Sinatra.application.errors[Sinatra::NotFound].should.not.be.nil
   end
 
-  specify "includes Static event" do
+  # Deprecated. No such thing as a Static event anymore.
+  xspecify "includes Static event" do
     assert Sinatra.application.events[:get].any? { |e| Sinatra::Static === e }
   end
 

data/compat/builder_test.rb

@@ -0,0 +1,101 @@
+require File.dirname(__FILE__) + '/helper'
+
+context "Builder" do
+
+  setup do
+    Sinatra.application = nil
+  end
+
+  context "without layouts" do
+
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify "should render" do
+
+      get '/no_layout' do
+        builder 'xml.instruct!'
+      end
+
+      get_it '/no_layout'
+      should.be.ok
+      body.should == %(<?xml version="1.0" encoding="UTF-8"?>\n)
+
+    end
+
+    specify "should render inline block" do
+
+      get '/no_layout_and_inlined' do
+        @name = "Frank & Mary"
+        builder do |xml|
+          xml.couple @name
+        end
+      end
+
+      get_it '/no_layout_and_inlined'
+      should.be.ok
+      body.should == %(<couple>Frank &amp; Mary</couple>\n)
+
+    end
+
+  end
+
+
+
+  context "Templates (in general)" do
+
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify "are read from files if Symbols" do
+
+      get '/from_file' do
+        @name = 'Blue'
+        builder :foo, :views_directory => File.dirname(__FILE__) + "/views"
+      end
+
+      get_it '/from_file'
+      should.be.ok
+      body.should.equal %(<exclaim>You rock Blue!</exclaim>\n)
+
+    end
+
+    specify "use layout.ext by default if available" do
+
+      get '/' do
+        builder :foo, :views_directory => File.dirname(__FILE__) + "/views/layout_test"
+      end
+
+      get_it '/'
+      should.be.ok
+      body.should.equal "<layout>\n<this>is foo!</this>\n</layout>\n"
+
+    end
+
+    specify "renders without layout" do
+
+      get '/' do
+        builder :no_layout, :views_directory => File.dirname(__FILE__) + "/views/no_layout"
+      end
+
+      get_it '/'
+      should.be.ok
+      body.should.equal "<foo>No Layout!</foo>\n"
+
+    end
+
+    specify "raises error if template not found" do
+
+      get '/' do
+        builder :not_found
+      end
+
+      lambda { get_it '/' }.should.raise(Errno::ENOENT)
+
+    end
+
+  end
+
+end

data/compat/erb_test.rb

@@ -0,0 +1,136 @@
+require File.dirname(__FILE__) + '/helper'
+
+context "Erb" do
+
+  setup do
+    Sinatra.application = nil
+  end
+
+  context "without layouts" do
+
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify "should render" do
+
+      get '/no_layout' do
+        erb '<%= 1 + 1 %>'
+      end
+
+      get_it '/no_layout'
+      should.be.ok
+      body.should == '2'
+
+    end
+
+    specify "should take an options hash with :locals set with a string" do
+      get '/locals' do
+        erb '<%= foo %>', :locals => {:foo => "Bar"}
+      end
+
+      get_it '/locals'
+      should.be.ok
+      body.should == 'Bar'
+    end
+
+    specify "should take an options hash with :locals set with a complex object" do
+      get '/locals-complex' do
+        erb '<%= foo[0] %>', :locals => {:foo => ["foo", "bar", "baz"]}
+      end
+
+      get_it '/locals-complex'
+      should.be.ok
+      body.should == 'foo'
+    end
+  end
+
+  context "with layouts" do
+
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify "can be inline" do
+
+      layout do
+        %Q{This is <%= yield %>!}
+      end
+
+      get '/lay' do
+        erb 'Blake'
+      end
+
+      get_it '/lay'
+      should.be.ok
+      body.should.equal 'This is Blake!'
+
+    end
+
+    specify "can use named layouts" do
+
+      layout :pretty do
+        %Q{<h1><%= yield %></h1>}
+      end
+
+      get '/pretty' do
+        erb 'Foo', :layout => :pretty
+      end
+
+      get '/not_pretty' do
+        erb 'Bar'
+      end
+
+      get_it '/pretty'
+      body.should.equal '<h1>Foo</h1>'
+
+      get_it '/not_pretty'
+      body.should.equal 'Bar'
+
+    end
+
+    specify "can be read from a file if they're not inlined" do
+
+      get '/foo' do
+        @title = 'Welcome to the Hello Program'
+        erb 'Blake', :layout => :foo_layout,
+                     :views_directory => File.dirname(__FILE__) + "/views"
+      end
+
+      get_it '/foo'
+      body.should.equal "Welcome to the Hello Program\nHi Blake\n"
+
+    end
+
+  end
+
+  context "Templates (in general)" do
+
+    specify "are read from files if Symbols" do
+
+      get '/from_file' do
+        @name = 'Alena'
+        erb :foo, :views_directory => File.dirname(__FILE__) + "/views"
+      end
+
+      get_it '/from_file'
+
+      body.should.equal 'You rock Alena!'
+
+    end
+
+    specify "use layout.ext by default if available" do
+
+      get '/layout_from_file' do
+        erb :foo, :views_directory => File.dirname(__FILE__) + "/views/layout_test"
+      end
+
+      get_it '/layout_from_file'
+      should.be.ok
+      body.should.equal "x This is foo! x \n"
+
+    end
+
+  end
+
+end

data/{test → compat}/events_test.rb

@@ -9,9 +9,19 @@ context "Simple Events" do
     })
   end
 
+  class MockResult < Struct.new(:block, :params)
+  end
+
   def invoke_simple(path, request_path, &b)
-    event = Sinatra::Event.new(path, &b)
-    event.invoke(simple_request_hash(:get, request_path))
+    params = nil
+    mock_app {
+      get path do
+        params = self.params
+        b.call if b
+      end
+    }
+    get_it request_path
+    MockResult.new(b, params)
   end
 
   specify "return last value" do

data/compat/filter_test.rb

@@ -0,0 +1,30 @@
+require File.dirname(__FILE__) + '/helper'
+
+context "before filters" do
+
+  setup do
+    Sinatra.application = nil
+    @app = Sinatra.application
+  end
+
+  specify "should be executed in the order defined" do
+    invoked = 0x0
+    @app.before { invoked = 0x01 }
+    @app.before { invoked |= 0x02 }
+    @app.get('/') { 'Hello World' }
+    get_it '/'
+    should.be.ok
+    body.should.be == 'Hello World'
+    invoked.should.be == 0x03
+  end
+
+  specify "should be capable of modifying the request" do
+    @app.get('/foo') { 'foo' }
+    @app.get('/bar') { 'bar' }
+    @app.before { request.path_info = '/bar' }
+    get_it '/foo'
+    should.be.ok
+    body.should.be == 'bar'
+  end
+
+end

data/compat/haml_test.rb

@@ -0,0 +1,233 @@
+require File.dirname(__FILE__) + '/helper'
+
+context "Haml" do
+
+  setup do
+    Sinatra.application = nil
+  end
+
+  context "without layouts" do
+
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify "should render" do
+
+      get '/no_layout' do
+        haml '== #{1+1}'
+      end
+
+      get_it '/no_layout'
+      should.be.ok
+      body.should == "2\n"
+
+    end
+  end
+
+  context "with layouts" do
+
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify "can be inline" do
+
+      layout do
+        '== This is #{yield}!'
+      end
+
+      get '/lay' do
+        haml 'Blake'
+      end
+
+      get_it '/lay'
+      should.be.ok
+      body.should.equal "This is Blake\n!\n"
+
+    end
+
+    specify "can use named layouts" do
+
+      layout :pretty do
+        '%h1== #{yield}'
+      end
+
+      get '/pretty' do
+        haml 'Foo', :layout => :pretty
+      end
+
+      get '/not_pretty' do
+        haml 'Bar'
+      end
+
+      get_it '/pretty'
+      body.should.equal "<h1>Foo</h1>\n"
+
+      get_it '/not_pretty'
+      body.should.equal "Bar\n"
+
+    end
+
+    specify "can be read from a file if they're not inlined" do
+
+      get '/foo' do
+        @title = 'Welcome to the Hello Program'
+        haml 'Blake', :layout => :foo_layout,
+                      :views_directory => File.dirname(__FILE__) + "/views"
+      end
+
+      get_it '/foo'
+      body.should.equal "Welcome to the Hello Program\nHi Blake\n"
+
+    end
+
+    specify "can be read from file and layout from text" do
+      get '/foo' do
+        haml 'Test', :layout => '== Foo #{yield}'
+      end
+
+      get_it '/foo'
+
+      body.should.equal "Foo Test\n"
+    end
+
+  end
+
+  context "Templates (in general)" do
+
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify "are read from files if Symbols" do
+
+      get '/from_file' do
+        @name = 'Alena'
+        haml :foo, :views_directory => File.dirname(__FILE__) + "/views"
+      end
+
+      get_it '/from_file'
+
+      body.should.equal "You rock Alena!\n"
+
+    end
+
+    specify "use layout.ext by default if available" do
+
+      get '/' do
+        haml :foo, :views_directory => File.dirname(__FILE__) + "/views/layout_test"
+      end
+
+      get_it '/'
+      should.be.ok
+      body.should.equal "x This is foo!\n x\n"
+
+    end
+
+    specify "renders without layout" do
+
+      get '/' do
+        haml :no_layout, :views_directory => File.dirname(__FILE__) + "/views/no_layout"
+      end
+
+      get_it '/'
+      should.be.ok
+      body.should.equal "<h1>No Layout!</h1>\n"
+
+    end
+
+    specify "can render with no layout" do
+      layout do
+        "X\n= yield\nX"
+      end
+
+      get '/' do
+        haml 'blake', :layout => false
+      end
+
+      get_it '/'
+
+      body.should.equal "blake\n"
+    end
+
+    specify "raises error if template not found" do
+      get '/' do
+        haml :not_found
+      end
+
+      lambda { get_it '/' }.should.raise(Errno::ENOENT)
+    end
+
+    specify "use layout.ext by default if available" do
+
+      template :foo do
+        'asdf'
+      end
+
+      get '/' do
+        haml :foo, :layout => false,
+                   :views_directory => File.dirname(__FILE__) + "/views/layout_test"
+      end
+
+      get_it '/'
+      should.be.ok
+      body.should.equal "asdf\n"
+
+    end
+
+  end
+
+  describe 'Options passed to the HAML interpreter' do
+    setup do
+      Sinatra.application = nil
+    end
+
+    specify 'are empty be default' do
+
+      get '/' do
+        haml 'foo'
+      end
+
+      Haml::Engine.expects(:new).with('foo', {}).returns(stub(:render => 'foo'))
+
+      get_it '/'
+      should.be.ok
+
+    end
+
+    specify 'can be configured by passing :options to haml' do
+
+      get '/' do
+        haml 'foo', :options => {:format => :html4}
+      end
+
+      Haml::Engine.expects(:new).with('foo', {:format => :html4}).returns(stub(:render => 'foo'))
+
+      get_it '/'
+      should.be.ok
+
+    end
+
+    specify 'can be configured using set_option :haml' do
+
+      configure do
+        set_option :haml, :format       => :html4,
+                          :escape_html  => true
+      end
+
+      get '/' do
+        haml 'foo'
+      end
+
+      Haml::Engine.expects(:new).with('foo', {:format => :html4,
+        :escape_html => true}).returns(stub(:render => 'foo'))
+
+      get_it '/'
+      should.be.ok
+
+    end
+
+  end
+
+end

data/compat/helper.rb

@@ -0,0 +1,21 @@
+require 'rubygems'
+require 'mocha'
+
+# disable warnings in compat specs.
+$VERBOSE = nil
+
+$:.unshift File.dirname(File.dirname(__FILE__)) + "/lib"
+
+ENV['RACK_ENV'] ||= 'test'
+
+require 'sinatra'
+require 'sinatra/test'
+require 'sinatra/test/unit'
+require 'sinatra/test/spec'
+
+class Test::Unit::TestCase
+  def setup
+    @app = lambda { |env| Sinatra::Application.call(env) }
+  end
+  include Sinatra::Test
+end