obitum-rails_admin 0.0.1 → 0.0.2

data/spec/{requests → integration}/config/edit/rails_admin_config_edit_spec.rb

@@ -6,6 +6,49 @@ describe "RailsAdmin Config DSL Edit Section" do
 
   subject { page }
 
+  describe "default_value" do
+
+    it "should be set for all types of input fields" do
+      RailsAdmin.config do |config|
+        config.excluded_models = []
+        config.model(FieldTest) do
+
+          field :string_field do
+            default_value 'string_field default_value'
+          end
+          field :text_field do
+            default_value 'string_field text_field'
+          end
+          field :boolean_field do
+            default_value true
+          end
+          field :date_field do
+            default_value Date.today.to_s
+          end
+        end
+      end
+
+      visit new_path(:model_name => "field_test")
+      find_field('field_test[string_field]').value.should == 'string_field default_value'
+      find_field('field_test[text_field]').value.should == 'string_field text_field'
+      find_field('field_test[date_field]').value.should == Date.today.to_s
+      has_checked_field?('field_test[boolean_field]').should be_true
+    end
+
+    it "should set default value for selects" do
+      RailsAdmin.config(Team) do
+        field :color, :enum do
+          default_value 'black'
+          enum do
+            ['black', 'white']
+          end
+        end
+      end
+      visit new_path(:model_name => "team")
+      find_field('team[color]').value.should == 'black'
+    end
+  end
+
   describe "attr_accessible" do
 
 
@@ -40,7 +83,7 @@ describe "RailsAdmin Config DSL Edit Section" do
   describe "css hooks" do
     it "should be present" do
       visit new_path(:model_name => "team")
-      should have_selector("#team_division_id_field.field.belongs_to_association_type.division_field")
+      should have_selector("#team_division_id_field.belongs_to_association_type.division_field")
     end
   end
 
@@ -112,7 +155,7 @@ describe "RailsAdmin Config DSL Edit Section" do
           end
         end
         visit new_path(:model_name => "team")
-        should have_selector('legend small', :text => "help paragraph to display")
+        should have_selector('fieldset>p', :text => "help paragraph to display")
       end
 
       it "should not show help if not present" do
@@ -124,7 +167,7 @@ describe "RailsAdmin Config DSL Edit Section" do
           end
         end
         visit new_path(:model_name => "team")
-        should_not have_selector('legend small')
+        should_not have_selector('fieldset>p')
       end
 
       it "should be able to display multiple help if there are multiple sections" do
@@ -142,9 +185,9 @@ describe "RailsAdmin Config DSL Edit Section" do
           end
         end
         visit new_path(:model_name => "team")
-        should have_selector("legend small", :text => 'help for default')
-        should have_selector("legend small", :text => 'help for other section')
-        should have_selector("legend small", :count => 2)
+        should have_selector("fieldset>p", :text => 'help for default')
+        should have_selector("fieldset>p", :text => 'help for other section')
+        should have_selector("fieldset>p", :count => 2)
       end
 
       it "should use the db column size for the maximum length" do
@@ -229,7 +272,7 @@ describe "RailsAdmin Config DSL Edit Section" do
       should have_selector("label", :text => "Name")
       should have_selector("label", :text => "Logo url")
       should have_selector("label", :text => "Division")
-      should have_selector(".field", :count => 3)
+      should have_selector(".control-group", :count => 3)
     end
 
     it "should have accessor for its fields by type" do
@@ -301,7 +344,7 @@ describe "RailsAdmin Config DSL Edit Section" do
       visit new_path(:model_name => "team")
       should have_selector("label", :text => "Division")
       should have_selector("label", :text => "Name")
-      should have_selector(".field", :count => 2)
+      should have_selector(".control-group", :count => 2)
     end
 
     it "should delegates the label option to the ActiveModel API and memoize I18n awarly" do
@@ -511,7 +554,7 @@ describe "RailsAdmin Config DSL Edit Section" do
     after(:each) do
       Time.zone = 'UTC'
     end
-    
+
     describe "a datetime field" do
 
       it "should default to %B %d, %Y %H:%M" do
@@ -534,7 +577,7 @@ describe "RailsAdmin Config DSL Edit Section" do
         fill_in "field_test[datetime_field]", :with => @time.strftime("%a, %d %b %Y %H:%M:%S")
         click_button "Save"
         @record = RailsAdmin::AbstractModel.new("FieldTest").first
-        @record.datetime_field.should eql(::DateTime.parse(@time.to_s))
+        @record.datetime_field.to_s(:rfc822).should eql(@time.to_s(:rfc822))
       end
 
       it "should have a customization option" do
