merb-core 0.9.2

data/spec/public/abstract_controller/controllers/helpers.rb

@@ -0,0 +1,31 @@
+module Merb::Test::Fixtures
+  module Abstract
+  
+    class HelperTesting < Merb::AbstractController
+      self._template_root = File.dirname(__FILE__) / "views"
+      
+      def _template_location(action, type = nil, controller = controller_name)
+        "helpers/#{File.basename(controller)}/#{action}"
+      end
+    end
+    
+    class Capture < HelperTesting
+      def index
+        render
+      end
+    end
+
+    module ConcatHelper
+      def concatter(&blk)
+        concat("Concat", blk.binding)
+      end
+    end
+    
+    class Concat < HelperTesting
+      def index
+        render
+      end
+    end
+       
+  end
+end

data/spec/public/abstract_controller/controllers/partial.rb

@@ -0,0 +1,106 @@
+
+
+module Merb::Test::Fixtures
+
+  module Abstract
+
+    class RenderIt < Merb::AbstractController
+      self._template_root = File.dirname(__FILE__) / "views"
+
+      def _template_location(action, type = nil, controller = controller_name)
+        "partial/#{File.basename(controller)}/#{action}"
+      end
+    end
+
+    class BasicPartial < RenderIt
+
+      def index
+        render
+      end
+    end
+
+    class WithPartial < RenderIt
+
+      def index
+        @foo = "With"
+        render
+      end
+    end
+
+    class WithNilPartial < RenderIt
+
+      def index
+        render
+      end
+    end
+
+    class WithAsPartial < RenderIt
+
+      def index
+        @foo = "With and As"
+        render
+      end
+    end
+
+    class PartialWithCollections < RenderIt
+
+      def index
+        @foo = %w{ c o l l e c t i o n }
+        render
+      end
+    end
+
+    class PartialWithCollectionsAndAs < RenderIt
+
+      def index
+        @foo = %w{ c o l l e c t i o n }
+        render
+      end
+    end
+
+    class PartialWithLocals < RenderIt
+
+      def index
+        @foo, @bar = %w{ local variables }
+        render
+      end
+    end
+
+    class PartialWithBoth < RenderIt
+
+      def index
+        @foo = %w{ c o l l e c t i o n }
+        @delimiter = "-"
+        render
+      end
+    end
+
+    class PartialWithWithAndLocals < RenderIt
+
+      def index
+        @foo, @bar = "with", "and locals"
+        render
+      end
+    end
+
+    class PartialInAnotherDirectory < RenderIt
+
+      def index
+        render
+      end
+    end
+
+    class NestedPartial < RenderIt
+      def index
+        render
+      end
+    end
+
+    class BasicPartialWithMultipleRoots < RenderIt
+      self._template_roots << [File.dirname(__FILE__) / "alt_views", :_template_location]
+      def index
+        render
+      end
+    end
+  end
+end

data/spec/public/abstract_controller/controllers/render.rb

@@ -0,0 +1,86 @@
+
+
+module Merb::Test::Fixtures
+
+  module Abstract
+    
+    class Testing < Merb::AbstractController
+      self._template_root = File.dirname(__FILE__) / "views"
+    end
+
+    class RenderString < Testing
+      
+      def index
+        render "the index"
+      end
+    end
+
+    class RenderStringCustomLayout < RenderString
+      layout :custom
+    end
+
+    class RenderStringAppLayout < RenderString
+      self._template_root = File.dirname(__FILE__) / "alt_views"      
+    end
+
+    class RenderStringControllerLayout < RenderString
+      self._template_root = File.dirname(__FILE__) / "alt_views"
+    end
+    
+    class RenderStringDynamicLayout < RenderString
+      layout :determine_layout
+      
+      def alt_index
+        render "the alt index"
+      end
+      
+      def determine_layout
+        action_name.index('alt') == 0 ? 'alt' : 'custom'
+      end
+    end
+
+    class RenderTemplate < Testing
+      
+      def index
+        render
+      end      
+    end
+
+    class RenderTemplateCustomLayout < RenderString
+      layout :custom
+    end
+
+    class RenderTemplateAppLayout < RenderString
+      self._template_root = File.dirname(__FILE__) / "alt_views"      
+    end
+
+    class RenderTemplateControllerLayout < RenderString
+      self._template_root = File.dirname(__FILE__) / "alt_views"      
+    end  
+    
+    class RenderTemplateCustomLocation < RenderTemplate      
+      def _template_location(action, type = nil, controller = controller_name)  
+        "wonderful/#{action}"
+      end
+    end
+    
+    class RenderTemplateMultipleRoots < RenderTemplate
+      self._template_roots << [File.dirname(__FILE__) / "alt_views", :_template_location]
+      
+      def show
+        render :layout=>false
+      end
+    end
+
+    class RenderTemplateMultipleRootsAndCustomLocation < RenderTemplate
+      self._template_roots = [[File.dirname(__FILE__) / "alt_views", :_custom_template_location]]
+      
+      def _custom_template_location(action, type = nil, controller = controller_name)
+        "#{self.class.name.split('::')[-1].to_const_path}/#{action}"
+      end
+    end
+    
+    class RenderTemplateMultipleRootsInherited < RenderTemplateMultipleRootsAndCustomLocation
+    end
+  end
+end

