simple-navigation 2.7.3 → 3.0.0.beta1

data/spec/lib/simple_navigation/{renderer → rendering/renderer}/breadcrumbs_spec.rb

@@ -1,19 +1,19 @@
-require File.dirname(__FILE__) + '/../../../spec_helper'
+require File.dirname(__FILE__) + '/../../../../spec_helper'
 require 'html/document' unless defined? HTML::Document
 
 describe SimpleNavigation::Renderer::Breadcrumbs do
-    
+
   describe 'render' do
-    
+
     def render(current_nav=nil, options={:level => :all})
       primary_navigation = primary_container
       select_item(current_nav) if current_nav
-      @renderer = SimpleNavigation::Renderer::Breadcrumbs.new(options)
+      setup_renderer_for SimpleNavigation::Renderer::Breadcrumbs, :rails, options
       HTML::Document.new(@renderer.render(primary_navigation)).root
     end
-      
+
     context 'regarding result' do
-      
+
       it "should render a div-tag around the items" do
         HTML::Selector.new('div').select(render).should have(1).entries
       end
@@ -29,7 +29,7 @@ describe SimpleNavigation::Renderer::Breadcrumbs do
           HTML::Selector.new('div a').select(render).should have(0).entries
         end
       end
-    
+
       context 'with current_navigation set' do
         before :all do
           @selection = HTML::Selector.new('div a').select(render(:invoices))
@@ -46,55 +46,13 @@ describe SimpleNavigation::Renderer::Breadcrumbs do
           end
         end
       end
-    
-    
+
+
       context 'nested sub_navigation' do
         it "should add an a tag for each selected item" do
           HTML::Selector.new('div a').select(render(:subnav1)).should have(2).entries
         end
       end
     end
-    
-    context 'regarding method calls' do
-      
-      context 'regarding the list_content' do
-        before(:each) do
-          @primary_navigation = primary_container
-          @list_content = stub(:list_content)
-          @list_items = stub(:list_items, :join => @list_content)
-          @items.stub!(:inject => @list_items)
-          @renderer = SimpleNavigation::Renderer::Breadcrumbs.new(options)
-        end
-      
-        it "should join the list_items" do
-          @list_items.should_receive(:join)
-        end
-      
-        it "should html_saferize the list_content" do
-          @renderer.should_receive(:html_safe).with(@list_content)
-        end
-
-        after(:each) do
-          @renderer.render(@primary_navigation)
-        end
-      end
-      
-      context 'regarding the items' do
-        before(:each) do
-          @primary_navigation = primary_container
-          select_item(:invoices)
-          @renderer = SimpleNavigation::Renderer::Breadcrumbs.new(options)
-        end
-        
-        it "should call html_safe on every rendered item's name" do
-          @items.each do |item|
-            @renderer.should_receive(:html_safe).with(item.name) if item.selected?
-          end
-          @renderer.should_receive(:html_safe).with(anything).twice()
-          @renderer.render(@primary_navigation)
-        end
-      end
-      
-    end
   end
 end

data/spec/lib/simple_navigation/rendering/renderer/links_spec.rb

