tennpipes-base 3.6.6

data/test/test_reloader_complex.rb

@@ -0,0 +1,74 @@
+require File.expand_path(File.dirname(__FILE__) + '/helper')
+require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/complex')
+
+describe "ComplexReloader" do
+
+  describe 'for complex reload functionality' do
+    before do
+      Tennpipes.clear!
+      Tennpipes.mount("complex_1_demo").to("/complex_1_demo")
+      Tennpipes.mount("complex_2_demo").to("/complex_2_demo")
+    end
+
+    it 'should correctly instantiate Complex(1-2)Demo fixture' do
+      assert_equal ["/complex_1_demo", "/complex_2_demo"], Tennpipes.mounted_apps.map(&:uri_root)
+      assert_equal ["complex_1_demo", "complex_2_demo"], Tennpipes.mounted_apps.map(&:name)
+      assert Complex1Demo.reload?
+      assert Complex2Demo.reload?
+      assert_match %r{fixtures/apps/complex.rb}, Complex1Demo.app_file
+      assert_match %r{fixtures/apps/complex.rb}, Complex2Demo.app_file
+    end
+
+    it 'should correctly reload Complex(1-2)Demo fixture' do
+      assert_match %r{fixtures/apps/complex.rb}, Complex1Demo.app_file
+      @app = Tennpipes.application
+
+      get "/"
+      assert_equal 404, status
+
+      get "/complex_1_demo"
+      assert_equal "Given random #{LibDemo.give_me_a_random}", body
+
+      get "/complex_2_demo"
+      assert_equal 200, status
+
+      get "/complex_1_demo/old"
+      assert_equal 200, status
+
+      get "/complex_2_demo/old"
+      assert_equal 200, status
+
+      get "/complex_2_demo/var/destroy"
+      assert_equal '{}', body
+
+      new_phrase = "The magick number is: #{rand(2**255)}!"
+      buffer     = File.read(Complex1Demo.app_file)
+      new_buffer = buffer.sub(/The magick number is: \d+!/, new_phrase)
+      new_buffer.sub!(/get\(:destroy\)/, 'get(:destroy, :with => :id)')
+      begin
+        File.open(Complex1Demo.app_file, "w") { |f| f.write(new_buffer) }
+        Time.stub(:now, Time.now + 2) { get "/complex_2_demo" }
+        assert_equal new_phrase, body
+
+        # Re-Check that we didn't forget any route
+        get "/complex_1_demo"
+        assert_equal "Given random #{LibDemo.give_me_a_random}", body
+
+        get "/complex_2_demo"
+        assert_equal 200, status
+
+        get "/complex_1_demo/old"
+        assert_equal 200, status
+
+        get "/complex_2_demo/old"
+        assert_equal 200, status
+
+        get "/complex_2_demo/var/destroy/variable"
+        assert_equal '{"id"=>"variable"}', body
+      ensure
+        # Now we need to prevent to commit a new changed file so we revert it
+        File.open(Complex1Demo.app_file, "w") { |f| f.write(buffer) }
+      end
+    end
+  end
+end

data/test/test_reloader_external.rb

@@ -0,0 +1,21 @@
+require File.expand_path(File.dirname(__FILE__) + '/helper')
+require File.expand_path(File.dirname(__FILE__) + '/fixtures/reloadable_apps/main/app')
+
+describe "ExternalReloader" do
+  describe "for external app" do
+    before do
+      Tennpipes.clear!
+      Tennpipes.mount("ReloadableApp::External").to("/reloadable/external")
+      Tennpipes.mount("ReloadableApp::Main").to("/reloadable")
+      Tennpipes.load!
+    end
+
+    it "should avoid reloading the file if its path is not started with Tennpipes.root" do
+      @app = Tennpipes.application
+      Tennpipes.stub(:root, File.expand_path(File.dirname(__FILE__) + '/fixtures/reloadable_apps/main')) do
+        get "/reloadable/external/base"
+      end
+      assert_equal "Hello External App", body
+    end
+  end
+end

data/test/test_reloader_simple.rb

