menu-rails 0.0.2 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 54995716b752ca3986bebdffa9bf21141217ce65
-  data.tar.gz: d53a501614e2b8650facd1f3e70b05c5f88cb749
+  metadata.gz: ade3caa8f049fcfc1c044546241a52b7371f71ad
+  data.tar.gz: 8cb89b4157e51547765c190f2816616f6a365231
 SHA512:
-  metadata.gz: 43ec47cda017c386a4265dafc53f4598a24711b8292c3b390464d5f7fcd4954d14ec0c7e30c19601702747b3269ba38300b9a795c23520c94e343f1efc1194df
-  data.tar.gz: fe61f77f6530eadda79e447a791a2ab6cca039a5efafa36cd216cbc4cacffa4116b884c7c50bfb251e4f45c40372cd6f4d3f012ad68a6a7b023ac5bc5d9bf218
+  metadata.gz: 035ea5bade29a8ee20e619ee369b03c5b7da41c799638a4ebd1bd10bb7ee3ff40911c5b0d2381d87fd5bf854819b25d5d0c22b5aff8736aec8408fc406a82efb
+  data.tar.gz: 1c0a7fe369d2a1864011ec74a4fca203c8c8a275aebc85c5c06ef40237f97d1e16fd4e815f8e849aac401df79101b91adad2a592daeca61f403248dffa5160df

data/.ruby-version

@@ -0,0 +1 @@
+2.0.0-p353

data/README.md

@@ -1,12 +1,43 @@
 # menu-rails
 
-TODO: Write a gem description
+This gem provide a set of helpers and methods that allow to easily generate a menu and identify
+the currently active link item.  
+Comes with support to generate menu from a YAML file in config directory, but supports generation
+even from database (or whatever you want).
+
+## Features
+
+- Threadsafe (not tested)
+- Menu generation from YAML file
+- Menu generation from any other source (database for example)
+- Check which menu item is active
+- Cancan support
+
+## How to generate menu from database
+
+You must create 2 classes which will respond to some methods:
+
+A `Menu` class which must respond to:
+
+- `all_menu_items` with an array of menu items
+- Must `include MenuRails::BaseMenu`
+
+A `MenuItem` class which must respond to:
+
+- `url` *String* which must be a valid url
+- `text` *String* which is text displayed to users
+- `authorization` should return `nil` (will be displayed without checking through `can?`) or a hash like this:
+  `{ can: :read, class_name: 'ApplicationController' }` which will be used in `can?` method (view) and if false, will skip the item
+- `active_controller_only` *Boolean* `false` if you want to check both against controller and action name if this link is currently active. If `true`, only controller will be checked
+- `active_method` *Symbol* if not `nil`, will be called instead of standard `active?` method (and `active_controller_only` ignored)
+
+The gem should be threadsafe
 
 ## Installation
 
 Add this line to your application's Gemfile:
 
-    gem 'menu-rails', '~> 0.0.1'
+    gem 'menu-rails', '~> 0.1.0'
 
 And then execute:
 
@@ -20,6 +51,11 @@ Or install it yourself as:
 
 TODO: Write usage instructions here
 
+## TODO
+
+- Split `Menu` and `MenuItem` classes into 2 pairs, `MenuBase` will not include anything related to `has_no_table`, same for `MenuItemBase`
+- I18n support (very important, for menu text string)
+
 ## Contributing
 
 1. Fork it

data/app/models/menu_rails/menu.rb

@@ -8,6 +8,7 @@ module MenuRails
 
   class Menu < ActiveRecord::Base
     include Symbolize::ActiveRecord
+    include MenuRails::BaseMenu
 
     has_no_table
 
@@ -18,13 +19,28 @@ module MenuRails
 
     symbolize :mrid
 
+    def get_menu_item_by_mriid(identifier)
+      self.menu_items.select{ |menu_item| menu_item.mriid == identifier.to_sym }.first
+    end
+
+    def all_menu_items
+      self.menu_items
+    end
+
     private
 
       def build_menu_items_from_yaml_data!(menu_items_data)
         menu_items_data.each do |menu_item_data|