@@ -0,0 +1,59 @@
+require File.dirname(__FILE__) + '/../../../../spec_helper'
+require 'html/document' unless defined? HTML::Document
+
+describe SimpleNavigation::Renderer::Links do
+
+    
+  describe 'render' do
+
+
+    def render(current_nav=nil, options={:level => :all})
+      primary_navigation = primary_container
+      select_item(current_nav) if current_nav
+      setup_renderer_for SimpleNavigation::Renderer::Links, :rails, options
+      HTML::Document.new(@renderer.render(primary_navigation)).root
+    end
+
+    context 'regarding result' do
+
+      it "should render a div-tag around the items" do
+        HTML::Selector.new('div').select(render).should have(1).entries
+      end
+      it "the rendered div-tag should have the specified dom_id" do
+        HTML::Selector.new('div#nav_dom_id').select(render).should have(1).entries
+      end
+      it "the rendered div-tag should have the specified class" do
+        HTML::Selector.new('div.nav_dom_class').select(render).should have(1).entries
+      end
+      it "should render an a-tag for each item" do
+        HTML::Selector.new('a').select(render).should have(3).entries
+      end
+      it "should pass the specified html_options to the a element" do
+        HTML::Selector.new('a[style=float:right]').select(render).should have(1).entries
+      end
+      it "should give the a-tag the id specified in the html_options" do
+        HTML::Selector.new('a#my_id').select(render).should have(1).entries
+      end
+      it "should give the a-tag the default id (stringified key) if no id is specified in the html_options" do
+        HTML::Selector.new('a#invoices').select(render).should have(1).entries
+      end
+      it "should not apply the the default id where there is an id specified in the html_options" do
+        HTML::Selector.new('a#users').select(render).should be_empty
+      end
+
+      context 'with current_navigation set' do
+        it "should mark the matching a-item as selected (with the css_class specified in configuration)" do
+          HTML::Selector.new('a.selected').select(render(:invoices)).should have(1).entries
+        end
+      end
+
+      context 'without current_navigation set' do
+        it "should not mark any of the items as selected" do
+          HTML::Selector.new('a.selected').select(render).should be_empty
+        end
+      end
+  
+    end
+    
+  end
+end

data/spec/lib/simple_navigation/{renderer → rendering/renderer}/list_spec.rb

@@ -1,19 +1,19 @@
-require File.dirname(__FILE__) + '/../../../spec_helper'
+require File.dirname(__FILE__) + '/../../../../spec_helper'
 require 'html/document' unless defined? HTML::Document
 
 describe SimpleNavigation::Renderer::List do
-    
+
   describe 'render' do
-    
+
     def render(current_nav=nil, options={:level => :all})
       primary_navigation = primary_container
       select_item(current_nav) if current_nav
-      @renderer = SimpleNavigation::Renderer::List.new(options)
+      setup_renderer_for SimpleNavigation::Renderer::List, :rails, options
       HTML::Document.new(@renderer.render(primary_navigation)).root
     end
-      
+
     context 'regarding result' do
-      
+
       it "should render a ul-tag around the items" do
         HTML::Selector.new('ul').select(render).should have(1).entries
       end
@@ -41,7 +41,7 @@ describe SimpleNavigation::Renderer::List do
       it "should not apply the the default id where there is an id specified in the html_options" do
         HTML::Selector.new('ul li#users').select(render).should be_empty
       end
-    
+
       context 'with current_navigation set' do
         it "should mark the matching li-item as selected (with the css_class specified in configuration)" do
           HTML::Selector.new('li.selected').select(render(:invoices)).should have(1).entries
@@ -49,9 +49,9 @@ describe SimpleNavigation::Renderer::List do
         it "should also mark the links inside the selected li's as selected" do
           HTML::Selector.new('li.selected a.selected').select(render(:invoices)).should have(1).entries
         end
-      
+
       end
-    
+
       context 'without current_navigation set' do
         it "should not mark any of the items as selected" do
           HTML::Selector.new('li.selected').select(render).should be_empty
@@ -60,7 +60,7 @@ describe SimpleNavigation::Renderer::List do
           HTML::Selector.new('a.selected').select(render).should be_empty
         end
       end
-    
+
       context 'nested sub_navigation' do
         it "should nest the current_primary's subnavigation inside the selected li-element" do
           HTML::Selector.new('li.selected ul li').select(render(:invoices)).should have(2).entries
@@ -74,23 +74,23 @@ describe SimpleNavigation::Renderer::List do
             HTML::Selector.new('#invoices #subnav2').select(render(:users, :level => :all, :expand_all => false)).should be_empty
           end
         end
-      
+
         context 'expand_all => true' do
           it "should render the invoices submenu even if the user-primary is active" do
             HTML::Selector.new('#invoices #subnav1').select(render(:users, :level => :all, :expand_all => true)).should have(1).entry
             HTML::Selector.new('#invoices #subnav2').select(render(:users, :level => :all, :expand_all => true)).should have(1).entry
           end
         end
-          
+
       end
-      
+
       context 'skip_if_empty' do
-        
+
         def render_container(options={})
