upstream-rails_admin 1.0.2

data/spec/integration/basic/create/rails_admin_namespaced_model_create_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe "RailsAdmin Namespaced Model Create" do
+
+  subject { page }
+
+  before(:each) do
+    visit new_path(:model_name => "cms~basic_page")
+
+    fill_in "cms_basic_page[title]", :with => "Hello"
+    fill_in "cms_basic_page[content]", :with => "World"
+  end
+
+  it 'should be successful' do
+    click_button "Save"
+  end
+
+  it 'should create object with correct attributes' do
+    expect {
+      click_button "Save"
+    }.to change(Cms::BasicPage, :count).by(1)
+  end
+end

data/spec/integration/basic/delete/rails_admin_basic_delete_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe "RailsAdmin Basic Delete" do
+
+  subject { page }
+
+  describe "delete" do
+    before(:each) do
+      @player = FactoryGirl.create :player
+      visit delete_path(:model_name => "player", :id => @player.id)
+    end
+
+    it "should show \"Delete model\"" do
+      should have_content("delete this player")
+    end
+  end
+
+  describe "delete with missing object" do
+    before(:each) do
+      visit delete_path(:model_name => "player", :id => 1)
+    end
+
+    it "should raise NotFound" do
+      page.driver.status_code.should eql(404)
+    end
+  end
+end

data/spec/integration/basic/destroy/rails_admin_basic_destroy_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+describe "RailsAdmin Basic Destroy" do
+
+  subject { page }
+
+  describe "destroy" do
+    before(:each) do
+      @player = FactoryGirl.create :player
+      visit delete_path(:model_name => "player", :id => @player.id)
+      click_button "Yes, I'm sure"
+      @player = RailsAdmin::AbstractModel.new("Player").first
+    end
+
+    it "should destroy an object" do
+      @player.should be_nil
+    end
+  end
+
+  describe "destroy with errors" do
+    before(:each) do
+      Player.any_instance.stub(:destroy).and_return false
+      @player = FactoryGirl.create :player
+      visit delete_path(:model_name => "player", :id => @player.id)
+      click_button "Yes, I'm sure"
+    end
+
+    it "should destroy an object" do
+      @player.reload.should be
+    end
+  end
+
+  describe "destroy" do
+    before(:each) do
+      @player = FactoryGirl.create :player
+      visit delete_path(:model_name => "player", :id => @player.id)
+      click_button "Cancel"
+      @player = RailsAdmin::AbstractModel.new("Player").first
+    end
+
+    it "should not destroy an object" do
+      @player.should be
+    end
+  end
+
+  describe "destroy with missing object" do
+    before(:each) do
+      page.driver.delete(delete_path(:model_name => "player", :id => 1))
+    end
+
+    it "should raise NotFound" do
+      page.driver.status_code.should eql(404)
+    end
+  end
+
+end

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

