nav_lynx 1.0.0 → 1.1.0

data/spec/helpers/nav_link_helper_spec.rb

@@ -11,14 +11,14 @@ describe NavLinkHelper do
     before { helper.stub(:request => :request) }
 
     it 'accepts only a title and a path' do
-      NavLinkHelper::LinkGenerator.should_receive(:new).with(:request, "My Title", "/path", {}, {})
+      NavLinkHelper::LinkGenerator.should_receive(:new).with(:request, "My Title", "/path", controller, {}, {})
         .and_return(link_generator)
 
       helper.nav_link_to("My Title", "/path")
     end
 
     it 'accepts a block that returns the title' do
-      NavLinkHelper::LinkGenerator.should_receive(:new).with(:request, "My Title", "/path", {}, {})
+      NavLinkHelper::LinkGenerator.should_receive(:new).with(:request, "My Title", "/path", controller, {}, {})
         .and_return(link_generator)
 
       helper.nav_link_to("/path") do
@@ -27,88 +27,97 @@ describe NavLinkHelper do
     end
 
     it "accepts options and html_options parameters" do
-      NavLinkHelper::LinkGenerator.should_receive(:new).with(:request, "My Title", "/path", :html_options, :options)
+      NavLinkHelper::LinkGenerator.should_receive(:new).with(:request, "My Title", "/path", controller, :html_options, :options)
         .and_return(link_generator)
 
       helper.nav_link_to("My Title", "/path", :html_options, :options)
     end
   end
 
-  require 'spec_helper'
-
   describe NavLinkHelper::LinkGenerator do
     describe '#to_html' do
       subject { described_class::LinkGenerator }
       let(:request){ helper.request }
+      after { NavLYNX.selected_class = 'selected' }
 
       it "outputs a simple link when appropriate" do
-        subject.new(request, 'Hi', '/projects').to_html.should == '<a href="/projects">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller).to_html.should == '<a href="/projects">Hi</a>'
+      end
+
+      it "accepts a hash for the URL parameter" do
+        subject.new(request, 'Hi', {:controller => 'projects', :action => 'index'}, controller).to_html.should == '<a href="/projects">Hi</a>'
       end
 
       it "knows when to flag the link as current" do
         request.stub(:fullpath).and_return('/projects')
-        subject.new(request, 'Hi', '/projects').to_html.should == '<a href="/projects" class="selected">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller).to_html.should have_tag("a.selected[href='/projects']", :text => "Hi")
+      end
+
+      it "knows use a globally defined selected class different from the default" do
+        NavLYNX.selected_class = 'current'
+        request.stub(:fullpath).and_return('/projects')
+        subject.new(request, 'Hi', '/projects', controller).to_html.should have_tag("a.current[href='/projects']", :text => "Hi")
       end
 
       it "does not flag the link as current if there are parameters in the path" do
         request.stub(:fullpath).and_return('/projects?all=true')
-        subject.new(request, 'Hi', '/projects').to_html.should == '<a href="/projects">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller).to_html.should == '<a href="/projects">Hi</a>'
       end
 
       it "flags the link as selected if there are parameters in the path but we are ignoring parameters" do
         request.stub(:fullpath).and_return('/projects?all=true')
-        subject.new(request, 'Hi', '/projects', {}, :ignore_params => true).to_html
-          .should == '<a href="/projects" class="selected">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller, {}, {:ignore_params => true}).to_html
+          .should have_tag("a.selected[href='/projects']", :text => "Hi")
       end
 
       it "allows the specification of an alternate selected class" do
         request.stub(:fullpath).and_return('/projects')
-        subject.new(request, 'Hi', '/projects', {}, :selected_class => 'current').to_html
-          .should == '<a href="/projects" class="current">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller, {}, {:selected_class => 'current'}).to_html
+          .should have_tag("a.current[href='/projects']", :text => "Hi")
       end
 
       it "generates a wrapper for the link when specified" do
-        subject.new(request, 'Hi', '/projects', {}, :wrapper => 'li').to_html
+        subject.new(request, 'Hi', '/projects', controller, {}, {:wrapper => 'li'}).to_html
           .should == '<li><a href="/projects">Hi</a></li>'
       end
 
       it "provides a class for the wrapper when specified" do