-          menu_item        = MenuItem.new(menu_item_data)
-          menu_item.menu   = self
-          self.menu_items << menu_item
+          menu_item_params          = menu_item_data.values.first.merge(mriid: menu_item_data.keys.first.to_sym)
+          overwrite_menu_item_class = menu_item_params.delete(:class_name)
+          if overwrite_menu_item_class.nil?
+            overwrite_menu_item_class = menu_item_class
+          else
+            overwrite_menu_item_class = overwrite_menu_item_class.constantize
+          end
+          menu_item                 = overwrite_menu_item_class.new(menu_item_params)
+          menu_item.menu            = self
+          self.menu_items          << menu_item
         end
         self.menu_items.freeze
 
@@ -32,6 +48,10 @@ module MenuRails
         self
       end
 
+      def menu_item_class
+        Rails.application.config.menu_rails.menu_item_class_name.constantize
+      end
+
     class << self
 
       SEMAPHORE = Mutex.new
@@ -44,7 +64,10 @@ module MenuRails
 
             base_menu.each do |menu_identifier, menu|
               mid_sym                = menu_identifier.to_sym
-              @menu_by_mrid[mid_sym] = new(mrid: mid_sym).send(:build_menu_items_from_yaml_data!, menu[:menu_items]).freeze
+              overwrite_menu_class   = menu.delete(:class_name)
+              overwrite_menu_class   = overwrite_menu_class.nil? ? menu_class : overwrite_menu_class.constantize
+              @menu_by_mrid[mid_sym] = overwrite_menu_class.new(mrid: mid_sym).send( :build_menu_items_from_yaml_data!,
+                                                                                     menu[:menu_items] ).freeze
             end
           end
 
@@ -52,6 +75,12 @@ module MenuRails
         end
       end
 
+      private
+
+        def menu_class
+          Rails.application.config.menu_rails.menu_class_name.constantize
+        end
+
     end
     
   end

data/app/models/menu_rails/menu_item.rb

@@ -1,21 +1,81 @@
 require 'activerecord-tableless'
 require 'symbolize'
+require 'active_support/all'
 
 module MenuRails
 
   class MenuItem < ActiveRecord::Base
     include Symbolize::ActiveRecord
+    include Rails.application.routes.url_helpers
 
     has_no_table
 
-    column :menu_id, :integer
+    column :menu_id,                  :integer
     # Stands for MenuRails Item ID
-    column :mriid,   :string
-    column :text,    :string
+    column :mriid,                    :string
+    column :text,                     :string
+    column :authorization_can,        :string
+    column :authorization_class_name, :string
+    column :url_method,               :string
+    column :url_text,                 :string
+    column :active_controller_only,   :boolean
+    column :active_method,            :string
     
     belongs_to :menu
 
-    symbolize :mriid
+    symbolize :mriid, :authorization_can, :url_method, :active_method
+
+    validates :text,       presence: true
+    validates :url_method, presence: { if: -> { self.url_text.blank?   } }
+    validates :url_text,   presence: { if: -> { self.url_method.blank? } }
+
+    def url=(value)
+      if value.kind_of? String
+        self.url_text   = value
+      elsif value.kind_of? Symbol
+        self.url_method = value
+      end
+    end
+
+    def url
+      self.url_is_string? ? url_for(self.url_text) : send(self.url_method)
+    end
+
+    def authorization=(value)
+      self.authorization_can        = value[:can]
+      self.authorization_class_name = value[:class_name]
+    end
+
+    def authorization
+      return nil if self.authorization_can.nil? || self.authorization_class_name.blank?
+
+      { can: self.authorization_can, class_name: self.authorization_class_name }
+    end
+
+    def active=(value)
+      self.active_controller_only = value[:controller_only]
+      self.active_method          = value[:method]
+    end
+
+    def active?
+      return send(self.active_method) unless self.active_method.nil?
+      return false if self.url_is_string?
+
+      result = self.menu.controller.controller_name == Rails.application.routes.recognize_path(self.url)[:controller]
+      unless self.active_controller_only?
+        result &&= self.menu.controller.action_name == Rails.application.routes.recognize_path(self.url)[:action]
+      end
+
+      result
+    end
+
+    def to_s
+      self.text
+    end
+
+    def url_is_string?
+      self.url_method.nil?
+    end
     
   end
 

data/lib/menu-rails.rb

