bmizerany-sinatra 0.9.0.4 → 0.9.0.5

data/lib/sinatra/compat.rb

@@ -9,7 +9,7 @@ require 'sinatra/base'
 require 'sinatra/main'
 
 # Like Kernel#warn but outputs the location that triggered the warning.
-def sinatra_warn(*message)
+def sinatra_warn(*message) #:nodoc:
   line = caller.
     detect { |line| line !~ /(?:lib\/sinatra\/|__DELEGATE__)/ }.
     sub(/:in .*/, '')
@@ -34,25 +34,27 @@ end
 # technically a Sinatra issue but many Sinatra apps access the old
 # MIME_TYPES constants due to Sinatra example code.
 require 'rack/file'
-class Rack::File
-  def self.const_missing(const_name)
-    if const_name == :MIME_TYPES
-      hash = Hash.new { |hash,key| Rack::Mime::MIME_TYPES[".#{key}"] }
-      const_set :MIME_TYPES, hash
-      sinatra_warn 'Rack::File::MIME_TYPES is deprecated; use Rack::Mime instead.'
-      hash
-    else
-      super
+module Rack #:nodoc:
+  class File #:nodoc:
+    def self.const_missing(const_name)
+      if const_name == :MIME_TYPES
+        hash = Hash.new { |hash,key| Rack::Mime::MIME_TYPES[".#{key}"] }
+        const_set :MIME_TYPES, hash
+        sinatra_warn 'Rack::File::MIME_TYPES is deprecated; use Rack::Mime instead.'
+        hash
+      else
+        super
+      end
     end
   end
 end
 
 module Sinatra
-  module Compat
+  module Compat #:nodoc:
   end
 
   # Make Sinatra::EventContext an alias for Sinatra::Default to unbreak plugins.
-  def self.const_missing(const_name)
+  def self.const_missing(const_name) #:nodoc:
     if const_name == :EventContext
       const_set :EventContext, Sinatra::Default
       sinatra_warn 'Sinatra::EventContext is deprecated; use Sinatra::Default instead.'
@@ -73,7 +75,7 @@ module Sinatra
   end
 
   class Default < Base
-    def self.const_missing(const_name)
+    def self.const_missing(const_name) #:nodoc:
       if const_name == :FORWARD_METHODS
         sinatra_warn 'Sinatra::Application::FORWARD_METHODS is deprecated;',
           'use Sinatra::Delegator::METHODS instead.'
@@ -107,7 +109,7 @@ module Sinatra
     # Throwing halt with a Symbol and the to_result convention are
     # deprecated. Override the invoke method to detect those types of return
     # values.
-    def invoke(&block)
+    def invoke(&block) #:nodoc:
       res = super
       case
       when res.kind_of?(Symbol)
@@ -121,7 +123,7 @@ module Sinatra
       res
     end
 
-    def options
+    def options #:nodoc:
       Options.new(self.class)
     end
 
@@ -184,7 +186,7 @@ module Sinatra
     end
 
     # Deprecated. Missing messages are no longer delegated to @response.
-    def method_missing(name, *args, &b)
+    def method_missing(name, *args, &b) #:nodoc:
       if @response.respond_to?(name)
         sinatra_warn "The '#{name}' method is deprecated; use 'response.#{name}' instead."
         @response.send(name, *args, &b)

data/lib/sinatra/main.rb

@@ -2,6 +2,10 @@ require 'sinatra/base'
 
 module Sinatra
   class Default < Base