-        subject.new(request, 'Hi', '/projects', {}, :wrapper => 'li', :wrapper_class => 'container').to_html
+        subject.new(request, 'Hi', '/projects', controller, {}, {:wrapper => 'li', :wrapper_class => 'container'}).to_html
           .should == '<li class="container"><a href="/projects">Hi</a></li>'
       end
 
       it "provides a class for the wrapper when specified along with a selected class if neeeded" do
         request.stub(:fullpath).and_return('/projects')
-        subject.new(request, 'Hi', '/projects', {}, :wrapper => 'li', :wrapper_class => 'container').to_html
+        subject.new(request, 'Hi', '/projects', controller, {}, {:wrapper => 'li', :wrapper_class => 'container'}).to_html
           .should == '<li class="selected container"><a href="/projects">Hi</a></li>'
       end
 
       it "allows specification of the link classes" do
-        subject.new(request, 'Hi', '/projects', :class => 'one two').to_html
-          .should == '<a href="/projects" class="one two">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller, {:class => 'one two'}, {}).to_html
+          .should have_tag("a.one.two[href='/projects']", :text => "Hi")
       end
 
       it "combines custom link classes when the link is selected" do
         request.stub(:fullpath).and_return('/projects')
-        subject.new(request, 'Hi', '/projects', :class => 'one two').to_html
-          .should == '<a href="/projects" class="one two selected">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller, {:class => 'one two'}, {}).to_html
+          .should have_tag("a.selected.one.two[href='/projects']", :text => "Hi")
       end
 
       it "ignores the ID in the path when provided with a segment" do
         request.stub(:fullpath).and_return('/projects/2')
-        subject.new(request, 'Hi', '/projects', {}, :url_segment => 1).to_html
-          .should == '<a href="/projects" class="selected">Hi</a>'
+        subject.new(request, 'Hi', '/projects', controller, {}, {:url_segment => 1}).to_html
+          .should have_tag("a.selected[href='/projects']", :text => "Hi")
       end
 
       it "handles deep URL segments" do
         request.stub(:fullpath).and_return('/projects/2/3')
-        subject.new(request, 'Hi', '/projects/2', {}, :url_segment => 2).to_html
-          .should == '<a href="/projects/2" class="selected">Hi</a>'
+        subject.new(request, 'Hi', '/projects/2', controller, {}, {:url_segment => 2}).to_html
+          .should have_tag("a.selected[href='/projects/2']", :text => "Hi")
       end
 
       it "knows to not match on the first segment when requested" do
         request.stub(:fullpath).and_return('/projects/2/3')
-        subject.new(request, 'Hi', '/projects/1', {}, :url_segment => 2).to_html
+        subject.new(request, 'Hi', '/projects/1', controller, {}, {:url_segment => 2}).to_html
           .should == '<a href="/projects/1">Hi</a>'
       end
 
@@ -119,8 +128,17 @@ describe NavLinkHelper do
         request.stub(:fullpath).and_return('/members/pages/1')
         request.stub(:path).and_return('/members/pages/1')
 
-        subject.new(request, 'Hi', '/members/1', {}, :controller_segment => 1).to_html
-          .should == '<a href="/members/1" class="selected">Hi</a>'
+        subject.new(request, 'Hi', '/members/1', controller, {}, {:controller_segment => 1}).to_html
+          .should have_tag("a.selected[href='/members/1']", :text => "Hi")
+      end
+
+      it "handles the case when the current request path is a POST route & has no GET route" do
+        # scenario where a route exists only for POST, so GET fails
+        Rails.application.routes.stub(:recognize_path).with('/projects/something').and_raise(ActionController::RoutingError.new("No route matches '/projects'"))
+        Rails.application.routes.stub(:recognize_path).with('/projects').and_return(:controller => 'projects', :action => 'index')
+
+        request.stub(:fullpath => '/projects/something', :path => '/projects/something')
+        subject.new(request, 'Hi', '/projects', controller).to_html.should == '<a href="/projects">Hi</a>'
       end
     end
   end

data/spec/spec_helper.rb

@@ -3,6 +3,7 @@ ENV["RAILS_ENV"] ||= 'test'
 require File.expand_path("../dummy/config/environment.rb",  __FILE__)
 require 'rspec/rails'
 require 'rspec/autorun'
+require 'rspec-html-matchers'
 
 Rails.backtrace_cleaner.remove_silencers!
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nav_lynx
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -11,14 +11,14 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-12 00:00:00.000000000 Z
+date: 2013-11-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 3.2.13
   type: :runtime