@@ -0,0 +1,109 @@
+require 'spec_helper'
+
+describe "RailsAdmin Basic Edit" do
+
+  subject { page }
+
+  describe "edit" do
+    before(:each) do
+      @player = FactoryGirl.create :player
+      visit edit_path(:model_name => "player", :id => @player.id)
+    end
+
+    it "should show \"Edit model\"" do
+      should have_content("Edit Player")
+    end
+
+    it "should show required fields as \"Required\"" do
+      should have_selector("div", :text => /Name\s*Required/)
+      should have_selector("div", :text => /Number\s*Required/)
+    end
+
+    it "should show non-required fields as \"Optional\"" do
+      find("#player_position_field .help-block").should have_content("Optional")
+      find("#player_born_on_field .help-block").should have_content("Optional")
+      find("#player_notes_field .help-block").should have_content("Optional")
+    end
+  end
+
+  describe "association with inverse_of option" do
+    it "should add a related id to the belongs_to create team link" do
+      @player = FactoryGirl.create :player
+      visit edit_path(:model_name => "player", :id => @player.id)
+      should have_selector("a", :href => 'admin/teams/new?associations[players]=' + @player.id.to_s)
+    end
+
+    it "should add a related id to the has_many create team link" do
+      @team = FactoryGirl.create :team
+      visit edit_path(:model_name => "team", :id => @team.id)
+      should have_selector("a", :href => 'admin/players/new?associations[team]=' + @team.id.to_s)
+    end
+  end
+
+  describe "readonly associations" do
+
+    it 'should not be editable' do
+      @league = FactoryGirl.create :league
+      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)
+    end
+
+    it "should show associated objects" do
+      should have_selector "#fan_team_ids" do |select|
+        select[0].should have_selector 'option[selected="selected"]'
+        select[1].should_not have_selector 'option[selected="selected"]'
+        select[2].should_not have_selector 'option[selected="selected"]'
+      end
+    end
+  end
+
+  describe "edit with missing object" do
+    before(:each) do
+      visit edit_path(:model_name => "player", :id => 1)
+    end
+
+    it "should raise NotFound" do
+      page.driver.status_code.should eql(404)
+    end
+  end
+
+  describe "edit with missing label", :given => ["a player exists", "three teams with no name exist"] do
+    before(:each) do
+      @player = FactoryGirl.create :player
+      @teams = 3.times.map { 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
+      @ball = FactoryGirl.create :ball
+      visit edit_path(:model_name => "ball", :id => @ball.id)
+    end
+
+    it "should display 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
+
+    it "should send back to previous URL" do
+      @ball = FactoryGirl.create :ball
+      visit '/admin/ball?sort=color'
+      click_link 'Edit'
+      click_button 'Cancel'
+      page.current_url.should == 'http://www.example.com/admin/ball?sort=color'
+    end
+  end
+end

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

@@ -0,0 +1,106 @@
+require 'spec_helper'
+require 'csv'
+
+describe "RailsAdmin Export" do
+
+  subject { page }
+
+  before(:each) do
+    Comment.all.map &:destroy # rspec bug => doesn't get destroyed with transaction
+
+    @players = 4.times.map { 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.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"]}
+      }
+    }
+  end
+
+  describe "POST /admin/players/export (prompt)" do
+
+    it "should allow 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
+          field :name do
+            export_value do
+              "#{value} exported"
+            end
+          end
+        end
+      end
+
+      visit export_path(:model_name => 'player')
+      should have_content 'Select fields to export'
+      select "<comma> ','", :from => "csv_options_generator_col_sep"
+      click_button 'Export to csv'
+      csv = CSV.parse find('body').text
+      csv[0].should =~ ["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]"]
+      csv.flatten.should include(@player.name + " exported")
+      csv.flatten.should include(@player.team.name)
+      csv.flatten.should include(@player.draft.college)
+
+      csv.flatten.join(' ').should include(@player.comments.first.content.split("\n").first.strip)
+      csv.flatten.join(' ').should include(@player.comments.second.content.split("\n").first.strip)
+    end
+
+    it "should allow to export to JSON" do
+      visit export_path(:model_name => 'player')
+      click_button 'Export to json'
+      should have_content @player.team.name
+    end
+
+    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?)
+      # 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"))
+        should have_content @player.team.name
+      end
+    end
+
+    it "should export 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 find('body').text
+      csv[0].should =~ ["Id", "Commentable", "Commentable type", "Content", "Created at", "Updated at"]
+      csv[1..-1].each do |line|
+        line[csv[0].index('Commentable')].should == @player.id.to_s
+        line[csv[0].index('Commentable type')].should == @player.class.to_s
+      end
+    end
+  end
+
+  describe "POST /admin/players/export :format => :csv" do
+    it "should export 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 find('body').text
+      csv[0].should_not include('Created at')
+    end
+  end
+end

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