@@ -1,6 +1,14 @@
-require "menu-rails/version"
+require 'menu-rails/version'
+require 'menu-rails/helpers'
 
 module MenuRails
+  autoload :BaseMenu, 'menu-rails/base_menu'
+
+  mattr_accessor :menu_class_name
+  @@menu_class_name = 'MenuRails::Menu'
+
+  mattr_accessor :menu_item_class_name
+  @@menu_item_class_name = 'MenuRails::MenuItem'
 end
 
 # load Rails/Railtie

data/lib/menu-rails/base_menu.rb

@@ -0,0 +1,22 @@
+module MenuRails
+
+  module BaseMenu
+
+    def display_for_controller(tmp_controller)
+      old_controller = @controller
+      @controller    = tmp_controller
+      block_result   = yield
+      @controller    = old_controller
+
+      block_result
+    end
+
+    def controller
+      raise 'Controller not valid if not displaying menu' if @controller.nil?
+
+      @controller
+    end
+
+  end
+
+end

data/lib/menu-rails/engine.rb

@@ -3,5 +3,13 @@ require 'rails'
 module MenuRails #:nodoc:
   class Engine < ::Rails::Engine #:nodoc:
     isolate_namespace MenuRails
+
+    config.menu_rails = MenuRails
+
+    initializer "menu-rails.helpers" do
+      ActiveSupport.on_load(:action_view) do
+        include MenuRails::Helpers::MenuHelper
+      end
+    end
   end
 end

data/lib/menu-rails/helpers.rb

@@ -0,0 +1,9 @@
+module MenuRails
+
+  module Helpers
+
+    autoload :MenuHelper, 'menu-rails/helpers/menu_helper'
+    
+  end
+
+end

data/lib/menu-rails/helpers/menu_helper.rb

@@ -0,0 +1,49 @@
+module MenuRails::Helpers
+
+  module MenuHelper
+
+    def menu_rails(menu)
+      raise 'Block must be provided' unless block_given?
+      
+      block_result = nil
+
+      menu.display_for_controller(self.controller) do
+        menu_items = menu_rails_all_items(menu)
+
+        block_result = yield(menu_items)
+      end
+
+      block_result
+    end
+
+    def menu_rails_each(menu)
+      raise 'Block must be provided' unless block_given?
+
+      menu_rails(menu) do |menu_items|
+        menu_items.each_with_index do |menu_item, index|
+          yield(menu_item, index)
+        end
+      end
+    end
+
+    def menu_rails_all_items(menu)
+      menu_items = menu.all_menu_items
+
+      menu_items = menu_items.select do |menu_item|
+        if respond_to?(:can?)
+          if menu_item.authorization.nil?
+            true
+          else
+            can?(menu_item.authorization[:can], menu_item.authorization[:class_name].constantize)
+          end
+        else
+          true
+        end
+      end
+
+      menu_items
+    end
+
+  end
+
+end

data/lib/menu-rails/version.rb

@@ -1,3 +1,3 @@
 module MenuRails
-  VERSION = "0.0.2"
+  VERSION = "0.1.0"
 end

data/spec/controllers/pages_controller_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+require 'internal/app/controllers/pages_controller'
+
+describe PagesController do
+
+  it "assigns @menu" do
+    get :home
+
+    assigns(:menu).should be_an_instance_of(MenuRails::Menu)
+  end
+
+end

data/spec/factories.rb

@@ -8,7 +8,8 @@ FactoryGirl.define do
 
   factory :menu_item, class: MenuRails::MenuItem do
     sequence(:mriid) { |n| "menu_item_#{ n }" }
-    text             { Faker::Lorem.word }
+    text             { Faker::Lorem.word      }
+    url              { Faker::Internet.url    }
     # name { Faker::Name.first_name }
 
     # factory :dummy_invalid do

data/spec/features/pages_spec.rb

@@ -1,11 +1,14 @@
 require 'spec_helper'
 require 'active_support/all'
 
-def menu_rails
-  @menu_rails ||= YAML::load_file(MenuRails::CONFIG_PATH).with_indifferent_access[:menu_rails]
-end
-
 feature "Menu generation" do
+  let(:menu) { MenuRails::Menu.get_menu_by_mrid(:client) }
+
+  before(:each) do
+    ActionView::Base.any_instance.stub(:can?) do |accessor, klass|
+      klass != NotVisibleController
+    end
+  end
 
   scenario "Ensure capybara is working correctly" do
     visit root_path
