rack-contrib-with-working-jsonp 0.9.2.1

data/test/spec_rack_relative_redirect.rb

@@ -0,0 +1,78 @@
+require 'test/spec'
+require 'rack/mock'
+require 'rack/contrib/relative_redirect'
+require 'fileutils'
+
+context Rack::RelativeRedirect do
+  def request(opts={}, &block)
+    @def_status = opts[:status] if opts[:status]
+    @def_location = opts[:location] if opts[:location]
+    yield Rack::MockRequest.new(Rack::RelativeRedirect.new(@def_app, &opts[:block])).get(opts[:path]||@def_path, opts[:headers]||{})
+  end
+
+  setup do
+    @def_path = '/path/to/blah'
+    @def_status = 301
+    @def_location = '/redirect/to/blah'
+    @def_app = lambda { |env| [@def_status, {'Location' => @def_location}, [""]]}
+  end
+
+  specify "should make the location url an absolute url if currently a relative url" do
+    request do |r|
+      r.status.should.equal(301)
+      r.headers['Location'].should.equal('http://example.org/redirect/to/blah')
+    end
+    request(:status=>302, :location=>'/redirect') do |r|
+      r.status.should.equal(302)
+      r.headers['Location'].should.equal('http://example.org/redirect')
+    end
+  end
+
+  specify "should use the request path if the relative url is given and doesn't start with a slash" do
+    request(:status=>303, :location=>'redirect/to/blah') do |r|
+      r.status.should.equal(303)
+      r.headers['Location'].should.equal('http://example.org/path/to/redirect/to/blah')
+    end
+    request(:status=>303, :location=>'redirect') do |r|
+      r.status.should.equal(303)
+      r.headers['Location'].should.equal('http://example.org/path/to/redirect')
+    end
+  end
+
+  specify "should use a given block to make the url absolute" do
+    request(:block=>proc{|env, res| "https://example.org"}) do |r|
+      r.status.should.equal(301)
+      r.headers['Location'].should.equal('https://example.org/redirect/to/blah')
+    end
+    request(:status=>303, :location=>'/redirect', :block=>proc{|env, res| "https://e.org:9999/blah"}) do |r|
+      r.status.should.equal(303)
+      r.headers['Location'].should.equal('https://e.org:9999/blah/redirect')
+    end
+  end
+
+  specify "should not modify the location url unless the response is a redirect" do
+    status = 200
+    @def_app = lambda { |env| [status, {'Content-Type' => "text/html"}, [""]]}
+    request do |r|
+      r.status.should.equal(200)
+      r.headers.should.not.include?('Location')
+    end
+    status = 404
+    @def_app = lambda { |env| [status, {'Content-Type' => "text/html", 'Location' => 'redirect'}, [""]]}
+    request do |r|
+      r.status.should.equal(404)
+      r.headers['Location'].should.equal('redirect')
+    end
+  end
+
+  specify "should not modify the location url if it is already an absolute url" do
+    request(:location=>'https://example.org/') do |r|
+      r.status.should.equal(301)
+      r.headers['Location'].should.equal('https://example.org/')
+    end
+    request(:status=>302, :location=>'https://e.org:9999/redirect') do |r|
+      r.status.should.equal(302)
+      r.headers['Location'].should.equal('https://e.org:9999/redirect')
+    end
+  end
+end

data/test/spec_rack_response_cache.rb

