rails_admin 0.6.0 → 0.6.1

data/spec/rails_admin/config/actions/base_spec.rb

@@ -2,8 +2,8 @@ require 'spec_helper'
 
 describe RailsAdmin::Config::Actions::Base do
 
-  describe "#visible?" do
-    it "excludes models not referenced in the only array" do
+  describe '#visible?' do
+    it 'excludes models not referenced in the only array' do
       RailsAdmin.config do |config|
         config.actions do
           index do
@@ -11,12 +11,12 @@ describe RailsAdmin::Config::Actions::Base do
           end
         end
       end
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Player)})).to be_visible
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Team)})).to be_nil
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Cms::BasicPage)})).to be_visible
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Player))).to be_visible
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Team))).to be_nil
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Cms::BasicPage))).to be_visible
     end
 
-    it "excludes models referenced in the except array" do
+    it 'excludes models referenced in the except array' do
       RailsAdmin.config do |config|
         config.actions do
           index do
@@ -24,9 +24,9 @@ describe RailsAdmin::Config::Actions::Base do
           end
         end
       end
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Player)})).to be_nil
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Team)})).to be_visible
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Cms::BasicPage)})).to be_nil
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Player))).to be_nil
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Team))).to be_visible
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Cms::BasicPage))).to be_nil
     end
   end
 end

data/spec/rails_admin/config/actions_spec.rb

@@ -2,14 +2,14 @@ require 'spec_helper'
 
 describe RailsAdmin::Config::Actions do
 
-  describe "default" do
-    it "is as before" do
-      expect(RailsAdmin::Config::Actions.all.map(&:key)).to eq([:dashboard, :index, :show, :new, :edit, :export, :delete, :bulk_delete, :history_show, :history_index, :show_in_app])
+  describe 'default' do
+    it 'is as before' do
+      expect(RailsAdmin::Config::Actions.all.collect(&:key)).to eq([:dashboard, :index, :show, :new, :edit, :export, :delete, :bulk_delete, :history_show, :history_index, :show_in_app])
     end
   end
 
-  describe "find" do
-    it "finds by custom key" do
+  describe 'find' do
+    it 'finds by custom key' do
       RailsAdmin.config do |config|
         config.actions do
           dashboard do
@@ -27,23 +27,23 @@ describe RailsAdmin::Config::Actions do
       expect(RailsAdmin::Config::Actions.find(:show)).to be_a(RailsAdmin::Config::Actions::Show)
     end
 
-    it "returns nil when no action is found by the custom key" do
+    it 'returns nil when no action is found by the custom key' do
       expect(RailsAdmin::Config::Actions.find(:non_existent_action_key)).to be_nil
     end
 
-    it "returns visible action passing binding if controller binding is given, and pass action visible or not if no" do
+    it 'returns visible action passing binding if controller binding is given, and pass action visible or not if no' do
       RailsAdmin.config do |config|
         config.actions do
           root :custom_root do
             visible do
-              bindings[:controller] == "controller"
+              bindings[:controller] == 'controller'
             end
           end
         end
       end
       expect(RailsAdmin::Config::Actions.find(:custom_root)).to be_a(RailsAdmin::Config::Actions::Base)
-      expect(RailsAdmin::Config::Actions.find(:custom_root, {:controller => "not_controller"})).to be_nil
-      expect(RailsAdmin::Config::Actions.find(:custom_root, {:controller => "controller"})).to be_a(RailsAdmin::Config::Actions::Base)
+      expect(RailsAdmin::Config::Actions.find(:custom_root, controller: 'not_controller')).to be_nil
+      expect(RailsAdmin::Config::Actions.find(:custom_root, controller: 'controller')).to be_a(RailsAdmin::Config::Actions::Base)
     end
 
     it "checks bindings[:abstract_model] visibility while checking action\'s visibility" do
@@ -51,21 +51,21 @@ describe RailsAdmin::Config::Actions do
         hide
       end
 
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Comment)})).to be_a(RailsAdmin::Config::Actions::Index) #decoy
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Team)})).to be_nil
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Comment))).to be_a(RailsAdmin::Config::Actions::Index) # decoy
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Team))).to be_nil
     end
 
     it "checks bindings[:abstract_model] presence while checking action\'s visibility" do
       RailsAdmin.config do |config|
         config.excluded_models << Team
       end
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Comment)})).to be_a(RailsAdmin::Config::Actions::Index) #decoy
-      expect(RailsAdmin::Config::Actions.find(:index, {:controller => double(:authorized? => true), :abstract_model => RailsAdmin::AbstractModel.new(Team)})).to be_nil
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Comment))).to be_a(RailsAdmin::Config::Actions::Index) # decoy
+      expect(RailsAdmin::Config::Actions.find(:index, controller: double(:authorized? => true), abstract_model: RailsAdmin::AbstractModel.new(Team))).to be_nil
     end
   end
 