-          @renderer = SimpleNavigation::Renderer::List.new(options)
+          setup_renderer_for SimpleNavigation::Renderer::List, :rails, options
           HTML::Document.new(@renderer.render(@container)).root
         end
-        
+
         context 'container is empty' do
           before(:each) do
             @container = SimpleNavigation::ItemContainer.new(0)
@@ -106,7 +106,7 @@ describe SimpleNavigation::Renderer::List do
             end
           end
         end
-        
+
         context 'container is not empty' do
           before(:each) do
             @container = primary_container
@@ -125,46 +125,5 @@ describe SimpleNavigation::Renderer::List do
       end
     end
     
-    context 'regarding method calls' do
-      
-      context 'regarding the list_content' do
-        before(:each) do
-          @primary_navigation = primary_container
-          @list_content = stub(:list_content)
-          @list_items = stub(:list_items, :join => @list_content)
-          @items.stub!(:inject => @list_items)
-          @renderer = SimpleNavigation::Renderer::List.new(options)
-        end
-      
-        it "should join the list_items" do
-          @list_items.should_receive(:join)
-        end
-      
-        it "should html_saferize the list_content" do
-          @renderer.should_receive(:html_safe).with(@list_content)
-        end
-      
-        after(:each) do
-          @renderer.render(@primary_navigation)
-        end
-      end
-      
-      context 'regarding the items' do
-        before(:each) do
-          @primary_navigation = primary_container
-          @renderer = SimpleNavigation::Renderer::List.new(options)
-        end
-        
-        it "should call html_safe on every item's name" do
-          @items.each do |item|
-            @renderer.should_receive(:html_safe).with(item.name)
-          end
-          @renderer.should_receive(:html_safe).with(anything)
-          @renderer.render(@primary_navigation)
-        end
-      end
-      
-    end
-    
   end
 end

data/spec/lib/simple_navigation_spec.rb

@@ -1,7 +1,11 @@
 require File.dirname(__FILE__) + '/../spec_helper'
 
 describe SimpleNavigation do
-    
+
+  before(:each) do
+    SimpleNavigation.config_file_path = 'path_to_config'
+  end
+
   describe 'config_file_name' do
     context 'for :default navigation_context' do
       it "should return the name of the default config file" do
@@ -18,33 +22,6 @@ describe SimpleNavigation do
     end
   end
   
-  describe 'config_file_paths_sentence' do
-    context 'no paths are set' do
-      before(:each) do
-        SimpleNavigation.config_file_paths = []
-      end
-      it {SimpleNavigation.config_file_paths_sentence.should == ''}
-    end
-    context 'one path is set' do
-      before(:each) do
-        SimpleNavigation.config_file_paths = ['first_path']
-      end
-      it {SimpleNavigation.config_file_paths_sentence.should == 'first_path'}
-    end
-    context 'two paths are set' do
-      before(:each) do
-        SimpleNavigation.config_file_paths = ['first_path', 'second_path']
-      end
-      it {SimpleNavigation.config_file_paths_sentence.should == 'first_path or second_path'}
-    end
-    context 'three pahts are set' do
-      before(:each) do
-        SimpleNavigation.config_file_paths = ['first_path', 'second_path', 'third_path']
-      end
-      it {SimpleNavigation.config_file_paths_sentence.should == 'first_path, second_path, or third_path'}
-    end
-  end
-  
   describe 'config_file_path=' do
     before(:each) do
       SimpleNavigation.config_file_paths = ['existing_path']
@@ -122,161 +99,19 @@ describe SimpleNavigation do
       it {SimpleNavigation.config_file?.should be_false}
     end
   end