@@ -0,0 +1,137 @@
+require 'test/spec'
+require 'rack/mock'
+require 'rack/contrib/response_cache'
+require 'fileutils'
+
+context Rack::ResponseCache do
+  F = ::File
+
+  def request(opts={}, &block)
+    Rack::MockRequest.new(Rack::ResponseCache.new(block||@def_app, opts[:cache]||@cache, &opts[:rc_block])).send(opts[:meth]||:get, opts[:path]||@def_path, opts[:headers]||{})
+  end
+
+  setup do
+    @cache = {}
+    @def_disk_cache = F.join(F.dirname(__FILE__), 'response_cache_test_disk_cache')
+    @def_value = ["rack-response-cache"]
+    @def_path = '/path/to/blah'
+    @def_app = lambda { |env| [200, {'Content-Type' => env['CT'] || 'text/html'}, @def_value]}
+  end
+  teardown do
+    FileUtils.rm_rf(@def_disk_cache)
+  end
+
+  specify "should cache results to disk if cache is a string" do
+    request(:cache=>@def_disk_cache)
+    F.read(F.join(@def_disk_cache, 'path', 'to', 'blah.html')).should.equal @def_value.first
+    request(:path=>'/path/3', :cache=>@def_disk_cache)
+    F.read(F.join(@def_disk_cache, 'path', '3.html')).should.equal @def_value.first
+  end
+
+  specify "should cache results to given cache if cache is not a string" do
+    request
+    @cache.should.equal('/path/to/blah.html'=>@def_value)
+    request(:path=>'/path/3')
+    @cache.should.equal('/path/to/blah.html'=>@def_value, '/path/3.html'=>@def_value)
+  end
+
+  specify "should not CACHE RESults if request method is not GET" do
+    request(:meth=>:post)
+    @cache.should.equal({})
+    request(:meth=>:put)
+    @cache.should.equal({})
+    request(:meth=>:delete)
+    @cache.should.equal({})
+  end
+
+  specify "should not cache results if there is a query string" do
+    request(:path=>'/path/to/blah?id=1')
+    @cache.should.equal({})
+    request(:path=>'/path/to/?id=1')
+    @cache.should.equal({})
+    request(:path=>'/?id=1')
+    @cache.should.equal({})
+  end
+
+  specify "should cache results if there is an empty query string" do
+    request(:path=>'/?')
+    @cache.should.equal('/index.html'=>@def_value)
+  end
+
+  specify "should not cache results if the request is not sucessful (status 200)" do
+    request{|env| [404, {'Content-Type' => 'text/html'}, ['']]}
+    @cache.should.equal({})
+    request{|env| [500, {'Content-Type' => 'text/html'}, ['']]}
+    @cache.should.equal({})
+    request{|env| [302, {'Content-Type' => 'text/html'}, ['']]}
+    @cache.should.equal({})
+  end
+
+  specify "should not cache results if the block returns nil or false" do
+    request(:rc_block=>proc{false})
+    @cache.should.equal({})
+    request(:rc_block=>proc{nil})
+    @cache.should.equal({})
+  end
+
+  specify "should cache results to path returned by block" do
+    request(:rc_block=>proc{"1"})
+    @cache.should.equal("1"=>@def_value)
+    request(:rc_block=>proc{"2"})
+    @cache.should.equal("1"=>@def_value, "2"=>@def_value)
+  end
+
+  specify "should pass the environment and response to the block" do
+    e, r = nil, nil
+    request(:rc_block=>proc{|env,res| e, r = env, res; nil})
+    e['PATH_INFO'].should.equal @def_path
+    e['REQUEST_METHOD'].should.equal 'GET'
+    e['QUERY_STRING'].should.equal ''
+    r.should.equal([200, {"Content-Type"=>"text/html"}, ["rack-response-cache"]])
+  end
+
+  specify "should unescape the path by default" do
+    request(:path=>'/path%20with%20spaces')
+    @cache.should.equal('/path with spaces.html'=>@def_value)
+    request(:path=>'/path%3chref%3e')
+    @cache.should.equal('/path with spaces.html'=>@def_value, '/path<href>.html'=>@def_value)
+  end
+
+  specify "should cache html, css, and xml responses by default" do
+    request(:path=>'/a')
+    @cache.should.equal('/a.html'=>@def_value)
+    request(:path=>'/b', :headers=>{'CT'=>'text/xml'})
+    @cache.should.equal('/a.html'=>@def_value, '/b.xml'=>@def_value)
+    request(:path=>'/c', :headers=>{'CT'=>'text/css'})
+    @cache.should.equal('/a.html'=>@def_value, '/b.xml'=>@def_value, '/c.css'=>@def_value)
+  end
+
+  specify "should cache responses by default with the extension added if not already present" do
+    request(:path=>'/a.html')
+    @cache.should.equal('/a.html'=>@def_value)
+    request(:path=>'/b.xml', :headers=>{'CT'=>'text/xml'})
+    @cache.should.equal('/a.html'=>@def_value, '/b.xml'=>@def_value)
+    request(:path=>'/c.css', :headers=>{'CT'=>'text/css'})
+    @cache.should.equal('/a.html'=>@def_value, '/b.xml'=>@def_value, '/c.css'=>@def_value)
+  end
+
+  specify "should not delete existing extensions" do
+    request(:path=>'/d.css', :headers=>{'CT'=>'text/html'})
+    @cache.should.equal('/d.css.html'=>@def_value)
+  end
+
+  specify "should cache html responses with empty basename to index.html by default" do
+    request(:path=>'/')
+    @cache.should.equal('/index.html'=>@def_value)
+    request(:path=>'/blah/')
+    @cache.should.equal('/index.html'=>@def_value, '/blah/index.html'=>@def_value)
+    request(:path=>'/blah/2/')
+    @cache.should.equal('/index.html'=>@def_value, '/blah/index.html'=>@def_value, '/blah/2/index.html'=>@def_value)
+  end
+
+  specify "should raise an error if a cache argument is not provided" do
+    app = Rack::Builder.new{use Rack::ResponseCache; run lambda { |env| [200, {'Content-Type' => 'text/plain'}, Rack::Request.new(env).POST]}}
+    proc{Rack::MockRequest.new(app).get('/')}.should.raise(ArgumentError)
+  end
+
+end

