semantic_navigation 0.0.14 → 0.1.0

data/lib/semantic_navigation/twitter_bootstrap/breadcrumb.rb

@@ -6,9 +6,9 @@ module SemanticNavigation
 
       style_accessor :last_as_link => false,
                      :breadcrumb_separator => '/'
-      
+
       navigation_default_classes [:breadcrumb]
-      
+
       show_navigation_active_class false
       show_node_active_class false
       show_leaf_active_class false
@@ -17,18 +17,18 @@ module SemanticNavigation
       show_node_id false
       show_leaf_id false
       show_link_id false
-      
+
       property_for :base, :ico
-      
+
       private
-      
+
       def navigation(object)
         content_tag :ul, nil, :id => show_id(:navigation, object.id),
                               :class => merge_classes(:navigation, object.active, object.classes) do
           yield
         end
       end
-      
+
       def node(object)
         content_tag(:li, nil, :id => show_id(:leaf, object.id),
                               :class => merge_classes(:leaf, object.active, object.classes)) do
@@ -39,7 +39,7 @@ module SemanticNavigation
         end +
         yield
       end
-     
+
       def leaf(object)
         content_tag :li, nil, :id => show_id(:leaf, object.id),
                               :class => merge_classes(:leaf, object.active, object.classes) do
@@ -52,7 +52,7 @@ module SemanticNavigation
           end].sum
         end
       end
-      
+
     end
   end
 end

data/lib/semantic_navigation/twitter_bootstrap/list.rb

@@ -3,24 +3,24 @@ module SemanticNavigation
     class List
       include SemanticNavigation::Renderers::RenderHelpers
       include SemanticNavigation::Renderers::ActsAsList
-      
+
       navigation_default_classes [:nav, 'nav-list']
       show_navigation_id false
       show_node_id false
       show_leaf_id false
-      show_link_id false      
-      
+      show_link_id false
+
       property_for :base, :ico
-      
+
       private
-      
+
       def navigation(object)
         content_tag :ul, nil, :id => show_id(:navigation, object.id),
                                 :class => merge_classes(:navigation, object.active, object.classes) do
           yield
-        end          
+        end
       end
-      
+
       def node(object)
         if object.ico
           name = [content_tag(:i,nil,:class => "icon-#{object.ico}"),
@@ -28,22 +28,22 @@ module SemanticNavigation
         else
           name = object_name(object)
         end
-        
+
         content_tag :li, nil, :id => show_id(:leaf, object.id),
                               :class => merge_classes(:leaf, object.active, object.classes) do
            link_to(name, object.url, :id => show_id(:link, object.id),
                                             :class => merge_classes(:link, object.active, object.link_classes)) +
           yield
-        end 
+        end
       end
-      
+
       def node_content(object)
         content_tag(:ul, nil, :id => show_id(:node, object.id),
                               :class => merge_classes(:node, object.active, object.node_classes)) do
           yield
         end
       end
-      
+
       def leaf(object)
         if object_name(object).empty? && object.url.nil?
           classes = 'divider'
@@ -52,25 +52,25 @@ module SemanticNavigation
         else
           classes = merge_classes(:leaf, object.active, object.classes)
         end
-        
+
         if object.ico
           name = [content_tag(:i,nil,:class => "icon-#{object.ico}"),
                   object_name(object)].sum
         else
           name = object_name(object)
         end
-                
+
         content_tag :li, nil, :id => show_id(:leaf, object.id),
                               :class => classes do
           if object.url.nil?
             name
           else
             link_to name, object.url, :id => show_id(:link, object.id),
-                                      :class => merge_classes(:link, object.active, object.link_classes)            
+                                      :class => merge_classes(:link, object.active, object.link_classes)
           end
         end
       end
-      
+
     end
   end
 end

data/lib/semantic_navigation/twitter_bootstrap/tabs.rb

@@ -3,26 +3,26 @@ module SemanticNavigation
     class Tabs
       include SemanticNavigation::Renderers::RenderHelpers
       include SemanticNavigation::Renderers::ActsAsList
-      
+
       style_accessor :direction => 'left'
-      
+
       navigation_default_classes [:nav, 'nav-tabs']
       show_navigation_id false
       show_node_id false
       show_leaf_id false
-      show_link_id false      
-      
+      show_link_id false
+
       property_for :base, :ico
-      
+
       private
-      
+
       def navigation(object)
         content_tag :ul, nil, :id => show_id(:navigation, object.id),
                               :class => merge_classes(:navigation, object.active, [object.classes,"pull-#{direction}"]) do
           yield
         end
       end
-      
+
       def node(object)
         content_tag :li, nil, :id => show_id(:leaf, object.id),
                               :class => merge_classes(:leaf, object.active, [object.classes,'dropdown'].flatten) do
@@ -33,19 +33,19 @@ module SemanticNavigation
             [object.ico ? content_tag(:i,nil,:class => "icon-#{object.ico}") : '',
              object_name(object),
              content_tag(:b,nil,:class => :caret)
-            ].sum.html_safe                     
+            ].sum.html_safe
           end +
           yield
