rails_admin 0.6.0 → 0.6.1

data/spec/integration/basic/edit/rails_admin_basic_edit_spec.rb

@@ -1,64 +1,64 @@
 require 'spec_helper'
 
-describe "RailsAdmin Basic Edit" do
+describe 'RailsAdmin Basic Edit' do
 
   subject { page }
 
-  describe "edit" do
-    before(:each) do
+  describe 'edit' do
+    before do
       @player = FactoryGirl.create :player
-      visit edit_path(:model_name => "player", :id => @player.id)
+      visit edit_path(model_name: 'player', id: @player.id)
     end
 
     it "shows \"Edit model\"" do
-      should have_content("Edit Player")
+      should have_content('Edit Player')
     end
 
     it "shows required fields as \"Required\"" do
-      should have_selector("div", :text => /Name\s*Required/)
-      should have_selector("div", :text => /Number\s*Required/)
+      should have_selector('div', text: /Name\s*Required/)
+      should have_selector('div', text: /Number\s*Required/)
     end
 
     it "shows non-required fields as \"Optional\"" do
-      expect(find("#player_position_field .help-block")).to have_content("Optional")
-      expect(find("#player_born_on_field .help-block")).to have_content("Optional")
-      expect(find("#player_notes_field .help-block")).to have_content("Optional")
+      expect(find('#player_position_field .help-block')).to have_content('Optional')
+      expect(find('#player_born_on_field .help-block')).to have_content('Optional')
+      expect(find('#player_notes_field .help-block')).to have_content('Optional')
     end
   end
 
-  describe "association with inverse_of option" do
-    it "adds a related id to the belongs_to create team link" do
+  describe 'association with inverse_of option' do
+    it 'adds a related id to the belongs_to create team link' do
       @player = FactoryGirl.create :player
-      visit edit_path(:model_name => "player", :id => @player.id)
+      visit edit_path(model_name: 'player', id: @player.id)
       should have_selector("a[data-link='/admin/team/new?associations%5Bplayers%5D=#{@player.id.to_s}&modal=true']")
     end
 
-    it "adds a related id to the has_many create team link" do
+    it 'adds a related id to the has_many create team link' do
       @team = FactoryGirl.create :team
-      visit edit_path(:model_name => "team", :id => @team.id)
+      visit edit_path(model_name: 'team', id: @team.id)
       should have_selector("a[data-link='/admin/player/new?associations%5Bteam%5D=#{@team.id.to_s}&modal=true']")
     end
   end
 
-  describe "readonly associations" do
+  describe 'readonly associations' do
 
-    it "is not editable" do
+    it 'is not editable' do
       @league = FactoryGirl.create :league
-      visit edit_path(:model_name => "league", :id => @league.id)
+      visit edit_path(model_name: 'league', id: @league.id)
       should_not have_selector('select#league_team_ids')
       should have_selector('select#league_division_ids') # decoy, fails if naming scheme changes
     end
   end
 
-  describe "edit with has-and-belongs-to-many association" do
-    before(:each) do
-      @teams = 3.times.map { FactoryGirl.create :team }
-      @fan = FactoryGirl.create :fan, :teams => [@teams[0]]
-      visit edit_path(:model_name => "fan", :id => @fan.id)
+  describe 'edit with has-and-belongs-to-many association' do
+    before do
+      @teams = 3.times.collect { FactoryGirl.create :team }
+      @fan = FactoryGirl.create :fan, teams: [@teams[0]]
+      visit edit_path(model_name: 'fan', id: @fan.id)
     end
 
-    it "shows associated objects" do
-      should have_selector "#fan_team_ids" do |select|
+    it 'shows associated objects' do
+      should have_selector '#fan_team_ids' do |select|
         expect(select[0]).to have_selector 'option[selected="selected"]'
         expect(select[1]).not_to have_selector 'option[selected="selected"]'
         expect(select[2]).not_to have_selector 'option[selected="selected"]'
@@ -66,39 +66,39 @@ describe "RailsAdmin Basic Edit" do
     end
   end
 
-  describe "edit with missing object" do
-    before(:each) do
-      visit edit_path(:model_name => "player", :id => 1)
+  describe 'edit with missing object' do
+    before do
+      visit edit_path(model_name: 'player', id: 1)
     end
 
-    it "raises NotFound" do
+    it 'raises NotFound' do
       expect(page.driver.status_code).to eq(404)
     end
   end
 