-  
-  describe 'set_template_from' do
-    before(:each) do
-      @context = stub :context
-      SimpleNavigation.stub!(:extract_controller_from => @controller)
-    end
-    context 'regarding setting the controller' do
-      it "should set the controller" do
-        @controller = Object.new
-        SimpleNavigation.should_receive(:extract_controller_from).with(@context).and_return(@controller)
-        SimpleNavigation.should_receive(:controller=).with(@controller)
-        SimpleNavigation.set_template_from @context
-      end
-    end
-    context 'regarding setting the template' do
-      before(:each) do
-        @template = stub :template
-        @controller = Object.new
-        SimpleNavigation.stub!(:controller => @controller)
-      end
-      context 'template is stored in controller as instance_var (Rails2)' do
-        context 'template is set' do
-          before(:each) do
-            @controller.stub!(:instance_variable_get => @template)
-          end
-          it {SimpleNavigation.should_receive(:template=).with(@template)}
-         end
-        context 'template is not set' do
-          before(:each) do
-            @controller.stub!(:instance_variable_get => nil)
-          end
-          it {SimpleNavigation.should_receive(:template=).with(nil)}
-        end
-      end
-      context 'template is stored in controller as view_context (Rails3)' do
-        context 'template is set' do
-          before(:each) do            
-            @controller.stub!(:view_context => @template)
-          end
-          it {SimpleNavigation.should_receive(:template=).with(@template)}
-        end
-        context 'template is not set' do
-          before(:each) do            
-            @controller.stub!(:view_context => nil)
-          end
-          it {SimpleNavigation.should_receive(:template=).with(nil)}
-        end
-      end
-      after(:each) do
-        SimpleNavigation.set_template_from @context
-      end
-      
-    end
-  end
-
-  describe 'self.init_rails' do
-    before(:each) do
-      SimpleNavigation.config_file_paths = []
-      SimpleNavigation.stub!(:default_config_file_path => 'default_path')
-      ActionController::Base.stub!(:include)
-    end
-    context 'SimpleNavigation.config_file_path is already set' do
-      before(:each) do
-        SimpleNavigation.config_file_path = 'my_path'
-      end
-      it "should have both the default_path and the new path" do
-        SimpleNavigation.init_rails
-        SimpleNavigation.config_file_paths.should == ['my_path', 'default_path']
-      end
-    end
-    context 'SimpleNavigation.config_file_paths are not set' do
-      it "should set the config_file_path to the default" do
-        SimpleNavigation.init_rails
-        SimpleNavigation.config_file_paths.should == ['default_path']
-      end
-    end
-    it "should extend the ActionController::Base" do
-      ActionController::Base.should_receive(:include).with(SimpleNavigation::ControllerMethods)
-      SimpleNavigation.init_rails
-    end
-  end
 
   describe 'self.default_config_file_path' do
-    it {SimpleNavigation.default_config_file_path.should == './config'}
-  end
-
-  describe 'self.extract_controller_from' do
     before(:each) do
-      @nav_context = stub(:nav_context)
-    end
-    
-    context 'object responds to controller' do
-      before(:each) do
-        @controller = stub(:controller)
-        @nav_context.stub!(:controller).and_return(@controller)
-      end
-      
-      it "should return the controller" do
-        SimpleNavigation.send(:extract_controller_from, @nav_context).should == @controller
-      end
-      
-    end
-    
-    context 'object does not respond to controller' do
-      it "should return the nav_context" do
-        SimpleNavigation.send(:extract_controller_from, @nav_context).should == @nav_context
-      end
-    end
-  end
-  
-  describe 'context_for_eval' do
-    context 'controller is present' do
-      before(:each) do
-        @controller = stub(:controller)
-        SimpleNavigation.stub!(:controller => @controller)
-      end
-      context 'template is present' do
-        before(:each) do
-          @template = stub(:template)
-          SimpleNavigation.stub!(:template => @template)
-        end
-        it {SimpleNavigation.context_for_eval.should == @template}
-      end
-      context 'template is not present' do
-        before(:each) do
-          SimpleNavigation.stub!(:template => nil)
-        end
-        it {SimpleNavigation.context_for_eval.should == @controller}
-      end
-    end
-    context 'controller is not present' do
-      before(:each) do
-        SimpleNavigation.stub!(:controller => nil)
-      end
-      context 'template is present' do
-        before(:each) do
-          @template = stub(:template)
-          SimpleNavigation.stub!(:template => @template)
-        end
-        it {SimpleNavigation.context_for_eval.should == @template}
-      end
-      context 'template is not present' do
-        before(:each) do
-          SimpleNavigation.stub!(:template => nil)
-        end
-        it {lambda {SimpleNavigation.context_for_eval}.should raise_error}
-      end
+      SimpleNavigation.stub!(:root => 'root')
     end
