sinatra 0.9.1.1 → 0.9.2

data/test/mapped_error_test.rb

@@ -6,155 +6,150 @@ end
 class FooNotFound < Sinatra::NotFound
 end
 
-describe 'Exception Mappings' do
-  it 'invokes handlers registered with ::error when raised' do
-    mock_app {
-      set :raise_errors, false
-      error(FooError) { 'Foo!' }
-      get '/' do
-        raise FooError
-      end
-    }
-    get '/'
-    assert_equal 500, status
-    assert_equal 'Foo!', body
+class MappedErrorTest < Test::Unit::TestCase
+  def test_default
+    assert true
   end
 
-  it 'uses the Exception handler if no matching handler found' do
-    mock_app {
-      set :raise_errors, false
-      error(Exception) { 'Exception!' }
-      get '/' do
-        raise FooError
-      end
-    }
-    get '/'
-    assert_equal 500, status
-    assert_equal 'Exception!', body
-  end
-
-  it "sets env['sinatra.error'] to the rescued exception" do
-    mock_app {
-      set :raise_errors, false
-      error(FooError) {
-        assert env.include?('sinatra.error')
-        assert env['sinatra.error'].kind_of?(FooError)
-        'looks good'
+  describe 'Exception Mappings' do
+    it 'invokes handlers registered with ::error when raised' do
+      mock_app {
+        set :raise_errors, false
+        error(FooError) { 'Foo!' }
+        get '/' do
+          raise FooError
+        end
+      }
+      get '/'
+      assert_equal 500, status
+      assert_equal 'Foo!', body
+    end
+
+    it 'uses the Exception handler if no matching handler found' do
+      mock_app {
+        set :raise_errors, false
+        error(Exception) { 'Exception!' }
+        get '/' do
+          raise FooError
+        end
       }
-      get '/' do
-        raise FooError
-      end
-    }
-    get '/'
-    assert_equal 'looks good', body
-  end
-
-  it 'dumps errors to rack.errors when dump_errors is enabled' do
-    mock_app {
-      set :raise_errors, false
-      set :dump_errors, true
-      get('/') { raise FooError, 'BOOM!' }
-    }
-
-    get '/'
-    assert_equal 500, status
-    assert @response.errors =~ /FooError - BOOM!:/
-  end
-
-  it "raises without calling the handler when the raise_errors options is set" do
-    mock_app {
-      set :raise_errors, true
-      error(FooError) { "she's not there." }
-      get '/' do
-        raise FooError
-      end
-    }
-    assert_raise(FooError) { get '/' }
-  end
-
-  it "never raises Sinatra::NotFound beyond the application" do
-    mock_app {
-      set :raise_errors, true
-      get '/' do
-        raise Sinatra::NotFound
-      end
-    }
-    assert_nothing_raised { get '/' }
-    assert_equal 404, status
-  end
-
-  it "cascades for subclasses of Sinatra::NotFound" do
-    mock_app {
-      set :raise_errors, true
-      error(FooNotFound) { "foo! not found." }
-      get '/' do
-        raise FooNotFound
-      end
-    }
-    assert_nothing_raised { get '/' }
-    assert_equal 404, status
-    assert_equal 'foo! not found.', body
-  end
-
-  it 'has a not_found method for backwards compatibility' do
-    mock_app {
-      not_found do
-        "Lost, are we?"
-      end
-    }
-
-    get '/test'
-    assert_equal 404, status
-    assert_equal "Lost, are we?", body
-  end
-end
-
-describe 'Custom Error Pages' do
-  it 'allows numeric status code mappings to be registered with ::error' do
-    mock_app {
-      set :raise_errors, false
-      error(500) { 'Foo!' }
-      get '/' do
-        [500, {}, 'Internal Foo Error']
-      end
-    }
-    get '/'
-    assert_equal 500, status
-    assert_equal 'Foo!', body
-  end
 
-  it 'allows ranges of status code mappings to be registered with :error' do
-    mock_app {
-      set :raise_errors, false
-      error(500..550) { "Error: #{response.status}" }
-      get '/' do
-        [507, {}, 'A very special error']
-      end
-    }
-    get '/'
-    assert_equal 507, status
-    assert_equal 'Error: 507', body
-  end
+      get '/'
+      assert_equal 500, status
+      assert_equal 'Exception!', body
+    end
+
+    it "sets env['sinatra.error'] to the rescued exception" do
+      mock_app {
+        set :raise_errors, false
+        error(FooError) {
+          assert env.include?('sinatra.error')
+          assert env['sinatra.error'].kind_of?(FooError)
+          'looks good'
+        }
+        get '/' do
+          raise FooError
+        end
+      }
+      get '/'
+      assert_equal 'looks good', body
+    end
+
+    it "raises without calling the handler when the raise_errors options is set" do
+      mock_app {
+        set :raise_errors, true
+        error(FooError) { "she's not there." }
+        get '/' do
+          raise FooError
+        end
+      }
+      assert_raise(FooError) { get '/' }
+    end
+
+    it "never raises Sinatra::NotFound beyond the application" do
+      mock_app {
+        set :raise_errors, true
+        get '/' do
+          raise Sinatra::NotFound
+        end
+      }
+      assert_nothing_raised { get '/' }
+      assert_equal 404, status
+    end
+
+    it "cascades for subclasses of Sinatra::NotFound" do
+      mock_app {
+        set :raise_errors, true
+        error(FooNotFound) { "foo! not found." }
+        get '/' do
+          raise FooNotFound
+        end
+      }
+      assert_nothing_raised { get '/' }
+      assert_equal 404, status
+      assert_equal 'foo! not found.', body
+    end
+
+    it 'has a not_found method for backwards compatibility' do
+      mock_app {
+        not_found do
+          "Lost, are we?"
+        end
+      }
 
-  class FooError < RuntimeError
+      get '/test'
+      assert_equal 404, status
+      assert_equal "Lost, are we?", body
+    end
   end
 
-  it 'runs after exception mappings and overwrites body' do
-    mock_app {
-      set :raise_errors, false
-      error FooError do
-        response.status = 502
-        'from exception mapping'
-      end
-      error(500) { 'from 500 handler' }
-      error(502) { 'from custom error page' }
-
-      get '/' do
-        raise FooError
-      end
-    }
-    get '/'
-    assert_equal 502, status
-    assert_equal 'from custom error page', body
+  describe 'Custom Error Pages' do
+    it 'allows numeric status code mappings to be registered with ::error' do
+      mock_app {
+        set :raise_errors, false
+        error(500) { 'Foo!' }
+        get '/' do
+          [500, {}, 'Internal Foo Error']
+        end
+      }
+      get '/'
+      assert_equal 500, status
+      assert_equal 'Foo!', body
+    end
+
+    it 'allows ranges of status code mappings to be registered with :error' do
+      mock_app {
+        set :raise_errors, false
+        error(500..550) { "Error: #{response.status}" }
+        get '/' do
+          [507, {}, 'A very special error']
+        end
+      }
+      get '/'
+      assert_equal 507, status
+      assert_equal 'Error: 507', body
+    end
+
+    class FooError < RuntimeError
+    end
+
+    it 'runs after exception mappings and overwrites body' do
+      mock_app {
+        set :raise_errors, false
+        error FooError do
+          response.status = 502
+          'from exception mapping'
+        end
+        error(500) { 'from 500 handler' }
+        error(502) { 'from custom error page' }
+
+        get '/' do
+          raise FooError
+        end
+      }
+      get '/'
+      assert_equal 502, status
+      assert_equal 'from custom error page', body
+    end
   end
 end

data/test/middleware_test.rb

@@ -1,7 +1,7 @@
 require File.dirname(__FILE__) + '/helper'
 
-describe "Middleware" do
-  before do
+class MiddlewareTest < Test::Unit::TestCase
+  setup do
     @app = mock_app(Sinatra::Default) {
       get '/*' do
         response.headers['X-Tests'] = env['test.ran'].

data/test/options_test.rb

@@ -1,37 +1,39 @@
 require File.dirname(__FILE__) + '/helper'
 
-describe 'Options' do
-  before do
-    restore_default_options
-    @app = Sinatra.new
+class OptionsTest < Test::Unit::TestCase
+  setup do
+    @base    = Sinatra.new(Sinatra::Base)
+    @default = Sinatra.new(Sinatra::Default)
+    @base.set    :environment, :development
+    @default.set :environment, :development
   end
 
   it 'sets options to literal values' do
-    @app.set(:foo, 'bar')
-    assert @app.respond_to?(:foo)
-    assert_equal 'bar', @app.foo
+    @base.set(:foo, 'bar')
+    assert @base.respond_to?(:foo)
+    assert_equal 'bar', @base.foo
   end
 
   it 'sets options to Procs' do
-    @app.set(:foo, Proc.new { 'baz' })
-    assert @app.respond_to?(:foo)
-    assert_equal 'baz', @app.foo
+    @base.set(:foo, Proc.new { 'baz' })
+    assert @base.respond_to?(:foo)
+    assert_equal 'baz', @base.foo
   end
 
   it "sets multiple options with a Hash" do
-    @app.set :foo => 1234,
+    @base.set :foo => 1234,
         :bar => 'Hello World',
         :baz => Proc.new { 'bizzle' }
-    assert_equal 1234, @app.foo
-    assert_equal 'Hello World', @app.bar
-    assert_equal 'bizzle', @app.baz
+    assert_equal 1234, @base.foo
+    assert_equal 'Hello World', @base.bar
+    assert_equal 'bizzle', @base.baz
   end
 
   it 'inherits option methods when subclassed' do
-    @app.set :foo, 'bar'
-    @app.set :biz, Proc.new { 'baz' }
+    @base.set :foo, 'bar'
+    @base.set :biz, Proc.new { 'baz' }
 
-    sub = Class.new(@app)
+    sub = Class.new(@base)
     assert sub.respond_to?(:foo)
     assert_equal 'bar', sub.foo
     assert sub.respond_to?(:biz)
@@ -39,31 +41,31 @@ describe 'Options' do
   end
 
   it 'overrides options in subclass' do
-    @app.set :foo, 'bar'
-    @app.set :biz, Proc.new { 'baz' }
-    sub = Class.new(@app)
+    @base.set :foo, 'bar'
+    @base.set :biz, Proc.new { 'baz' }
+    sub = Class.new(@base)
     sub.set :foo, 'bling'
     assert_equal 'bling', sub.foo
-    assert_equal 'bar', @app.foo
+    assert_equal 'bar', @base.foo
   end
 
   it 'creates setter methods when first defined' do
-    @app.set :foo, 'bar'
-    assert @app.respond_to?('foo=')
-    @app.foo = 'biz'
-    assert_equal 'biz', @app.foo
+    @base.set :foo, 'bar'
+    assert @base.respond_to?('foo=')
+    @base.foo = 'biz'
+    assert_equal 'biz', @base.foo
   end
 
   it 'creates predicate methods when first defined' do
-    @app.set :foo, 'hello world'
-    assert @app.respond_to?(:foo?)
-    assert @app.foo?
-    @app.set :foo, nil
-    assert !@app.foo?
+    @base.set :foo, 'hello world'
+    assert @base.respond_to?(:foo?)
+    assert @base.foo?
+    @base.set :foo, nil
+    assert !@base.foo?
   end
 
   it 'uses existing setter methods if detected' do
-    class << @app
+    class << @base
       def foo
         @foo
       end
@@ -72,303 +74,294 @@ describe 'Options' do
       end
     end
 
-    @app.set :foo, 'bam'
-    assert_equal 'oops', @app.foo
+    @base.set :foo, 'bam'
+    assert_equal 'oops', @base.foo
   end
 
   it "sets multiple options to true with #enable" do
-    @app.enable :sessions, :foo, :bar
-    assert @app.sessions
-    assert @app.foo
-    assert @app.bar
+    @base.enable :sessions, :foo, :bar
+    assert @base.sessions
+    assert @base.foo
+    assert @base.bar
   end
 
   it "sets multiple options to false with #disable" do
-    @app.disable :sessions, :foo, :bar
-    assert !@app.sessions
-    assert !@app.foo
-    assert !@app.bar
+    @base.disable :sessions, :foo, :bar
+    assert !@base.sessions
+    assert !@base.foo
+    assert !@base.bar
   end
 
   it 'enables MethodOverride middleware when :methodoverride is enabled' do
-    @app.set :methodoverride, true
-    @app.put('/') { 'okay' }
+    @base.set :methodoverride, true
+    @base.put('/') { 'okay' }
+    @app = @base
     post '/', {'_method'=>'PUT'}, {}
     assert_equal 200, status
     assert_equal 'okay', body
   end
-end
 
-describe_option 'clean_trace' do
-  def clean_backtrace(trace)
-    @base.new.send(:clean_backtrace, trace)
-  end
+  describe 'clean_trace' do
+    def clean_backtrace(trace)
+      Sinatra::Base.new.send(:clean_backtrace, trace)
+    end
 
-  it 'is enabled on Base' do
-    assert @base.clean_trace?
-  end
+    it 'is enabled on Base' do
+      assert @base.clean_trace?
+    end
 
-  it 'is enabled on Default' do
-    assert @default.clean_trace?
-  end
+    it 'is enabled on Default' do
+      assert @default.clean_trace?
+    end
 
-  it 'does nothing when disabled' do
-    backtrace = [
-      "./lib/sinatra/base.rb",
-      "./myapp:42",
-      ("#{Gem.dir}/some/lib.rb" if defined?(Gem))
-    ].compact
-    @base.set :clean_trace, false
-    assert_equal backtrace, clean_backtrace(backtrace)
-  end
+    it 'does nothing when disabled' do
+      backtrace = [
+        "./lib/sinatra/base.rb",
+        "./myapp:42",
+        ("#{Gem.dir}/some/lib.rb" if defined?(Gem))
+      ].compact
 
-  it 'removes sinatra lib paths from backtrace when enabled' do
-    backtrace = [
-      "./lib/sinatra/base.rb",
-      "./lib/sinatra/compat.rb:42",
-      "./lib/sinatra/main.rb:55 in `foo'"
-    ]
-    assert clean_backtrace(backtrace).empty?
-  end
+      klass = Class.new(Sinatra::Base)
+      klass.disable :clean_trace
 
-  it 'removes ./ prefix from backtrace paths when enabled' do
-    assert_equal ['myapp.rb:42'], clean_backtrace(['./myapp.rb:42'])
-  end
+      assert_equal backtrace, klass.new.send(:clean_backtrace, backtrace)
+    end
 
-  if defined?(Gem)
-    it 'removes gem lib paths from backtrace when enabled' do
-      assert clean_backtrace(["#{Gem.dir}/some/lib"]).empty?
+    it 'removes sinatra lib paths from backtrace when enabled' do
+      backtrace = [
+        "./lib/sinatra/base.rb",
+        "./lib/sinatra/compat.rb:42",
+        "./lib/sinatra/main.rb:55 in `foo'"
+      ]
+      assert clean_backtrace(backtrace).empty?
+    end
+
+    it 'removes ./ prefix from backtrace paths when enabled' do
+      assert_equal ['myapp.rb:42'], clean_backtrace(['./myapp.rb:42'])
     end
-  end
-end
 
-describe_option 'run' do
-  it 'is disabled on Base' do
-    assert ! @base.run?
+    if defined?(Gem)
+      it 'removes gem lib paths from backtrace when enabled' do
+        assert clean_backtrace(["#{Gem.dir}/some/lib"]).empty?
+      end
+    end
   end
 
-  it 'is enabled on Default when not in test environment' do
-    assert @default.development?
-    assert @default.run?
+  describe 'run' do
+    it 'is disabled on Base' do
+      assert ! @base.run?
+    end
 
-    @default.set :environment, :development
-    assert @default.run?
-  end
+    it 'is enabled on Default when not in test environment' do
+      @default.set :environment, :development
+      assert @default.development?
+      assert @default.run?
 
-  # TODO: it 'is enabled when $0 == app_file'
-end
+      @default.set :environment, :development
+      assert @default.run?
+    end
 
-describe_option 'raise_errors' do
-  it 'is enabled on Base' do
-    assert @base.raise_errors?
+    # TODO: it 'is enabled when $0 == app_file'
   end
 
-  it 'is enabled on Default only in test' do
-    @default.set(:environment, :development)
-    assert @default.development?
-    assert ! @default.raise_errors?, "disabled development"
+  describe 'raise_errors' do
+    it 'is enabled on Base' do
+      assert @base.raise_errors?
+    end
 
-    @default.set(:environment, :production)
-    assert ! @default.raise_errors?
+    it 'is enabled on Default only in test' do
+      @default.set(:environment, :development)
+      assert @default.development?
+      assert ! @default.raise_errors?
 
-    @default.set(:environment, :test)
-    assert @default.raise_errors?
-  end
-end
+      @default.set(:environment, :production)
+      assert ! @default.raise_errors?
 
-describe_option 'dump_errors' do
-  it 'is disabled on Base' do
-    assert ! @base.dump_errors?
+      @default.set(:environment, :test)
+      assert @default.raise_errors?
+    end
   end
 
-  it 'is enabled on Default' do
-    assert @default.dump_errors?
-  end
+  describe 'show_exceptions' do
+    it 'is enabled on Default only in development' do
+      @base.set(:environment, :development)
+      assert @base.show_exceptions?
 
-  it 'dumps exception with backtrace to rack.errors' do
-    Sinatra::Default.disable(:raise_errors)
+      assert @default.development?
+      assert @default.show_exceptions?
 
-    mock_app(Sinatra::Default) {
-      error do
-        error = @env['rack.errors'].instance_variable_get(:@error)
-        error.rewind
+      @default.set(:environment, :test)
+      assert ! @default.show_exceptions?
 
-        error.read
-      end
+      @base.set(:environment, :production)
+      assert ! @base.show_exceptions?
+    end
 
-      get '/' do
-        raise
-      end
-    }
+    it 'returns a friendly 500' do
+      klass = Sinatra.new(Sinatra::Default)
+      mock_app(klass) {
+        enable :show_exceptions
 
-    get '/'
-    assert body.include?("RuntimeError") && body.include?("options_test.rb")
-  end
-end
+        get '/' do
+          raise StandardError
+        end
+      }
 
-describe_option 'sessions' do
-  it 'is disabled on Base' do
-    assert ! @base.sessions?
+      get '/'
+      assert_equal 500, status
+      assert body.include?("StandardError")
+      assert body.include?("<code>show_exceptions</code> option")
+    end
   end
 
-  it 'is disabled on Default' do
-    assert ! @default.sessions?
-  end
+  describe 'dump_errors' do
+    it 'is disabled on Base' do
+      assert ! @base.dump_errors?
+    end
 
-  # TODO: it 'uses Rack::Session::Cookie when enabled' do
-end
+    it 'is enabled on Default' do
+      assert @default.dump_errors?
+    end
 
-describe_option 'logging' do
-  it 'is disabled on Base' do
-    assert ! @base.logging?
-  end
+    it 'dumps exception with backtrace to rack.errors' do
+      klass = Sinatra.new(Sinatra::Default)
 
-  it 'is enabled on Default when not in test environment' do
-    assert @default.logging?
+      mock_app(klass) {
+        disable :raise_errors
 
-    @default.set :environment, :test
-    assert ! @default.logging
-  end
+        error do
+          error = @env['rack.errors'].instance_variable_get(:@error)
+          error.rewind
 
-  # TODO: it 'uses Rack::CommonLogger when enabled' do
-end
+          error.read
+        end
 
-describe_option 'static' do
-  it 'is disabled on Base' do
-    assert ! @base.static?
-  end
+        get '/' do
+          raise
+        end
+      }
 
-  it 'is enabled on Default' do
-    assert @default.static?
+      get '/'
+      assert body.include?("RuntimeError") && body.include?("options_test.rb")
+    end
   end
 
-  # TODO: it setup static routes if public is enabled
-  # TODO: however, that's already tested in static_test so...
-end
-
-describe_option 'host' do
-  it 'defaults to 0.0.0.0' do
-    assert_equal '0.0.0.0', @base.host
-    assert_equal '0.0.0.0', @default.host
-  end
-end
+  describe 'sessions' do
+    it 'is disabled on Base' do
+      assert ! @base.sessions?
+    end
 
-describe_option 'port' do
-  it 'defaults to 4567' do
-    assert_equal 4567, @base.port
-    assert_equal 4567, @default.port
-  end
-end
+    it 'is disabled on Default' do
+      assert ! @default.sessions?
+    end
 
-describe_option 'server' do
-  it 'is one of thin, mongrel, webrick' do
-    assert_equal %w[thin mongrel webrick], @base.server
-    assert_equal %w[thin mongrel webrick], @default.server
+    # TODO: it 'uses Rack::Session::Cookie when enabled' do
   end
-end
 
-describe_option 'app_file' do
-  it 'is nil' do
-    assert @base.app_file.nil?
-    assert @default.app_file.nil?
-  end
-end
+  describe 'logging' do
+    it 'is disabled on Base' do
+      assert ! @base.logging?
+    end
 
-describe_option 'root' do
-  it 'is nil if app_file is not set' do
-    assert @base.root.nil?
-    assert @default.root.nil?
-  end
+    it 'is enabled on Default when not in test environment' do
+      assert @default.logging?
 
-  it 'is equal to the expanded basename of app_file' do
-    @base.app_file = __FILE__
-    assert_equal File.expand_path(File.dirname(__FILE__)), @base.root
+      @default.set :environment, :test
+      assert ! @default.logging
+    end
 
-    @default.app_file = __FILE__
-    assert_equal File.expand_path(File.dirname(__FILE__)), @default.root
+    # TODO: it 'uses Rack::CommonLogger when enabled' do
   end
-end
 
-describe_option 'views' do
-  it 'is nil if root is not set' do
-    assert @base.views.nil?
-    assert @default.views.nil?
-  end
+  describe 'static' do
+    it 'is disabled on Base' do
+      assert ! @base.static?
+    end
 
-  it 'is set to root joined with views/' do
-    @base.root = File.dirname(__FILE__)
-    assert_equal File.dirname(__FILE__) + "/views", @base.views
+    it 'is enabled on Default' do
+      assert @default.static?
+    end
 
-    @default.root = File.dirname(__FILE__)
-    assert_equal File.dirname(__FILE__) + "/views", @default.views
+    # TODO: it setup static routes if public is enabled
+    # TODO: however, that's already tested in static_test so...
   end
-end
 
-describe_option 'public' do
-  it 'is nil if root is not set' do
-    assert @base.public.nil?
-    assert @default.public.nil?
+  describe 'host' do
+    it 'defaults to 0.0.0.0' do
+      assert_equal '0.0.0.0', @base.host
+      assert_equal '0.0.0.0', @default.host
+    end
   end
 
-  it 'is set to root joined with public/' do
-    @base.root = File.dirname(__FILE__)
-    assert_equal File.dirname(__FILE__) + "/public", @base.public
-
-    @default.root = File.dirname(__FILE__)
-    assert_equal File.dirname(__FILE__) + "/public", @default.public
+  describe 'port' do
+    it 'defaults to 4567' do
+      assert_equal 4567, @base.port
+      assert_equal 4567, @default.port
+    end
   end
-end
 
-describe_option 'reload' do
-  it 'is enabled when
-        app_file is set,
-        is not a rackup file,
-        and we are in development' do
-    @base.app_file = __FILE__
-    @base.set(:environment, :development)
-    assert @base.reload?
-
-    @default.app_file = __FILE__
-    @default.set(:environment, :development)
-    assert @default.reload?
+  describe 'server' do
+    it 'is one of thin, mongrel, webrick' do
+      assert_equal %w[thin mongrel webrick], @base.server
+      assert_equal %w[thin mongrel webrick], @default.server
+    end
   end
 
-  it 'is disabled if app_file is not set' do
-    assert ! @base.reload?
-    assert ! @default.reload?
+  describe 'app_file' do
+    it 'is nil' do
+      assert @base.app_file.nil?
+      assert @default.app_file.nil?
+    end
   end
 
-  it 'is disabled if app_file is a rackup file' do
-    @base.app_file = 'config.ru'
-    assert ! @base.reload?
+  describe 'root' do
+    it 'is nil if app_file is not set' do
+      assert @base.root.nil?
+      assert @default.root.nil?
+    end
+
+    it 'is equal to the expanded basename of app_file' do
+      @base.app_file = __FILE__
+      assert_equal File.expand_path(File.dirname(__FILE__)), @base.root
 
-    @default.app_file = 'config.ru'
-    assert ! @base.reload?
+      @default.app_file = __FILE__
+      assert_equal File.expand_path(File.dirname(__FILE__)), @default.root
+    end
   end
 
-  it 'is disabled if we are not in development' do
-    @base.set(:environment, :foo)
-    assert ! @base.reload
+  describe 'views' do
+    it 'is nil if root is not set' do
+      assert @base.views.nil?
+      assert @default.views.nil?
+    end
+
+    it 'is set to root joined with views/' do
+      @base.root = File.dirname(__FILE__)
+      assert_equal File.dirname(__FILE__) + "/views", @base.views
 
-    @default.set(:environment, :bar)
-    assert ! @default.reload
+      @default.root = File.dirname(__FILE__)
+      assert_equal File.dirname(__FILE__) + "/views", @default.views
+    end
   end
-end
 
-describe_option 'lock' do
-  it 'is enabled when reload is enabled' do
-    @base.enable(:reload)
-    assert @base.lock?
+  describe 'public' do
+    it 'is nil if root is not set' do
+      assert @base.public.nil?
+      assert @default.public.nil?
+    end
 
-    @default.enable(:reload)
-    assert @default.lock?
-  end
+    it 'is set to root joined with public/' do
+      @base.root = File.dirname(__FILE__)
+      assert_equal File.dirname(__FILE__) + "/public", @base.public
 
-  it 'is disabled when reload is disabled' do
-    @base.disable(:reload)
-    assert ! @base.lock?
+      @default.root = File.dirname(__FILE__)
+      assert_equal File.dirname(__FILE__) + "/public", @default.public
+    end
+  end
 
-    @default.disable(:reload)
-    assert ! @default.lock?
+  describe 'lock' do
+    it 'is disabled by default' do
+      assert ! @base.lock?
+    end
   end
 end