-  describe "edit with missing label", :given => ["a player exists", "three teams with no name exist"] do
-    before(:each) do
+  describe 'edit with missing label', given: ['a player exists', 'three teams with no name exist'] do
+    before do
       @player = FactoryGirl.create :player
-      @teams = 3.times.map { FactoryGirl.create :team, :name => "" }
-      visit edit_path(:model_name => "player", :id => @player.id)
+      @teams = 3.times.collect { FactoryGirl.create :team, name: '' }
+      visit edit_path(model_name: 'player', id: @player.id)
     end
   end
 
-  describe "edit object with overridden to_param" do
-    before(:each) do
+  describe 'edit object with overridden to_param' do
+    before do
       @ball = FactoryGirl.create :ball
-      visit edit_path(:model_name => "ball", :id => @ball.id)
+      visit edit_path(model_name: 'ball', id: @ball.id)
     end
 
-    it "displays a link to the delete page" do
+    it 'displays a link to the delete page' do
       should have_selector "a[href='/admin/ball/#{@ball.id}/delete']"
     end
 
   end
 
-  describe "clicking cancel when editing an object" do
+  describe 'clicking cancel when editing an object' do
 
-    it "sends back to previous URL" do
+    it 'sends back to previous URL' do
       @ball = FactoryGirl.create :ball
       visit '/admin/ball?sort=color'
       click_link 'Edit'

data/spec/integration/basic/export/rails_admin_basic_export_spec.rb

@@ -1,36 +1,36 @@
 require 'spec_helper'
 require 'csv'
 
-describe "RailsAdmin Export" do
+describe 'RailsAdmin Export' do
 
   subject { page }
 
-  before(:each) do
-    Comment.all.map &:destroy # rspec bug => doesn't get destroyed with transaction
+  before do
+    Comment.all.collect(&:destroy) # rspec bug => doesn't get destroyed with transaction
 
-    @players = 4.times.map { FactoryGirl.create :player }
+    @players = 4.times.collect { FactoryGirl.create :player }
     @player = @players.first
     @player.team = FactoryGirl.create :team
     @player.draft = FactoryGirl.create :draft