@@ -0,0 +1,101 @@
+require File.expand_path(File.dirname(__FILE__) + '/helper')
+require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/simple')
+
+describe "SimpleReloader" do
+
+  describe 'for simple reset functionality' do
+
+    it 'should reset routes' do
+      mock_app do
+        (1..10).each do |i|
+          get("/#{i}") { "Foo #{i}" }
+        end
+      end
+      (1..10).each do |i|
+        get "/#{i}"
+        assert_equal "Foo #{i}", body
+      end
+      @app.reset_routes!
+      (1..10).each do |i|
+        get "/#{i}"
+        assert_equal 404, status
+      end
+    end
+
+    it 'should keep sinatra routes on development' do
+      mock_app do
+        set :environment, :development
+        get("/"){ "ok" }
+      end
+      assert_equal :development, @app.environment
+      get "/"
+      assert_equal 200, status
+      get "/__sinatra__/404.png"
+      assert_equal 200, status
+      assert_match /image\/png/, response["Content-Type"]
+      @app.reset_routes!
+      get "/"
+      assert_equal 404, status
+      get "/__sinatra__/404.png"
+      assert_equal 200, status
+      assert_match /image\/png/, response["Content-Type"]
+    end
+  end
+
+  describe 'for simple reload functionality' do
+    before do
+      Tennpipes.clear!
+      Tennpipes.mount("simple_demo").to("/")
+      Tennpipes.reload!
+    end
+
+    it 'should correctly instantiate SimpleDemo fixture' do
+      assert_equal ["simple_demo"], Tennpipes.mounted_apps.map(&:name)
+      assert SimpleDemo.reload?
+      assert_match %r{fixtures/apps/simple.rb}, SimpleDemo.app_file
+    end
+
+    it 'should correctly reload SimpleDemo fixture' do
+      @app = SimpleDemo
+      get "/"
+      assert ok?
+      new_phrase = "The magick number is: #{rand(2**255)}!"
+      buffer     = File.read(SimpleDemo.app_file)
+      new_buffer = buffer.sub(/The magick number is: \d+!/, new_phrase)
+      begin
+        File.open(SimpleDemo.app_file, "w") { |f| f.write(new_buffer) }
+        Time.stub(:now, Time.now + 2) { get "/" }
+        assert_equal new_phrase, body
+      ensure
+        File.open(SimpleDemo.app_file, "w") { |f| f.write(buffer) }
+        Tennpipes.reload!
+      end
+    end
+
+    it 'should correctly reset SimpleDemo fixture' do
+      @app = SimpleDemo
+      get "/"
+      @app.reload!
+      get "/rand"
+      assert ok?
+      last_body = body
+      assert_equal 1, @app.filters[:before].size
+      assert_equal 1, @app.errors.size
+      assert_equal 2, @app.filters[:after].size # app + content-type + tennpipes-flash
+      assert_equal 0, @app.middleware.size
+      assert_equal 4, @app.routes.size # GET+HEAD of "/" + GET+HEAD of "/rand" = 4
+      assert_equal 4, @app.extensions.size # [Tennpipes::ApplicationSetup, Tennpipes::ParamsProtection, Tennpipes::Routing, Tennpipes::Flash]
+      assert_equal 0, @app.templates.size
+      @app.reload!
+      get "/rand"
+      refute_equal last_body, body
+      assert_equal 1, @app.filters[:before].size
+      assert_equal 1, @app.errors.size
+      assert_equal 2, @app.filters[:after].size
+      assert_equal 0, @app.middleware.size
+      assert_equal 4, @app.routes.size # GET+HEAD of "/" = 2
+      assert_equal 4, @app.extensions.size # [Tennpipes::ApplicationSetup, Tennpipes::ParamsProtection, Tennpipes::Routing, Tennpipes::Flash]
+      assert_equal 0, @app.templates.size
+    end
+  end
+end

data/test/test_reloader_system.rb