@@ -26,7 +26,7 @@ dependencies:
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 3.2.13
 - !ruby/object:Gem::Dependency
@@ -77,6 +77,38 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-html-matchers
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Rails helper to generate navigation links with a selected class.
 email:
 - dan.tello@viget.com
@@ -129,6 +161,20 @@ files:
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
 - spec/dummy/script/rails
+- spec/dummy/tmp/cache/assets/test/sprockets/061366614f53b2e629f4ce5af7376503
+- spec/dummy/tmp/cache/assets/test/sprockets/12e93292435b9fd9a8ead1d08efcc40b
+- spec/dummy/tmp/cache/assets/test/sprockets/13fe41fee1fe35b49d145bcc06610705
+- spec/dummy/tmp/cache/assets/test/sprockets/16813814f2e3c416d514746de8977f71
+- spec/dummy/tmp/cache/assets/test/sprockets/20bdd97a4de949dfc1ccc5b4881d5de6
+- spec/dummy/tmp/cache/assets/test/sprockets/2f5173deea6c795b8fdde723bb4b63af
+- spec/dummy/tmp/cache/assets/test/sprockets/357970feca3ac29060c1e3861e2c0953
+- spec/dummy/tmp/cache/assets/test/sprockets/52ef98542a240e988c164c670087be82
+- spec/dummy/tmp/cache/assets/test/sprockets/8528046f7875341a083dca3a0349725a
+- spec/dummy/tmp/cache/assets/test/sprockets/a45664b91391c11181b20e5903d4aa8f
+- spec/dummy/tmp/cache/assets/test/sprockets/cffd775d018f68ce5dba1ee0d951a994
+- spec/dummy/tmp/cache/assets/test/sprockets/d771ace226fc8215a3572e0aa35bb0d6
+- spec/dummy/tmp/cache/assets/test/sprockets/f7cbd26ba1d28d48de824f0e94586655
+- spec/dummy/tmp/cache/assets/test/sprockets/fe314a8ef926518b95394d23d7967ac4
 - spec/features/navigation_spec.rb
 - spec/helpers/nav_link_helper_spec.rb
 - spec/spec_helper.rb
@@ -146,7 +192,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3753548243184339803
+      hash: 3905832851249229231
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -155,10 +201,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3753548243184339803
+      hash: 3905832851249229231
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.21
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Rails helper to generate navigation links with a selected class.
@@ -199,6 +245,20 @@ test_files:
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
 - spec/dummy/script/rails
+- spec/dummy/tmp/cache/assets/test/sprockets/061366614f53b2e629f4ce5af7376503
+- spec/dummy/tmp/cache/assets/test/sprockets/12e93292435b9fd9a8ead1d08efcc40b
+- spec/dummy/tmp/cache/assets/test/sprockets/13fe41fee1fe35b49d145bcc06610705
+- spec/dummy/tmp/cache/assets/test/sprockets/16813814f2e3c416d514746de8977f71
+- spec/dummy/tmp/cache/assets/test/sprockets/20bdd97a4de949dfc1ccc5b4881d5de6
+- spec/dummy/tmp/cache/assets/test/sprockets/2f5173deea6c795b8fdde723bb4b63af
+- spec/dummy/tmp/cache/assets/test/sprockets/357970feca3ac29060c1e3861e2c0953
+- spec/dummy/tmp/cache/assets/test/sprockets/52ef98542a240e988c164c670087be82
+- spec/dummy/tmp/cache/assets/test/sprockets/8528046f7875341a083dca3a0349725a
+- spec/dummy/tmp/cache/assets/test/sprockets/a45664b91391c11181b20e5903d4aa8f
+- spec/dummy/tmp/cache/assets/test/sprockets/cffd775d018f68ce5dba1ee0d951a994
+- spec/dummy/tmp/cache/assets/test/sprockets/d771ace226fc8215a3572e0aa35bb0d6
+- spec/dummy/tmp/cache/assets/test/sprockets/f7cbd26ba1d28d48de824f0e94586655
+- spec/dummy/tmp/cache/assets/test/sprockets/fe314a8ef926518b95394d23d7967ac4
 - spec/features/navigation_spec.rb
 - spec/helpers/nav_link_helper_spec.rb
 - spec/spec_helper.rb