semantic_navigation 0.1.1 → 0.1.3

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

@@ -1,23 +1,23 @@
 require 'spec_helper'
 
 describe SemanticNavigation::Core::Navigation do
-  
+
   describe '#initialize' do
-    
-    it 'should create instance of navigation with level 0 and empty subitems' do
+
+    it 'creates instance of navigation with level 0 and empty subitems' do
       navigation = SemanticNavigation::Core::Navigation.new({})
       navigation.level.should == 0
       navigation.sub_elements.should be_empty
     end
 
-    it 'should create instance and save sended options' do
+    it 'creates instance and save sended options' do
       navigation = SemanticNavigation::Core::Navigation.new :id => :some_id,
                                                             :classes => [:one, :two]
       navigation.id.should == :some_id
       navigation.classes.should == [:one,:two]
     end
 
-    it 'should create instance and save unintended properties' do
+    it 'creates instance and save unintended properties' do
       navigation = SemanticNavigation::Core::Navigation.new :some_attribute => :some_value
       navigation.instance_variable_get("@some_attribute").should == :some_value
     end
@@ -25,25 +25,25 @@ describe SemanticNavigation::Core::Navigation do
   end
 
   describe '#item' do
-  	before :each do
-  	  @navigation = SemanticNavigation::Core::Navigation.new({})
-  	end
+    before :each do
+      @navigation = SemanticNavigation::Core::Navigation.new({})
+    end
 
-  	it "should receive item method and make the leaf" do
+    it "receives item method and make the leaf" do
       @navigation.item :some_id, 'some_url'
       @navigation.sub_elements.size.should == 1
       @navigation.sub_elements.first.is_a?(SemanticNavigation::Core::Leaf).should be_true
-  	end
+    end
 
-  	it "should receive item method with block and create node" do
+    it "receives item method with block and create node" do
       @navigation.item :node_id, 'node_url' do
-      	item :leaf_id, 'leaf_url'
+        item :leaf_id, 'leaf_url'
       end
       @navigation.sub_elements.size.should == 1
       @navigation.sub_elements.first.is_a?(SemanticNavigation::Core::Node).should be_true
-  	end
+    end
 
-    it "should receive item method with array of urls and save them properly" do
+    it "receives item method with array of urls and save them properly" do
       @navigation.item :leaf_id, ['string/url',"controller#action",:symbolic_name,['array','like','url']]
       urls = @navigation.sub_elements.first.instance_variable_get("@url")
       urls.should == ['string/url',
@@ -52,18 +52,18 @@ describe SemanticNavigation::Core::Navigation do
                       ['array','like','url']]
     end
 
-    it "should receive item with Proc url and decode it to normal url (leaf)" do
+    it "receives item with Proc url and decode it to normal url (leaf)" do
       @navigation.item :leaf_id, proc{'some' + 'func'}
       @navigation.sub_elements.first.url.should == 'somefunc'
     end
 
-    it 'should receive item with array of urls one of each is a Proc (leaf)' do
+    it 'receives item with array of urls one of each is a Proc (leaf)' do
       @navigation.item :leaf_id, ['string_url',proc{'some' + 'func'}]
       urls = @navigation.sub_elements.first.send :urls
       urls.should == ['string_url', 'somefunc']
     end
 
-    it "should receive item with Proc url and decode it to normal url (node)" do
+    it "receives item with Proc url and decode it to normal url (node)" do
       @navigation.item :leaf_id, proc{'some' + 'func'} do
         item :first_value, '#'
         item :second_value, '#'
@@ -71,59 +71,59 @@ describe SemanticNavigation::Core::Navigation do
       @navigation.sub_elements.first.url.should == 'somefunc'
     end
 
-    it 'should receive item with array of urls one of each is a Proc (node)' do
+    it 'receives item with array of urls one of each is a Proc (node)' do
       @navigation.item :leaf_id, ['string_url',proc{'some' + 'func'}] do
         item :first_value, '#'
         item :second_value, '#'
       end
       urls = @navigation.sub_elements.first.send :urls
       urls.should == ['string_url', 'somefunc']