@@ -13,13 +16,29 @@ feature "Menu generation" do
     page.should have_content("Capybara working correctly")
   end
 
-  # # scenario "Print Home menu item" do
-  #   menu_rails.each do |menu_rails_item|
-  #     scenario "Print #{ menu_rails_item[:text] } menu item" do
-  #       visit root_path
+  context ":client menu" do
+
+    # scenario "Print Home menu item" do
+    ['Home', 'About'].each do |menu_rails_item|
+      scenario "Print #{ menu_rails_item } menu item" do
+        visit root_path
+
+        page.should have_content(menu_rails_item)
+      end
+    end
 
-  #       page.should have_content(menu_rails_item[:text])
-  #     end
-  #   end
+    scenario "Home is active" do
+      visit root_path
+
+      find('#menu a.is-active').should have_content('Home')
+    end
+
+    scenario "Dummy1 active? not called, instead it's called always_inactive" do
+      menu.get_menu_item_by_mriid(:dummy1).should_receive(:always_inactive).with(no_args).and_call_original
+
+      visit root_path
+    end
+
+  end
 
 end

data/spec/helpers/menu_helper_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe MenuRails::Helpers::MenuHelper do
+  let(:menu) { MenuRails::Menu.get_menu_by_mrid(:client) }
+
+  it { view.menu_rails_all_items(menu).first.text.should == 'Home'               }
+  it { view.menu_rails_all_items(menu).map(&:text).should include('Not visible') }
+
+  it "is not active because url it's a string" do
+    view.menu_rails(menu){ |menu_items| menu_items.second.active? }.should be_false
+  end
+
+  context "with cancan" do
+    before(:each) do
+      view.stub(:can?) do |accessor, klass|
+        klass != NotVisibleController
+      end
+    end
+
+    it { view.menu_rails_all_items(menu).map(&:text).should_not include('Not visible') }
+
+  end
+
+end

data/spec/internal/app/controllers/not_visible_controller.rb

@@ -0,0 +1,2 @@
+class NotVisibleController < ApplicationController  
+end

data/spec/internal/app/controllers/pages_controller.rb

@@ -1,6 +1,13 @@
 class PagesController < ApplicationController
 
   def home
+    @menu = MenuRails::Menu.get_menu_by_mrid(:client)
+  end
+
+  def about
+  end
+
+  def dummy1
   end
   
 end

data/spec/internal/app/models/dummy_menu_item.rb

@@ -0,0 +1,7 @@
+class DummyMenuItem < MenuRails::MenuItem
+
+  def always_inactive
+    false
+  end
+
+end

data/spec/internal/app/models/second_dummy_menu.rb

@@ -0,0 +1,2 @@
+class SecondDummyMenu < MenuRails::Menu
+end

data/spec/internal/app/models/second_dummy_menu_item.rb

@@ -0,0 +1,2 @@
+class SecondDummyMenuItem < MenuRails::MenuItem
+end

data/spec/internal/app/views/pages/home.html.erb

@@ -1 +1,7 @@
-Capybara working correctly
+Capybara working correctly
+
+<div id="menu">
+  <% menu_rails_each(@menu) do |menu_item| %>
+    <%= link_to(menu_item.to_s, menu_item.url, class: menu_item.active? ? ['is-active'] : []) %>
+  <% end %>
+</div>

data/spec/internal/config/menu-rails.yml

@@ -1,42 +1,44 @@
 menu_rails:
   client:
     menu_items:
-      - mriid: :home
-        text: Home
+      - home:
+          text: Home
+          url:  :root_path
+          authorization:
+            can:        :read
+            class_name: PagesController
+      - contact:
+          text: Contact
+          url:  "http://www.myrandomurl.com/"
+          authorization:
+            class_name: MissingCanKey
+          active:
+            # It's ignored because url is a string
+            controller_only: true
+      - about:
+          text: About
+          url:  :about_path
+      - dummy1:
+          text: Dummy1
+          url:  :dummy1_path
+          active:
+            method: :always_inactive
+      - not_visible:
+          text: Not visible
+          url:  :root_path
+          authorization:
+            can:        :read
+            class_name: NotVisibleController
   admin:
     menu_items:
-      - mriid: :home
-        text: Home
-  # app_config:
-  # client:
-  #   menu:
-  #     - text:       Home
-  #       class_name: 'PagesController'
-  #       can:        :read
-  #       url:        :root_path
-  #     - text:       About
-  #       class_name: 'PagesController'
-  #       can:        :read
-  #       url:        :root_path
-  #     - text:       Archives
-  #       class_name: 'PagesController'
-  #       can:        :read
-  #       url:        :root_path
-  #     - text:       Dropdown
-  #       class_name: 'PagesController'
-  #       can:        :read
-  #       url:        :root_path
-  #     - text:       Templates
-  #       class_name: 'PagesController'
-  #       can:        :read
-  #       url:        :root_path
-  #     - text:       Contact
-  #       class_name: 'PagesController'
-  #       can:        :read
-  #       url:        :root_path
-  #       last:       true
-  # admin:
-  #   menu:
-  #     - text:       Home
-  #       class_name: 'Admin::PagesController'
-  #       can:        :read
+      - home:
+          text: Home
+          url:  :root_path
+
+  custom_class_menu:
+    class_name: SecondDummyMenu
+    menu_items:
+      - home:
+          class_name: SecondDummyMenuItem
+          text:       Home
+          url:        :root_path

data/spec/internal/config/routes.rb

@@ -1,3 +1,6 @@
 Rails.application.routes.draw do
   root to: 'pages#home'
+
+  get 'about'  => 'pages#about'
+  get 'dummy1' => 'pages#dummy1'
 end

data/spec/models/menu_item_spec.rb

@@ -5,19 +5,47 @@ describe MenuRails::MenuItem do
   it "has a valid factory" do
     -> {FactoryGirl.build(:menu_item)}.should_not raise_error
   end
-  
+
+  it { FactoryGirl.build(:menu_item, text: 'Dummy', url: 'http://dummy.com/').should be_valid }
+  it { FactoryGirl.build(:menu_item, url: nil).should_not be_valid                            }
+
   let(:menu_item) { FactoryGirl.build(:menu_item) }
 
-  it { menu_item.should respond_to(:mriid, :text, :menu) }
-  it { menu_item.text.should be_an_instance_of String    }
-  it { menu_item.mriid.should be_an_instance_of Symbol   }
+  it do
+    menu_item.should respond_to(:mriid,
+                                :text,
+                                :menu,
+                                :authorization,
+                                :url)
+  end
+
+  it { menu_item.text.should be_an_instance_of String  }
+  it { menu_item.mriid.should be_an_instance_of Symbol }
 
   context "with menu :client as parent" do
-    let(:menu)      { MenuRails::Menu.get_menu_by_mrid(:client) }
-    let(:menu_item) { menu.menu_items.first                     }
+    let(:menu)             { MenuRails::Menu.get_menu_by_mrid(:client) }
+    let(:first_menu_item)  { menu.all_menu_items.first                 }
+    let(:second_menu_item) { menu.all_menu_items.second                }
+
+    it { menu.get_menu_item_by_mriid(:home).should be first_menu_item                          }
+    it { menu.get_menu_item_by_mriid(:dummy1).should be_an_instance_of(DummyMenuItem)          }
+
+    it { first_menu_item.menu.should be menu                                                   }
+    it { first_menu_item.authorization.should == { can: :read, class_name: 'PagesController' } }
+    it { first_menu_item.url.should == '/'                                                     }
+
+    it { second_menu_item.authorization.should be nil                                          }
+    it { second_menu_item.url.should == second_menu_item.url_text                              }
+    it { second_menu_item.active_controller_only?.should be_true                               }
+
+  end
 
-    it { menu_item.menu.should be menu }
+  context "with menu :custom_class_menu as parent" do
+    let(:menu)             { MenuRails::Menu.get_menu_by_mrid(:custom_class_menu) }
+    let(:first_menu_item)  { menu.all_menu_items.first                            }
 
+    it { menu.should be_an_instance_of SecondDummyMenu                }
+    it { first_menu_item.should be_an_instance_of SecondDummyMenuItem }
   end
 
 end

data/spec/models/menu_spec.rb

