obitum-rails_admin 0.0.1 → 0.0.2

data/spec/helpers/application_helper_spec.rb

@@ -1,18 +1,20 @@
 require 'spec_helper'
 
 describe RailsAdmin::ApplicationHelper do
-  
-  subject { page }
-  
+
+  before do
+    controller.stub(:authorized?).and_return(true)
+  end
+
   describe '#current_action?' do
     it 'should return true if current_action, false otherwise' do
       @action = RailsAdmin::Config::Actions.find(:index)
-      
+
       helper.current_action?(RailsAdmin::Config::Actions.find(:index)).should be_true
       helper.current_action?(RailsAdmin::Config::Actions.find(:show)).should_not be_true
     end
   end
-  
+
   describe '#action' do
     it 'should return action by :custom_key' do
       RailsAdmin.config do |config|
@@ -22,10 +24,9 @@ describe RailsAdmin::ApplicationHelper do
           end
         end
       end
-      
-      helper.action(:my_custom_dashboard_key).should == RailsAdmin::Config::Actions.find(:my_custom_dashboard_key)
+      helper.action(:my_custom_dashboard_key).should be
     end
-    
+
     it 'should return only visible actions' do
       RailsAdmin.config do |config|
         config.actions do
@@ -34,10 +35,10 @@ describe RailsAdmin::ApplicationHelper do
           end
         end
       end
-      
+
       helper.action(:dashboard).should == nil
     end
-    
+
     it 'should return only visible actions, passing all bindings' do
       RailsAdmin.config do |config|
         config.actions do
@@ -50,25 +51,22 @@ describe RailsAdmin::ApplicationHelper do
           end
         end
       end
-      
-      helper.action(:test_bindings, RailsAdmin::AbstractModel.new(Team), Team.new).should == RailsAdmin::Config::Actions.find(:test_bindings)
-      helper.action(:test_bindings, RailsAdmin::AbstractModel.new(Team), Player.new).should == nil
-      helper.action(:test_bindings, RailsAdmin::AbstractModel.new(Player), Team.new).should == nil
+      helper.action(:test_bindings, RailsAdmin::AbstractModel.new(Team), Team.new).should be
+      helper.action(:test_bindings, RailsAdmin::AbstractModel.new(Team), Player.new).should be_nil
+      helper.action(:test_bindings, RailsAdmin::AbstractModel.new(Player), Team.new).should be_nil
     end
   end
-  
+
   describe "#actions" do
     it 'should return actions by type' do
       abstract_model = RailsAdmin::AbstractModel.new(Player)
       object = FactoryGirl.create :player
       helper.actions(:all, abstract_model, object).map(&:custom_key).should == [:dashboard, :index, :show, :new, :edit, :export, :delete, :bulk_delete, :history_show, :history_index, :show_in_app]
-      helper.actions(:all).map(&:custom_key).should == [:dashboard, :index, :show, :new, :edit, :export, :delete, :bulk_delete, :history_show, :history_index]
-      helper.actions.map(&:custom_key).should == [:dashboard, :index, :show, :new, :edit, :export, :delete, :bulk_delete, :history_show, :history_index]
-      helper.actions(:root).map(&:custom_key).should == [:dashboard]
-      helper.actions(:collection).map(&:custom_key).should == [:index, :new, :export, :bulk_delete, :history_index]
-      helper.actions(:member).map(&:custom_key).should == [:show, :edit, :delete, :history_show]
+      helper.actions(:root, abstract_model, object).map(&:custom_key).should == [:dashboard]
+      helper.actions(:collection, abstract_model, object).map(&:custom_key).should == [:index, :new, :export, :bulk_delete, :history_index]
+      helper.actions(:member, abstract_model, object).map(&:custom_key).should == [:show, :edit, :delete, :history_show, :show_in_app]
     end
-    
+
     it 'should only return visible actions, passing bindings correctly' do
       RailsAdmin.config do |config|
         config.actions do
@@ -81,13 +79,13 @@ describe RailsAdmin::ApplicationHelper do
           end
         end
       end