-  describe "all" do
-    it "returns all defined actions" do
+  describe 'all' do
+    it 'returns all defined actions' do
       RailsAdmin.config do |config|
         config.actions do
           dashboard
@@ -73,10 +73,10 @@ describe RailsAdmin::Config::Actions do
         end
       end
 
-      expect(RailsAdmin::Config::Actions.all.map(&:key)).to eq([:dashboard, :index])
+      expect(RailsAdmin::Config::Actions.all.collect(&:key)).to eq([:dashboard, :index])
     end
 
-    it "restricts by scope" do
+    it 'restricts by scope' do
       RailsAdmin.config do |config|
         config.actions do
           root :custom_root
@@ -84,29 +84,29 @@ describe RailsAdmin::Config::Actions do
           member :custom_member
         end
       end
-      expect(RailsAdmin::Config::Actions.all(:root).map(&:key)).to eq([:custom_root])
-      expect(RailsAdmin::Config::Actions.all(:collection).map(&:key)).to eq([:custom_collection])
-      expect(RailsAdmin::Config::Actions.all(:member).map(&:key)).to eq([:custom_member])
+      expect(RailsAdmin::Config::Actions.all(:root).collect(&:key)).to eq([:custom_root])
+      expect(RailsAdmin::Config::Actions.all(:collection).collect(&:key)).to eq([:custom_collection])
+      expect(RailsAdmin::Config::Actions.all(:member).collect(&:key)).to eq([:custom_member])
     end
 
-    it "returns all visible actions passing binding if controller binding is given, and pass all actions if no" do
+    it 'returns all visible actions passing binding if controller binding is given, and pass all actions if no' do
       RailsAdmin.config do |config|
         config.actions do
           root :custom_root do
             visible do
-              bindings[:controller] == "controller"
+              bindings[:controller] == 'controller'
             end
           end
         end
       end
-      expect(RailsAdmin::Config::Actions.all(:root).map(&:custom_key)).to eq([:custom_root])
-      expect(RailsAdmin::Config::Actions.all(:root, {:controller => "not_controller"}).map(&:custom_key)).to eq([])
-      expect(RailsAdmin::Config::Actions.all(:root, {:controller => "controller"}).map(&:custom_key)).to eq([:custom_root])
+      expect(RailsAdmin::Config::Actions.all(:root).collect(&:custom_key)).to eq([:custom_root])
+      expect(RailsAdmin::Config::Actions.all(:root, controller: 'not_controller').collect(&:custom_key)).to eq([])
+      expect(RailsAdmin::Config::Actions.all(:root, controller: 'controller').collect(&:custom_key)).to eq([:custom_root])
     end
   end
 
-  describe "customized through DSL" do
-    it "adds the one asked" do
+  describe 'customized through DSL' do
+    it 'adds the one asked' do
       RailsAdmin.config do |config|
         config.actions do
           dashboard
@@ -115,10 +115,10 @@ describe RailsAdmin::Config::Actions do
         end
       end
 
-      expect(RailsAdmin::Config::Actions.all.map(&:key)).to eq([:dashboard, :index, :show])
+      expect(RailsAdmin::Config::Actions.all.collect(&:key)).to eq([:dashboard, :index, :show])
     end
 
-    it "allows to customize the custom_key when customizing an existing action" do
+    it 'allows to customize the custom_key when customizing an existing action' do
       RailsAdmin.config do |config|
         config.actions do
           dashboard do
@@ -126,11 +126,11 @@ describe RailsAdmin::Config::Actions do
           end
         end
       end
-      expect(RailsAdmin::Config::Actions.all.map(&:custom_key)).to eq([:my_dashboard])
-      expect(RailsAdmin::Config::Actions.all.map(&:key)).to eq([:dashboard])
+      expect(RailsAdmin::Config::Actions.all.collect(&:custom_key)).to eq([:my_dashboard])
+      expect(RailsAdmin::Config::Actions.all.collect(&:key)).to eq([:dashboard])
     end
 