-    @player.comments = (@comments = 2.times.map { FactoryGirl.create(:comment) })
+    @player.comments = (@comments = 2.times.collect { FactoryGirl.create(:comment) })
     @player.save
 
     @abstract_model = RailsAdmin::AbstractModel.new(Player)
 
     # removed schema=>only=>created_at
     @non_default_schema = {
-      "only"=>[PK_COLUMN.to_s, "updated_at", "deleted_at", "name", "position", "number", "retired", "injured", "born_on", "notes", "suspended"],
-      "include"=>{
-        "team"=>{"only"=>[PK_COLUMN.to_s, "created_at", "updated_at", "name", "logo_url", "manager", "ballpark", "mascot", "founded", "wins", "losses", "win_percentage", "revenue", "color"]},
-        "draft"=>{"only"=>[PK_COLUMN.to_s, "created_at", "updated_at", "date", "round", "pick", "overall", "college", "notes"]},
-        "comments"=>{"only"=>[PK_COLUMN.to_s, "content", "created_at", "updated_at"]}
+      'only' => [PK_COLUMN.to_s, 'updated_at', 'deleted_at', 'name', 'position', 'number', 'retired', 'injured', 'born_on', 'notes', 'suspended'],
+      'include' => {
+        'team' => {'only' => [PK_COLUMN.to_s, 'created_at', 'updated_at', 'name', 'logo_url', 'manager', 'ballpark', 'mascot', 'founded', 'wins', 'losses', 'win_percentage', 'revenue', 'color']},
+        'draft' => {'only' => [PK_COLUMN.to_s, 'created_at', 'updated_at', 'date', 'round', 'pick', 'overall', 'college', 'notes']},
+        'comments' => {'only' => [PK_COLUMN.to_s, 'content', 'created_at', 'updated_at']}
       }
     }
   end
 
-  describe "POST /admin/players/export (prompt)" do
+  describe 'POST /admin/players/export (prompt)' do
 
-    it "allows to export to CSV with associations and default schema, containing properly translated header and follow configuration" do
+    it 'allows to export to CSV with associations and default schema, containing properly translated header and follow configuration' do
       RailsAdmin.config do |c|
         c.model Player do
           include_all_fields
@@ -42,20 +42,20 @@ describe "RailsAdmin Export" do
         end
       end
 
-      visit export_path(:model_name => 'player')
+      visit export_path(model_name: 'player')
       should have_content 'Select fields to export'
-      select "<comma> ','", :from => "csv_options_generator_col_sep"
+      select "<comma> ','", from: 'csv_options_generator_col_sep'
       click_button 'Export to csv'
-      csv = CSV.parse page.driver.response.body.force_encoding("utf-8") # comes through as us-ascii on some platforms
-      expect(csv[0]).to match_array ["Id", "Created at", "Updated at", "Deleted at", "Name", "Position",
-        "Number", "Retired", "Injured", "Born on", "Notes", "Suspended", "Id [Team]", "Created at [Team]",
-        "Updated at [Team]", "Name [Team]", "Logo url [Team]", "Team Manager [Team]", "Ballpark [Team]",
-        "Mascot [Team]", "Founded [Team]", "Wins [Team]", "Losses [Team]", "Win percentage [Team]",
-        "Revenue [Team]", "Color [Team]", "Custom field [Team]", "Id [Draft]", "Created at [Draft]",
-        "Updated at [Draft]", "Date [Draft]", "Round [Draft]", "Pick [Draft]", "Overall [Draft]",
-        "College [Draft]", "Notes [Draft]", "Id [Comments]", "Content [Comments]", "Created at [Comments]",
-        "Updated at [Comments]"]
-      expect(csv.flatten).to include(@player.name + " exported")
+      csv = CSV.parse page.driver.response.body.force_encoding('utf-8') # comes through as us-ascii on some platforms
+      expect(csv[0]).to match_array ['Id', 'Created at', 'Updated at', 'Deleted at', 'Name', 'Position',
+                                     'Number', 'Retired', 'Injured', 'Born on', 'Notes', 'Suspended', 'Id [Team]', 'Created at [Team]',
+                                     'Updated at [Team]', 'Name [Team]', 'Logo url [Team]', 'Team Manager [Team]', 'Ballpark [Team]',
+                                     'Mascot [Team]', 'Founded [Team]', 'Wins [Team]', 'Losses [Team]', 'Win percentage [Team]',
+                                     'Revenue [Team]', 'Color [Team]', 'Custom field [Team]', 'Id [Draft]', 'Created at [Draft]',
+                                     'Updated at [Draft]', 'Date [Draft]', 'Round [Draft]', 'Pick [Draft]', 'Overall [Draft]',
+                                     'College [Draft]', 'Notes [Draft]', 'Id [Comments]', 'Content [Comments]', 'Created at [Comments]',
+                                     'Updated at [Comments]']
+      expect(csv.flatten).to include(@player.name + ' exported')
       expect(csv.flatten).to include(@player.team.name)
       expect(csv.flatten).to include(@player.draft.college)
 
@@ -63,14 +63,14 @@ describe "RailsAdmin Export" do
       expect(csv.flatten.join(' ')).to include(@player.comments.second.content.split("\n").first.strip)
     end
 
-    it "allows to export to JSON" do
-      visit export_path(:model_name => 'player')
+    it 'allows to export to JSON' do
+      visit export_path(model_name: 'player')
       click_button 'Export to json'
       should have_content @player.team.name
     end
 
-    it "allows to export to XML" do
-      visit export_path(:model_name => 'player')
+    it 'allows 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
@@ -78,17 +78,17 @@ describe "RailsAdmin Export" do
       # and Mongoid with ActiveModel 3.1 does not support to_xml's :include options
       # (due change of implementation in ActiveModel::Serializers between 3.1 and 3.2)
       if RUBY_VERSION =~ /1\.9/ &&
-          (CI_ORM != :mongoid || (CI_ORM == :mongoid && ActiveModel::VERSION::STRING >= "3.2"))
+          (CI_ORM != :mongoid || (CI_ORM == :mongoid && ActiveModel::VERSION::STRING >= '3.2'))
         should have_content @player.team.name
       end
     end
 
-    it "exports polymorphic fields the easy way for now" do
-      visit export_path(:model_name => 'comment')
-      select "<comma> ','", :from => "csv_options_generator_col_sep"
+    it 'exports polymorphic fields the easy way for now' do
+      visit export_path(model_name: 'comment')
+      select "<comma> ','", from: 'csv_options_generator_col_sep'
       click_button 'Export to csv'
       csv = CSV.parse page.driver.response.body
-      expect(csv[0]).to match_array ["Id", "Commentable", "Commentable type", "Content", "Created at", "Updated at"]
+      expect(csv[0]).to match_array ['Id', 'Commentable', 'Commentable type', 'Content', 'Created at', 'Updated at']
       csv[1..-1].each do |line|
         expect(line[csv[0].index('Commentable')]).to eq(@player.id.to_s)
         expect(line[csv[0].index('Commentable type')]).to eq(@player.class.to_s)
@@ -96,9 +96,9 @@ describe "RailsAdmin Export" do
     end
   end
 
-  describe "POST /admin/players/export :format => :csv" do
-    it "exports with modified schema" do
-      page.driver.post(export_path(:model_name => 'player', :schema => @non_default_schema, :csv => true, :all => true, :csv_options => { :generator => { :col_sep => "," } }))
+  describe 'POST /admin/players/export :format => :csv' do
+    it 'exports with modified schema' do
+      page.driver.post(export_path(model_name: 'player', schema: @non_default_schema, csv: true, all: true, csv_options: {generator: {col_sep: ','}}))
       csv = CSV.parse page.driver.response.body
       expect(csv[0]).not_to include('Created at')
     end

data/spec/integration/basic/list/rails_admin_basic_list_spec.rb

@@ -2,17 +2,17 @@
 
 require 'spec_helper'
 
-describe "RailsAdmin Basic List" do
+describe 'RailsAdmin Basic List' do
 
   subject { page }
 
-  describe "GET /admin" do
-    it "responds successfully" do
+  describe 'GET /admin' do
+    it 'responds successfully' do
       visit dashboard_path
     end
   end
 
-  describe "GET /admin/typo" do
+  describe 'GET /admin/typo' do
     it "redirects to dashboard and inform the user the model wasn't found" do
       visit '/admin/whatever'
       expect(page.driver.status_code).to eq(404)
@@ -20,7 +20,7 @@ describe "RailsAdmin Basic List" do
     end
   end
 
-  describe "GET /admin/balls/545-typo" do
+  describe 'GET /admin/balls/545-typo' do
     it "redirects to balls index and inform the user the id wasn't found" do
       visit '/admin/ball/545-typo'
       expect(page.driver.status_code).to eq(404)
@@ -28,14 +28,14 @@ describe "RailsAdmin Basic List" do
     end
   end
 
-  describe "GET /admin/player as list" do
+  describe 'GET /admin/player as list' do
     it "shows \"List of Models\", should show filters and should show column headers" do
       RailsAdmin.config.default_items_per_page = 1
       2.times { FactoryGirl.create :player } # two pages of players
-      visit index_path(:model_name => "player")
-      should have_content("List of Players")
-      should have_content("Created at")
-      should have_content("Updated at")
+      visit index_path(model_name: 'player')
+      should have_content('List of Players')
+      should have_content('Created at')
+      should have_content('Updated at')
 
       # it "shows the show, edit and delete links" do
       should have_selector("li[title='Show'] a")
@@ -54,20 +54,20 @@ describe "RailsAdmin Basic List" do
     end
   end
 
-  describe "GET /admin/player" do
+  describe 'GET /admin/player' do
     before do
-      @teams = 2.times.map do
+      @teams = 2.times.collect do
         FactoryGirl.create(:team)
       end
       @players = [
-        FactoryGirl.create(:player, :retired => true, :injured => true, :team => @teams[0]),
-        FactoryGirl.create(:player, :retired => true, :injured => false, :team => @teams[0]),
-        FactoryGirl.create(:player, :retired => false, :injured => true, :team => @teams[1]),
-        FactoryGirl.create(:player, :retired => false, :injured => false, :team => @teams[1]),
+        FactoryGirl.create(:player, retired: true, injured: true, team: @teams[0]),
+        FactoryGirl.create(:player, retired: true, injured: false, team: @teams[0]),
+        FactoryGirl.create(:player, retired: false, injured: true, team: @teams[1]),
+        FactoryGirl.create(:player, retired: false, injured: false, team: @teams[1]),
       ]
     end
 
-    it "allows to query on any attribute" do
+    it 'allows to query on any attribute' do
       RailsAdmin.config Player do
         list do
           field :name
@@ -77,14 +77,14 @@ describe "RailsAdmin Basic List" do
         end
       end
 
-      visit index_path(:model_name => "player", :query => @players[0].name)
+      visit index_path(model_name: 'player', query: @players[0].name)
       should have_content(@players[0].name)
       (1..3).each do |i|
         should have_no_content(@players[i].name)
       end
     end
 
-    it "allows to filter on one attribute" do
+    it 'allows to filter on one attribute' do
       RailsAdmin.config Player do
         list do
           field :name
@@ -94,14 +94,14 @@ describe "RailsAdmin Basic List" do
         end
       end
 
-      visit index_path(:model_name => "player", :f => {:injured => {"1" => {:v => "true"}}})
+      visit index_path(model_name: 'player', f: {injured: {'1' => {v: 'true'}}})
       should have_content(@players[0].name)
       should have_no_content(@players[1].name)
       should have_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "allows to combine filters on two different attributes" do
+    it 'allows to combine filters on two different attributes' do
       RailsAdmin.config Player do
         list do
           field :name
@@ -111,14 +111,14 @@ describe "RailsAdmin Basic List" do
         end
       end
 
-      visit index_path(:model_name => "player", :f => {:retired => {"1" => {:v => "true"}}, :injured => {"1" => {:v => "true"}}})
+      visit index_path(model_name: 'player', f: {retired: {'1' => {v: 'true'}}, injured: {'1' => {v: 'true'}}})
       should have_content(@players[0].name)
       (1..3).each do |i|
         should have_no_content(@players[i].name)
       end
     end
 
-    it "allows to filter on belongs_to relationships" do
+    it 'allows to filter on belongs_to relationships' do
       RailsAdmin.config Player do
         list do
           field :name
@@ -128,14 +128,14 @@ describe "RailsAdmin Basic List" do
         end
       end
 
-      visit index_path(:model_name => "player", :f => {:team => {"1" => { :v => @teams[0].name }}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams[0].name}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "allows to disable search on attributes" do
+    it 'allows to disable search on attributes' do
       RailsAdmin.config Player do
         list do
           field :position
@@ -144,11 +144,11 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :query => @players[0].name)
+      visit index_path(model_name: 'player', query: @players[0].name)
       should have_no_content(@players[0].name)
     end
 
-    it "allows to search a belongs_to attribute over the base table" do
+    it 'allows to search a belongs_to attribute over the base table' do
       RailsAdmin.config Player do
         list do
           field PK_COLUMN
@@ -158,15 +158,14 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.id}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.id}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-
-    it "allows to search a belongs_to attribute over the target table" do
+    it 'allows to search a belongs_to attribute over the target table' do
       RailsAdmin.config Player do
         list do
           field PK_COLUMN