-        end 
+        end
       end
-      
+
       def node_content(object)
         content_tag(:ul, nil, :id => show_id(:node, object.id),
                               :class => merge_classes(:node, false, [object.node_classes,'dropdown-menu'].flatten)) do
           yield
         end
       end
-      
+
       def leaf(object)
         if object.ico
           name = [content_tag(:i,nil,:class => "icon-#{object.ico}"),
@@ -53,18 +53,18 @@ module SemanticNavigation
         else
           name = object_name(object)
         end
-                
+
         content_tag :li, nil, :id => show_id(:leaf, object.id),
                               :class => merge_classes(:leaf, object.active, object.classes) do
           if object.url.nil?
             name
           else
             link_to name, object.url, :id => show_id(:link, object.id),
-                                             :class => merge_classes(:link, object.active, object.link_classes)            
+                                             :class => merge_classes(:link, object.active, object.link_classes)
           end
         end
       end
-           
+
     end
   end
 end

data/lib/semantic_navigation/version.rb

@@ -1,3 +1,3 @@
 module SemanticNavigation
-  VERSION = "0.0.14"
+  VERSION = "0.1.0"
 end

data/spec/lib/semantic_navigation/configuration_spec.rb

@@ -1,35 +1,35 @@
 require 'spec_helper'
 
 describe SemanticNavigation::Configuration do
-  
+
   before :each do
     SemanticNavigation::Configuration.class_variable_set("@@navigations",{})
   end
 
   describe "#run" do
-  	it 'should receive class eval if block passed' do
+    it 'should receive class eval if block passed' do
       SemanticNavigation::Configuration.should_receive(:class_eval)
       SemanticNavigation::Configuration.run do
-      	do_something
+        do_something
       end
-  	end
+    end
 
-  	it 'should not receive class_eval if block not passed' do
+    it 'should not receive class_eval if block not passed' do
       SemanticNavigation::Configuration.run
-  	end
+    end
   end
 
   describe '#navigate' do
-  	it 'should receive at least only id and save navigation instance in class variables' do
-  	  nav_instance = SemanticNavigation::Configuration.navigate :some_menu
-  	  navigations = SemanticNavigation::Configuration.class_variable_get("@@navigations")
-  	  navigations.keys.should == [:some_menu]
-  	  navigations[:some_menu].should == nav_instance
-  	end
+    it 'should receive at least only id and save navigation instance in class variables' do
+      nav_instance = SemanticNavigation::Configuration.navigate :some_menu
+      navigations = SemanticNavigation::Configuration.class_variable_get("@@navigations")
+      navigations.keys.should == [:some_menu]
+      navigations[:some_menu].should == nav_instance
+    end
 
     it 'should receive id and pass to Navigation instance create options hash' do
-      SemanticNavigation::Core::Navigation.should_receive(:new).with({:id=>:some_menu, 
-      	                                                              :i18n_name=>"semantic_navigation.some_menu"})
+      SemanticNavigation::Core::Navigation.should_receive(:new).with({:id=>:some_menu,
+                                                                      :i18n_name=>"semantic_navigation.some_menu"})
       SemanticNavigation::Configuration.navigate :some_menu
     end
 
@@ -38,40 +38,40 @@ describe SemanticNavigation::Configuration do
       navigation.should_receive(:instance_eval)
       SemanticNavigation::Core::Navigation.should_receive(:new).and_return navigation
       SemanticNavigation::Configuration.navigate :some_menu do
-      	do_something
+        do_something
       end
     end
 
     it 'should merge id, i18n_name and received params and pass them to navigation instance create method' do
       SemanticNavigation::Core::Navigation.should_receive(:new).with({:id => :some_menu,
-      	                                                              :i18n_name=>"semantic_navigation.some_menu",
-      	                                                              :some_attr => 1,
-      	                                                              :some_attr2 => 2})
+                                                                      :i18n_name=>"semantic_navigation.some_menu",
+                                                                      :some_attr => 1,
+                                                                      :some_attr2 => 2})
       SemanticNavigation::Configuration.navigate :some_menu, :some_attr => 1, :some_attr2 => 2
     end
   end
 
   describe '#navigation' do
