building-blocks 1.0.0 → 1.1.0

data/CHANGELOG.rdoc

@@ -0,0 +1,6 @@
+1.1.0 (February 4, 2012)
+
+* Ability to disable use of partials when rendering a block
+* Ability to disable use of partials for before and after hooks
+* Complete test coverage
+* :template_folder and :variable options are no longer being passed in as part of the options hash to defined blocks and partials

data/VERSION

@@ -1 +1 @@
-1.0.0
+1.1.0

data/lib/building_blocks/base.rb

@@ -1,11 +1,16 @@
 module BuildingBlocks
-  BUILDING_BLOCKS_TEMPLATE_FOLDER = "blocks"
+  TEMPLATE_FOLDER = "blocks"
+  USE_PARTIALS = true
+  USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS = true
 
   class Base
+    # a pointer to the ActionView that called BuildingBlocks
     attr_accessor :view
 
+    # Hash of block names to BuildingBlocks::Container objects
     attr_accessor :blocks
 
+    # the block that is passed in for the templating feature
     attr_accessor :block
 
     # Array of BuildingBlocks::Container objects, storing the order of blocks as they were queued
@@ -14,11 +19,18 @@ module BuildingBlocks
     # counter, used to give unnamed blocks a unique name
     attr_accessor :anonymous_block_number
 
+    # A Hash of queued_blocks arrays; a new array is started when method_missing is invoked
     attr_accessor :block_groups
 
     # These are the options that are passed into the initalize method
     attr_accessor :global_options
 
+    # The default folder to look in for global partials
+    attr_accessor :templates_folder
+
+    # The variable to use when rendering the partial for the templating feature (by default, "blocks")
+    attr_accessor :variable
+
     # Checks if a particular block has been defined within the current block scope.
     #   <%= blocks.defined? :some_block_name %>
     # Options:
@@ -86,7 +98,7 @@ module BuildingBlocks
       raise "Must specify :template parameter in order to render" unless global_options[:template]
 
       render_options = global_options.clone
-      render_options[render_options[:variable] ? render_options[:variable].to_sym : :blocks] = self
+      render_options[self.variable] = self
       render_options[:captured_block] = view.capture(self, &self.block) if self.block
 
       view.render global_options[:template], render_options
@@ -129,8 +141,8 @@ module BuildingBlocks
     protected
 
     def initialize(view, options={}, &block)
-      options[:templates_folder] = BuildingBlocks::BUILDING_BLOCKS_TEMPLATE_FOLDER if options[:templates_folder].nil?
-
+      self.templates_folder = options[:templates_folder] ? options.delete(:templates_folder) : BuildingBlocks::TEMPLATE_FOLDER
+      self.variable = (options[:variable] ? options.delete(:variable) : :blocks).to_sym
       self.view = view
       self.global_options = options
       self.block = block
@@ -162,17 +174,20 @@ module BuildingBlocks
         block_container = blocks[name]
         args.push(global_options.merge(block_container.options).merge(block_options).merge(options))
         buffer << view.capture(*(args[0, block_container.block.arity]), &block_container.block)
-      else
+      elsif BuildingBlocks::USE_PARTIALS
         begin
           begin
             buffer << view.render("#{name.to_s}", global_options.merge(block_options).merge(options))
           rescue ActionView::MissingTemplate
-            buffer << view.render("#{self.global_options[:templates_folder]}/#{name.to_s}", global_options.merge(block_options).merge(options))
+            buffer << view.render("#{self.templates_folder}/#{name.to_s}", global_options.merge(block_options).merge(options))
           end
         rescue ActionView::MissingTemplate
           args.push(global_options.merge(options))
           buffer << view.capture(*(args[0, block.arity]), &block) if block_given?
         end
+      else
+        args.push(global_options.merge(options))
+        buffer << view.capture(*(args[0, block.arity]), &block) if block_given?
       end
 
       buffer