@@ -176,31 +175,31 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.name}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.name}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "allows to search a belongs_to attribute over the target table with a table name specified as a hash" do
+    it 'allows to search a belongs_to attribute over the target table with a table name specified as a hash' do
       RailsAdmin.config Player do
         list do
           field PK_COLUMN
           field :name
           field :team do
-            searchable :teams => :name
+            searchable teams: :name
           end
         end
       end
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.name}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.name}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "allows to search a belongs_to attribute over the target table with a table name specified as a string" do
+    it 'allows to search a belongs_to attribute over the target table with a table name specified as a string' do
       RailsAdmin.config Player do
         list do
           field PK_COLUMN
@@ -210,14 +209,14 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.name}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.name}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "allows to search a belongs_to attribute over the label method by default" do
+    it 'allows to search a belongs_to attribute over the label method by default' do
       RailsAdmin.config Player do
         list do
           field PK_COLUMN
@@ -225,14 +224,14 @@ describe "RailsAdmin Basic List" do
           field :team
         end
       end
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.name}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.name}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "allows to search a belongs_to attribute over the target table when an attribute is specified" do
+    it 'allows to search a belongs_to attribute over the target table when an attribute is specified' do
       RailsAdmin.config Player do
         list do
           field PK_COLUMN
@@ -242,14 +241,14 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.name}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.name}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "allows to search over more than one attribute" do
+    it 'allows to search over more than one attribute' do
       RailsAdmin.config Player do
         list do
           field PK_COLUMN