data/test/spec_rack_sendfile.rb

@@ -0,0 +1,86 @@
+require 'test/spec'
+require 'rack/mock'
+require 'rack/contrib/sendfile'
+
+context "Rack::File" do
+  specify "should respond to #to_path" do
+    Rack::File.new(Dir.pwd).should.respond_to :to_path
+  end
+end
+
+context "Rack::Sendfile" do
+  def sendfile_body
+    res = ['Hello World']
+    def res.to_path ; "/tmp/hello.txt" ; end
+    res
+  end
+
+  def simple_app(body=sendfile_body)
+    lambda { |env| [200, {'Content-Type' => 'text/plain'}, body] }
+  end
+
+  def sendfile_app(body=sendfile_body)
+    Rack::Sendfile.new(simple_app(body))
+  end
+
+  setup do
+    @request = Rack::MockRequest.new(sendfile_app)
+  end
+
+  def request(headers={})
+    yield @request.get('/', headers)
+  end
+
+  specify "does nothing when no X-Sendfile-Type header present" do
+    request do |response|
+      response.should.be.ok
+      response.body.should.equal 'Hello World'
+      response.headers.should.not.include 'X-Sendfile'
+    end
+  end
+
+  specify "sets X-Sendfile response header and discards body" do
+    request 'HTTP_X_SENDFILE_TYPE' => 'X-Sendfile' do |response|
+      response.should.be.ok
+      response.body.should.be.empty
+      response.headers['X-Sendfile'].should.equal '/tmp/hello.txt'
+    end
+  end
+
+  specify "sets X-Lighttpd-Send-File response header and discards body" do
+    request 'HTTP_X_SENDFILE_TYPE' => 'X-Lighttpd-Send-File' do |response|
+      response.should.be.ok
+      response.body.should.be.empty
+      response.headers['X-Lighttpd-Send-File'].should.equal '/tmp/hello.txt'
+    end
+  end
+
+  specify "sets X-Accel-Redirect response header and discards body" do
+    headers = {
+      'HTTP_X_SENDFILE_TYPE' => 'X-Accel-Redirect',
+      'HTTP_X_ACCEL_MAPPING' => '/tmp/=/foo/bar/'
+    }
+    request headers do |response|
+      response.should.be.ok
+      response.body.should.be.empty
+      response.headers['X-Accel-Redirect'].should.equal '/foo/bar/hello.txt'
+    end
+  end
+
+  specify 'writes to rack.error when no X-Accel-Mapping is specified' do
+    request 'HTTP_X_SENDFILE_TYPE' => 'X-Accel-Redirect' do |response|
+      response.should.be.ok
+      response.body.should.equal 'Hello World'
+      response.headers.should.not.include 'X-Accel-Redirect'
+      response.errors.should.include 'X-Accel-Mapping'
+    end
+  end
+
+  specify 'does nothing when body does not respond to #to_path' do
+    @request = Rack::MockRequest.new(sendfile_app(['Not a file...']))
+    request 'HTTP_X_SENDFILE_TYPE' => 'X-Sendfile' do |response|
+      response.body.should.equal 'Not a file...'
+      response.headers.should.not.include 'X-Sendfile'
+    end
+  end
+end

metadata