@@ -199,12 +214,12 @@ module BuildingBlocks
           args_clone.push(global_options.merge(block_options).merge(block_container.options).merge(options))
           buffer << view.capture(*(args_clone[0, block_container.block.arity]), &block_container.block)
         end
-      else
+      elsif BuildingBlocks::USE_PARTIALS && BuildingBlocks::USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS
         begin
           begin
             buffer << view.render("before_#{name.to_s}", global_options.merge(block_options).merge(options))
           rescue ActionView::MissingTemplate
-            buffer << view.render("#{self.global_options[:templates_folder]}/before_#{name.to_s}", global_options.merge(block_options).merge(options))
+            buffer << view.render("#{self.templates_folder}/before_#{name.to_s}", global_options.merge(block_options).merge(options))
           end
         rescue ActionView::MissingTemplate
         end
@@ -234,12 +249,12 @@ module BuildingBlocks
           args_clone.push(global_options.merge(block_options).merge(block_container.options).merge(options))
           buffer << view.capture(*(args_clone[0, block_container.block.arity]), &block_container.block)
         end
-      else
+      elsif BuildingBlocks::USE_PARTIALS && BuildingBlocks::USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS
         begin
           begin
             buffer << view.render("after_#{name.to_s}", global_options.merge(block_options).merge(options))
           rescue ActionView::MissingTemplate
-            buffer << view.render("#{self.global_options[:templates_folder]}/after_#{name.to_s}", global_options.merge(block_options).merge(options))
+            buffer << view.render("#{self.templates_folder}/after_#{name.to_s}", global_options.merge(block_options).merge(options))
           end
         rescue ActionView::MissingTemplate
         end

data/spec/building-blocks/base_spec.rb

@@ -6,6 +6,20 @@ describe BuildingBlocks::Base do
     @builder = BuildingBlocks::Base.new(@view)
   end
 