@@ -0,0 +1,400 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe "RailsAdmin Basic List" do
+
+  subject { page }
+
+  describe "GET /admin" do
+    it "should respond successfully" 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-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-error').should have_content("Ball with id '545-typo' could not be found")
+    end
+  end
+
+  describe "GET /admin/player as list" do
+    before do
+      21.times { FactoryGirl.create :player } # two pages of players
+      visit index_path(:model_name => "player")
+    end
+
+    it "should show \"List of Models\", should show filters and should show column headers" do
+      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("td a", :text => 'Show')
+      should have_selector("td a", :text => 'Edit')
+      should have_selector("td a", :text => 'Delete')
+
+      # it "has the search box with some prompt text" do
+      should have_selector("input[placeholder='Filter']")
+
+      # 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" do
+    before do
+      RailsAdmin.config Player do
+        list do
+          field :name
+          field :team
+          field :injured
+          field :retired
+        end
+      end
+      @teams = 2.times.map 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]),
+      ]
+    end
+
+    it "should allow to query on any attribute" do
+      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 "should allow to filter on one attribute" do
+      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 "should allow to combine filters on two different attributes" do
+      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 "should allow to filter on belongs_to relationships" do
+      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 "should allow to disable search on attributes" do
+      RailsAdmin.config Player do
+        list do
+          field :position
+          field :name do
+            searchable false
+          end
+        end
+      end
+      visit index_path(:model_name => "player", :query => @players[0].name)
+      should have_no_content(@players[0].name)
+    end
+
+    it "should allow to search a belongs_to attribute over the base table" do
+      RailsAdmin.config Player do
+        list do
+          field PK_COLUMN
+          field :name
+          field :team do
+            searchable Player => :team_id
+          end
+        end
+      end
+      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 "should allow to search a belongs_to attribute over the target table" do
+      RailsAdmin.config Player do
+        list do
+          field PK_COLUMN
+          field :name
+          field :team do
+            searchable Team => :name
+          end
+        end
+      end
+      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 "should allow 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
+          end
+        end
+      end
+      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 "should allow 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
+          field :name
+          field :team do
+            searchable 'teams.name'
+          end
+        end
+      end
+      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 "should allow to search a belongs_to attribute over the label method by default" do
+      RailsAdmin.config Player do
+        list do
+          field PK_COLUMN
+          field :name
+          field :team
+        end
+      end
+      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 "should allow 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
+          field :name
+          field :team do
+            searchable :name
+          end
+        end
+      end
+      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 "should allow to search over more than one attribute" do
+      RailsAdmin.config Player do
+        list do
+          field PK_COLUMN
+          field :name
+          field :team do
+            searchable [:name, {Player => :team_id}]
+          end
+        end
+      end
+      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'}}})
+      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 "should display base filters when no filters are present in the params" do
+      RailsAdmin.config Player do
+        list do
+          filters [:name, :team]
+        end
+      end
+
+      visit index_path(:model_name => "player")
+      should have_content(%{$.filters.append("Name", "name", "string", null, null, "", 1);})
+      should have_content(%{$.filters.append("Team", "team", "belongs_to_association", null, 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")
+    end
+
+    it "should show \"2 results\"" do
+      should have_content("2 players")
+    end
+  end
+
+  describe "GET /admin/player with 20 objects" do
+    before(:each) do
+      @players = 20.times.map { FactoryGirl.create :player }
+      visit index_path(:model_name => "player")
+    end
+
+    it "should show \"20 results\"" do
+      should have_content("20 players")
+    end
+  end
+
+  describe "GET /admin/player with 20 pages, page 8" do
+    before(:each) do
+      items_per_page = RailsAdmin.config.default_items_per_page
+      (items_per_page * 20).times { FactoryGirl.create(:player) }
+      visit index_path(:model_name => "player", :page => 8)
+    end
+
+    it "should paginate correctly" do
+      find('.pagination ul li:first').should have_content("« Prev")
+      find('.pagination ul li:last').should have_content("Next »")
+      find('.pagination ul li.active').should have_content("8")
+    end
+  end
+
+  describe "list with 20 pages, page 20" do
+    before(:each) do
+      items_per_page = RailsAdmin.config.default_items_per_page
+      @players = (items_per_page * 20).times.map { FactoryGirl.create(:player) }
+      visit index_path(:model_name => "player", :page => 20)
+    end
+
+    it "should paginate correctly and contain the right item" do
+      find('.pagination ul li:first').should have_content("« Prev")
+      find('.pagination ul li:last').should have_content("Next »")
+      find('.pagination ul li.active').should have_content("20")
+    end
+  end
+
+  describe "GET /admin/player show all" do
+    it "should respond successfully" do
+      2.times.map { FactoryGirl.create :player }
+      visit index_path(:model_name => "player", :all => true)
+    end
+  end
+
+  describe "list as compact json" do
+    it "should have_content an array with 2 elements and contain an array of elements with keys id and label" do
+      2.times.map { FactoryGirl.create :player }
+      response = page.driver.get(index_path(:model_name => "player", :compact => true, :format => :json))
+      ActiveSupport::JSON.decode(response.body).length.should eql(2)
+      ActiveSupport::JSON.decode(response.body).each do |object|
+        object.should have_key("id")
+        object.should have_key("label")
+      end
+    end
+  end
+
+  describe "search operator" do
+    let(:player) { FactoryGirl.create :player }
+
+    before do
+      Player.count.should == 0
+    end
+
+    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])
+      should have_content(player.name)
+    end
+
+    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])
+      should have_no_content(player.name)
+    end
+
+    it "finds the player if the query matches the specified search operator" do
+      RailsAdmin.config Player do
+        list do
+          field :name do
+            search_operator 'starts_with'
+          end
+        end
+      end
+      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
+      RailsAdmin.config Player do
+        list do
+          field :name do
+            search_operator 'starts_with'
+          end
+        end
+      end
+      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
+      @ball = FactoryGirl.create :ball
+
+      visit index_path(:model_name => "ball")
+    end
+
+    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']")
+    end
+
+  end
+
+end