@@ -0,0 +1,174 @@
+--- !ruby/object:Gem::Specification 
+name: rack-contrib-with-working-jsonp
+version: !ruby/object:Gem::Version 
+  version: 0.9.2.1
+platform: ruby
+authors: 
+- rack-devel
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2009-03-07 00:00:00 -05:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: rack
+  type: :runtime
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: 0.9.1
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: test-spec
+  type: :development
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        version: 0.9.0
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: tmail
+  type: :development
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "1.2"
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: json
+  type: :development
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "1.1"
+    version: 
+description: Contributed Rack Middleware and Utilities with working JSON-P (edge rack-contrib as of Oct. 26)
+email: rack-devel@googlegroups.com
+executables: []
+
+extensions: []
+
+extra_rdoc_files: 
+- README.rdoc
+- COPYING
+files: 
+- COPYING
+- README.rdoc
+- Rakefile
+- lib/rack/contrib.rb
+- lib/rack/contrib/accept_format.rb
+- lib/rack/contrib/backstage.rb
+- lib/rack/contrib/bounce_favicon.rb
+- lib/rack/contrib/callbacks.rb
+- lib/rack/contrib/config.rb
+- lib/rack/contrib/cookies.rb
+- lib/rack/contrib/csshttprequest.rb
+- lib/rack/contrib/deflect.rb
+- lib/rack/contrib/etag.rb
+- lib/rack/contrib/evil.rb
+- lib/rack/contrib/garbagecollector.rb
+- lib/rack/contrib/jsonp.rb
+- lib/rack/contrib/lighttpd_script_name_fix.rb
+- lib/rack/contrib/locale.rb
+- lib/rack/contrib/mailexceptions.rb
+- lib/rack/contrib/nested_params.rb
+- lib/rack/contrib/not_found.rb
+- lib/rack/contrib/post_body_content_type_parser.rb
+- lib/rack/contrib/proctitle.rb
+- lib/rack/contrib/profiler.rb
+- lib/rack/contrib/relative_redirect.rb
+- lib/rack/contrib/response_cache.rb
+- lib/rack/contrib/route_exceptions.rb
+- lib/rack/contrib/sendfile.rb
+- lib/rack/contrib/signals.rb
+- lib/rack/contrib/time_zone.rb
+- rack-contrib.gemspec
+- test/404.html
+- test/Maintenance.html
+- test/mail_settings.rb
+- test/spec_rack_accept_format.rb
+- test/spec_rack_backstage.rb
+- test/spec_rack_callbacks.rb
+- test/spec_rack_config.rb
+- test/spec_rack_contrib.rb
+- test/spec_rack_csshttprequest.rb
+- test/spec_rack_deflect.rb
+- test/spec_rack_etag.rb
+- test/spec_rack_evil.rb
+- test/spec_rack_garbagecollector.rb
+- test/spec_rack_jsonp.rb
+- test/spec_rack_lighttpd_script_name_fix.rb
+- test/spec_rack_mailexceptions.rb
+- test/spec_rack_nested_params.rb
+- test/spec_rack_not_found.rb
+- test/spec_rack_post_body_content_type_parser.rb
+- test/spec_rack_proctitle.rb
+- test/spec_rack_profiler.rb
+- test/spec_rack_relative_redirect.rb
+- test/spec_rack_response_cache.rb
+- test/spec_rack_sendfile.rb
+has_rdoc: true
+homepage: http://github.com/rack/rack-contrib/
+licenses: []
+
+post_install_message: 
+rdoc_options: 
+- --line-numbers
+- --inline-source
+- --title
+- rack-contrib
+- --main
+- README
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
+requirements: []
+
+rubyforge_project: 
+rubygems_version: 1.3.5
+signing_key: 
+specification_version: 2
+summary: Didn't have time to wait for a new release.
+test_files: 
+- test/spec_rack_accept_format.rb
+- test/spec_rack_backstage.rb
+- test/spec_rack_callbacks.rb
+- test/spec_rack_config.rb
+- test/spec_rack_contrib.rb
+- test/spec_rack_csshttprequest.rb
+- test/spec_rack_deflect.rb
+- test/spec_rack_etag.rb
+- test/spec_rack_evil.rb
+- test/spec_rack_garbagecollector.rb
+- test/spec_rack_jsonp.rb
+- test/spec_rack_lighttpd_script_name_fix.rb
+- test/spec_rack_mailexceptions.rb
+- test/spec_rack_nested_params.rb
+- test/spec_rack_not_found.rb
+- test/spec_rack_post_body_content_type_parser.rb
+- test/spec_rack_proctitle.rb
+- test/spec_rack_profiler.rb
+- test/spec_rack_relative_redirect.rb
+- test/spec_rack_response_cache.rb
+- test/spec_rack_sendfile.rb