+  it "should be able change the default global partials directory" do
+    BuildingBlocks.send(:remove_const, "TEMPLATE_FOLDER")
+    BuildingBlocks.const_set("TEMPLATE_FOLDER", "shared")
+    @builder = BuildingBlocks::Base.new(@view)
+    @builder.expects(:render_before_blocks).at_least_once
+    @builder.expects(:render_after_blocks).at_least_once
+    @view.expects(:capture).with(:value1 => 1, :value2 => 2).never
+    @view.expects(:render).with("some_block", :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+    @view.expects(:render).with("shared/some_block", :value1 => 1, :value2 => 2).once
+    @builder.use :some_block, :value1 => 1, :value2 => 2
+    BuildingBlocks.send(:remove_const, "TEMPLATE_FOLDER")
+    BuildingBlocks.const_set("TEMPLATE_FOLDER", "blocks")
+  end
+
   describe "defined? method" do
     it "should be able to determine if a block by a specific name is already defined" do
       @builder.defined?(:test_block).should be_false
@@ -150,7 +164,7 @@ describe BuildingBlocks::Base do
     it "should set all of the global options as local variables to the partial it renders" do
       view = mock()
       builder = BuildingBlocks::Base.new(view, :template => "some_template")
-      view.expects(:render).with { |template, options| options.should eql :templates_folder => 'blocks', :template => 'some_template', :blocks => builder }
+      view.expects(:render).with { |template, options| options.should eql :template => 'some_template', :blocks => builder }
       builder.render
     end
 
@@ -283,9 +297,9 @@ describe BuildingBlocks::Base do
     end
 
     it "should automatically pass in an options hash to a defined block that takes one paramter when that block is used" do
-      block = Proc.new {|options| "Templates folder is #{options[:templates_folder]}"}
+      block = Proc.new {|options| "Options are #{options.inspect}"}
       @builder.define :some_block, &block
-      @builder.use(:some_block).should eql "Templates folder is blocks"
+      @builder.use(:some_block).should eql "Options are {}"
     end
 
     it "should be able to use a defined block by its name and pass in runtime arguments as a hash" do
@@ -293,7 +307,7 @@ describe BuildingBlocks::Base do
         print_hash(options)
       end
       @builder.define :some_block, &block
-      @builder.use(:some_block, :param1 => 1, :param2 => "value2").should eql print_hash(:templates_folder => 'blocks', :param1 => 1, :param2 => "value2")
+      @builder.use(:some_block, :param1 => 1, :param2 => "value2").should eql print_hash(:param1 => 1, :param2 => "value2")
     end
 
     it "should be able to use a defined block by its name and pass in runtime arguments one by one" do
@@ -301,7 +315,7 @@ describe BuildingBlocks::Base do
         "first_param: #{first_param}, second_param: #{second_param}, #{print_hash options}"
       end
       @builder.define :some_block, &block
-      @builder.use(:some_block, 3, 4, :param1 => 1, :param2 => "value2").should eql("first_param: 3, second_param: 4, #{print_hash(:templates_folder => 'blocks', :param1 => 1, :param2 => "value2")}")
+      @builder.use(:some_block, 3, 4, :param1 => 1, :param2 => "value2").should eql("first_param: 3, second_param: 4, #{print_hash(:param1 => 1, :param2 => "value2")}")
     end
 
     it "should match up the number of arguments to a defined block with the parameters passed when a block is used" do
@@ -317,45 +331,57 @@ describe BuildingBlocks::Base do
         "first_param: #{first_param}, second_param: #{second_param}, #{print_hash options}"
       end
       @builder.replace :some_block, &block
-      @builder.use(:some_block, 3, 4, :param1 => 1, :param2 => "value2").should eql("first_param: 3, second_param: 4, #{print_hash(:templates_folder => 'blocks', :param1 => 1, :param2 => "value2")}")
+      @builder.use(:some_block, 3, 4, :param1 => 1, :param2 => "value2").should eql("first_param: 3, second_param: 4, #{print_hash(:param1 => 1, :param2 => "value2")}")
     end
 
     it "should not render anything if using a block that has been defined" do
       @view.expects(:capture).never
-      @view.expects(:render).with("some_block", :templates_folder => 'blocks').raises(ActionView::MissingTemplate.new([],[],[],[],[]))
-      @view.expects(:render).with("blocks/some_block", :templates_folder => 'blocks').raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+      @view.expects(:render).with("some_block", {}).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+      @view.expects(:render).with("blocks/some_block", {}).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
       @builder.use :some_block
     end
 
     it "should first attempt to capture a block's contents when blocks.use is called" do
       block = Proc.new {|options|}
-      @view.expects(:capture).with(:templates_folder => 'blocks', :value1 => 1, :value2 => 2)
-      @view.expects(:render).with("some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).never
-      @view.expects(:render).with("blocks/some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).never
+      @view.expects(:capture).with(:value1 => 1, :value2 => 2)
+      @view.expects(:render).with("some_block", :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("blocks/some_block", :value1 => 1, :value2 => 2).never
       @builder.define :some_block, &block
       @builder.use :some_block, :value1 => 1, :value2 => 2
     end
 
     it "should second attempt to render a local partial by the block's name when blocks.use is called" do
-      @view.expects(:capture).with(:templates_folder => 'blocks', :value1 => 1, :value2 => 2).never
-      @view.expects(:render).with("some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).once
-      @view.expects(:render).with("blocks/some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).never
+      @view.expects(:capture).with(:value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("some_block", :value1 => 1, :value2 => 2).once
+      @view.expects(:render).with("blocks/some_block", :value1 => 1, :value2 => 2).never
       @builder.use :some_block, :value1 => 1, :value2 => 2
     end
 
     it "should third attempt to render a global partial by the block's name when blocks.use is called" do
-      @view.expects(:capture).with(:templates_folder => 'blocks', :value1 => 1, :value2 => 2).never
-      @view.expects(:render).with("some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
-      @view.expects(:render).with("blocks/some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).once
+      @view.expects(:capture).with(:value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("some_block", :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+      @view.expects(:render).with("blocks/some_block", :value1 => 1, :value2 => 2).once
       @builder.use :some_block, :value1 => 1, :value2 => 2
     end
 
     it "should fourth attempt to render a default block when blocks.use is called" do
       block = Proc.new {|options|}
-      @view.expects(:render).with("some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
-      @view.expects(:render).with("blocks/some_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
-      @view.expects(:capture).with(:templates_folder => 'blocks', :value1 => 1, :value2 => 2)
+      @view.expects(:render).with("some_block", :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+      @view.expects(:render).with("blocks/some_block", :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+      @view.expects(:capture).with(:value1 => 1, :value2 => 2)
+      @builder.use :some_block, :value1 => 1, :value2 => 2, &block
+    end
+
+    it "should not attempt to render a partial if BuildingBlocks::USE_PARTIALS is set to false" do
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS")
+      BuildingBlocks.const_set("USE_PARTIALS", false)
+      block = Proc.new {|options|}
+      @view.expects(:render).with("some_block", :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("blocks/some_block", :value1 => 1, :value2 => 2).never
+      @view.expects(:capture).with(:value1 => 1, :value2 => 2)
       @builder.use :some_block, :value1 => 1, :value2 => 2, &block
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS")
+      BuildingBlocks.const_set("USE_PARTIALS", true)
     end
 
     it "should override hash options for a block by merging the runtime options the define default options into the queue level options into the global options" do
@@ -364,7 +390,7 @@ describe BuildingBlocks::Base do
       @builder.queue(:my_before_block, :param1 => "queue level", :param2 => "queue level")
       @builder.define(:my_before_block, :param1 => "define level", :param2 => "define level", :param3 => "define level", &block)
       block_container = @builder.queued_blocks.first
-      @view.expects(:capture).with(:param4 => 'global level', :param1 => 'use level', :templates_folder => 'blocks', :param2 => 'queue level', :param3 => 'define level')
+      @view.expects(:capture).with(:param4 => 'global level', :param1 => 'use level', :param2 => 'queue level', :param3 => 'define level')
       @builder.use block_container, :param1 => "use level"
     end
 
@@ -386,32 +412,56 @@ describe BuildingBlocks::Base do
     it "should render before blocks when using a block" do
       block = Proc.new {|value1, value2, options|}
       @builder.before("my_before_block", &block)
-      @view.expects(:capture).with(1, 2, :templates_folder => 'blocks', :value3 => 3, :value4 => 4)
+      @view.expects(:capture).with(1, 2, :value3 => 3, :value4 => 4)
       @builder.use :my_before_block, 1, 2, :value3 => 3, :value4 => 4
     end
 
     it "should try and render a before block as a local partial if no before blocks are specified" do
       block = Proc.new {}
       @view.expects(:capture).never
-      @view.expects(:render).with("before_my_before_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).once
-      @view.expects(:render).with("blocks/before_my_before_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("before_my_before_block", :value1 => 1, :value2 => 2).once
+      @view.expects(:render).with("blocks/before_my_before_block", :value1 => 1, :value2 => 2).never
       @builder.use :my_before_block, :value1 => 1, :value2 => 2
     end
 
     it "should try and render a before block as a global partial if no after blocks are specified and the local partial does not exist" do
       block = Proc.new {}
       @view.expects(:capture).never
-      @view.expects(:render).with("before_my_before_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
-      @view.expects(:render).with("blocks/before_my_before_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).once
+      @view.expects(:render).with("before_my_before_block", :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+      @view.expects(:render).with("blocks/before_my_before_block", :value1 => 1, :value2 => 2).once
       @builder.use :my_before_block, :value1 => 1, :value2 => 2
     end
 
+    it "should not attempt to render a before block as a partial if BuildingBlocks::USE_PARTIALS is set to false" do
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS")
+      BuildingBlocks.const_set("USE_PARTIALS", false)
+      block = Proc.new {}
+      @view.expects(:capture).never
+      @view.expects(:render).with("before_my_before_block", :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("blocks/before_my_before_block", :value1 => 1, :value2 => 2).never
+      @builder.use :my_before_block, :value1 => 1, :value2 => 2
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS")
+      BuildingBlocks.const_set("USE_PARTIALS", true)
+    end
+
+    it "should not attempt to render a before block as a partial if BuildingBlocks::USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS is set to false" do
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS")
+      BuildingBlocks.const_set("USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS", false)
+      block = Proc.new {}
+      @view.expects(:capture).never
+      @view.expects(:render).with("before_my_before_block", :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("blocks/before_my_before_block", :value1 => 1, :value2 => 2).never
+      @builder.use :my_before_block, :value1 => 1, :value2 => 2
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS")
+      BuildingBlocks.const_set("USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS", true)
+    end
+
     it "should override hash options for before blocks by merging the runtime options into the before block options into the block options into the global options" do
       block = Proc.new {|options|}
       @builder.global_options.merge!(:param1 => "global level", :param2 => "global level", :param3 => "global level", :param4 => "global level")
       @builder.define(:my_before_block, :param1 => "block level", :param2 => "block level", :param3 => "block level", &block)
       @builder.before(:my_before_block, :param1 => "before block level", :param2 => "before block level", &block)
-      @view.expects(:capture).with(:param4 => 'global level', :param1 => 'top level', :templates_folder => 'blocks', :param2 => 'before block level', :param3 => 'block level')
+      @view.expects(:capture).with(:param4 => 'global level', :param1 => 'top level', :param2 => 'before block level', :param3 => 'block level')
       @builder.use :my_before_block, :param1 => "top level"
     end
   end
@@ -425,24 +475,48 @@ describe BuildingBlocks::Base do
     it "should render after blocks when using a block" do
       block = Proc.new {|value1, value2, options|}
       @builder.after("my_after_block", &block)
-      @view.expects(:capture).with(1, 2, :templates_folder => 'blocks', :value3 => 3, :value4 => 4)
+      @view.expects(:capture).with(1, 2, :value3 => 3, :value4 => 4)
       @builder.use :my_after_block, 1, 2, :value3 => 3, :value4 => 4
     end
 
     it "should try and render a after block as a local partial if no after blocks are specified" do
       block = Proc.new {}
       @view.expects(:capture).never
-      @view.expects(:render).with("after_my_after_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).once
-      @view.expects(:render).with("blocks/after_my_after_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("after_my_after_block", :value1 => 1, :value2 => 2).once
+      @view.expects(:render).with("blocks/after_my_after_block", :value1 => 1, :value2 => 2).never
       @builder.use :my_after_block, :value1 => 1, :value2 => 2
     end
 
     it "should try and render a after block as a global partial if no after blocks are specified and the local partial does not exist" do
       block = Proc.new {}
       @view.expects(:capture).never
-      @view.expects(:render).with("after_my_after_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
-      @view.expects(:render).with("blocks/after_my_after_block", :templates_folder => 'blocks', :value1 => 1, :value2 => 2).once
+      @view.expects(:render).with("after_my_after_block", :value1 => 1, :value2 => 2).raises(ActionView::MissingTemplate.new([],[],[],[],[]))
+      @view.expects(:render).with("blocks/after_my_after_block", :value1 => 1, :value2 => 2).once
+      @builder.use :my_after_block, :value1 => 1, :value2 => 2
+    end
+
+    it "should not attempt to render a after block as a partial if BuildingBlocks::USE_PARTIALS is set to false" do
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS")
+      BuildingBlocks.const_set("USE_PARTIALS", false)
+      block = Proc.new {}
+      @view.expects(:capture).never
+      @view.expects(:render).with("after_my_after_block", :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("blocks/after_my_after_block", :value1 => 1, :value2 => 2).never
+      @builder.use :my_after_block, :value1 => 1, :value2 => 2
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS")
+      BuildingBlocks.const_set("USE_PARTIALS", true)
+    end
+
+    it "should not attempt to render a after block as a partial if BuildingBlocks::USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS is set to false" do
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS")
+      BuildingBlocks.const_set("USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS", false)
+      block = Proc.new {}
+      @view.expects(:capture).never
+      @view.expects(:render).with("after_my_after_block", :value1 => 1, :value2 => 2).never
+      @view.expects(:render).with("blocks/after_my_after_block", :value1 => 1, :value2 => 2).never
       @builder.use :my_after_block, :value1 => 1, :value2 => 2
+      BuildingBlocks.send(:remove_const, "USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS")
+      BuildingBlocks.const_set("USE_PARTIALS_FOR_BEFORE_AND_AFTER_HOOKS", true)
     end
 
     it "should override hash options for after blocks by merging the runtime options into the after block options into the block options into the global options" do
@@ -450,7 +524,7 @@ describe BuildingBlocks::Base do
       @builder.global_options.merge!(:param1 => "global level", :param2 => "global level", :param3 => "global level", :param4 => "global level")
       @builder.define(:my_after_block, :param1 => "block level", :param2 => "block level", :param3 => "block level", &block)
       @builder.after(:my_after_block, :param1 => "after block level", :param2 => "after block level", &block)
-      @view.expects(:capture).with(:param4 => 'global level', :param1 => 'top level', :templates_folder => 'blocks', :param2 => 'after block level', :param3 => 'block level')
+      @view.expects(:capture).with(:param4 => 'global level', :param1 => 'top level', :param2 => 'after block level', :param3 => 'block level')
       @builder.use :my_after_block, :param1 => "top level"
     end
   end

data/spec/building-blocks/view_additions_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+describe BuildingBlocks::ViewAdditions do
+  before(:each) do
+    @view_class = Class.new
+    @view = @view_class.new
+    @view_class.send(:include, BuildingBlocks::ViewAdditions::ClassMethods)
+  end
+
+  describe "blocks method" do
+    it "should pass the view as the only parameter to BuildingBlocks::Base initialization" do
+      BuildingBlocks::Base.expects(:new).with {|view| view == @view}
+      @view.blocks
+    end
+
+    it "should memoize the BuildingBlocks::Base instance for 'blocks' call" do
+      BuildingBlocks::Base.expects(:new).once.with {|view| view == @view}.returns "something"
+      @view.blocks.should eql "something"
+      @view.blocks.should eql "something"
+    end
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: building-blocks
 version: !ruby/object:Gem::Version 
-  hash: 23
+  hash: 19
   prerelease: 
   segments: 
   - 1
+  - 1
   - 0
-  - 0
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors: 
 - Andrew Hunter
@@ -132,6 +132,34 @@ dependencies:
   prerelease: false
   type: :development
   requirement: *id008
+- !ruby/object:Gem::Dependency 
+  name: jeweler
+  version_requirements: &id009 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  prerelease: false
+  type: :development
+  requirement: *id009
+- !ruby/object:Gem::Dependency 
+  name: jeweler
+  version_requirements: &id010 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  prerelease: false
+  type: :development
+  requirement: *id010
 description: ""
 email: hunterae@gmail.com
 executables: []
@@ -141,6 +169,7 @@ extensions: []
 extra_rdoc_files: 
 - README.rdoc
 files: 
+- CHANGELOG.rdoc
 - README.rdoc
 - Rakefile
 - VERSION
@@ -150,6 +179,7 @@ files:
 - lib/building_blocks/view_additions.rb
 - rails/init.rb
 - spec/building-blocks/base_spec.rb
+- spec/building-blocks/view_additions_spec.rb
 - spec/spec_helper.rb
 has_rdoc: true
 homepage: http://github.com/hunterae/building-blocks