-    it "allows to change the key and the custom_key when subclassing an existing action" do
+    it 'allows to change the key and the custom_key when subclassing an existing action' do
       RailsAdmin.config do |config|
         config.actions do
           root :my_dashboard_key, :dashboard do
@@ -138,12 +138,12 @@ describe RailsAdmin::Config::Actions do
           end
         end
       end
-      expect(RailsAdmin::Config::Actions.all.map(&:custom_key)).to eq([:my_dashboard_custom_key])
-      expect(RailsAdmin::Config::Actions.all.map(&:key)).to eq([:my_dashboard_key])
-      expect(RailsAdmin::Config::Actions.all.map(&:class)).to eq([RailsAdmin::Config::Actions::Dashboard])
+      expect(RailsAdmin::Config::Actions.all.collect(&:custom_key)).to eq([:my_dashboard_custom_key])
+      expect(RailsAdmin::Config::Actions.all.collect(&:key)).to eq([:my_dashboard_key])
+      expect(RailsAdmin::Config::Actions.all.collect(&:class)).to eq([RailsAdmin::Config::Actions::Dashboard])
     end
 
-    it "does not add the same custom_key twice" do
+    it 'does not add the same custom_key twice' do
       expect do
         RailsAdmin.config do |config|
           config.actions do
@@ -151,7 +151,7 @@ describe RailsAdmin::Config::Actions do
             dashboard
           end
         end
-      end.to raise_error("Action dashboard already exists. Please change its custom key.")
+      end.to raise_error('Action dashboard already exists. Please change its custom key.')
 
       expect do
         RailsAdmin.config do |config|
@@ -160,10 +160,10 @@ describe RailsAdmin::Config::Actions do
             collection :index
           end
         end
-      end.to raise_error("Action index already exists. Please change its custom key.")
+      end.to raise_error('Action index already exists. Please change its custom key.')
     end
 
-    it "adds the same key with different custom key" do
+    it 'adds the same key with different custom key' do
       RailsAdmin.config do |config|
         config.actions do
           dashboard
@@ -173,8 +173,8 @@ describe RailsAdmin::Config::Actions do
         end
       end
 
-      expect(RailsAdmin::Config::Actions.all.map(&:custom_key)).to eq([:dashboard, :my_dashboard])
-      expect(RailsAdmin::Config::Actions.all.map(&:key)).to eq([:dashboard, :dashboard])
+      expect(RailsAdmin::Config::Actions.all.collect(&:custom_key)).to eq([:dashboard, :my_dashboard])
+      expect(RailsAdmin::Config::Actions.all.collect(&:key)).to eq([:dashboard, :dashboard])
     end
   end
 end

data/spec/rails_admin/config/fields/base_spec.rb

@@ -2,25 +2,25 @@ require 'spec_helper'
 
 describe RailsAdmin::Config::Fields::Base do
 
-  describe "#required" do
-    it "reads the :on => :create/:update validate option" do
+  describe '#required' do
+    it 'reads the :on => :create/:update validate option' do
       RailsAdmin.config Ball do
         field 'color'
       end
 
-      expect(RailsAdmin.config('Ball').fields.first.with(:object => Ball.new)).to be_required
-      expect(RailsAdmin.config('Ball').fields.first.with(:object => FactoryGirl.create(:ball))).not_to be_required
+      expect(RailsAdmin.config('Ball').fields.first.with(object: Ball.new)).to be_required
+      expect(RailsAdmin.config('Ball').fields.first.with(object: FactoryGirl.create(:ball))).not_to be_required
     end
 
     context 'on a Paperclip installation' do
-      it "should detect required fields" do
-        expect(RailsAdmin.config('Image').fields.find{ |f| f.name == :file }.with(:object => Image.new)).to be_required
+      it 'should detect required fields' do
+        expect(RailsAdmin.config('Image').fields.detect { |f| f.name == :file }.with(object: Image.new)).to be_required
       end
     end
   end
 
-  describe "#name" do
-    it "is normalized to Symbol" do
+  describe '#name' do
+    it 'is normalized to Symbol' do
       RailsAdmin.config Team do
         field 'name'
       end
@@ -28,58 +28,58 @@ describe RailsAdmin::Config::Fields::Base do
     end
   end
 
-  describe "#children_fields" do
+  describe '#children_fields' do
     POLYMORPHIC_CHILDREN = [:commentable_id, :commentable_type]
 
-    it "is empty by default" do
-      expect(RailsAdmin.config(Team).fields.find{ |f| f.name == :name }.children_fields).to eq([])
+    it 'is empty by default' do
+      expect(RailsAdmin.config(Team).fields.detect { |f| f.name == :name }.children_fields).to eq([])
     end
 