@@ -259,149 +258,149 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.name}, "2" => {:v => @teams.first.id, :o => 'is'}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.name}, '2' => {v: @teams.first.id, o: 'is'}}})
       should have_content(@players[0].name)
       should have_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
       # same with a different id
-      visit index_path(:model_name => "player", :f => {:team => {"1" => {:v => @teams.first.name}, "2" => {:v => @teams.last.id, :o => 'is'}}})
+      visit index_path(model_name: 'player', f: {team: {'1' => {v: @teams.first.name}, '2' => {v: @teams.last.id, o: 'is'}}})
       should have_no_content(@players[0].name)
       should have_no_content(@players[1].name)
       should have_no_content(@players[2].name)
       should have_no_content(@players[3].name)
     end
 
-    it "displays base filters when no filters are present in the params" do
+    it 'displays base filters when no filters are present in the params' do
       RailsAdmin.config Player do
         list do
           filters [:name, :team]
         end
       end
 
-      get index_path(:model_name => "player")
+      get index_path(model_name: 'player')
       expect(response.body).to include(%{$.filters.append("Name", "name", "string", "", null, "", 1);})
       expect(response.body).to include(%{$.filters.append("Team", "team", "belongs_to_association", "", null, "", 2);})
     end
   end
 
-  describe "GET /admin/player with 2 objects" do
-    before(:each) do
-      @players = 2.times.map { FactoryGirl.create :player }
-      visit index_path(:model_name => "player")
+  describe 'GET /admin/player with 2 objects' do
+    before do
+      @players = 2.times.collect { FactoryGirl.create :player }
+      visit index_path(model_name: 'player')
     end
 
     it "shows \"2 results\"" do
