page_navigation 0.2 → 0.3

data/ChangeLog

@@ -1,4 +1,7 @@
-=== Version 0.2 / 2012-1-25
+=== Version 0.3 / 2013-1-26
+* Fixed issue when navigating from a page to the same page
+
+=== Version 0.2 / 2013-1-25
 * Fixed minor issue when module is included in module that is included in Class
 
 === Version 0.1 / 2013-1-25

data/lib/page_navigation/version.rb

@@ -1,3 +1,3 @@
 module PageNavigation
-  VERSION = "0.2"
+  VERSION = "0.3"
 end

data/lib/page_navigation.rb

@@ -73,9 +73,10 @@ module PageNavigation
   #
   def continue_navigation_to(page_cls, how = {:using => :default}, &block)
     path = path_for how
-    from_index = find_index_for(path, @current_page.class)+1
+    from_index = find_index_for(path, @current_page.class)
     to_index = find_index_for(path, page_cls)-1
-    navigate_through_pages(path[from_index..to_index])
+    navigate_through_pages([path[from_index]]) if from_index == to_index
+    navigate_through_pages(path[from_index..to_index]) unless from_index == to_index
     on(page_cls, &block)
   end
 

data/spec/lib/page_navigation_spec.rb

@@ -78,12 +78,29 @@ describe PageNavigation do
                    [AnotherPage, :b_method],
                    [YetAnotherPage, :c_method]]
     }
-    fake_page = double('a_page')
+    @navigator.current_page = FactoryTestPage.new
+    f_page = FactoryTestPage.new
+    FactoryTestPage.should_receive(:new).and_return(f_page)
+    f_page.should_receive(:respond_to?).with(:a_method).and_return(true)
+    f_page.should_receive(:a_method)
+    a_page = AnotherPage.new
+    AnotherPage.should_receive(:new).and_return(a_page)
+    a_page.should_receive(:respond_to?).with(:b_method).and_return(true)
+    a_page.should_receive(:b_method)
+    @navigator.continue_navigation_to(YetAnotherPage).class.should == YetAnotherPage
+  end
+
+  it "should know how to continue routng from a location that is one page from the end" do
+    TestNavigator.routes = {
+      :default => [[FactoryTestPage, :a_method],
+                   [AnotherPage, :b_method],
+                   [YetAnotherPage, :c_method]]
+    }
+    @navigator.current_page = AnotherPage.new
+    fake_page = AnotherPage.new
     AnotherPage.should_receive(:new).and_return(fake_page)
     fake_page.should_receive(:respond_to?).with(:b_method).and_return(true)
     fake_page.should_receive(:b_method)
-    @navigator.current_page = FactoryTestPage.new
-    FactoryTestPage.should_not_receive(:new)
     @navigator.continue_navigation_to(YetAnotherPage).class.should == YetAnotherPage
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: page_navigation
 version: !ruby/object:Gem::Version
-  version: '0.2'
+  version: '0.3'
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-26 00:00:00.000000000 Z
+date: 2013-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -63,7 +63,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1381246942640339342
+      hash: -751630789489587890
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -72,7 +72,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1381246942640339342
+      hash: -751630789489587890
 requirements: []
 rubyforge_project: page_naigation
 rubygems_version: 1.8.24