-    it "contains child key for belongs to associations" do
-      expect(RailsAdmin.config(Team).fields.find{ |f| f.name == :division }.children_fields).to eq([:division_id])
+    it 'contains child key for belongs to associations' do
+      expect(RailsAdmin.config(Team).fields.detect { |f| f.name == :division }.children_fields).to eq([:division_id])
     end
 
-    it "contains child keys for polymorphic belongs to associations" do
-      expect(RailsAdmin.config(Comment).fields.find{ |f| f.name == :commentable }.children_fields).to match_array POLYMORPHIC_CHILDREN
+    it 'contains child keys for polymorphic belongs to associations' do
+      expect(RailsAdmin.config(Comment).fields.detect { |f| f.name == :commentable }.children_fields).to match_array POLYMORPHIC_CHILDREN
     end
 
-    it "has correct fields when polymorphic_type column comes ahead of polymorphic foreign_key column" do
+    it 'has correct fields when polymorphic_type column comes ahead of polymorphic foreign_key column' do
       class CommentReversed < Tableless
         column :commentable_type, :string
         column :commentable_id, :integer
-        belongs_to :commentable, :polymorphic => true
+        belongs_to :commentable, polymorphic: true
       end
-      expect(RailsAdmin.config(CommentReversed).fields.map{|f| f.name.to_s}.select{|f| /^comment/ =~ f}).to match_array ['commentable'].concat(POLYMORPHIC_CHILDREN.map(&:to_s))
+      expect(RailsAdmin.config(CommentReversed).fields.collect { |f| f.name.to_s }.select { |f| /^comment/ =~ f }).to match_array ['commentable'].concat(POLYMORPHIC_CHILDREN.collect(&:to_s))
     end
 
-    context "of a Paperclip installation" do
-      it "is a _file_name field" do
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :paperclip_asset }.children_fields.include?(:paperclip_asset_file_name)).to be_true
+    context 'of a Paperclip installation' do
+      it 'is a _file_name field' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :paperclip_asset }.children_fields.include?(:paperclip_asset_file_name)).to be_true
       end
 
-      it "is hidden, not filterable" do
-        f = RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :paperclip_asset_file_name }
-        expect(f.hidden?).to be_true
-        expect(f.filterable?).to be_false
+      it 'is hidden, not filterable' do
+        field = RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :paperclip_asset_file_name }
+        expect(field.hidden?).to be_true
+        expect(field.filterable?).to be_false
       end
     end
 
-    context "of a Dragonfly installation" do
-      it "is a _name field and _uid field" do
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :dragonfly_asset }.children_fields).to eq([:dragonfly_asset_name, :dragonfly_asset_uid])
+    context 'of a Dragonfly installation' do
+      it 'is a _name field and _uid field' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :dragonfly_asset }.children_fields).to eq([:dragonfly_asset_name, :dragonfly_asset_uid])
       end
     end
 
-    context "of a Carrierwave installation" do
-      it "is the parent field itself" do
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :carrierwave_asset }.children_fields).to eq([:carrierwave_asset])
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :carrierwave_asset }.hidden?).to be_false
+    context 'of a Carrierwave installation' do
+      it 'is the parent field itself' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :carrierwave_asset }.children_fields).to eq([:carrierwave_asset])
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :carrierwave_asset }.hidden?).to be_false
       end
     end
   end
 
-  describe "#form_default_value" do
-    it "is default_value for new records when value is nil" do
+  describe '#form_default_value' do
+    it 'is default_value for new records when value is nil' do
       RailsAdmin.config Team do
         list do
           field :name do
@@ -88,220 +88,219 @@ describe RailsAdmin::Config::Fields::Base do
         end
       end
       @team = Team.new
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :name}.with(:object => @team).form_default_value).to eq('default value')
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :name }.with(object: @team).form_default_value).to eq('default value')
       @team.name = 'set value'
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :name}.with(:object => @team).form_default_value).to be_nil
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :name }.with(object: @team).form_default_value).to be_nil
       @team = FactoryGirl.create :team
       @team.name = nil
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :name}.with(:object => @team).form_default_value).to be_nil
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :name }.with(object: @team).form_default_value).to be_nil
     end
   end
 
-  describe "#default_value" do
-    it "is nil by default" do
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :name}.default_value).to be_nil
+  describe '#default_value' do
+    it 'is nil by default' do
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :name }.default_value).to be_nil
     end
   end
 