@@ -0,0 +1,113 @@
+require File.expand_path(File.dirname(__FILE__) + '/helper')
+require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/kiq')
+require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/system')
+require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/static')
+
+describe "SystemReloader" do
+  describe 'for wierd and difficult reload events' do
+    before do
+      @app = SystemDemo
+      get '/'
+    end
+
+    it 'should reload system features if they were required only in helper' do
+      @app.reload!
+      get '/'
+      assert_equal 'Resolv', body
+    end
+
+    it 'should reload children on parent change' do
+      Tennpipes.mount(SystemDemo).to("/")
+      assert_equal Child.new.family, 'Danes'
+      parent_file = File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/models/parent.rb')
+      new_class = <<-DOC
+        class Parent
+          def family
+            'Dancy'
+          end
+          def shmamily
+            'Shmancy'
+          end
+        end
+      DOC
+      begin
+        backup = File.read(parent_file)
+        Tennpipes::Reloader.reload!
+        assert_equal 'Danes', Parent.new.family
+        assert_equal 'Danes', Child.new.family
+        File.open(parent_file, "w") { |f| f.write(new_class) }
+        Tennpipes::Reloader.reload!
+        assert_equal 'Dancy', Parent.new.family
+        assert_equal 'Shmancy', Parent.new.shmamily
+        assert_equal 'Dancy', Child.new.family
+        assert_equal 'Shmancy', Child.new.shmamily
+      ensure
+        File.open(parent_file, "w") { |f| f.write(backup) }
+      end
+    end
+
+    it 'should not fail horribly on reload event with non-tennpipes apps' do
+      Tennpipes.mount("kiq").to("/")
+      Tennpipes.reload!
+    end
+
+    it 'should not reload apps with disabled reload' do
+      Tennpipes.mount(StaticDemo).to("/")
+      Tennpipes.reload!
+    end
+  end
+
+  describe 'reloading external constants' do
+    it 'should not touch external constants defining singleton methods' do
+      new_class = <<-DOC
+        class SingletonClassTest
+          def self.external_test
+          end
+        end
+      DOC
+      tmp_file = '/tmp/tennpipes_class_demo.rb'
+      begin
+        File.open(tmp_file, "w") { |f| f.write(new_class) }
+        Tennpipes.clear!
+        require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/system_class_methods_demo.rb')
+        @app = SystemClassMethodsDemo
+        Tennpipes.mount(SystemClassMethodsDemo).to("/")
+        get '/'
+        assert defined?(SingletonClassTest), 'SingletonClassTest undefined'
+        assert_includes SingletonClassTest.singleton_methods, :external_test
+        FileUtils.touch File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/helpers/class_methods_helpers.rb')
+        Tennpipes.reload!
+        assert defined?(SingletonClassTest), 'SingletonClassTest undefined'
+        assert_includes SingletonClassTest.singleton_methods, :external_test
+      ensure
+        FileUtils.rm tmp_file
+      end
+    end
+
+    it 'should not touch external constants defining instance methods' do
+      new_class = <<-DOC
+        class InstanceTest
+          def instance_test
+          end
+        end
+      DOC
+      tmp_file = '/tmp/tennpipes_instance_demo.rb'
+      begin
+        File.open(tmp_file, "w") { |f| f.write(new_class) }
+        Tennpipes.clear!
+        require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/system_instance_methods_demo.rb')
+        @app = SystemInstanceMethodsDemo
+        Tennpipes.mount(SystemInstanceMethodsDemo).to("/")
+        get '/'
+        assert defined?(InstanceTest), 'InstanceTest undefined'
+        assert_includes InstanceTest.new.methods, :instance_test
+        FileUtils.touch File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/helpers/instance_methods_helpers.rb')
+        Tennpipes.reload!
+        assert defined?(InstanceTest), 'InstanceTest undefined'
+        assert_includes InstanceTest.new.methods, :instance_test
+      ensure
+        FileUtils.rm tmp_file
+      end
+    end
+  end
+end

data/test/test_restful_routing.rb

@@ -0,0 +1,33 @@
+require File.expand_path(File.dirname(__FILE__) + '/helper')
+
+describe "Routing" do
+  it 'should perform restul routing' do
+    mock_app do
+      controller :parent => :parents do
+        get :index, :with => :asset_id do
+          "#{url_for(:index, params[:parent_id], :asset_id => params[:asset_id])} get"
+        end
+
+        put :index, :with => :asset_id do
+          "#{url_for(:index, params[:parent_id], :asset_id => params[:asset_id])} put"
+        end
+
+        post :index, :with => :asset_id do
+          "#{url_for(:index, :parent_id => params[:parent_id], :asset_id => params[:asset_id])} post"
+        end
+
+        delete :index, :with => :asset_id do
+          "#{url_for(:index, params[:parent_id], :asset_id => params[:asset_id])} delete"
+        end
+      end
+    end
+    get "/parents/1/hi"
+    assert_equal "/parents/1/hi get", body
+    put "/parents/1/hi"
+    assert_equal "/parents/1/hi put", body
+    post "/parents/1/hi"
+    assert_equal "/parents/1/hi post", body
+    delete "/parents/1/hi"
+    assert_equal "/parents/1/hi delete", body
+  end
+end

data/test/test_router.rb