@@ -4,6 +4,7 @@ describe MenuRails::Menu do
 
   it { described_class.should respond_to(:get_menu_by_mrid)                               }
   it { described_class.get_menu_by_mrid(:client).should be_an_instance_of described_class }
+  it { described_class.get_menu_by_mrid(:admin).should be_an_instance_of described_class  }
   it { described_class.get_menu_by_mrid(:not_exist).should be_nil                         }
   it { described_class.get_menu_by_mrid(:client).mrid.should be :client                   }
 
@@ -22,6 +23,18 @@ describe MenuRails::Menu do
   it { menu.should respond_to(:menu_items, :mrid)      }
   it { menu.menu_items.should respond_to(:size, :each) }
   it { menu.mrid.should be_a_kind_of Symbol            }
-  it { menu.menu_items.first.text.should match 'Home'  }
+  it { menu.menu_items.first.text.should == 'Home'     }
+  it { menu.all_menu_items.should be menu.menu_items   }
+
+  it "uses given instance as controller while displaying in controller" do
+    fake_controller = double('Fake controller')
+    used_controller = nil
+
+    menu.display_for_controller(fake_controller) do
+      used_controller = menu.controller
+    end
+
+    used_controller.should be fake_controller
+  end
 
 end

data/spec/spec_helper.rb

@@ -8,7 +8,8 @@ require 'capybara/rspec'
 Combustion.initialize! :active_record,
                        :action_controller,
                        :action_view do
-  config.i18n.enforce_available_locales = false
+  config.i18n.enforce_available_locales   = false
+  config.menu_rails.menu_item_class_name = 'DummyMenuItem'
 end
 
 Bundler.require(:default, :development, :test)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: menu-rails
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.1.0
 platform: ruby
 authors:
 - Fire-Dragon-DoL
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-11 00:00:00.000000000 Z
+date: 2013-12-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n
@@ -230,6 +230,7 @@ extra_rdoc_files: []
 files:
 - .gitignore
 - .rspec
+- .ruby-version
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -238,14 +239,25 @@ files:
 - app/models/menu_rails/menu_item.rb
 - config.ru
 - lib/menu-rails.rb
+- lib/menu-rails/base_menu.rb
 - lib/menu-rails/engine.rb
+- lib/menu-rails/helpers.rb
+- lib/menu-rails/helpers/menu_helper.rb
 - lib/menu-rails/version.rb
 - menu-rails.gemspec
+- spec/controllers/pages_controller_spec.rb
 - spec/factories.rb
 - spec/features/pages_spec.rb
+- spec/helpers/menu_helper_spec.rb
 - spec/internal/app/controllers/application_controller.rb
+- spec/internal/app/controllers/not_visible_controller.rb
 - spec/internal/app/controllers/pages_controller.rb
+- spec/internal/app/models/dummy_menu_item.rb
+- spec/internal/app/models/second_dummy_menu.rb
+- spec/internal/app/models/second_dummy_menu_item.rb
 - spec/internal/app/views/layouts/application.html.erb
+- spec/internal/app/views/pages/about.html.erb
+- spec/internal/app/views/pages/dummy1.html.erb
 - spec/internal/app/views/pages/home.html.erb
 - spec/internal/config/database.yml
 - spec/internal/config/menu-rails.yml
@@ -276,17 +288,25 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 2.1.11
 signing_key: 
 specification_version: 4
 summary: Gem that helps creating menu in views and setting the currently active link
   item
 test_files:
+- spec/controllers/pages_controller_spec.rb
 - spec/factories.rb
 - spec/features/pages_spec.rb
+- spec/helpers/menu_helper_spec.rb
 - spec/internal/app/controllers/application_controller.rb
+- spec/internal/app/controllers/not_visible_controller.rb
 - spec/internal/app/controllers/pages_controller.rb
+- spec/internal/app/models/dummy_menu_item.rb
+- spec/internal/app/models/second_dummy_menu.rb
+- spec/internal/app/models/second_dummy_menu_item.rb
 - spec/internal/app/views/layouts/application.html.erb
+- spec/internal/app/views/pages/about.html.erb
+- spec/internal/app/views/pages/dummy1.html.erb
 - spec/internal/app/views/pages/home.html.erb
 - spec/internal/config/database.yml
 - spec/internal/config/menu-rails.yml