-      should have_content("2 players")
+      should have_content('2 players')
     end
   end
 
-  describe "GET /admin/player with 2 objects" do
-    before(:each) do
-      @players = 2.times.map { FactoryGirl.create :player }
-      visit index_path(:model_name => "player")
+  describe 'GET /admin/player with 2 objects' do
+    before do
+      @players = 2.times.collect { FactoryGirl.create :player }
+      visit index_path(model_name: 'player')
     end
 
     it "shows \"2 results\"" do
-      should have_content("2 players")
+      should have_content('2 players')
     end
   end
 
-  describe "GET /admin/player with 3 pages, page 2" do
+  describe 'GET /admin/player with 3 pages, page 2' do
     before do
       RailsAdmin.config.default_items_per_page = 1
       items_per_page = RailsAdmin.config.default_items_per_page
       (items_per_page * 3).times { FactoryGirl.create(:player) }
-      visit index_path(:model_name => "player", :page => 2)
+      visit index_path(model_name: 'player', page: 2)
     end
 
-    it "paginates correctly" do
-      expect(find('.pagination ul li:first')).to have_content("« Prev")
-      expect(find('.pagination ul li:last')).to have_content("Next »")
-      expect(find('.pagination ul li.active')).to have_content("2")
+    it 'paginates correctly' do
+      expect(find('.pagination ul li:first')).to have_content('« Prev')
+      expect(find('.pagination ul li:last')).to have_content('Next »')
+      expect(find('.pagination ul li.active')).to have_content('2')
     end
   end
 
-  describe "list with 3 pages, page 3" do
-    before(:each) do
+  describe 'list with 3 pages, page 3' do
+    before do
       items_per_page = RailsAdmin.config.default_items_per_page
-      @players = (items_per_page * 3).times.map { FactoryGirl.create(:player) }
-      visit index_path(:model_name => "player", :page => 3)
+      @players = (items_per_page * 3).times.collect { FactoryGirl.create(:player) }
+      visit index_path(model_name: 'player', page: 3)
     end
 
-    it "paginates correctly and contain the right item" do
-      expect(find('.pagination ul li:first')).to have_content("« Prev")
-      expect(find('.pagination ul li:last')).to have_content("Next »")
-      expect(find('.pagination ul li.active')).to have_content("3")
+    it 'paginates correctly and contain the right item' do
+      expect(find('.pagination ul li:first')).to have_content('« Prev')
+      expect(find('.pagination ul li:last')).to have_content('Next »')
+      expect(find('.pagination ul li.active')).to have_content('3')
     end
   end
 
-  describe "GET /admin/player show all" do
-    it "responds successfully with a single model" do
+  describe 'GET /admin/player show all' do
+    it 'responds successfully with a single model' do
       FactoryGirl.create :player