@@ -0,0 +1,281 @@
+require File.expand_path(File.dirname(__FILE__) + '/helper')
+require File.expand_path(File.dirname(__FILE__) + '/fixtures/apps/simple')
+
+describe "Router" do
+
+  def setup
+    Tennpipes.clear!
+  end
+
+  it 'should dispatch paths correctly' do
+    app = lambda { |env|
+      [200, {
+        'X-ScriptName' => env['SCRIPT_NAME'],
+        'X-PathInfo' => env['PATH_INFO'],
+        'Content-Type' => 'text/plain'
+      }, [""]]
+    }
+    map = Tennpipes::Router.new(
+      { :path => '/bar',     :to => app },
+      { :path => '/foo/bar', :to => app },
+      { :path => '/foo',     :to => app }
+    )
+
+    res = Rack::MockRequest.new(map).get("/")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(map).get("/qux")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(map).get("/foo")
+    assert res.ok?
+    assert_equal "/foo", res["X-ScriptName"]
+    assert_equal "/", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/foo/")
+    assert res.ok?
+    assert_equal "/foo", res["X-ScriptName"]
+    assert_equal "/", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/foo/bar")
+    assert res.ok?
+    assert_equal "/foo/bar", res["X-ScriptName"]
+    assert_equal "/", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/foo/bar/")
+    assert res.ok?
+    assert_equal "/foo/bar", res["X-ScriptName"]
+    assert_equal "/", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/foo///bar//quux")
+    assert_equal 200, res.status
+    assert res.ok?
+    assert_equal "/foo/bar", res["X-ScriptName"]
+    assert_equal "//quux", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/foo/quux", "SCRIPT_NAME" => "/bleh")
+    assert res.ok?
+    assert_equal "/bleh/foo", res["X-ScriptName"]
+    assert_equal "/quux", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/bar", 'HTTP_HOST' => 'foo.org')
+    assert res.ok?
+    assert_equal "/bar", res["X-ScriptName"]
+    assert_equal "/", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/bar/", 'HTTP_HOST' => 'foo.org')
+    assert res.ok?
+    assert_equal "/bar", res["X-ScriptName"]
+    assert_equal "/", res["X-PathInfo"]
+  end
+
+  it 'should dispatch requests to cascade mounted apps' do
+    app = lambda { |env|
+      scary = !!env['PATH_INFO'].match(/scary/)
+      [scary ? 404 : 200, {
+        'X-ScriptName' => env['SCRIPT_NAME'],
+        'X-PathInfo' => env['PATH_INFO'],
+        'Content-Type' => 'text/plain'
+      }, [""]]
+    }
+    api = lambda { |env|
+      spooky = !!env['QUERY_STRING'].match(/spooky/)
+      [spooky ? 200 : 404, {
+        'X-API' => spooky,
+        'X-ScriptName' => env['SCRIPT_NAME'],
+        'X-PathInfo' => env['PATH_INFO'],
+        'Content-Type' => 'application/json'
+      }, [""]]
+    }
+    map = Tennpipes::Router.new(
+      { :path => '/bar',     :to => api },
+      { :path => '/bar',     :to => app }
+    )
+
+    res = Rack::MockRequest.new(map).get("/werewolf")
+    assert_equal 404, res.status
+    assert_equal nil, res["X-API"]
+    assert_equal nil, res["X-ScriptName"]
+    assert_equal nil, res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/bar/mitzvah")
+    assert res.ok?
+    assert_equal nil, res["X-API"]
+    assert_equal 'text/plain', res["Content-Type"]
+    assert_equal "/bar", res["X-ScriptName"]
+    assert_equal "/mitzvah", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/bar?spooky")
+    assert res.ok?
+    assert_equal true, res["X-API"]
+    assert_equal 'application/json', res["Content-Type"]
+    assert_equal "/bar", res["X-ScriptName"]
+    assert_equal "/", res["X-PathInfo"]
+
+    res = Rack::MockRequest.new(map).get("/bar/scary")
+    assert_equal 404, res.status
+    assert_equal nil, res["X-API"]
+    assert_equal 'text/plain', res["Content-Type"]
+    assert_equal "/bar", res["X-ScriptName"]
+    assert_equal "/scary", res["X-PathInfo"]
+  end
+
+  it 'should dispatch requests to cascade mounted apps and not cascade ok statuses' do
+
+    api = mock_app do
+      get 'scary' do
+        "1"
+      end
+      set :cascade, true
+    end
+
+    app = mock_app do
+      get 'scary' do
+        "2"
+      end
+      set :cascade, false
+    end
+
+    app2 = mock_app do
+      get 'terrifying' do
+        ""
+      end
+    end
+
+    map = Tennpipes::Router.new(
+        { :path => '/bar',     :to => api },
+        { :path => '/bar',   :to => app  },
+        { :path => '/bar',     :to => app2 }
+    )
+
+    res = Rack::MockRequest.new(map).get("/bar/scary")
+    assert res.ok?
+    #asserting that on ok we're good to go
+    assert_equal "1", res.body
+
+    res = Rack::MockRequest.new(map).get("/bar/terrifying")
+    assert !res.ok?
+
+  end
+
+  it 'should dispatch requests to cascade mounted apps until it sees a cascade == false or []g' do
+    app = mock_app do
+      get 'scary' do
+        ""
+      end
+      set :cascade, []
+    end
+
+    app2 = mock_app do
+      get 'terrifying' do
+        ""
+      end
+    end
+
+    map = Tennpipes::Router.new(
+        { :path => '/bar',   :to => app  },
+        { :path => '/bar',     :to => app2 }
+    )
+
+    request_case = lambda {
+      Rack::MockRequest.new(map).get("/bar/terrifying")
+    }
+
+    app.cascade = false
+    assert !request_case.call.ok?
+
+    app.cascade = true
+    assert request_case.call.ok?
+  end
+
+  it 'should dispatches hosts correctly' do
+    map = Tennpipes::Router.new(
+     { :host => "foo.org", :to => lambda { |env|
+       [200,
+        { "Content-Type" => "text/plain",
+          "X-Position" => "foo.org",
+          "X-Host" => env["HTTP_HOST"] || env["SERVER_NAME"],
+        }, [""]]}},
+     { :host => "subdomain.foo.org", :to => lambda { |env|
+       [200,
+        { "Content-Type" => "text/plain",
+          "X-Position" => "subdomain.foo.org",
+          "X-Host" => env["HTTP_HOST"] || env["SERVER_NAME"],
+        }, [""]]}},
+     { :host => /.*\.bar.org/, :to => lambda { |env|
+       [200,
+        { "Content-Type" => "text/plain",
+          "X-Position" => "bar.org",
+          "X-Host" => env["HTTP_HOST"] || env["SERVER_NAME"],
+        }, [""]]}}
+     )
+
+     res = Rack::MockRequest.new(map).get("/", "HTTP_HOST" => "bar.org")
+     assert res.not_found?
+
+     res = Rack::MockRequest.new(map).get("/", "HTTP_HOST" => "at.bar.org")
+     assert res.ok?
+     assert_equal "bar.org", res["X-Position"]
+
+     res = Rack::MockRequest.new(map).get("/", "HTTP_HOST" => "foo.org")
+     assert res.ok?
+     assert_equal "foo.org", res["X-Position"]
+
+     res = Rack::MockRequest.new(map).get("/", "HTTP_HOST" => "subdomain.foo.org", "SERVER_NAME" => "foo.org")
+     assert res.ok?
+     assert_equal "subdomain.foo.org", res["X-Position"]
+  end
+
+  it 'should works with tennpipes core applications' do
+    Tennpipes.mount("simple_demo").host("tennpipes.org")
+    assert_equal ["simple_demo"], Tennpipes.mounted_apps.map(&:name)
+    assert_equal ["tennpipes.org"], Tennpipes.mounted_apps.map(&:app_host)
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/", "HTTP_HOST" => "bar.org")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/", "HTTP_HOST" => "tennpipes.org")
+    assert res.ok?
+  end
+
+  it 'should works with tennpipes applications' do
+    Tennpipes.mount("simple_demo").to("/foo").host(/.*\.tennpipes.org/)
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/", "HTTP_HOST" => "bar.org")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/", "HTTP_HOST" => "tennpipes.org")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/none", "HTTP_HOST" => "foo.tennpipes.org")
+    assert res.not_found?
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/foo", "HTTP_HOST" => "bar.tennpipes.org")
+    assert res.ok?
+
+    res = Rack::MockRequest.new(Tennpipes.application).get("/foo/", "HTTP_HOST" => "bar.tennpipes.org")
+    assert res.ok?
+  end
+
+  it 'should keep the same environment object' do
+    app = lambda { |env|
+      env['path'] = env['PATH_INFO']
+      [200, {'Content-Type' => 'text/plain'}, [""]]
+    }
+    map = Tennpipes::Router.new(
+      { :path => '/bar',     :to => app },
+      { :path => '/foo/bar', :to => app },
+      { :path => '/foo',     :to => app }
+    )
+
+    env = Rack::MockRequest.env_for("/bar/foo")
+    map.call(env)
+    assert_equal "/foo", env["path"]
+  end
+end