-  	it 'should return navigation instance by name' do
-  	  nav_instance = SemanticNavigation::Configuration.navigate :some_menu
+    it 'should return navigation instance by name' do
+      nav_instance = SemanticNavigation::Configuration.navigate :some_menu
       SemanticNavigation::Configuration.navigation(:some_menu).should == nav_instance
-  	end
+    end
   end
 
   describe '#styles_for' do
-  	it 'should save styles block as a proc' do
-  	  proc_object = SemanticNavigation::Configuration.styles_for :some_renderer do
-  	  	do_something1 true
-  	  	do_something2 'some_attr'
-  	  end
-  	  styles = SemanticNavigation::Configuration.class_variable_get("@@render_styles")
-  	  styles[:some_renderer].should == proc_object
-
-  	  mock_renderer = mock
-  	  mock_renderer.should_receive(:do_something1).with(true)
-  	  mock_renderer.should_receive(:do_something2).with('some_attr')
-  	  mock_renderer.instance_eval &proc_object
-  	end
+    it 'should save styles block as a proc' do
+      proc_object = SemanticNavigation::Configuration.styles_for :some_renderer do
+        do_something1 true
+        do_something2 'some_attr'
+      end
+      styles = SemanticNavigation::Configuration.class_variable_get("@@render_styles")
+      styles[:some_renderer].should == proc_object
+
+      mock_renderer = mock
+      mock_renderer.should_receive(:do_something1).with(true)
+      mock_renderer.should_receive(:do_something2).with('some_attr')
+      mock_renderer.instance_eval &proc_object
+    end
   end
 
   describe '#register_renderer' do
@@ -93,10 +93,18 @@ describe SemanticNavigation::Configuration do
       renderers[:some_renderer].should == SomeRenderer
     end
 
+    it 'should reregister the renderer class regigstered before' do
+      SemanticNavigation::Configuration.register_renderer SomeRenderer
+      SemanticNavigation::Configuration.register_renderer :another_name, :some_renderer
+      renderers = SemanticNavigation::Configuration.class_variable_get("@@renderers")
+      renderers[:another_name].should_not be_nil
+      renderers[:another_name].should == SomeRenderer
+    end
+
     it 'should register a new helper method based on name of a renderer and depending on method navigation_for' do
       SemanticNavigation::Configuration.register_renderer SomeRenderer
       SemanticNavigation::HelperMethods.method_defined?(:some_renderer_for).should be_true
-      
+
       class SomeClass
         include SemanticNavigation::HelperMethods
       end
@@ -109,7 +117,7 @@ describe SemanticNavigation::Configuration do
 
   describe '#render' do
 
-    before :each do 
+    before :each do
       @view_object = mock
       @renderer = mock
       @renderer_instance = mock
@@ -123,7 +131,7 @@ describe SemanticNavigation::Configuration do
     it 'should send render method to Renderer class; mark active navigation elements;' do
       @navigation.should_receive(:mark_active)
       @navigation.should_receive(:render).with(@renderer_instance)