-      visit index_path(:model_name => "player", :all => true)
-      expect(find('div.total-count')).to have_content("1 player")
-      expect(find('div.total-count')).not_to have_content("1 players")
+      visit index_path(model_name: 'player', all: true)
+      expect(find('div.total-count')).to have_content('1 player')
+      expect(find('div.total-count')).not_to have_content('1 players')
     end
 
-    it "responds successfully with multiple models" do
-      2.times.map { FactoryGirl.create :player }
-      visit index_path(:model_name => "player", :all => true)
-      expect(find('div.total-count')).to have_content("2 players")
+    it 'responds successfully with multiple models' do
+      2.times.collect { FactoryGirl.create :player }
+      visit index_path(model_name: 'player', all: true)
+      expect(find('div.total-count')).to have_content('2 players')
     end
   end
 
-  describe "GET /admin/player show with pagination disabled by :associated_collection" do
-    it "responds successfully" do
+  describe 'GET /admin/player show with pagination disabled by :associated_collection' do
+    it 'responds successfully' do
       @team = FactoryGirl.create :team
-      2.times.map { FactoryGirl.create :player, :team => @team }
-      visit index_path(:model_name => "player", :associated_collection => "players", :compact => true, :current_action => 'update', :source_abstract_model => 'team', :source_object_id => @team.id)
-      expect(find('div.total-count')).to have_content("2 players")
+      2.times.collect { FactoryGirl.create :player, team: @team }
+      visit index_path(model_name: 'player', associated_collection: 'players', compact: true, current_action: 'update', source_abstract_model: 'team', source_object_id: @team.id)
+      expect(find('div.total-count')).to have_content('2 players')
     end
   end
 
-  describe "list as compact json" do
-    it "has_content an array with 2 elements and contain an array of elements with keys id and label" do
-      2.times.map { FactoryGirl.create :player }
-      get index_path(:model_name => "player", :compact => true, :format => :json)
+  describe 'list as compact json' do
+    it 'has_content an array with 2 elements and contain an array of elements with keys id and label' do
+      2.times.collect { FactoryGirl.create :player }
+      get index_path(model_name: 'player', compact: true, format: :json)
       expect(ActiveSupport::JSON.decode(response.body).length).to eq(2)
       ActiveSupport::JSON.decode(response.body).each do |object|
-        expect(object).to have_key("id")
-        expect(object).to have_key("label")
+        expect(object).to have_key('id')
+        expect(object).to have_key('label')
       end
     end
   end
 
-  describe "search operator" do
+  describe 'search operator' do
     let(:player) { FactoryGirl.create :player }
 
     before do
       expect(Player.count).to eq(0)
     end
 
-    it "finds the player if the query matches the default search opeartor" do
+    it 'finds the player if the query matches the default search opeartor' do
       RailsAdmin.config do |config|
         config.default_search_operator = 'ends_with'
         config.model Player do
           list { field :name }
         end
       end
-      visit index_path(:model_name => "player", :query => player.name[2, -1])
+      visit index_path(model_name: 'player', query: player.name[2, -1])
       should have_content(player.name)
     end
 
-    it "does not find the player if the query does not match the default search opeartor" do
+    it 'does not find the player if the query does not match the default search opeartor' do
       RailsAdmin.config do |config|
         config.default_search_operator = 'ends_with'
         config.model Player do
           list { field :name }
         end
       end
-      visit index_path(:model_name => "player", :query => player.name[0, 2])
+      visit index_path(model_name: 'player', query: player.name[0, 2])
       should have_no_content(player.name)
     end
 
-    it "finds the player if the query matches the specified search operator" do
+    it 'finds the player if the query matches the specified search operator' do
       RailsAdmin.config Player do
         list do
           field :name do
@@ -409,11 +408,11 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :query => player.name[0, 2])
+      visit index_path(model_name: 'player', query: player.name[0, 2])
       should have_content(player.name)
     end
 
-    it "does not find the player if the query does not match the specified search operator" do
+    it 'does not find the player if the query does not match the specified search operator' do
       RailsAdmin.config Player do
         list do
           field :name do
@@ -421,19 +420,19 @@ describe "RailsAdmin Basic List" do
           end
         end
       end
-      visit index_path(:model_name => "player", :query => player.name[1..-1])
+      visit index_path(model_name: 'player', query: player.name[1..-1])
       should have_no_content(player.name)
     end
   end
 