-      
+
       helper.actions(:all, RailsAdmin::AbstractModel.new(Team), Team.new).map(&:custom_key).should == [:test_bindings]
       helper.actions(:all, RailsAdmin::AbstractModel.new(Team), Player.new).map(&:custom_key).should == []
       helper.actions(:all, RailsAdmin::AbstractModel.new(Player), Team.new).map(&:custom_key).should == []
     end
   end
-  
+
   describe "#wording_for" do
     it "gives correct wording even if action is not visible" do
       RailsAdmin.config do |config|
@@ -97,35 +95,33 @@ describe RailsAdmin::ApplicationHelper do
           end
         end
       end
-      
+
       helper.wording_for(:menu, :index).should == "List"
     end
-    
+
     it "passes correct bindings" do
       helper.wording_for(:title, :edit, RailsAdmin::AbstractModel.new(Team), Team.new(:name => 'the avengers')).should == "Edit Team 'the avengers'"
     end
-    
+
     it "defaults correct bindings" do
       @action = RailsAdmin::Config::Actions.find :edit
       @abstract_model = RailsAdmin::AbstractModel.new(Team)
       @object = Team.new(:name => 'the avengers')
       helper.wording_for(:title).should == "Edit Team 'the avengers'"
     end
-    
+
     it "does not try to use the wrong :label_metod" do
       @abstract_model = RailsAdmin::AbstractModel.new(Draft)
       @object = Draft.new
-      
+
       helper.wording_for(:link, :new, RailsAdmin::AbstractModel.new(Team)).should == "Add a new Team"
     end
-    
+
   end
-  
+
   describe "#breadcrumb" do
-    it "gives us a breadcrumb, dammit" do
-      @action = RailsAdmin::Config::Actions.find :edit
-      @abstract_model = RailsAdmin::AbstractModel.new(Team)
-      @object = Team.new(:name => 'the avengers')
+    it "gives us a breadcrumb" do
+      @action = RailsAdmin::Config::Actions.find(:edit, {:abstract_model => RailsAdmin::AbstractModel.new(Team), :object => Team.new(:name => 'the avengers')})
       bc = helper.breadcrumb
       bc.should match /Dashboard/ # dashboard
       bc.should match /Teams/ # list
@@ -133,7 +129,7 @@ describe RailsAdmin::ApplicationHelper do
       bc.should match /Edit/ # current (edit)
     end
   end
-  
+
   describe "#menu_for" do
     it 'passes model and object as bindings and generates a menu, excluding non-get actions' do
       RailsAdmin.config do |config|
@@ -154,7 +150,7 @@ describe RailsAdmin::ApplicationHelper do
           end
         end
       end
-      
+
       @action = RailsAdmin::Config::Actions.find :show
       @abstract_model = RailsAdmin::AbstractModel.new(Team)
       @object = Team.new(:name => 'the avengers')
@@ -162,13 +158,13 @@ describe RailsAdmin::ApplicationHelper do
       helper.menu_for(:root).should match /Dashboard/
       helper.menu_for(:collection, @abstract_model).should match /List/
       helper.menu_for(:member, @abstract_model, @object).should match /Show/
-      
+
       @abstract_model = RailsAdmin::AbstractModel.new(Player)
       @object = Player.new
       helper.menu_for(:collection, @abstract_model).should_not match /List/
       helper.menu_for(:member, @abstract_model, @object).should_not match /Show/
     end
-    
+
     it "excludes non-get actions" do
       RailsAdmin.config do |config|
         config.actions do
@@ -177,12 +173,82 @@ describe RailsAdmin::ApplicationHelper do
           end
         end
       end
-      
+
       @action = RailsAdmin::Config::Actions.find :dashboard
       helper.menu_for(:root).should_not match /Dashboard/
     end
   end