@@ -549,7 +592,7 @@ describe "RailsAdmin Config DSL Edit Section" do
         fill_in "field_test[datetime_field]", :with => @time.strftime("%Y-%m-%d %H:%M:%S")
         click_button "Save"
         @record = RailsAdmin::AbstractModel.new("FieldTest").first
-        @record.datetime_field.should eql(::DateTime.parse(@time.to_s))
+        @record.datetime_field.to_s(:rfc822).should eql(@time.to_s(:rfc822))
       end
     end
 
@@ -575,7 +618,7 @@ describe "RailsAdmin Config DSL Edit Section" do
         fill_in "field_test[timestamp_field]", :with => @time.strftime("%a, %d %b %Y %H:%M:%S")
         click_button "Save"
         @record = RailsAdmin::AbstractModel.new("FieldTest").first
-        @record.timestamp_field.should eql(::DateTime.parse(@time.to_s))
+        @record.timestamp_field.to_s(:rfc822).should eql(@time.to_s(:rfc822))
       end
 
       it "should have a customization option" do
@@ -590,7 +633,7 @@ describe "RailsAdmin Config DSL Edit Section" do
         fill_in "field_test[timestamp_field]", :with => @time.strftime("%Y-%m-%d %H:%M:%S")
         click_button "Save"
         @record = RailsAdmin::AbstractModel.new("FieldTest").first
-        @record.timestamp_field.should eql(::DateTime.parse(@time.to_s))
+        @record.timestamp_field.to_s(:rfc822).should eql(@time.to_s(:rfc822))
       end
     end
 
@@ -698,7 +741,7 @@ describe "RailsAdmin Config DSL Edit Section" do
       end
     end
   end
-  
+
   describe 'bindings' do
     it 'should be present at creation time' do
       RailsAdmin.config do |config|
@@ -711,7 +754,7 @@ describe "RailsAdmin Config DSL Edit Section" do
           end
         end
       end
-      
+
       visit new_path(:model_name => 'category')
       should have_no_css('#category_parent_category_id')
       click_button 'Save'
@@ -721,8 +764,8 @@ describe "RailsAdmin Config DSL Edit Section" do
       should have_content('Category successfully updated')
     end
   end
-  
-  describe 'nested form' do 
+
+  describe 'nested form' do
 
     it 'should work' do
       RailsAdmin::Config.excluded_models = [RelTest]
@@ -734,15 +777,13 @@ describe "RailsAdmin Config DSL Edit Section" do
       @record.nested_field_tests = [NestedFieldTest.create!(:title => 'title 1'), NestedFieldTest.create!(:title => 'title 2')]
       visit edit_path(:model_name => "field_test", :id => @record.id)
       fill_in "field_test_nested_field_tests_attributes_0_title", :with => 'nested field test title 1 edited'
-      page.find('#field_test_comment_attributes__destroy').set('true')
       page.find('#field_test_nested_field_tests_attributes_1__destroy').set('true')
       click_button "Save"
       @record.reload
-      @record.comment.should == nil
       @record.nested_field_tests.length.should == 1
       @record.nested_field_tests[0].title.should == 'nested field test title 1 edited'
     end
-    
+
     it 'should set bindings[:object] to nested object' do
       RailsAdmin::Config.excluded_models = [RelTest]
       RailsAdmin.config(NestedFieldTest) do
@@ -759,19 +800,19 @@ describe "RailsAdmin Config DSL Edit Section" do
       visit edit_path(:model_name => "field_test", :id => @record.id)
       find('#field_test_nested_field_tests_attributes_0_title_field').should have_content('NestedFieldTest')
     end
-    
+
     it 'should be desactivable' do
       RailsAdmin::Config.excluded_models = [RelTest]
-      
+
       visit new_path(:model_name => "field_test")
-      should have_selector('.label.add_nested_fields')
+      should have_selector('.add_nested_fields')
       RailsAdmin.config(FieldTest) do
         configure :nested_field_tests do
           nested_form false
         end
       end
       visit new_path(:model_name => "field_test")
-      should have_no_selector('.label.add_nested_fields')
+      should have_no_selector('.add_nested_fields')
     end
   end
 

data/spec/{requests → integration}/config/list/rails_admin_config_list_spec.rb

@@ -421,7 +421,7 @@ describe "RailsAdmin Config DSL List Section" do
     let(:player_names_by_date){ players.sort_by{|p| p[:created_at]}.map{|p| p[:name]} }
     let(:league_names_by_date){ leagues.sort_by{|l| l[:created_at]}.map{|l| l[:name]} }
 