-  describe "#hint" do
-    it "is user customizable" do
+  describe '#hint' do
+    it 'is user customizable' do
       RailsAdmin.config Team do
         list do
           field :division do
-            hint "Great Division"
+            hint 'Great Division'
           end
           field :name
         end
       end
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :division}.hint).to eq("Great Division") # custom
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :name}.hint).to eq("") # default
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :division }.hint).to eq('Great Division') # custom
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :name }.hint).to eq('') # default
     end
   end
 
-  describe "#help" do
-    it "has a default and be user customizable via i18n" do
+  describe '#help' do
+    it 'has a default and be user customizable via i18n' do
       RailsAdmin.config Team do
         list do
           field :division
           field :name
         end
       end
-      field_specific_i18n = RailsAdmin.config('Team').list.fields.find{|f| f.name == :name}
-      expect(field_specific_i18n.help).to eq(I18n.translate("admin.help.team.name")) # custom via locales yml
-      field_no_specific_i18n = RailsAdmin.config('Team').list.fields.find{|f| f.name == :division}
+      field_specific_i18n = RailsAdmin.config('Team').list.fields.detect { |f| f.name == :name }
+      expect(field_specific_i18n.help).to eq(I18n.translate('admin.help.team.name')) # custom via locales yml
+      field_no_specific_i18n = RailsAdmin.config('Team').list.fields.detect { |f| f.name == :division }
       expect(field_no_specific_i18n.help).to eq(field_no_specific_i18n.generic_help) # rails_admin generic fallback
     end
   end
 
-  describe "#css_class" do
-    it "has a default and be user customizable" do
+  describe '#css_class' do
+    it 'has a default and be user customizable' do
       RailsAdmin.config Team do
         list do
           field :division do
-            css_class "custom"
+            css_class 'custom'
           end
           field :name
         end
       end
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :division}.css_class).to eq("custom") # custom
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :division}.type_css_class).to eq("belongs_to_association_type") # type css class, non-customizable
-      expect(RailsAdmin.config('Team').list.fields.find{|f| f.name == :name}.css_class).to eq("name_field") # default
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :division }.css_class).to eq('custom') # custom
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :division }.type_css_class).to eq('belongs_to_association_type') # type css class, non-customizable
+      expect(RailsAdmin.config('Team').list.fields.detect { |f| f.name == :name }.css_class).to eq('name_field') # default
     end
   end
 
-  describe "#associated_collection_cache_all" do
-    it "defaults to true if associated collection count < 100" do
-      expect(RailsAdmin.config(Team).edit.fields.find{|f| f.name == :players}.associated_collection_cache_all).to be_true
+  describe '#associated_collection_cache_all' do
+    it 'defaults to true if associated collection count < 100' do
+      expect(RailsAdmin.config(Team).edit.fields.detect { |f| f.name == :players }.associated_collection_cache_all).to be_true
     end
 
-    it "defaults to false if associated collection count >= 100" do
-      @players = 100.times.map do
+    it 'defaults to false if associated collection count >= 100' do
+      @players = 100.times.collect do
         FactoryGirl.create :player
       end
-      expect(RailsAdmin.config(Team).edit.fields.find{|f| f.name == :players}.associated_collection_cache_all).to be_false
+      expect(RailsAdmin.config(Team).edit.fields.detect { |f| f.name == :players }.associated_collection_cache_all).to be_false
     end
   end
 
-  describe "#searchable_columns" do
-    describe "for belongs_to fields" do
-      it "finds label method on the opposite side for belongs_to associations by default" do
-        expect(RailsAdmin.config(Team).fields.find{|f| f.name == :division}.searchable_columns.map{|c| c[:column]}).to eq(["divisions.name", "teams.division_id"])
+  describe '#searchable_columns' do
+    describe 'for belongs_to fields' do
+      it 'finds label method on the opposite side for belongs_to associations by default' do
+        expect(RailsAdmin.config(Team).fields.detect { |f| f.name == :division }.searchable_columns.collect { |c| c[:column] }).to eq(['divisions.name', 'teams.division_id'])
       end
 
-      it "searches on opposite table for belongs_to" do
+      it 'searches on opposite table for belongs_to' do
         RailsAdmin.config(Team) do
           field :division do
             searchable :custom_id
           end
         end