-  
+
+  describe "#main_navigation" do
+    it 'should show included models' do
+      RailsAdmin.config do |config|
+        config.included_models = [Ball, Comment]
+      end
+      helper.main_navigation.should match /(nav\-header).*(Navigation).*(Balls).*(Comments)/m
+    end
+
+    it 'should not show unvisible models' do
+      RailsAdmin.config do |config|
+        config.included_models = [Ball, Comment]
+        config.model Comment do
+          hide
+        end
+      end
+      result = helper.main_navigation
+      result.should match /(nav\-header).*(Navigation).*(Balls)/m
+      result.should_not match "Comments"
+    end
+
+    it "should show children of hidden models" do # https://github.com/sferik/rails_admin/issues/978
+      RailsAdmin.config do |config|
+        config.included_models = [Ball, Hardball]
+        config.model Ball do
+          hide
+        end
+      end
+      helper.main_navigation.should match /(nav\-header).*(Navigation).*(Hardballs)/m
+    end
+
+    it "should show children of excluded models" do
+      RailsAdmin.config do |config|
+        config.included_models = [Hardball]
+      end
+      helper.main_navigation.should match /(nav\-header).*(Navigation).*(Hardballs)/m
+    end
+
+    it 'should "nest" in navigation label' do
+      RailsAdmin.config do |config|
+        config.included_models = [Comment]
+        config.model Comment do
+          navigation_label 'commentable'
+        end
+      end
+      helper.main_navigation.should match /(nav\-header).*(commentable).*(Comments)/m
+    end
+
+    it 'should "nest" in parent model' do
+      RailsAdmin.config do |config|
+        config.included_models = [Player, Comment]
+        config.model Comment do
+          parent Player
+        end
+      end
+      helper.main_navigation.should match /(Players).*(nav\-level\-1).*(Comments)/m
+    end
+
+    it 'should order' do
+      RailsAdmin.config do |config|
+        config.included_models = [Player, Comment]
+      end
+      helper.main_navigation.should match /(Comments).*(Players)/m
+
+      RailsAdmin.config(Comment) do
+        weight 1
+      end
+      helper.main_navigation.should match /(Players).*(Comments)/m
+    end
+  end
+
   describe "#bulk_menu" do
     it 'should include all visible bulkable actions' do
       RailsAdmin.config do |config|
@@ -201,7 +267,6 @@ describe RailsAdmin::ApplicationHelper do
         end
       end
       @action = RailsAdmin::Config::Actions.find :index
-      
       result = helper.bulk_menu(RailsAdmin::AbstractModel.new(Team))
       result.should match "zorg_action"
       result.should match "blub"
@@ -209,4 +274,4 @@ describe RailsAdmin::ApplicationHelper do
       helper.bulk_menu(RailsAdmin::AbstractModel.new(Player)).should_not match "blub"
     end
   end
-end
+end

data/spec/{requests → integration}/authorization/cancan_spec.rb

@@ -77,7 +77,7 @@ describe "RailsAdmin CanCan Authorization" do
       ]
 
       visit index_path(:model_name => "player")
-      
+
       should have_content(@players[0].name)
       should_not have_content(@players[1].name)
       should_not have_content("Add new")
@@ -112,7 +112,7 @@ describe "RailsAdmin CanCan Authorization" do
       should_not have_content("Delete")
 
       should have_content("Save and add another")
-      fill_in "player[name]", :with => "Jackie Robinson"
+      fill_in "player[name]", :with   => "Jackie Robinson"
       fill_in "player[number]", :with => "42"
       fill_in "player[position]", :with => "Second baseman"
       click_button "Save"
@@ -141,7 +141,6 @@ describe "RailsAdmin CanCan Authorization" do
     it "GET /admin/player/1/edit should render and update record upon submission" do
       @player = FactoryGirl.create :player
       visit edit_path(:model_name => "player", :id => @player.id)
-
       should have_content("Save and edit")
       should_not have_content("Save and add another")
       should_not have_content("Add new")
@@ -246,7 +245,7 @@ describe "RailsAdmin CanCan Authorization" do
       visit delete_path(:model_name => "player", :id => player_id)
 
       click_button "Yes, I'm sure"
-      
+
       Player.exists?(player_id).should be_false
     end
 
@@ -347,4 +346,4 @@ describe "RailsAdmin CanCan Authorization" do
     end
   end
 
-end
+end

data/spec/{requests → integration}/basic/bulk_destroy/rails_admin_basic_bulk_destroy_spec.rb

@@ -5,7 +5,7 @@ describe "RailsAdmin Basic Bulk Destroy" do
   subject { page }
 
   describe "successful bulk delete of records" do