+    it {SimpleNavigation.default_config_file_path.should == 'root/config'}
   end
   
   describe 'regarding renderers' do
     it "should have registered the builtin renderers by default" do
       SimpleNavigation.registered_renderers.should_not be_empty
     end
-    
+
     describe 'register_renderer' do
       before(:each) do
         @renderer = stub(:renderer)
@@ -286,9 +121,25 @@ describe SimpleNavigation do
         SimpleNavigation.registered_renderers[:my_renderer].should == @renderer
       end
     end
-    
+
   end
 
+  describe 'set_env' do
+    before(:each) do
+      SimpleNavigation.config_file_paths = []
+      SimpleNavigation.stub!(:default_config_file_path => 'default_path')
+      SimpleNavigation.set_env('root', 'my_env')
+    end
+    it "should set the root" do
+      SimpleNavigation.root.should == 'root'
+    end
+    it "should set the environment" do
+      SimpleNavigation.environment.should == 'my_env'
+    end
+    it "should add the default-config path to the list of config_file_paths" do
+      SimpleNavigation.config_file_paths.should == ['default_path']
+    end
+  end
 
   describe 'load_config' do
     context 'config_file_path is set' do
@@ -340,9 +191,9 @@ describe SimpleNavigation do
         SimpleNavigation.config_file_path = 'path_to_config'
         File.stub!(:exists? => true)
       end
-      context "RAILS_ENV undefined" do
+      context "environment undefined" do
         before(:each) do
-          SimpleNavigation.rails_env = nil
+          SimpleNavigation.stub!(:environment => nil)
         end
         it "should load the config file twice" do
           IO.should_receive(:read).twice
@@ -350,11 +201,11 @@ describe SimpleNavigation do
           SimpleNavigation.load_config
         end
       end
-      context "RAILS_ENV defined" do
+      context "environment defined" do
         before(:each) do
-          SimpleNavigation.rails_env = 'production'
+          SimpleNavigation.stub!(:environment => 'production')
         end
-        context "RAILS_ENV=production" do
+        context "environment=production" do
           it "should load the config file only once" do
             IO.should_receive(:read).once
             SimpleNavigation.load_config
@@ -362,9 +213,9 @@ describe SimpleNavigation do
           end
         end
         
-        context "RAILS_ENV=development" do
+        context "environment=development" do
           before(:each) do
-            SimpleNavigation.rails_env = 'development'
+            SimpleNavigation.stub!(:environment => 'development')
           end
           it "should load the config file twice" do
             IO.should_receive(:read).twice
@@ -373,9 +224,9 @@ describe SimpleNavigation do
           end
         end
         
-        context "RAILS_ENV=test" do
+        context "environment=test" do
           before(:each) do
-            SimpleNavigation.rails_env = 'test'
+            SimpleNavigation.stub!(:environment => 'test')
           end
           it "should load the config file twice" do
             IO.should_receive(:read).twice
@@ -390,59 +241,10 @@ describe SimpleNavigation do
     end
   end
   
-  describe 'request' do
-    context 'template is set' do
-      before(:each) do
-        @request = stub(:request)
-        SimpleNavigation.stub!(:template => stub(:template, :request => @request))
-      end
-      it {SimpleNavigation.request.should == @request}
-    end
-    context 'template is not set' do
-      it {SimpleNavigation.request.should be_nil}
-    end
-  end
-  
-  describe 'request_uri' do
-    context 'request is set' do
-      context 'fullpath is defined on request' do
-        before(:each) do
-          @request = stub(:request, :fullpath => '/fullpath')
-          SimpleNavigation.stub!(:request => @request)
-        end
-        it {SimpleNavigation.request_uri.should == '/fullpath'}
-      end
-      context 'fullpath is not defined on request' do
-        before(:each) do
-          @request = stub(:request, :request_uri => '/request_uri')
-          SimpleNavigation.stub!(:request => @request)
-        end
-        it {SimpleNavigation.request_uri.should == '/request_uri'}
-      end
-    end
-    context 'request is not set' do
-      before(:each) do
-        SimpleNavigation.stub!(:request => nil)
-      end
-      it {SimpleNavigation.request_uri.should == ''}
-    end
-  end
-  
   describe 'config' do
     it {SimpleNavigation.config.should == SimpleNavigation::Configuration.instance}
   end
   