-  describe "list for objects with overridden to_param" do
-    before(:each) do
+  describe 'list for objects with overridden to_param' do
+    before do
       @ball = FactoryGirl.create :ball
 
-      visit index_path(:model_name => "ball")
+      visit index_path(model_name: 'ball')
     end
 
-    it "shows the show, edit and delete links with valid url" do
+    it 'shows the show, edit and delete links with valid url' do
       should have_selector("td a[href='/admin/ball/#{@ball.id}']")
       should have_selector("td a[href='/admin/ball/#{@ball.id}/edit']")
       should have_selector("td a[href='/admin/ball/#{@ball.id}/delete']")
@@ -441,4 +440,97 @@ describe "RailsAdmin Basic List" do
 
   end
 
+  describe 'Scopes' do
+    before do
+      RailsAdmin.config do |config|
+        config.model Team do
+          list do
+            scopes [nil, :red, :white]
+          end
+        end
+      end
+      @teams = [
+        FactoryGirl.create(:team, color: 'red'),
+        FactoryGirl.create(:team, color: 'red'),
+        FactoryGirl.create(:team, color: 'white'),
+        FactoryGirl.create(:team, color: 'black')
+      ]
+    end
+
+    it 'displays configured scopes' do
+      visit index_path(model_name: 'team')
+      expect(find('#scope_selector li:first')).to have_content('All')
+      expect(find('#scope_selector li:nth-child(2)')).to have_content('Red')
+      expect(find('#scope_selector li:nth-child(3)')).to have_content('White')
+      expect(find('#scope_selector li:last')).to have_content('White')
+      expect(find('#scope_selector li.active')).to have_content('All')
+    end
+
+    it 'shows only scoped records' do
+      visit index_path(model_name: 'team')
+      should have_content(@teams[0].name)
+      should have_content(@teams[1].name)
+      should have_content(@teams[2].name)
+      should have_content(@teams[3].name)
+
+      visit index_path(model_name: 'team', scope: 'red')
+      expect(find('#scope_selector li.active')).to have_content('Red')
+      should have_content(@teams[0].name)
+      should have_content(@teams[1].name)
+      should have_no_content(@teams[2].name)
+      should have_no_content(@teams[3].name)
+
+      visit index_path(model_name: 'team', scope: 'white')
+      expect(find('#scope_selector li.active')).to have_content('White')
+      should have_no_content(@teams[0].name)
+      should have_no_content(@teams[1].name)
+      should have_content(@teams[2].name)
+      should have_no_content(@teams[3].name)
+    end
+    it 'shows all records instead when scope not in list' do
+      visit index_path(model_name: 'team', scope: 'green')
+      should have_content(@teams[0].name)
+      should have_content(@teams[1].name)
+      should have_content(@teams[2].name)
+      should have_content(@teams[3].name)
+    end
+    describe 'i18n' do
+      before :each do
+        en = {admin: {scopes: {
+          _all: 'every',
+          red: 'krasnyj'
+        }}}
+        I18n.backend.store_translations(:en, en)
+      end
+      after { I18n.reload! }
+      context 'global' do
+        it 'displays configured scopes' do
+          visit index_path(model_name: 'team')
+          expect(find('#scope_selector li:first')).to have_content('every')
+          expect(find('#scope_selector li:nth-child(2)')).to have_content('krasnyj')
+          expect(find('#scope_selector li:nth-child(3)')).to have_content('White')
+          expect(find('#scope_selector li:last')).to have_content('White')
+          expect(find('#scope_selector li.active')).to have_content('every')
+        end
+      end
+      context 'per model' do
+        before :each do
+          en = {admin: {scopes: {team: {
+            _all: 'any',
+            red: 'kr'
+          }}}}
+          I18n.backend.store_translations(:en, en)
+        end
+        after { I18n.reload! }
+        it 'displays configured scopes' do
+          visit index_path(model_name: 'team')
+          expect(find('#scope_selector li:first')).to have_content('any')
+          expect(find('#scope_selector li:nth-child(2)')).to have_content('kr')
+          expect(find('#scope_selector li:nth-child(3)')).to have_content('White')
+          expect(find('#scope_selector li:last')).to have_content('White')
+          expect(find('#scope_selector li.active')).to have_content('any')
+        end
+      end
+    end
+  end
 end