-    before(:each) { @players = RailsAdmin::AbstractModel.new("Player").create(players) }
+    before(:each) { @players = Player.create(players) }
 
     context "should be configurable" do
       it "globaly" do
@@ -453,7 +453,7 @@ describe "RailsAdmin Config DSL List Section" do
       end
 
       it "globaly and overrideable per model" do
-        RailsAdmin::AbstractModel.new("League").create(leagues)
+        League.create(leagues)
 
         RailsAdmin::Config.models do
           list do

data/spec/{requests → integration}/config/show/rails_admin_config_show_spec.rb

@@ -13,6 +13,26 @@ describe "RailsAdmin Config DSL Show Section" do
     visit show_path(:model_name => "team", :id => team.id)
   end
 
+  describe "compact_show_view" do
+
+    it 'should hide empty fields in show view by default' do
+      @player = FactoryGirl.create :player
+      visit show_path(:model_name => "league", :id => @player.id)
+      should_not have_css(".born_on_field")
+    end
+
+
+    it 'should be disactivable' do
+      RailsAdmin.config do |c|
+        c.compact_show_view = false
+      end
+
+      @player = FactoryGirl.create :player
+      visit show_path(:model_name => "player", :id => @player.id)
+      should have_css(".born_on_field")
+    end
+  end
+
   describe "css hooks" do
     it "should be present" do
       do_request

data/spec/{requests → integration}/history/rails_admin_history_spec.rb

@@ -18,7 +18,7 @@ describe "RailsAdmin History" do
       histories.total_count.should == 30
       histories.count.should == 20
     end
-    
+
     it "should respect RailsAdmin::Config.default_items_per_page" do
       RailsAdmin.config.default_items_per_page = 15
       histories = RailsAdmin::History.history_for_model @model, nil, false, false, false, nil
@@ -45,13 +45,13 @@ describe "RailsAdmin History" do
 
       context "with a lot of histories" do
         before :each do
-          player = @model.create(:team_id => -1, :number => -1, :name => "Player 1")
+          player = Player.create(:team_id => -1, :number => -1, :name => "Player 1")
           101.times do |i|
             player.number = i
             RailsAdmin::History.create_history_item "change #{i}", player, @model, nil
           end
         end
-        
+
         it 'should get latest ones' do
           RailsAdmin::History.latest.count.should == 100
         end

data/spec/{requests → integration}/rails_admin_spec.rb

@@ -39,9 +39,8 @@ describe "RailsAdmin" do
       should have_selector('head script[src^="/assets/rails_admin/rails_admin.js"]')
     end
   end
-  
+
   describe 'hidden fields with default values' do
-    
     it "should show up with default value, hidden" do
       RailsAdmin.config Player do
         include_all_fields
@@ -53,16 +52,16 @@ describe "RailsAdmin" do
           end
         end
       end
-      
+
       visit new_path(:model_name => "player")
       should have_selector("#player_name[type=hidden][value='username@example.com']")
       should_not have_selector("#player_name[type=hidden][value='toto@example.com']")
     end
   end
-  
-  describe '_current_user should be accessible from the list view' do # https://github.com/sferik/rails_admin/issues/549
-    
-    it 'should work' do
+
+  describe '_current_user' do # https://github.com/sferik/rails_admin/issues/549
+
+    it 'should be accessible from the list view' do
       RailsAdmin.config Player do
         list do
           field :name do
@@ -70,7 +69,7 @@ describe "RailsAdmin" do
               bindings[:view]._current_user.email == 'username@example.com'
             end
           end
-          
+
           field :team do
             visible do
               bindings[:view]._current_user.email == 'foo@example.com'
@@ -78,7 +77,7 @@ describe "RailsAdmin" do
           end
         end
       end
-      
+
       visit index_path(:model_name => "player")
       should have_selector(".header.name_field")
       should_not have_selector(".header.team_field")
@@ -111,36 +110,4 @@ describe "RailsAdmin" do
     end
   end
 
