menu_maker 0.1.0 → 0.2.0

data/test/menu_item_test.rb

@@ -10,6 +10,11 @@ module MenuMaker
       assert_equal 'my/path', item.path
     end
 
+    test 'accepts many paths' do
+      item = Menu::MenuItem.new 'My title', 'path/1', 'path/2', 'path/3'
+      assert_equal ['path/1', 'path/2', 'path/3'], item.paths.map(&:address)
+    end
+
     test 'submenu_paths returns submenu paths recursively' do
       item = Menu::MenuItem.new 'Level 1', 'level/1'
 
@@ -28,7 +33,7 @@ module MenuMaker
 
       expected = %w[level/2 level/2/1 level/2/2 level/3 level/4 level/5 level/6]
 
-      assert_equal expected, item.submenu_paths.sort
+      assert_equal expected, item.submenu_paths.map(&:address).sort
     end
 
     test 'all_paths returns submenu paths + current menu path' do
@@ -47,7 +52,7 @@ module MenuMaker
 
       expected = %w[level/1 level/2 level/3 level/4 level/5 level/6]
 
-      assert_equal expected, item.all_paths.sort
+      assert_equal expected, item.all_paths.map(&:address).sort
     end
 
     test 'has_path? also checks for submenus' do
@@ -69,6 +74,25 @@ module MenuMaker
       refute item.has_path?('level/8')
     end
 
+    test 'Menu#add accepts multiple paths and options' do
+      menu = Menu.new proc {}
+      menu.add('Item', 'path/1', [:post, 'path/2'], [:put, 'path/3'], option: 'optional')
+
+      result = menu.items.first.paths.map(&:address)
+
+      expected = %w[path/1 path/2 path/3]
+
+      assert_equal expected, result
+      assert_equal 'optional', menu.items.first.option
+    end
+
+    test 'has_path? matches on other restful paths' do
+      item = Menu::MenuItem.new 'Item', 'main_path', [:post, 'other/path']
+
+      assert item.has_path? [:post, 'other/path']
+      assert item.has_path? Path.new(:post, 'other/path')
+    end
+
     test "has_submenu? when returns false" do
       item = Menu::MenuItem.new 'Level 1', 'level/1'
       refute item.has_submenu?

data/test/menu_renderer_test.rb

@@ -1,4 +1,5 @@
 require 'test_helper'
+require 'ostruct'
 
 module MenuMaker
   class MenuRendererTest < ActiveSupport::TestCase
@@ -60,5 +61,19 @@ module MenuMaker
         end
       end
     end
+
+    context 'with a MenuRenderer class renderer' do
+      should 'detect the path when not provided and the context responds to request' do
+        request = Class.new do
+          def method; 'POST'  end
+          def path;   '/path' end
+        end.new
+
+        context  = OpenStruct.new   request: request
+        renderer = MenuRenderer.new context
+
+        assert_equal Path.new(:post, '/path'), renderer.current_path
+      end
+    end
   end
 end

data/test/path_test.rb

@@ -0,0 +1,84 @@
+require 'test_helper'
+
+module MenuMaker
+  class PathTest < ActionView::TestCase
+    test 'accepts get, post, put, patch and delete' do
+      %i[get post put patch delete].each do |method|
+        Path.new method, '/path'
+      end
+    end
+
+    test "doesn't accept a method other than get, post, put, patch and delete" do
+      assert_raise Path::PathError do
+        Path.new :ouch, '/path'
+      end
+    end
+
+    test 'to_s returns the address' do
+      assert_equal '/path', Path.new(:get, '/path').to_s
+    end
+
+    test 'creates a get path from a given string' do
+      assert_equal Path.new(:get, '/path'), Path.convert('/path')
+    end
+
+    test 'creates a path from an array' do
+      assert_equal Path.new(:post, '/path'), Path.convert([:post, '/path'])
+    end
+
+    test "creates a path from an array assumes get when can't find request method" do
+      assert_equal Path.new(:get, '/path'), Path.convert(['/path'])
+    end
+
+    test "creates a path from an array on empty array creates empty get path" do
+      assert_equal Path.new(:get, ''), Path.convert([])
+    end
+
+    test "returns back the path if already a Path" do
+      path = Path.new :put, '/path'
+      assert_equal path, Path.convert(path)
+    end
+
+    test "creates a path from an object wich responds to path and method" do
+      request = Class.new do
+        def path
+          '/path'
+        end
+
+        def method
+          'PUT'
+        end
+      end.new
+
+      assert_equal Path.new(:put, '/path'), Path.convert(request)
+    end
+
+    test "fails if can't create path from object which responds to path and method" do
+      assert_raise Path::PathError do
+        Path.convert(Object.new)
+      end
+    end
+
+    test "from_path fails when not a path" do
+      assert_raise Path::PathError do
+        Path.from_path(Object.new)
+      end
+    end
+
+    test "equality works with an equal path" do
+      assert_equal Path.new(:put, '/path'), Path.new(:put, '/path')
+    end
+
+    test "equality works with a convertible string" do
+      assert_equal Path.new(:get, '/path'), '/path'
+    end
+
+    test "equality works with a convertible array" do
+      assert Path.new(:post, '/path') == [:post, '/path']
+    end
+
+    test "a Path global conversion method is also available" do
+      assert_equal Path.new(:post, '/path'), Path(:post, '/path')
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: menu_maker
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Thiago A. Silva
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-07 00:00:00.000000000 Z
+date: 2014-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -57,6 +57,7 @@ files:
 - lib/menu_maker/menu.rb
 - lib/menu_maker/menu_renderer.rb
 - lib/menu_maker/menu_renderer_container.rb
+- lib/menu_maker/path.rb
 - lib/menu_maker/version.rb
 - lib/tasks/menu_maker_tasks.rake
 - test/dummy/README.rdoc
@@ -100,6 +101,7 @@ files:
 - test/menu_item_test.rb
 - test/menu_maker_test.rb
 - test/menu_renderer_test.rb
+- test/path_test.rb
 - test/test_helper.rb
 homepage: http://github.com/thiagoa/menu_maker
 licenses:
@@ -167,4 +169,5 @@ test_files:
 - test/menu_item_test.rb
 - test/menu_maker_test.rb
 - test/menu_renderer_test.rb
+- test/path_test.rb
 - test/test_helper.rb