-      @renderer_instance.should_receive(:name=).with(:renderer)      
+      @renderer_instance.should_receive(:name=).with(:renderer)
       SemanticNavigation::Configuration.new.render(:some_menu, :renderer, {}, @view_object)
     end
 
@@ -131,15 +139,15 @@ describe SemanticNavigation::Configuration do
       @navigation.should_receive(:mark_active)
       @navigation.should_receive(:render).with(@renderer_instance)
       @renderer_instance.should_receive(:level=).with(1)
-      @renderer_instance.should_receive(:name=).with(:renderer)      
-      SemanticNavigation::Configuration.new.render(:some_menu, :renderer, {:level => 1}, @view_object)      
+      @renderer_instance.should_receive(:name=).with(:renderer)
+      SemanticNavigation::Configuration.new.render(:some_menu, :renderer, {:level => 1}, @view_object)
     end
 
     it 'should exec default renderer styles from configuration' do
       render_styles = proc{}
       SemanticNavigation::Configuration.class_variable_set "@@render_styles", {:renderer => render_styles}
       @renderer_instance.should_receive(:instance_eval).with(&render_styles)
-      @renderer_instance.should_receive(:name=).with(:renderer)      
+      @renderer_instance.should_receive(:name=).with(:renderer)
       @navigation.should_receive(:mark_active)
       @navigation.should_receive(:render).with(@renderer_instance)
       SemanticNavigation::Configuration.new.render(:some_menu, :renderer, {}, @view_object)
@@ -148,11 +156,11 @@ describe SemanticNavigation::Configuration do
     it 'should set @@view_object in configuration class' do
       @navigation.should_receive(:mark_active)
       @navigation.should_receive(:render).with(@renderer_instance)
-      @renderer_instance.should_receive(:name=).with(:renderer)      
+      @renderer_instance.should_receive(:name=).with(:renderer)
       SemanticNavigation::Configuration.new.render(:some_menu, :renderer, {}, @view_object)
       SemanticNavigation::Configuration.view_object.should == @view_object
     end
 
   end
 
-end
+end

data/spec/lib/semantic_navigation/core/navigation_spec.rb

@@ -216,6 +216,64 @@ describe SemanticNavigation::Core::Navigation do
 
   end
 
+  describe "#render_if scope" do
+    before :each do
+      @navigation = SemanticNavigation::Core::Navigation.new({})
+
+      @view_object = mock
+      SemanticNavigation::Configuration.stub(:view_object).and_return @view_object
+    end
+
+    it "should scope the items" do
+      @navigation.scope do
+        item :first_item,  "controller1#action"
+        item :second_item, "controller2#action"
+      end
+
+      @navigation.sub_elements.size.should == 2
+    end
+
+    it "should scope the creating items options" do
+      some_condition = mock
+      @navigation.scope :render_if => some_condition do
+        item :first_item, "controller1#action"
+        item :second_item, "controller2#action"
+      end
+
+      @navigation.sub_elements[0..1].each do |element|
+        element.instance_variable_get(:"@render_if").should == some_condition
+      end
+    end
+
+    it "should not scope unscoped items" do
+      some_condition = mock
+      @navigation.scope render_if: some_condition do
+        item :first_item, "controller1#action"
+        item :second_item, "controller2#action"
+      end
+      @navigation.item :third_item, "controller3#action"
+
+      last_item = @navigation.sub_elements.last
+      last_item.instance_variable_get(:"@render_if").should == nil
+    end
+
+    it "should not override defined scopes" do
+      some_condition = mock
+      some_condition2 = mock
+      @navigation.scope :render_if => some_condition do
+        item :first_item, "controller1#action", render_if: some_condition2
+        item :second_item, "controller2#action"
+      end
+
+      first_item = @navigation.sub_elements.first
+      second_item = @navigation.sub_elements.last
+
+      first_item.instance_variable_get(:"@render_if").should == some_condition2
+      second_item.instance_variable_get(:"@render_if").should == some_condition
+    end
+
+  end
+
   describe '#method_missing' do
     it 'should get unknown method and call super' do
       @navigation = SemanticNavigation::Core::Navigation.new({})