data/spec/public/abstract_controller/controllers/views/helpers/capture/index.erb

@@ -0,0 +1 @@
+<% x = capture do -%>Capture<% end -%><%= x %>

data/spec/public/abstract_controller/controllers/views/helpers/concat/index.erb

@@ -0,0 +1 @@
+<% concatter do -%>Hello<% end -%>

data/spec/public/abstract_controller/controllers/views/layout/alt.erb

@@ -0,0 +1 @@
+Alt: <%= catch_content(:for_layout) %>

data/spec/public/abstract_controller/controllers/views/layout/custom.erb

@@ -0,0 +1 @@
+Custom: <%= catch_content(:for_layout) %>

data/spec/public/abstract_controller/controllers/views/merb/test/fixtures/abstract/display_object/index.erb

@@ -0,0 +1 @@
+the index

data/spec/public/abstract_controller/controllers/views/merb/test/fixtures/abstract/display_object_with_action/new.erb

@@ -0,0 +1 @@
+new action

data/spec/public/abstract_controller/controllers/views/merb/test/fixtures/abstract/render_template/index.erb

@@ -0,0 +1 @@
+the index

data/spec/public/abstract_controller/controllers/views/merb/test/fixtures/abstract/render_template_custom_layout/index.erb

@@ -0,0 +1 @@
+the index

data/spec/public/abstract_controller/controllers/views/merb/test/fixtures/abstract/render_template_multiple_roots/index.erb

@@ -0,0 +1 @@
+This is overriden by the custom template root

data/spec/public/abstract_controller/controllers/views/merb/test/fixtures/abstract/render_template_multiple_roots/show.erb

@@ -0,0 +1 @@
+default show

data/spec/public/abstract_controller/controllers/views/partial/another_directory/_partial.erb

@@ -0,0 +1 @@
+Partial in another directory

data/spec/public/abstract_controller/controllers/views/partial/basic_partial/_partial.erb

@@ -0,0 +1 @@
+Partial

data/spec/public/abstract_controller/controllers/views/partial/basic_partial/index.erb

@@ -0,0 +1 @@
+Index <%= partial :partial %>

data/spec/public/abstract_controller/controllers/views/partial/basic_partial_with_multiple_roots/index.erb

@@ -0,0 +1 @@
+Base Index: <%= partial :partial %>

data/spec/public/abstract_controller/controllers/views/partial/nested_partial/_first.erb

@@ -0,0 +1 @@
+<%= msg %> <%= partial 'second', :another => 'second' %> <%= msg %>

data/spec/public/abstract_controller/controllers/views/partial/nested_partial/_second.erb

@@ -0,0 +1 @@
+<%= another %>

data/spec/public/abstract_controller/controllers/views/partial/nested_partial/index.erb

@@ -0,0 +1 @@
+<%= partial 'first', :msg => 'first' %>

data/spec/public/abstract_controller/controllers/views/partial/partial_in_another_directory/index.erb

@@ -0,0 +1 @@
+Index <%= partial 'another_directory/partial' %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_both/_collection.erb

@@ -0,0 +1 @@
+<%= letter %><%= delimiter %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_both/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :collection, :with => @foo, :as => :letter, :delimiter => @delimiter %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_collections/_collection.erb

@@ -0,0 +1 @@
+<%= collection %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_collections/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :collection, :with => @foo %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_collections_and_as/_collection.erb

@@ -0,0 +1 @@
+<%= letter %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_collections_and_as/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :collection, :with => @foo, :as => :letter %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_locals/_variables.erb

@@ -0,0 +1 @@
+<%= local %> <%= variables %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_locals/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :variables, :local => @foo, :variables => @bar %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_with_and_locals/_both.erb

@@ -0,0 +1 @@
+<%= first %> <%= second %>

data/spec/public/abstract_controller/controllers/views/partial/partial_with_with_and_locals/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :both, :with => @foo, :as => :first, :second => @bar %>

data/spec/public/abstract_controller/controllers/views/partial/with_as_partial/_with_partial.erb

@@ -0,0 +1 @@
+<%= w00t %>

data/spec/public/abstract_controller/controllers/views/partial/with_as_partial/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :with_partial, :with => @foo, :as => :w00t %>

data/spec/public/abstract_controller/controllers/views/partial/with_nil_partial/_with_partial.erb

@@ -0,0 +1 @@
+<%= with_partial || "nil local" %>

data/spec/public/abstract_controller/controllers/views/partial/with_nil_partial/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :with_partial, :with => nil %>

data/spec/public/abstract_controller/controllers/views/partial/with_partial/_with_partial.erb

@@ -0,0 +1 @@
+<%= with_partial || "nil local" %>

data/spec/public/abstract_controller/controllers/views/partial/with_partial/index.erb

@@ -0,0 +1 @@
+Partial with <%= partial :with_partial, :with => @foo %>

data/spec/public/abstract_controller/controllers/views/test_display/foo.html.erb

@@ -0,0 +1 @@
+fooness