-    before(:each) do
+    before do
       RailsAdmin::History.destroy_all
       RailsAdmin.config { |c| c.audit_with :history }
       @players = 3.times.map { FactoryGirl.create(:player) }
@@ -17,32 +17,19 @@ describe "RailsAdmin Basic Bulk Destroy" do
 
     it "should not contain deleted records" do
       RailsAdmin::AbstractModel.new("Player").count.should == 1
-    end
-
-    it "history count should be equal to number of deleted records" do
       RailsAdmin::History.count.should == @delete_ids.count
-    end
-
-    it "history items should be for proper table" do
       RailsAdmin::History.all.each do |history|
         history.table.should == "Player"
       end
-    end
-
-    it "history items should be for proper items" do
       RailsAdmin::History.all.each do |history|
         @delete_ids.should include(history.item)
       end
-    end
-
-    it "displays a flash notice stating the number of records destroyed" do
-      # 2 Players because @delete_ids.count == 2:
-      page.should have_selector(".alert-message", :text => "2 Players successfully deleted")
+      page.should have_selector(".alert-success", :text => "2 Players successfully deleted")
     end
   end
 
   describe "cancelled bulk_deletion" do
-    before(:each) do
+    before do
       RailsAdmin::History.destroy_all
       @players = 3.times.map { FactoryGirl.create(:player) }
       @delete_ids = @players[0..1].map(&:id)

data/spec/{requests → integration}/basic/create/rails_admin_basic_create_spec.rb

@@ -6,12 +6,10 @@ describe "RailsAdmin Basic Create" do
   describe "create" do
     before(:each) do
       visit new_path(:model_name => "player")
-
       fill_in "player[name]", :with => "Jackie Robinson"
       fill_in "player[number]", :with => "42"
       fill_in "player[position]", :with => "Second baseman"
       click_button "Save"
-
       @player = RailsAdmin::AbstractModel.new("Player").first
     end
 

data/spec/{requests → integration}/basic/create/rails_admin_namespaced_model_create_spec.rb

@@ -21,19 +21,3 @@ describe "RailsAdmin Namespaced Model Create" do
     }.to change(Cms::BasicPage, :count).by(1)
   end
 end
-
-
-describe "RailsAdmin Mounted Engine Model Create" do
-
-  subject { page }
-
-  before(:each) do
-    visit new_path(:model_name => "foo~bar")
-    fill_in "foo_bar[title]", :with => "Hello"
-    click_button "Save"
-  end
-
-  it 'should be successful' do
-    Foo::Bar.first.title.should == 'Hello'
-  end
-end

data/spec/{requests → integration}/basic/edit/rails_admin_basic_edit_spec.rb

@@ -95,9 +95,9 @@ describe "RailsAdmin Basic Edit" do
     end
 
   end
-  
+
   describe "clicking cancel when editing an object" do
-    
+
     it "should send back to previous URL" do
       @ball = FactoryGirl.create :ball
       visit '/admin/ball?sort=color'

data/spec/{requests → integration}/basic/export/rails_admin_basic_export_spec.rb

@@ -63,7 +63,7 @@ describe "RailsAdmin Export" do
     it "should allow to export to XML" do
       visit export_path(:model_name => 'player')
       click_button 'Export to xml'
-      
+
       # spec fails on non 1.9 mri rubies because of this https://github.com/rails/rails/pull/2076
       # waiting for fix (rails-3.1.4?)
       should have_content @player.team.name if RUBY_VERSION =~ /1\.9/

data/spec/{requests → integration}/basic/list/rails_admin_basic_list_spec.rb

@@ -11,25 +11,25 @@ describe "RailsAdmin Basic List" do
       visit dashboard_path
     end
   end
-  
+
   describe "GET /admin/typo" do
     it "should redirect to dashboard and inform the user the model wasn't found" do
       visit '/admin/whatever'
       page.driver.status_code.should eql(404)
-      find('.alert-message.error').should have_content("Model 'Whatever' could not be found")
+      find('.alert-error').should have_content("Model 'Whatever' could not be found")
     end
   end