-  describe 'current_navigation_for' do
-    before(:each) do
-      @controller = stub(:controller)
-      SimpleNavigation.stub!(:controller => @controller)
-    end
-    it "should access the correct instance_var in the controller" do
-      @controller.should_receive(:instance_variable_get).with(:@sn_current_navigation_1)
-      SimpleNavigation.current_navigation_for(1)
-    end
-  end
-
   describe 'active_item_container_for' do
     before(:each) do
       @primary = stub(:primary)
@@ -472,77 +274,28 @@ describe SimpleNavigation do
     end
   end
 
-  describe 'handle_explicit_navigation' do
-    def args(*args)
-      SimpleNavigation.stub!(:explicit_navigation_args => args.compact.empty? ? nil : args)
-    end
-
-    before(:each) do
-      @controller = stub(:controller)
-      SimpleNavigation.stub!(:controller => @controller)
-    end
-
-    context 'with explicit navigation set' do
-      context 'list of navigations' do
-        before(:each) do
-          args :first, :second, :third
-        end
-        it "should set the correct instance var in the controller" do
-          @controller.should_receive(:instance_variable_set).with(:@sn_current_navigation_3, :third)
-          SimpleNavigation.handle_explicit_navigation
-        end
-      end
-      context 'single navigation' do
-        context 'specified key is a valid navigation item' do
-          before(:each) do
-            @primary = stub(:primary, :level_for_item => 2)
-            SimpleNavigation.stub!(:primary_navigation => @primary)
-            args :key
-          end
-          it "should set the correct instance var in the controller" do
-            @controller.should_receive(:instance_variable_set).with(:@sn_current_navigation_2, :key)
-            SimpleNavigation.handle_explicit_navigation
-          end
-        end
-        context 'specified key is an invalid navigation item' do
-          before(:each) do
-            @primary = stub(:primary, :level_for_item => nil)
-            SimpleNavigation.stub!(:primary_navigation => @primary)
-            args :key
-          end
-          it "should not raise an ArgumentError" do
-            lambda {SimpleNavigation.handle_explicit_navigation}.should_not raise_error(ArgumentError)
-          end
-        end
-      end
-      context 'hash with level' do
-        before(:each) do
-          args :level_2 => :key
-        end
-        it "should set the correct instance var in the controller" do
-          @controller.should_receive(:instance_variable_set).with(:@sn_current_navigation_2, :key)
-          SimpleNavigation.handle_explicit_navigation
-        end
-      end
-      context 'hash with multiple_levels' do
-        before(:each) do
-          args :level_2 => :key, :level_1 => :bla
-        end
-        it "should set the correct instance var in the controller" do
-          @controller.should_receive(:instance_variable_set).with(:@sn_current_navigation_2, :key)
-          SimpleNavigation.handle_explicit_navigation
-        end
+  describe 'load_adapter' do
+    context 'Rails' do
+      before(:each) do
+        SimpleNavigation.stub!(:framework => :rails)
+        SimpleNavigation.load_adapter
       end
+      it {SimpleNavigation.adapter_class.should == SimpleNavigation::Adapters::Rails}
     end
-    context 'without explicit navigation set' do
+    context 'Padrino' do
       before(:each) do
-        args nil
+        SimpleNavigation.stub!(:framework => :padrino)
+        SimpleNavigation.load_adapter
       end
-      it "should not set the current_navigation instance var in the controller" do
-        @controller.should_not_receive(:instance_variable_set)
-        SimpleNavigation.handle_explicit_navigation
+      it {SimpleNavigation.adapter_class.should == SimpleNavigation::Adapters::Padrino}
+    end
+    context 'Sinatra' do
+      before(:each) do
+        SimpleNavigation.stub!(:framework => :sinatra)
+        SimpleNavigation.load_adapter
       end
+      it {SimpleNavigation.adapter_class.should == SimpleNavigation::Adapters::Sinatra}
     end
   end
-
+  
 end