data/spec/public/abstract_controller/controllers/views/wonderful/index.erb

@@ -0,0 +1 @@
+Wonderful

data/spec/public/abstract_controller/display_spec.rb

@@ -0,0 +1,33 @@
+require File.join(File.dirname(__FILE__), "spec_helper")
+
+describe Merb::AbstractController, " displaying objects with templates" do
+
+  before do
+    Merb.push_path(:layout, File.dirname(__FILE__) / "controllers" / "views" / "layouts")
+  end
+  
+  it "should allow you to pass an object" do
+    dispatch_should_make_body("DisplayObject", "the index")
+  end
+  
+  it "should allow you to pass an object with an action specified" do
+    dispatch_should_make_body("DisplayObjectWithAction", "new action", :create)
+  end
+  
+  it "should allow you to pass an object with a path specified for the template" do
+    dispatch_should_make_body("DisplayObjectWithPath", "fooness")
+  end
+  
+  it "should allow you to pass an object using multiple template root" do
+    dispatch_should_make_body("DisplayObjectWithMultipleRoots", "App: new index")
+  end
+
+  it "should allow you to pass an object using multiple template root, with layout" do
+    dispatch_should_make_body("DisplayObjectWithMultipleRoots", "Alt: new show", "show")
+  end
+
+  it "should allow you to pass an object using multiple template root, without layout" do
+    dispatch_should_make_body("DisplayObjectWithMultipleRoots", "fooness", "another")
+  end
+
+end

data/spec/public/abstract_controller/filter_spec.rb

@@ -0,0 +1,80 @@
+# ==== Public API
+# Merb::AbstractController.before(filter<Symbol, Proc>, opts<Hash>)
+# Merb::AbstractController.after(filter<Symbol, Proc>, opts<Hash>)
+# Merb::AbstractController.skip_before(filter<Symbol>)
+# Merb::AbstractController.skip_after(filter<Symbol>)
+#
+# ==== Semipublic API
+# Merb::AbstractController#_body
+# Merb::AbstractController#_dispatch(action<~to_s>)
+
+require File.join(File.dirname(__FILE__), "spec_helper")
+
+describe Merb::AbstractController, " should support before and after filters" do
+    
+  it "should support before filters" do
+    dispatch_should_make_body("TestBeforeFilter", "foo filter")
+  end
+  
+  it "should support after filters" do
+    dispatch_should_make_body("TestAfterFilter", "foo filter")
+  end
+  
+  it "should support skipping filters that were defined in a superclass" do
+    dispatch_should_make_body("TestSkipFilter", "")
+  end
+  
+  it "should append before filters when added" do
+    dispatch_should_make_body("TestBeforeFilterOrder", "bar filter")
+  end
+
+  it "should append after filters when added" do
+    dispatch_should_make_body("TestAfterFilterOrder", "bar filter")
+  end
+  
+  it "should support proc arguments to filters evaluated in the controller's instance" do
+    dispatch_should_make_body("TestProcFilter", "proc filter1 proc filter2")
+  end
+  
+  it "should support filters that skip specific actions via :exclude" do
+    dispatch_should_make_body("TestExcludeFilter", " ", :index)
+    dispatch_should_make_body("TestExcludeFilter", "foo filter bar filter", :show)    
+  end
+  
+  it "should support filters that work only on specific actions via :only" do
+    dispatch_should_make_body("TestOnlyFilter", "foo filter bar filter", :index)        
+    dispatch_should_make_body("TestOnlyFilter", " ", :show)
+  end
+  
+  it "should throw an error if both :exclude and :only are passed to a filter" do
+    running { Merb::Test::Fixtures::Abstract.class_eval do
+      
+      class TestErrorFilter < Merb::Test::Fixtures::Abstract::Testing
+        before :foo, :only => :index, :exclude => :show
+      end 
+    end }.should raise_error(ArgumentError, /either :only or :exclude/)
+  end
+
+  it "should support filters that work only when a condition is met via :if" do
+    dispatch_should_make_body("TestConditionalFilterWithMethod", "foo filter", :index, :presets => {:bar= => true})
+    dispatch_should_make_body("TestConditionalFilterWithMethod", "", :index, :presets => {:bar= => false})
+  end
+  
+  it "should support filters that work only when a condition is met via :unless" do
+    dispatch_should_make_body("TestConditionalFilterWithProc", "foo filter", :index, :presets => {:bar= => 'baz'})
+    dispatch_should_make_body("TestConditionalFilterWithProc", "index action", :index, :presets => {:bar= => 'bar'})
+  end
+  
+  it "should throw an error if both :if and :unless are passed to a filter" do
+    running { Merb::Test::Fixtures::Abstract.class_eval do
+      
+      class TestErrorFilter < Merb::Test::Fixtures::Abstract::Testing
+        before :foo, :if => :index, :unless => :show
+      end 
+    end }.should raise_error(ArgumentError, /either :if or :unless/)
+  end
+  
+  it "should throw an error" do
+    running { dispatch_should_make_body("TestConditionalFilterWithNoProcOrSymbol", "") }.should raise_error(ArgumentError, /a Symbol or a Proc/)
+  end
+end