-    end 
+    end
   end
 
   describe '#header' do
-  	before :each do
-  	  @navigation = SemanticNavigation::Core::Navigation.new({})
-  	  @navigation.header :some_id
-  	end
-
-  	it "should create item with nil url" do
-   	  @navigation.sub_elements.size.should == 1
-   	  @navigation.sub_elements.first.url.should be_nil
-  	end
+    before :each do
+      @navigation = SemanticNavigation::Core::Navigation.new({})
+      @navigation.header :some_id
+    end
+
+    it "creates item with nil url" do
+      @navigation.sub_elements.size.should == 1
+      @navigation.sub_elements.first.url.should be_nil
+    end
   end
 
   describe "#divider" do
-  	before :each do
-  	  @navigation = SemanticNavigation::Core::Navigation.new({})
-  	end
+    before :each do
+      @navigation = SemanticNavigation::Core::Navigation.new({})
+    end
 
-  	it "should create divider item with nil url and name" do
+    it "creates divider item with nil url and name" do
       @navigation.divider
-  	  @navigation.sub_elements.size.should == 1
-  	  @navigation.sub_elements.first.url.should be_nil
-  	  @navigation.sub_elements.first.name.should be_empty
-  	end
-
-    it "should receive any length method containing char `_` and create divider" do
-  	  
-  	  (1..10).each do |c|
+      @navigation.sub_elements.size.should == 1
+      @navigation.sub_elements.first.url.should be_nil
+      @navigation.sub_elements.first.name.should be_empty
+    end
+
+    it "receives any length method containing char `_` and create divider" do
+
+      (1..10).each do |c|
         @navigation.send ('_'*c).to_sym
-  	  end
-  	  @navigation.sub_elements.size.should == 10
-  	  @navigation.sub_elements.map(&:id).should == [:divider]*10
-  	end
+      end
+      @navigation.sub_elements.size.should == 10
+      @navigation.sub_elements.map(&:id).should == [:divider]*10
+    end
   end
 
   describe "#decode_url" do
-    
+
     before :each do
       @navigation = SemanticNavigation::Core::Navigation.new({})
     end
 
     it "while creating the item make support for urls in format controller#action" do
       view_object = mock
-     
+
       @navigation.item :some_id, 'controller#action'
       @navigation.sub_elements.size.should == 1
       @navigation.sub_elements.first.url.should == {:controller => "controller", :action => "action"}
@@ -131,31 +131,36 @@ describe SemanticNavigation::Core::Navigation do
   end
 
   describe '#render_if' do
-    it 'should return true if render_if proc is nil' do
-      navigation = SemanticNavigation::Core::Navigation.new({})
-      navigation.render_if.should be_true  
-    end
+    context :returns do
+      it 'true if render_if proc is nil' do
+        navigation = SemanticNavigation::Core::Navigation.new({})
+        navigation.render_if.should be_true
+      end
 
-    it 'should return true if render_if proc return true' do
-      navigation = SemanticNavigation::Core::Navigation.new({:render_if => proc{true}})
-      navigation.render_if.should be_true
-    end
+      it 'true if render_if proc return true' do
+        navigation = SemanticNavigation::Core::Navigation.new({:render_if => proc{true}})
+        navigation.render_if.should be_true
+      end
+
+      it 'false if render_if proc return false' do
+        navigation = SemanticNavigation::Core::Navigation.new({:render_if => proc{false}})
+        navigation.render_if.should be_false
+      end
 
-    it 'should return false if render_if proc return false' do
-      navigation = SemanticNavigation::Core::Navigation.new({:render_if => proc{false}})
-      navigation.render_if.should be_false
     end
 