-        expect(RailsAdmin.config(Team).fields.find{|f| f.name == :division}.searchable_columns.map{|c| c[:column]}).to eq(["divisions.custom_id"])
+        expect(RailsAdmin.config(Team).fields.detect { |f| f.name == :division }.searchable_columns.collect { |c| c[:column] }).to eq(['divisions.custom_id'])
       end
 
-      it "searches on asked table with model name" do
+      it 'searches on asked table with model name' do
         RailsAdmin.config(Team) do
           field :division do
             searchable League => :name
           end
         end
-        expect(RailsAdmin.config(Team).fields.find{|f| f.name == :division}.searchable_columns).to eq([{:column=>"leagues.name", :type=>:string}])
+        expect(RailsAdmin.config(Team).fields.detect { |f| f.name == :division }.searchable_columns).to eq([{column: 'leagues.name', type: :string}])
       end
 
-      it "searches on asked table with table name" do
+      it 'searches on asked table with table name' do
         RailsAdmin.config(Team) do
           field :division do
-            searchable :leagues => :name
+            searchable leagues: :name
           end
         end
-        expect(RailsAdmin.config(Team).fields.find{|f| f.name == :division}.searchable_columns).to eq([{:column=>"leagues.name", :type=>:string}])
+        expect(RailsAdmin.config(Team).fields.detect { |f| f.name == :division }.searchable_columns).to eq([{column: 'leagues.name', type: :string}])
       end
     end
 
-    describe "for basic type fields" do
+    describe 'for basic type fields' do
 
-      it "uses base table and find correct column type" do
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :text_field}.searchable_columns).to eq([{:column=>"field_tests.text_field", :type=>:text}])
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :integer_field}.searchable_columns).to eq([{:column=>"field_tests.integer_field", :type=>:integer}])
+      it 'uses base table and find correct column type' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :text_field }.searchable_columns).to eq([{column: 'field_tests.text_field', type: :text}])
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :integer_field }.searchable_columns).to eq([{column: 'field_tests.integer_field', type: :integer}])
       end
 
-      it "is customizable to another field on the same table" do
+      it 'is customizable to another field on the same table' do
         RailsAdmin.config(FieldTest) do
           field :time_field do
             searchable :date_field
           end
         end
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :time_field}.searchable_columns).to eq([{:column=>"field_tests.date_field", :type=>:date}])
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :time_field }.searchable_columns).to eq([{column: 'field_tests.date_field', type: :date}])
       end
 
-      it "is customizable to another field on another table with :table_name" do
+      it 'is customizable to another field on another table with :table_name' do
         RailsAdmin.config(FieldTest) do
           field :string_field do
-            searchable :nested_field_tests => :title
+            searchable nested_field_tests: :title
           end
         end
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :string_field}.searchable_columns).to eq([{:column=>"nested_field_tests.title", :type=>:string}])
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :string_field }.searchable_columns).to eq([{column: 'nested_field_tests.title', type: :string}])
       end
 
-      it "is customizable to another field on another model with ModelClass" do
+      it 'is customizable to another field on another model with ModelClass' do
         RailsAdmin.config(FieldTest) do
           field :string_field do
             searchable NestedFieldTest => :title
           end
         end
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :string_field}.searchable_columns).to eq([{:column=>"nested_field_tests.title", :type=>:string}])
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :string_field }.searchable_columns).to eq([{column: 'nested_field_tests.title', type: :string}])
       end
     end
 
-    describe "for mapped fields" do
-      it "of paperclip should find the underlying column on the base table" do
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :paperclip_asset}.searchable_columns.map{|c| c[:column]}).to eq(["field_tests.paperclip_asset_file_name"])
+    describe 'for mapped fields' do
+      it 'of paperclip should find the underlying column on the base table' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :paperclip_asset }.searchable_columns.collect { |c| c[:column] }).to eq(['field_tests.paperclip_asset_file_name'])
       end
 
-      it "of dragonfly should find the underlying column on the base table" do
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :dragonfly_asset}.searchable_columns.map{|c| c[:column]}).to eq(["field_tests.dragonfly_asset_name"])
+      it 'of dragonfly should find the underlying column on the base table' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :dragonfly_asset }.searchable_columns.collect { |c| c[:column] }).to eq(['field_tests.dragonfly_asset_name'])
       end
 
-      it "of carrierwave should find the underlying column on the base table" do
-        expect(RailsAdmin.config(FieldTest).fields.find{|f| f.name == :carrierwave_asset}.searchable_columns.map{|c| c[:column]}).to eq(["field_tests.carrierwave_asset"])
+      it 'of carrierwave should find the underlying column on the base table' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :carrierwave_asset }.searchable_columns.collect { |c| c[:column] }).to eq(['field_tests.carrierwave_asset'])
       end
     end
   end
 