+
+    # we assume that the first file that requires 'sinatra' is the
+    # app_file. all other path related options are calculated based
+    # on this path by default.
     set :app_file, lambda {
       ignore = [
         /lib\/sinatra.*\.rb$/, # all sinatra code
@@ -32,10 +36,6 @@ end
 
 include Sinatra::Delegator
 
-def helpers(&block)
-  Sinatra::Application.send :class_eval, &block
-end
-
 def mime(ext, type)
   ext = ".#{ext}" unless ext.to_s[0] == ?.
   Rack::Mime::MIME_TYPES[ext.to_s] = type

data/lib/sinatra/test.rb

@@ -7,7 +7,7 @@ module Sinatra
 
     attr_reader :app, :request, :response
 
-    def test_request(verb, path, *args)
+    def make_request(verb, path, *args)
       @app = Sinatra::Application if @app.nil? && defined?(Sinatra::Application)
       fail "@app not set - cannot make request" if @app.nil?
       @request = Rack::MockRequest.new(@app)
@@ -38,14 +38,14 @@ module Sinatra
       @response = @request.request(verb, path, opts)
     end
 
-    def get(path, *args, &b)  ; test_request('GET', path, *args, &b) ; end
-    def head(path, *args, &b) ; test_request('HEAD', path, *args, &b) ; end
-    def post(path, *args, &b) ; test_request('POST', path, *args, &b) ; end
-    def put(path, *args, &b)  ; test_request('PUT', path, *args, &b) ; end
-    def delete(path, *args, &b) ; test_request('DELETE', path, *args, &b) ; end
+    def get(path, *args, &b)  ; make_request('GET', path, *args, &b) ; end
+    def head(path, *args, &b) ; make_request('HEAD', path, *args, &b) ; end
+    def post(path, *args, &b) ; make_request('POST', path, *args, &b) ; end
+    def put(path, *args, &b)  ; make_request('PUT', path, *args, &b) ; end
+    def delete(path, *args, &b) ; make_request('DELETE', path, *args, &b) ; end
 
     def follow!
-      test_request 'GET', @response.location
+      make_request 'GET', @response.location
     end
 
     def body ; @response.body ; end
@@ -65,6 +65,8 @@ module Sinatra
       super || (@response && @response.respond_to?(symbol, include_private))
     end
 
+  private
+
     RACK_OPT_NAMES = {
       :accept => "HTTP_ACCEPT",
       :agent => "HTTP_USER_AGENT",
@@ -103,7 +105,7 @@ module Sinatra
         eval <<-RUBY, binding, __FILE__, __LINE__
         def #{verb}_it(*args, &block)
           sinatra_warn "The #{verb}_it method is deprecated; use #{verb} instead."
-          test_request('#{verb.upcase}', *args, &block)
+          make_request('#{verb.upcase}', *args, &block)
         end
         RUBY
       end

data/sinatra.gemspec

@@ -3,8 +3,8 @@ Gem::Specification.new do |s|
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
 
   s.name = 'sinatra'
-  s.version = '0.9.0.4'
-  s.date = '2009-01-25'
+  s.version = '0.9.0.5'
+  s.date = '2009-02-10'
 
   s.description = "Classy web-development dressed in a DSL"
   s.summary     = "Classy web-development dressed in a DSL"
@@ -22,6 +22,7 @@ Gem::Specification.new do |s|
     compat/app_test.rb
     compat/application_test.rb
     compat/builder_test.rb
+    compat/compat_test.rb
     compat/custom_error_test.rb
     compat/erb_test.rb
     compat/events_test.rb
@@ -69,6 +70,7 @@ Gem::Specification.new do |s|
     test/builder_test.rb
     test/data/reload_app_file.rb
     test/erb_test.rb
+    test/extensions_test.rb
     test/filter_test.rb
     test/haml_test.rb
     test/helper.rb
@@ -78,12 +80,15 @@ Gem::Specification.new do |s|
     test/options_test.rb
     test/reload_test.rb
     test/request_test.rb
+    test/response_test.rb
     test/result_test.rb
     test/routing_test.rb
     test/sass_test.rb
+    test/server_test.rb
     test/sinatra_test.rb
     test/static_test.rb
     test/templates_test.rb
+    test/test_test.rb
     test/views/hello.builder
     test/views/hello.erb
     test/views/hello.haml

data/test/base_test.rb

@@ -65,4 +65,17 @@ describe 'Sinatra::Base' do
     assert response.ok?
     assert_equal 'not foo', response.body
   end
+
+  it "makes redirecting back pretty" do
+    app = mock_app {
+      get '/foo' do
+        redirect back
+      end
+    }
+
+    request  = Rack::MockRequest.new(app)
+    response = request.get('/foo', 'HTTP_REFERER' => 'http://github.com')
+    assert response.redirect?
+    assert_equal "http://github.com", response.location
+  end
 end

data/test/extensions_test.rb

@@ -0,0 +1,63 @@
+require File.dirname(__FILE__) + '/helper'
+
+describe 'Registering extensions' do
+  module FooExtensions
+    def foo
+    end
+
+    private
+      def im_hiding_in_ur_foos
+      end
+  end
+
+  module BarExtensions
+    def bar
+    end
+  end
+
+  module BazExtensions
+    def baz
+    end
+  end
+
+  module QuuxExtensions
+    def quux
+    end
+  end
+
+  it 'will add the methods to the DSL for the class in which you register them and its subclasses' do
+    Sinatra::Base.register FooExtensions
+    assert Sinatra::Base.respond_to?(:foo)
+
+    Sinatra::Default.register BarExtensions
+    assert Sinatra::Default.respond_to?(:bar)
+    assert Sinatra::Default.respond_to?(:foo)
+    assert !Sinatra::Base.respond_to?(:bar)
+  end
+
+  it 'allows extending by passing a block' do
+    Sinatra::Base.register {
+      def im_in_ur_anonymous_module; end
+    }
+    assert Sinatra::Base.respond_to?(:im_in_ur_anonymous_module)
+  end
+
+  it 'will make sure any public methods added via Default#register are delegated to Sinatra::Delegator' do
+    Sinatra::Default.register FooExtensions
+    assert Sinatra::Delegator.private_instance_methods.
+      map { |m| m.to_sym }.include?(:foo)
+    assert !Sinatra::Delegator.private_instance_methods.
+      map { |m| m.to_sym }.include?(:im_hiding_in_ur_foos)
+  end
+
+  it 'will not delegate methods on Base#register' do
+    Sinatra::Base.register QuuxExtensions
+    assert !Sinatra::Delegator.private_instance_methods.include?("quux")
+  end
+
+  it 'will extend the Sinatra::Default application by default' do
+    Sinatra.register BazExtensions
+    assert !Sinatra::Base.respond_to?(:baz)
+    assert Sinatra::Default.respond_to?(:baz)
+  end
+end

data/test/helper.rb

@@ -32,9 +32,12 @@ end
 #
 def describe(*args, &block)
   return super unless (name = args.first) && block
-  klass = Class.new(Test::Unit::TestCase) do
+  name = "#{name.gsub(/\W/, '')}Test"
+  Object.send :const_set, name, Class.new(Test::Unit::TestCase)
+  klass = Object.const_get(name)
+  klass.class_eval do
     def self.it(name, &block)
-      define_method("test_#{name.gsub(/\W/,'_')}", &block)
+      define_method("test_#{name.gsub(/\W/,'_').downcase}", &block)
     end
     def self.xspecify(*args) end
     def self.before(&block) define_method(:setup, &block)    end
@@ -42,3 +45,11 @@ def describe(*args, &block)
   end
   klass.class_eval &block
 end
+
+# Do not output warnings for the duration of the block.
+def silence_warnings
+  $VERBOSE, v = nil, $VERBOSE
+  yield
+ensure
+  $VERBOSE = v
+end

data/test/helpers_test.rb

@@ -379,3 +379,61 @@ describe 'Helpers#etag' do
     assert_equal 'W/"FOO"', response['ETag']
   end
 end
+
+module HelperOne; def one; '1'; end; end
+module HelperTwo; def two; '2'; end; end
+
+describe 'Adding new helpers' do
+  it 'takes a list of modules to mix into the app' do
+    mock_app {
+      helpers HelperOne, HelperTwo
+
+      get '/one' do
+        one
+      end
+
+      get '/two' do
+        two
+      end
+    }
+
+    get '/one'
+    assert_equal '1', body
+
+    get '/two'
+    assert_equal '2', body
+  end
+
+  it 'takes a block to mix into the app' do
+    mock_app {
+      helpers do
+        def foo
+          'foo'
+        end
+      end
+
+      get '/' do
+        foo
+      end
+    }
+
+    get '/'
+    assert_equal 'foo', body
+  end
+
+  it 'evaluates the block in class context so that methods can be aliased' do
+    mock_app {
+      helpers do
+        alias_method :h, :escape_html
+      end
+
+      get '/' do
+        h('42 < 43')
+      end
+    }
+
+    get '/'
+    assert ok?
+    assert_equal '42 &lt; 43', body
+  end
+end

data/test/mapped_error_test.rb

@@ -1,9 +1,12 @@
 require File.dirname(__FILE__) + '/helper'
 
-describe 'Exception Mappings' do
-  class FooError < RuntimeError
-  end
+class FooError < RuntimeError
+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
@@ -80,9 +83,6 @@ describe 'Exception Mappings' do
     assert_equal 404, status
   end
 
-  class FooNotFound < Sinatra::NotFound
-  end
-
   it "cascades for subclasses of Sinatra::NotFound" do
     mock_app {
       set :raise_errors, true

data/test/middleware_test.rb

@@ -4,7 +4,9 @@ describe "Middleware" do
   before do
     @app = mock_app(Sinatra::Default) {
       get '/*' do
-        response.headers['X-Tests'] = env['test.ran'].join(', ')
+        response.headers['X-Tests'] = env['test.ran'].
+          map { |n| n.split('::').last }.
+          join(', ')
         env['PATH_INFO']
       end
     }

data/test/response_test.rb

@@ -0,0 +1,42 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/helper'
+
+describe 'Sinatra::Response' do
+  before do
+    @response = Sinatra::Response.new
+  end
+
+  it "initializes with 200, text/html, and empty body" do
+    assert_equal 200, @response.status
+    assert_equal 'text/html', @response['Content-Type']
+    assert_equal [], @response.body
+  end
+
+  it 'uses case insensitive headers' do
+    @response['content-type'] = 'application/foo'
+    assert_equal 'application/foo', @response['Content-Type']
+    assert_equal 'application/foo', @response['CONTENT-TYPE']
+  end
+
+  it 'writes to body' do
+    @response.body = 'Hello'
+    @response.write ' World'
+    assert_equal 'Hello World', @response.body
+  end
+
+  [204, 304].each do |status_code|
+    it "removes the Content-Type header and body when response status is #{status_code}" do
+      @response.status = status_code
+      @response.body = ['Hello World']
+      assert_equal [status_code, {}, []], @response.finish
+    end
+  end
+
+  it 'Calculates the Content-Length using the bytesize of the body' do
+    @response.body = ['Hello', 'World!', '✈']
+    status, headers, body = @response.finish
+    assert_equal '14', headers['Content-Length']
+    assert_equal @response.body, body
+  end
+end