-  
+
   describe "GET /admin/balls/545-typo" do
     it "should redirect to balls index and inform the user the id wasn't found" do
       visit '/admin/ball/545-typo'
       page.driver.status_code.should eql(404)
-      find('.alert-message.error').should have_content("Ball with id '545-typo' could not be found")
+      find('.alert-error').should have_content("Ball with id '545-typo' could not be found")
     end
   end
 
   describe "GET /admin/player as list" do
-    before(:each) do
+    before do
       21.times { FactoryGirl.create :player } # two pages of players
       visit index_path(:model_name => "player")
     end
@@ -38,52 +38,26 @@ describe "RailsAdmin Basic List" do
       should have_content("List of Players")
       should have_content("Created at")
       should have_content("Updated at")
-    end
 
-    it "shows the show, edit and delete links" do
+      # it "shows the show, edit and delete links" do
       should have_selector("td a", :text => 'Show')
       should have_selector("td a", :text => 'Edit')
       should have_selector("td a", :text => 'Delete')
-    end
 
-    it "has the search box with some prompt text" do
+      # it "has the search box with some prompt text" do
       should have_selector("input[placeholder='Filter']")
-    end
 
-    # https://github.com/sferik/rails_admin/issues/362
-    # test that no link uses the "wildcard route" with the main
-    # controller and list method
-    it "should not use the 'wildcard route'" do
+      # https://github.com/sferik/rails_admin/issues/362
+      # test that no link uses the "wildcard route" with the main
+      # controller and list method
+      # it "should not use the 'wildcard route'" do
       should have_selector("a[href*='all=true']") # make sure we're fully testing pagination
       should have_no_selector("a[href^='/rails_admin/main/list']")
     end
   end
 
-  describe "GET /admin/player with sort" do
-    before(:each) do
-      @players = 2.times.map { FactoryGirl.create :player }
-      visit index_path(:model_name => "player", :sort => "name")
-    end
-
-    it "should be sorted correctly" do
-      2.times { |i| should have_selector("td", :text => /#{@players[i].name}/) }
-    end
-  end
-
-  describe "GET /admin/player with reverse sort" do
-
-    before(:each) do
-      @players = 2.times.map { FactoryGirl.create :player }
-      visit index_path(:model_name => "player", :sort => "name", :sort_reverse => "true")
-    end
-
-    it "should be sorted correctly" do
-      2.times { |i| should have_selector("td", :text => /#{@players.reverse[i].name}/) }
-    end
-  end
-
   describe "GET /admin/player" do
-    before(:each) do
+    before do
       @teams = 2.times.map do
         FactoryGirl.create(:team)
       end

data/spec/{requests → integration}/basic/show/rails_admin_basic_show_spec.rb

@@ -5,7 +5,6 @@ describe "RailsAdmin Basic Show" do
 
   describe "show" do
     before(:each) do
-      
       @player = FactoryGirl.create :player
       visit show_path(:model_name => "player", :id => @player.id)
     end
@@ -27,7 +26,7 @@ describe "RailsAdmin Basic Show" do
       should have_content(@player.number)
     end
   end
-  
+
   describe "GET /admin/players/123this-id-doesnt-exist" do
     it "should raise NotFound" do
       visit '/admin/players/123this-id-doesnt-exist'

data/spec/{requests → integration}/basic/update/rails_admin_basic_update_spec.rb

@@ -85,7 +85,7 @@ describe "RailsAdmin Basic Update" do
       RailsAdmin.config do |c|
         c.audit_with :history
       end
-      
+
       @league = FactoryGirl.create :league
       @divisions = 3.times.map { Division.create!(:name => "div #{Time.now.to_f}", :league => League.create!(:name => "league #{Time.now.to_f}")) }
 
@@ -97,7 +97,7 @@ describe "RailsAdmin Basic Update" do
       @league.divisions.should include(@divisions[0])
       @league.divisions.should_not include(@divisions[1])
       @league.divisions.should_not include(@divisions[2])
-      
+
       RailsAdmin::History.where(:item => @league.id).collect(&:message).should include("Added Divisions ##{@divisions[0].id} associations, Changed name")
 
       page.driver.put edit_path(:model_name => "league", :id => @league.id, :league => {:division_ids => [""]})