-
-  describe "#searchable and #sortable" do
-    it "is false if column is virtual, true otherwise" do
+  describe '#searchable and #sortable' do
+    it 'is false if column is virtual, true otherwise' do
       RailsAdmin.config League do
         field :virtual_column
         field :name
       end
       @league = FactoryGirl.create :league
-      expect(RailsAdmin.config('League').export.fields.find{ |f| f.name == :virtual_column }.sortable).to be_false
-      expect(RailsAdmin.config('League').export.fields.find{ |f| f.name == :virtual_column }.searchable).to be_false
-      expect(RailsAdmin.config('League').export.fields.find{ |f| f.name == :name }.sortable).to be_true
-      expect(RailsAdmin.config('League').export.fields.find{ |f| f.name == :name }.searchable).to be_true
+      expect(RailsAdmin.config('League').export.fields.detect { |f| f.name == :virtual_column }.sortable).to be_false
+      expect(RailsAdmin.config('League').export.fields.detect { |f| f.name == :virtual_column }.searchable).to be_false
+      expect(RailsAdmin.config('League').export.fields.detect { |f| f.name == :name }.sortable).to be_true
+      expect(RailsAdmin.config('League').export.fields.detect { |f| f.name == :name }.searchable).to be_true
     end
 
-    context "of a virtual field with children fields" do
-      it "of paperclip should target the first children field" do
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :paperclip_asset }.searchable).to eq(:paperclip_asset_file_name)
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :paperclip_asset }.sortable).to eq(:paperclip_asset_file_name)
+    context 'of a virtual field with children fields' do
+      it 'of paperclip should target the first children field' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :paperclip_asset }.searchable).to eq(:paperclip_asset_file_name)
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :paperclip_asset }.sortable).to eq(:paperclip_asset_file_name)
       end
 
-      it "of dragonfly should target the first children field" do
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :dragonfly_asset }.searchable).to eq(:dragonfly_asset_name)
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :dragonfly_asset }.sortable).to eq(:dragonfly_asset_name)
+      it 'of dragonfly should target the first children field' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :dragonfly_asset }.searchable).to eq(:dragonfly_asset_name)
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :dragonfly_asset }.sortable).to eq(:dragonfly_asset_name)
       end
 
-      it "of carrierwave should target the first children field" do
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :carrierwave_asset }.searchable).to eq(:carrierwave_asset)
-        expect(RailsAdmin.config(FieldTest).fields.find{ |f| f.name == :carrierwave_asset }.sortable).to eq(:carrierwave_asset)
+      it 'of carrierwave should target the first children field' do
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :carrierwave_asset }.searchable).to eq(:carrierwave_asset)
+        expect(RailsAdmin.config(FieldTest).fields.detect { |f| f.name == :carrierwave_asset }.sortable).to eq(:carrierwave_asset)
       end
     end
   end
 
-  describe "#virtual?" do
-    it "is true if column has no properties, false otherwise" do
+  describe '#virtual?' do
+    it 'is true if column has no properties, false otherwise' do
       RailsAdmin.config League do
         field :virtual_column
         field :name
       end
       @league = FactoryGirl.create :league
-      expect(RailsAdmin.config('League').export.fields.find{ |f| f.name == :virtual_column }.virtual?).to be_true
-      expect(RailsAdmin.config('League').export.fields.find{ |f| f.name == :name }.virtual?).to be_false
+      expect(RailsAdmin.config('League').export.fields.detect { |f| f.name == :virtual_column }.virtual?).to be_true
+      expect(RailsAdmin.config('League').export.fields.detect { |f| f.name == :name }.virtual?).to be_false
     end
   end
 
-  describe "#default_search_operator" do
+  describe '#default_search_operator' do
     let(:abstract_model) { RailsAdmin::AbstractModel.new('Player') }
     let(:model_config) { RailsAdmin.config(abstract_model) }
     let(:queryable_fields) { model_config.list.fields.select(&:queryable?) }
 
-    context "when no search operator is specified for the field" do
+    context 'when no search operator is specified for the field' do
       it "uses 'default' search operator" do
         expect(queryable_fields).to have_at_least(1).field
         expect(queryable_fields.first.search_operator).to eq(RailsAdmin::Config.default_search_operator)
       end
 