-  describe "model whitelist:" do
-
-    it 'should only use included models' do
-      RailsAdmin.config.included_models = [Team, League]
-      RailsAdmin::AbstractModel.all.map(&:model).should == [League, Team] #it gets sorted
-    end
-
-    it 'should not restrict models if included_models is left empty' do
-      RailsAdmin.config.included_models = []
-      RailsAdmin::AbstractModel.all.map(&:model).should include(Team, League)
-    end
-
-    it 'should further remove excluded models (whitelist - blacklist)' do
-      RailsAdmin.config.excluded_models = [Team]
-      RailsAdmin.config.included_models = [Team, League]
-      RailsAdmin::AbstractModel.all.map(&:model).should == [League]
-    end
-
-    it 'should always exclude history' do
-      RailsAdmin::AbstractModel.all.map(&:model).should_not include(RailsAdmin::History)
-    end
-
-    it 'excluded? returns true for any model not on the list' do
-      RailsAdmin.config.included_models = [Team, League]
-
-      team_config = RailsAdmin.config(RailsAdmin::AbstractModel.new('Team'))
-      fan_config = RailsAdmin.config(RailsAdmin::AbstractModel.new('Fan'))
-
-      fan_config.should be_excluded
-      team_config.should_not be_excluded
-    end
-  end
 end

data/spec/spec_helper.rb

@@ -1,20 +1,12 @@
 # Configure Rails Envinronment
 ENV["RAILS_ENV"] = "test"
-
-require 'simplecov'
-SimpleCov.start 'rails'
-
 ENV['SKIP_RAILS_ADMIN_INITIALIZER'] = 'true'
 require File.expand_path('../dummy_app/config/environment', __FILE__)
 
-require 'generator_spec/test_case'
-require 'generators/rails_admin/install_generator'
-require 'generators/rails_admin/uninstall_generator'
 require 'rspec/rails'
 require 'factory_girl'
 require 'factories'
 require 'database_helpers'
-require 'generator_helpers'
 
 ActionMailer::Base.delivery_method = :test
 ActionMailer::Base.perform_deliveries = true
@@ -23,13 +15,9 @@ ActionMailer::Base.default_url_options[:host] = "example.com"
 Rails.backtrace_cleaner.remove_silencers!
 
 include DatabaseHelpers
-# Run any available migration
-puts 'Setting up database...'
 drop_all_tables
 migrate_database
 ENV['SKIP_RAILS_ADMIN_INITIALIZER'] = 'false'
-# Load support files
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each{|f| require f}
 
 # Don't need passwords in test DB to be secure, but we would like 'em to be
 # fast -- and the stretches mechanism is intended to make passwords
@@ -45,6 +33,7 @@ module Devise
     end
   end
 end
+
 Devise.setup do |config|
   config.stretches = 0
 end
@@ -54,38 +43,31 @@ RSpec.configure do |config|
 
   config.include RSpec::Matchers
   config.include DatabaseHelpers
-  config.include GeneratorHelpers
   config.include RailsAdmin::Engine.routes.url_helpers
 
   config.include Warden::Test::Helpers
 
   config.before(:each) do
-    RailsAdmin.reset
+    RailsAdmin::Config.reset
+    RailsAdmin::AbstractModel.reset
     RailsAdmin::Config.excluded_models = [RelTest, FieldTest, Category]
     RailsAdmin::Config.audit_with :history