-    it 'should pass self to passed proc' do
-      navigation = SemanticNavigation::Core::Navigation.new({:id => :some_id, :render_if => proc{|o| o.id == :some_id}})
-      navigation.render_if.should be_true
-      navigation = SemanticNavigation::Core::Navigation.new({:id => :another_id, :render_if => proc{|o| o.id == :some_id}})
-      navigation.render_if.should be_false
+    context :passes do
+      it 'self to passed proc' do
+        navigation = SemanticNavigation::Core::Navigation.new({:id => :some_id, :render_if => proc{|o| o.id == :some_id}})
+        navigation.render_if.should be_true
+        navigation = SemanticNavigation::Core::Navigation.new({:id => :another_id, :render_if => proc{|o| o.id == :some_id}})
+        navigation.render_if.should be_false
+      end
     end
   end
 
   describe '#render' do
-    it 'should call renderer class :render_navigation method' do
+    it 'calls renderer class :render_navigation method' do
       renderer = mock
       navigation = SemanticNavigation::Core::Navigation.new({})
       renderer.should_receive(:render_navigation).with(navigation)
@@ -171,21 +176,21 @@ describe SemanticNavigation::Core::Navigation do
       SemanticNavigation::Configuration.stub(:view_object).and_return @view_object
     end
 
-    it 'should define @active variable to false if no sub_elemetns' do
+    it 'defines @active variable to false if no sub_elemetns' do
       @navigation.mark_active
       @navigation.instance_variable_get("@active").should be false
     end
 
-    it 'should define @active variable to true if at least one sub_element is active' do
+    it 'defines @active variable to true if at least one sub_element is active' do
       @navigation.item :first_item, '111'
       @navigation.item :second_item, '222'
       @view_object.should_receive(:current_page?).with('111').and_return true
       @view_object.should_receive(:current_page?).with('222').and_return false
-      
+
       @navigation.mark_active
     end
 
-    it 'should define @active variable to false if all sub_elements is unactive' do
+    it 'defines @active variable to false if all sub_elements is unactive' do
       @navigation.item :first_item, '333'
       @navigation.item :secodn_item, '444'
       @view_object.should_receive(:current_page?).with('333').and_return false
@@ -194,17 +199,17 @@ describe SemanticNavigation::Core::Navigation do
       @navigation.mark_active
     end
 
-    it 'should use custom current_page? for Hash url params' do
+    it 'uses custom current_page? for Hash url params' do
       @navigation.item :first_item, {:controller => "controller1", :action => "action"}
       @navigation.item :second_item, {:controller => "controller2", :action => "action"}
       @view_object.stub(:params).and_return({:controller => "controller1", :action => "action", :some_other_params => "blablabla"})
-      
+
       @navigation.mark_active
       @navigation.sub_elements[0].active.should be_true
       @navigation.sub_elements[1].active.should be_false
     end
 
-    it 'should work for route like urls as good as for Hash url params' do
+    it 'works for route like urls as good as for Hash url params' do
       @navigation.item :first_item, "controller1#action"
       @navigation.item :second_item, "controller2#action"
       @view_object.stub(:params).and_return({:controller => "controller1", :action => "action", :some_other_params => "blablabla"})
@@ -212,7 +217,7 @@ describe SemanticNavigation::Core::Navigation do
       @navigation.mark_active
       @navigation.sub_elements[0].active.should be_true
       @navigation.sub_elements[1].active.should be_false
-    end    
+    end
 
   end
 
@@ -224,7 +229,7 @@ describe SemanticNavigation::Core::Navigation do
       SemanticNavigation::Configuration.stub(:view_object).and_return @view_object
     end
 
-    it "should scope the items" do
+    it "scopes the items" do
       @navigation.scope do
         item :first_item,  "controller1#action"
         item :second_item, "controller2#action"
@@ -233,7 +238,7 @@ describe SemanticNavigation::Core::Navigation do
       @navigation.sub_elements.size.should == 2
     end
 
-    it "should scope the creating items options" do
+    it "scopes the creating items options" do
       some_condition = mock
       @navigation.scope :render_if => some_condition do
         item :first_item, "controller1#action"
@@ -245,7 +250,7 @@ describe SemanticNavigation::Core::Navigation do
       end
     end
 
-    it "should not scope unscoped items" do
+    it "doesnt scope unscoped items" do
       some_condition = mock
       @navigation.scope render_if: some_condition do
         item :first_item, "controller1#action"