-      it "uses config.default_search_operator if set" do
+      it 'uses config.default_search_operator if set' do
         RailsAdmin.config do |config|
           config.default_search_operator = 'starts_with'
         end
@@ -310,39 +309,39 @@ describe RailsAdmin::Config::Fields::Base do
       end
     end
 
-    context "when search operator is specified for the field" do
-      it "uses specified search operator" do
+    context 'when search operator is specified for the field' do
+      it 'uses specified search operator' do
         RailsAdmin.config Player do
           list do
             fields do
-              search_operator "starts_with"
+              search_operator 'starts_with'
             end
           end
         end
         expect(queryable_fields).to have_at_least(1).field
-        expect(queryable_fields.first.search_operator).to eq("starts_with")
+        expect(queryable_fields.first.search_operator).to eq('starts_with')
       end
 
-      it "uses specified search operator even if config.default_search_operator set" do
+      it 'uses specified search operator even if config.default_search_operator set' do
         RailsAdmin.config do |config|
           config.default_search_operator = 'starts_with'
 
           config.model Player do
             list do
               fields do
-                search_operator "ends_with"
+                search_operator 'ends_with'
               end
             end
           end
         end
         expect(queryable_fields).to have_at_least(1).field
-        expect(queryable_fields.first.search_operator).to eq("ends_with")
+        expect(queryable_fields.first.search_operator).to eq('ends_with')
       end
     end
   end
 
-  describe "#render" do
-    it "is configurable" do
+  describe '#render' do
+    it 'is configurable' do
       RailsAdmin.config Team do
         field :name do
           render do
@@ -354,20 +353,20 @@ describe RailsAdmin::Config::Fields::Base do
     end
   end
 
-  describe "#active" do
-    it "is false by default" do
+  describe '#active' do
+    it 'is false by default' do
       expect(RailsAdmin.config(Team).field(:division).active?).to be_false
     end
   end
 
-  describe "#associated_collection" do
-    it "returns [] when type is blank?" do
-      expect(RailsAdmin.config(Comment).fields.find{|f|f.name == :commentable}.associated_collection('')).to be_empty
+  describe '#associated_collection' do
+    it 'returns [] when type is blank?' do
+      expect(RailsAdmin.config(Comment).fields.detect { |f|f.name == :commentable }.associated_collection('')).to be_empty
     end
   end
 
-  describe "#visible?" do
-    it "is false when fields have specific name " do
+  describe '#visible?' do
+    it 'is false when fields have specific name ' do
       class FieldVisibilityTest < Tableless
         column :id, :integer
         column :_id, :integer
@@ -380,14 +379,14 @@ describe RailsAdmin::Config::Fields::Base do
         column :updated_on, :datetime
         column :deleted_on, :datetime
       end
-      expect(RailsAdmin.config(FieldVisibilityTest).base.fields.select{|f| f.visible? }.map(&:name)).to match_array [:_id, :created_at, :created_on, :deleted_at, :deleted_on, :id, :name, :updated_at, :updated_on]
-      expect(RailsAdmin.config(FieldVisibilityTest).list.fields.select{|f| f.visible? }.map(&:name)).to match_array [:_id, :created_at, :created_on, :deleted_at, :deleted_on, :id, :name, :updated_at, :updated_on]
-      expect(RailsAdmin.config(FieldVisibilityTest).edit.fields.select{|f| f.visible? }.map(&:name)).to match_array [:name]
-      expect(RailsAdmin.config(FieldVisibilityTest).show.fields.select{|f| f.visible? }.map(&:name)).to match_array [:name]
+      expect(RailsAdmin.config(FieldVisibilityTest).base.fields.select { |f| f.visible? }.collect(&:name)).to match_array [:_id, :created_at, :created_on, :deleted_at, :deleted_on, :id, :name, :updated_at, :updated_on]
+      expect(RailsAdmin.config(FieldVisibilityTest).list.fields.select { |f| f.visible? }.collect(&:name)).to match_array [:_id, :created_at, :created_on, :deleted_at, :deleted_on, :id, :name, :updated_at, :updated_on]
+      expect(RailsAdmin.config(FieldVisibilityTest).edit.fields.select { |f| f.visible? }.collect(&:name)).to match_array [:name]
+      expect(RailsAdmin.config(FieldVisibilityTest).show.fields.select { |f| f.visible? }.collect(&:name)).to match_array [:name]
     end
   end
 
-  describe "#allowed_methods" do
+  describe '#allowed_methods' do
     it 'includes method_name' do
       RailsAdmin.config do |config|
         config.model Team do