-    RailsAdmin::AbstractModel.all_models = nil
-    RailsAdmin::AbstractModel.all_abstract_models = nil
-    RailsAdmin::AbstractModel.new("Category").destroy_all!
-    RailsAdmin::AbstractModel.new("Division").destroy_all!
-    RailsAdmin::AbstractModel.new("Draft").destroy_all!
-    RailsAdmin::AbstractModel.new("Fan").destroy_all!
-    RailsAdmin::AbstractModel.new("League").destroy_all!
-    RailsAdmin::AbstractModel.new("Player").destroy_all!
-    RailsAdmin::AbstractModel.new("Team").destroy_all!
-    RailsAdmin::AbstractModel.new("User").destroy_all!
-    RailsAdmin::AbstractModel.new("Foo::Bar").destroy_all!
-    RailsAdmin::AbstractModel.new("FieldTest").destroy_all!
-
-    user = RailsAdmin::AbstractModel.new("User").create(
+    Category.delete_all
+    Division.delete_all
+    Draft.delete_all
+    Fan.delete_all
+    League.delete_all
+    Player.delete_all
+    Team.delete_all
+    User.delete_all
+    FieldTest.delete_all
+    login_as User.create(
       :email => "username@example.com",
       :password => "password"
     )
-
-    login_as user
   end
 
   config.after(:each) do
-    
     Warden.test_reset!
   end
 end

data/spec/{lib → unit/adapters/active_record}/abstract_object_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe "AbstractObject" do
   describe "proxy" do
     let(:object) { mock("object") }
-    let(:abstract_object) { RailsAdmin::AbstractObject.new(object) }
+    let(:abstract_object) { RailsAdmin::Adapters::ActiveRecord::AbstractObject.new(object) }
 
     it "should act like a proxy" do
       object.should_receive(:method_call)
@@ -13,7 +13,7 @@ describe "AbstractObject" do
 
   describe "create" do
     let(:player) { Player.new }
-    let(:object) { RailsAdmin::AbstractObject.new player }
+    let(:object) { RailsAdmin::Adapters::ActiveRecord::AbstractObject.new player }
     let(:name) { "Stefan Kiszonka" }
     let(:number) { 87 }
     let(:position) { "Fifth baseman" }
@@ -59,7 +59,7 @@ describe "AbstractObject" do
 
     describe "a record with has_many associations" do
       let(:league) { League.new }
-      let(:object) { RailsAdmin::AbstractObject.new league }
+      let(:object) { RailsAdmin::Adapters::ActiveRecord::AbstractObject.new league }
       let(:name) { "Awesome League" }
       let(:teams) { [Factory(:team)] }
       let(:divisions) { [Division.create!(:name => 'div 1', :league => League.create!(:name => 'north')), Division.create!(:name => 'div 2', :league => League.create!(:name => 'south'))] }
@@ -82,7 +82,7 @@ describe "AbstractObject" do
       let(:name) { "Stefan Koza" }
       let(:suspended) { true }
       let(:player) { Factory :player, :suspended => true, :name => name, :draft => Factory(:draft) }
-      let(:object) { RailsAdmin::AbstractObject.new player }
+      let(:object) { RailsAdmin::Adapters::ActiveRecord::AbstractObject.new player }
       let(:new_team) { Factory :team }
       let(:new_suspended) { false }
       let(:new_draft) { nil }
@@ -106,7 +106,7 @@ describe "AbstractObject" do
 
   describe "destroy" do
     let(:player) { Factory :player }
-    let(:object) { RailsAdmin::AbstractObject.new player }
+    let(:object) { RailsAdmin::Adapters::ActiveRecord::AbstractObject.new player }
 
     before do
       object.destroy
@@ -116,4 +116,16 @@ describe "AbstractObject" do
       Player.exists?(player.id).should == false
     end
   end
+
+  describe "object_label_method" do
+    it 'should be configurable' do
+      RailsAdmin.config League do
+        object_label_method { :custom_name }
+      end
+
+      @league = FactoryGirl.create :league
+
+      RailsAdmin.config('League').with(:object => @league).object_label.should == "League '#{@league.name}'"
+    end
+  end
 end

data/spec/unit/adapters/active_record_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+require 'rails_admin/adapters/active_record'
+
+
+describe RailsAdmin::Adapters::ActiveRecord do
+
+  before :all do
+
+    class ARBlog < ActiveRecord::Base
+      has_many :a_r_posts
+      has_many :a_r_comments, :as => :commentable
+    end
+
+    class ARPost < ActiveRecord::Base
+      belongs_to :a_r_blog
+      has_and_belongs_to_many :a_r_categories
+      has_many :a_r_comments, :as => :commentable
+    end
+
+    class ARCategory < ActiveRecord::Base
+      has_and_belongs_to_many :a_r_posts
+    end
+
+    class ARUser < ActiveRecord::Base
+      has_one :a_r_profile
+    end
+
+    class ARProfile < ActiveRecord::Base
+      belongs_to :a_r_user
+    end
+
+    class ARComment < ActiveRecord::Base
+      belongs_to :commentable, :polymorphic => true
+    end
+
+    @blog = RailsAdmin::AbstractModel.new(ARBlog)
+    @post = RailsAdmin::AbstractModel.new(ARPost)
+    @category = RailsAdmin::AbstractModel.new(ARCategory)
+    @user = RailsAdmin::AbstractModel.new(ARUser)
+    @profile = RailsAdmin::AbstractModel.new(ARProfile)
+    @comment = RailsAdmin::AbstractModel.new(ARComment)
+  end
+
+  describe '#associations' do
+    it 'lists associations' do
+      @post.associations.map{|a|a[:name].to_s}.sort.should == ['a_r_blog', 'a_r_categories', 'a_r_comments']
+    end
+
+    it 'list associations types in supported [:belongs_to, :has_and_belongs_to_many, :has_many, :has_one]' do
+      (@post.associations + @blog.associations + @user.associations).map{|a|a[:type]}.uniq.map(&:to_s).sort.should == ['belongs_to', 'has_and_belongs_to_many', 'has_many', 'has_one']
+    end
+  end
+end