@@ -257,7 +262,7 @@ describe SemanticNavigation::Core::Navigation do
       last_item.instance_variable_get(:"@render_if").should == nil
     end
 
-    it "should not override defined scopes" do
+    it "doesnt override defined scopes" do
       some_condition = mock
       some_condition2 = mock
       @navigation.scope :render_if => some_condition do
@@ -275,7 +280,7 @@ describe SemanticNavigation::Core::Navigation do
   end
 
   describe '#method_missing' do
-    it 'should get unknown method and call super' do
+    it 'gets unknown method and call super' do
       @navigation = SemanticNavigation::Core::Navigation.new({})
       lambda{@navigation.unknow_method}.should raise_exception
     end

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

@@ -1,58 +1,58 @@
 require 'spec_helper'
 
 describe SemanticNavigation::Core::Node do
-  
+
   describe '#name' do
 
-    it 'should return basic name even if renderer name sended' do
+    it 'returns basic name even if renderer name sended' do
       node = SemanticNavigation::Core::Node.new({:id => :first, :name => 'first'},1)
-      node.name(:renderer_name).should == 'first'      
+      node.name(:renderer_name).should == 'first'
     end
 
-    it 'should return the name for renderer' do
-      node = SemanticNavigation::Core::Node.new({:id => :first, 
+    it 'returns the name for renderer' do
+      node = SemanticNavigation::Core::Node.new({:id => :first,
                                                  :name => {:some_renderer => 'some_renderer_name'}},
                                                  1)
       node.name(:some_renderer).should == 'some_renderer_name'
     end
 
-    it 'should return default name for unexpected renderer' do
-      node = SemanticNavigation::Core::Node.new({:id => :first, 
+    it 'returns default name for unexpected renderer' do
+      node = SemanticNavigation::Core::Node.new({:id => :first,
                                                  :name => {:default => 'default_name',
                                                            :some_renderer => 'some_renderer_name'}},
                                                  1)
       node.name(:unexpected_renderer).should == 'default_name'
     end
 
-    it 'should return nil if no name defined' do
+    it 'returns nil if no name defined' do
       node = SemanticNavigation::Core::Node.new({:id => :first}, 1)
       node.name.should == ''
-    end    
+    end
 
-    it 'should return passed name while creating node' do
+    it 'returns passed name while creating node' do
       node = SemanticNavigation::Core::Node.new({:name => 'some name'}, 1)
       node.name.should == 'some name'
     end
-   
-    it 'should return i18n name if passed name is nil' do
+
+    it 'returns i18n name if passed name is nil' do
       node = SemanticNavigation::Core::Node.new({:id => :some_id, :i18n_name => :parent_name}, 1)
       I18n.should_receive(:t).with("parent_name.some_id", {:default=>""}).and_return 'some name'
       node.name.should == 'some name'
     end
 
-    it 'should return result of proc if passed name is a proc' do
+    it 'returns result of proc if passed name is a proc' do
       node = SemanticNavigation::Core::Node.new({:name => proc{'some name'}},1)
       node.name.should == 'some name'
     end
   end
 
-  describe '#url' do	
-    it 'should return passed url' do
+  describe '#url' do
+    it 'returns passed url' do
       node = SemanticNavigation::Core::Node.new({:url => {:controller => 'controller', :action => 'action'}},1)
       node.url.should == {:controller => 'controller', :action => 'action'}
     end
 
-    it 'should return first url if passed array of urls' do
+    it 'returns first url if passed array of urls' do
       node = SemanticNavigation::Core::Node.new({:url => [{:controller => 'controller1', :action => 'action'},
                                                           {:controller => 'controller2', :action => 'action'}]},1)
       node.url.should == {:controller => 'controller1', :action => 'action'}
@@ -65,51 +65,54 @@ describe SemanticNavigation::Core::Node do
                                                            {:controller => :node_controller2, :action => :action}]},1)
 
       @view_object = mock
-      @view_object.stub(:params).and_return({:action => "some", :action => "some"})      
+      @view_object.stub(:params).and_return({:action => "some", :action => "some"})
       SemanticNavigation::Configuration.stub(:view_object).and_return @view_object
     end
 
-    it 'should define @active variable to false if no active sub_elements and node url is not active' do
-      @node.mark_active
-      @node.instance_variable_get("@active").should be_false
-    end
+    context :defines do
 
-    it 'should define @active variable to true if at least one sub_element is active' do
-      @node.item :first_item, '111'
-      @node.item :second_item, '222'
-      @view_object.should_receive(:current_page?).with('111').and_return true
-      @view_object.should_receive(:current_page?).with('222').and_return false
-      
-      @node.mark_active
-      @node.sub_elements[0].active.should be_true
-      @node.sub_elements[1].active.should be_false
-      @node.active.should be_true
-    end
+      it '@active variable to false if no active sub_elements and node url is not active' do
+        @node.mark_active
+        @node.instance_variable_get("@active").should be_false
+      end
 
-    it 'should define @active variable to false if all sub_elements is unactive' do
-      @node.item :first_item, '333'
-      @node.item :second_item, '444'
-      @view_object.should_receive(:current_page?).with('333').and_return false
-      @view_object.should_receive(:current_page?).with('444').and_return false
+      it '@active variable to true if at least one sub_element is active' do
+        @node.item :first_item, '111'
+        @node.item :second_item, '222'
+        @view_object.should_receive(:current_page?).with('111').and_return true
+        @view_object.should_receive(:current_page?).with('222').and_return false
 
-      @node.mark_active
-      @node.sub_elements[0].active.should be_false
-      @node.sub_elements[1].active.should be_false
-      @node.active.should be_false
+        @node.mark_active
+        @node.sub_elements[0].active.should be_true
+        @node.sub_elements[1].active.should be_false
+        @node.active.should be_true
+      end
+
+      it '@active variable to false if all sub_elements is unactive' do
+        @node.item :first_item, '333'
+        @node.item :second_item, '444'
+        @view_object.should_receive(:current_page?).with('333').and_return false
+        @view_object.should_receive(:current_page?).with('444').and_return false
+
+        @node.mark_active
+        @node.sub_elements[0].active.should be_false
+        @node.sub_elements[1].active.should be_false
+        @node.active.should be_false
+      end
     end
 
-    it 'should use custom current_page? for Hash url params' do
+    it 'uses custom current_page? for Hash url params' do
       @node.item :first_item, {:controller => "controller1", :action => "action"}
       @node.item :second_item, {:controller => "controller2", :action => "action"}
       @view_object.stub(:params).and_return({:controller => "controller1", :action => "action", :some_other_params => "blablabla"})
-      
+
       @node.mark_active
       @node.sub_elements[0].active.should be_true
       @node.sub_elements[1].active.should be_false
       @node.active.should be true
     end
 
-    it 'should work for route like urls as good as for Hash url params' do
+    it 'works for route like urls as good as for Hash url params' do
       @node.item :first_item, "controller1#action"
       @node.item :second_item, "controller2#action"
       @view_object.stub(:params).and_return({:controller => "controller1", :action => "action", :some_other_params => "blablabla"})
@@ -117,16 +120,16 @@ describe SemanticNavigation::Core::Node do
       @node.mark_active
       @node.sub_elements[0].active.should be_true
       @node.sub_elements[1].active.should be_false
-      @node.active.should be true      
+      @node.active.should be true
     end
 
-    it 'should be active if at least one url in passed array is active' do
+    it 'is active if at least one url in passed array is active' do
       @view_object.stub(:params).and_return(:controller => 'node_controller2', :action => 'action')
       @node.mark_active
       @node.active.should be_true
     end
 
-    it 'should accept array like urls with other urls' do
+    it 'accepts array like urls with other urls' do
       node = SemanticNavigation::Core::Node.new({:url => [['url','with','id'],
                                                           :symbol_url_name,
                                                           {:controller => 'hash', :action => 'url'},
@@ -138,6 +141,6 @@ describe SemanticNavigation::Core::Node do
       node.mark_active
     end
 
-  end  
+  end
 
-end
+end