activeadmin 0.3.4 → 0.4.0

data/lib/generators/active_admin/install/templates/dashboards.rb

@@ -34,5 +34,11 @@ ActiveAdmin::Dashboards.build do
   #   section "Recent User", :priority => 1
   #
   # Will render the "Recent Users" then the "Recent Posts" sections on the dashboard.
+  
+  # == Conditionally Display
+  # Provide a method name or Proc object to conditionally render a section at run time.
+  #
+  # section "Membership Summary", :if => :memberships_enabled?
+  # section "Membership Summary", :if => Proc.new { current_admin_user.account.memberships.any? }
 
 end

data/script/use_rails

@@ -3,6 +3,8 @@
 # Switches the development environment to use the given
 # version of rails. Caches the Gemfile.locks so that 
 # switching it very fast.
+#
+require File.expand_path("../../spec/support/detect_rails_version", __FILE__)
 
 def cmd(command)
   puts command
@@ -34,13 +36,18 @@ if File.exists?(gem_lock_file) && ARGV.include?('--clobber')
   cmd "rm #{gem_lock_file}"
 end
 
-unless File.exists?(gem_lock_file)
-  # Generate it
+write_rails_version(version)
+
+# Ensure that bundler installs
+ENV['RUBYOPT'] = ''
+
+if File.exists?(gem_lock_file)
+  cmd("rm Gemfile.lock") if file_or_symlink?("Gemfile.lock")
+  cmd("ln -s #{gem_lock_file} Gemfile.lock")
+  cmd("bundle")
+else
   cmd "rm Gemfile.lock" if file_or_symlink?("Gemfile.lock")
-  cmd "export RAILS=#{version} && bundle install"
+  cmd "bundle install"
   cmd "mv Gemfile.lock #{gem_lock_file}"
+  cmd("ln -s #{gem_lock_file} Gemfile.lock")
 end
-
-cmd("rm Gemfile.lock") if file_or_symlink?("Gemfile.lock")
-cmd("ln -s #{gem_lock_file} Gemfile.lock")
-cmd("bundle")

data/spec/spec_helper.rb

@@ -1,16 +1,4 @@
-$LOAD_PATH.unshift(File.dirname(__FILE__))
-$LOAD_PATH << File.expand_path('../support', __FILE__)
-
-ENV['BUNDLE_GEMFILE'] = File.expand_path('../../Gemfile', __FILE__)
-
-require 'detect_rails_version'
-ENV['RAILS'] = detect_rails_version
-
-require "bundler"
-Bundler.setup
-
-require 'shoulda/active_record'
-include Shoulda::ActiveRecord::Macros
+require File.expand_path("../spec_helper_without_rails", __FILE__)
 
 module ActiveAdminIntegrationSpecHelper
   extend self
@@ -90,10 +78,17 @@ module ActiveAdminIntegrationSpecHelper
   end  
   alias_method :action_view, :mock_action_view
 
+  # A mock resource to register
+  class MockResource
+  end
+
 end
 
 ENV['RAILS_ENV'] = 'test'
-ENV['RAILS_ROOT'] = File.expand_path("../rails/rails-#{ENV["RAILS"]}", __FILE__)
+
+require 'detect_rails_version'
+rails_version = detect_rails_version
+ENV['RAILS_ROOT'] = File.expand_path("../rails/rails-#{rails_version}", __FILE__)
 
 # Create the test app if it doesn't exists
 unless File.exists?(ENV['RAILS_ROOT'])

data/spec/spec_helper_without_rails.rb

@@ -0,0 +1,10 @@
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+$LOAD_PATH << File.expand_path('../support', __FILE__)
+
+ENV['BUNDLE_GEMFILE'] = File.expand_path('../../Gemfile', __FILE__)
+
+require 'detect_rails_version'
+ENV['RAILS'] = detect_rails_version
+
+require "bundler"
+Bundler.setup

data/spec/support/detect_rails_version.rb

@@ -2,9 +2,28 @@
 #
 # You can pass it in as an ENV variable or it will use
 # the current Gemfile.lock to find it
+
+unless defined?(RAILS_VERSION_FILE)
+  RAILS_VERSION_FILE = File.expand_path("../../../.rails-version", __FILE__)
+end
+
+unless defined?(DEFAULT_RAILS_VERSION)
+  DEFAULT_RAILS_VERSION = "3.2.0"
+end
+
 def detect_rails_version
-  return nil unless (File.exists?("Gemfile.lock") || File.symlink?("Gemfile.lock"))
+  detected_version = if File.exists?(RAILS_VERSION_FILE)
+    version = File.read(RAILS_VERSION_FILE).chomp.strip
+    version != "" ? version : DEFAULT_RAILS_VERSION
+  else
+    DEFAULT_RAILS_VERSION
+  end
+
+  puts "Detected Rails: #{detected_version}" if ENV['DEBUG']
+
+  detected_version
+end
 
-  File.read("Gemfile.lock").match(/^\W*rails \(([a-z\d.]*)\)/)
-  return $1
+def write_rails_version(version)
+  File.open(RAILS_VERSION_FILE, "w+"){|f| f << version }
 end

data/spec/support/integration_example_group.rb

@@ -13,14 +13,12 @@ module RSpec
       include Capybara::DSL
       include RSpec::Matchers
 
-      module InstanceMethods
-        def app
-          ::Rails.application
-        end
+      def app
+        ::Rails.application
+      end
 
-        def last_response
-          page
-        end
+      def last_response
+        page
       end
 
       included do

data/spec/support/rails_template.rb

@@ -2,8 +2,11 @@
 
 # Create a cucumber database and environment
 copy_file File.expand_path('../templates/cucumber.rb', __FILE__), "config/environments/cucumber.rb"
+copy_file File.expand_path('../templates/cucumber_with_reloading.rb', __FILE__), "config/environments/cucumber_with_reloading.rb"
+
 gsub_file 'config/database.yml', /^test:.*\n/, "test: &test\n"
 gsub_file 'config/database.yml', /\z/, "\ncucumber:\n  <<: *test\n  database: db/cucumber.sqlite3"
+gsub_file 'config/database.yml', /\z/, "\ncucumber_with_reloading:\n  <<: *test\n  database: db/cucumber.sqlite3"
 
 # Generate some test models
 generate :model, "post title:string body:text published_at:datetime author_id:integer category_id:integer"
@@ -22,6 +25,14 @@ run "rm -r test"
 run "rm -r spec"
 
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+
+# we need this routing path, named "logout_path", for testing
+route <<-EOS
+  devise_scope :user do
+    match '/admin/logout' => 'active_admin/devise/sessions#destroy', :as => :logout
+  end
+EOS
+
 generate :'active_admin:install'
 
 # Setup a root path for devise

data/spec/support/rails_template_with_data.rb

@@ -6,6 +6,27 @@ apply File.expand_path("../rails_template.rb", __FILE__)
   generate :'active_admin:resource', type
 end
 
+scopes = <<-EOF
+  scope :all, :default => true
+
+  scope :drafts do |posts|
+    posts.where(["published_at IS NULL"])
+  end
+
+  scope :scheduled do |posts|
+    posts.where(["posts.published_at IS NOT NULL AND posts.published_at > ?", Time.now.utc])
+  end
+
+  scope :published do |posts|
+    posts.where(["posts.published_at IS NOT NULL AND posts.published_at < ?", Time.now.utc])
+  end
+
+  scope :my_posts do |posts|
+    posts.where(:author_id => current_admin_user.id)
+  end
+EOF
+inject_into_file 'app/admin/posts.rb', scopes , :after => "ActiveAdmin.register Post do\n"
+
 # Setup some default data
 append_file "db/seeds.rb", <<-EOF
   users = ["Jimi Hendrix", "Jimmy Page", "Yngwie Malmsteen", "Eric Clapton", "Kirk Hammett"].collect do |name|
@@ -20,12 +41,16 @@ append_file "db/seeds.rb", <<-EOF
     Category.create! :name => name
   end
 
+  published_at_values = [Time.now.utc - 5.days, Time.now.utc - 1.day, nil, Time.now.utc + 3.days]
+
   1_000.times do |i|
     user = users[i % users.size]
     cat = categories[i % categories.size]
+    published_at = published_at_values[i % published_at_values.size]
     Post.create :title => "Blog Post \#{i}",
                 :body => "Blog post \#{i} is written by \#{user.username} about \#{cat.name}",
                 :category => cat,
+                :published_at => published_at,
                 :author => user
   end
 EOF

data/spec/support/templates/cucumber_with_reloading.rb

@@ -0,0 +1,5 @@
+require File.expand_path('config/environments/cucumber', Rails.root)
+
+Rails.application.class.configure do
+  config.cache_classes = false
+end

data/spec/unit/active_admin_spec.rb

@@ -14,4 +14,12 @@ describe ActiveAdmin do
       ActiveAdmin.default_namespace
     end
   end
+
+  %w(register register_page unload! load! routes).each do |method|
+    it "delegates ##{method} to application" do
+      ActiveAdmin.application.should_receive(method)
+
+      ActiveAdmin.send(method)
+    end
+  end
 end

data/spec/unit/application_spec.rb

@@ -39,6 +39,15 @@ describe ActiveAdmin::Application do
     application.site_title_link = "http://www.mygreatsite.com"
     application.site_title_link.should == "http://www.mygreatsite.com"
   end
+  
+  it "should store the site's title image" do
+    application.site_title_image.should == ""
+  end
+  
+  it "should set the site's title image" do
+    application.site_title_image = "http://railscasts.com/assets/episodes/stills/284-active-admin.png?1316476106"
+    application.site_title_image.should == "http://railscasts.com/assets/episodes/stills/284-active-admin.png?1316476106"
+  end
 
   it "should have a view factory" do
     application.view_factory.should be_an_instance_of(ActiveAdmin::ViewFactory)
@@ -48,8 +57,8 @@ describe ActiveAdmin::Application do
     application.admin_notes.should be_nil
   end
 
-  it "should have admin notes in admin namespace by default" do
-    application.allow_comments_in.should == [:admin]
+  it "should allow comments by default" do
+    application.allow_comments.should == true
   end
 
   describe "authentication settings" do
@@ -84,4 +93,41 @@ describe ActiveAdmin::Application do
     end
   end
 
+  describe "adding an inheritable setting" do
+
+    it "should add a setting to Application and Namespace" do
+      ActiveAdmin::Application.inheritable_setting :inheritable_setting, "inheritable_setting"
+      app = ActiveAdmin::Application.new
+      app.inheritable_setting.should == "inheritable_setting"
+      ns = ActiveAdmin::Namespace.new(app, :admin)
+      ns.inheritable_setting.should == "inheritable_setting"
+    end
+
+  end
+
+  describe "#namespace" do
+    it "should yield a new namespace" do
+      application.namespace :new_namespace do |ns|
+        ns.name.should == :new_namespace
+      end
+    end
+
+    it "should return an instantiated namespace" do
+      admin = application.find_or_create_namespace :admin
+      application.namespace :admin do |ns|
+        ns.should == admin
+      end
+    end
+  end
+
+  describe "#register_page" do
+    it "finds or create the namespace and register the page to it" do
+      namespace = mock
+      application.should_receive(:find_or_create_namespace).with("public").and_return namespace
+      namespace.should_receive(:register_page).with("My Page", {:namespace => "public"})
+
+      application.register_page("My Page", :namespace => "public")
+    end
+  end
+
 end

data/spec/unit/arbre/context_spec.rb

@@ -27,7 +27,7 @@ describe Arbre::Context do
   end
 
   it "should use a cached version of the HTML for method delegation" do
-    current_dom_context.should_receive(:to_html).once.and_return("<h1>札幌市北区</h1>")
+    current_dom_context.should_receive(:to_s).once.and_return("<h1>札幌市北区</h1>")
     current_dom_context.index('<').should == 0
     current_dom_context.index('<').should == 0
   end

data/spec/unit/arbre/html/element_finder_methods_spec.rb

@@ -49,6 +49,62 @@ describe Arbre::HTML::Element, "Finder Methods" do
     end
   end
 
-  describe "finding an element by id"
-  describe "finding an element by a class name"
+  #TODO: describe "finding an element by id"
+  
+  describe "finding an element by a class name" do
+    it "should return 0 when no elements exist" do
+      div.get_elements_by_class_name("my_class").size.should == 0
+    end
+    it "should return a child element" do
+      html = div do
+        div :class => "some_class"
+        div :class => "my_class"
+      end
+      elements = html.get_elements_by_class_name("my_class")
+      elements.size.should == 1
+      elements[0].should be_instance_of(Arbre::HTML::Div)
+    end
+
+    it "should return multple child elements" do
+      html = div do
+        div :class => "some_class"
+        div :class => "my_class"
+        div :class => "my_class"
+      end
+      elements = html.get_elements_by_class_name("my_class")
+      elements.size.should == 2
+      elements[0].should be_instance_of(Arbre::HTML::Div)
+      elements[1].should be_instance_of(Arbre::HTML::Div)
+    end
+    
+    it "should return elements that match one of several classes" do
+      html = div do
+        div :class => "some_class this_class"
+        div :class => "some_class"
+        div :class => "other_class"
+        
+      end
+      elements = html.get_elements_by_class_name("this_class")
+      elements.size.should == 1
+      elements[0].should be_instance_of(Arbre::HTML::Div)
+    end
+    
+    #TODO: find children's children by class name
+    
+    #it "should return elements that are nested" do
+    #  html = div do
+    #    div :class => "my_class"
+    #    div :class => "some_class" do
+    #      ul :class => "other_class" do
+    #        li :class => "special_awesome_class"
+    #      end
+    #    end
+    #  end
+    #  
+    #  elements = html.get_elements_by_class_name("special_awesome_class")
+    #
+    #  elements.size.should == 1
+    #  elements[0].should be_instance_of(Arbre::HTML::Li)
+    #end
+  end
 end

data/spec/unit/arbre/html/element_spec.rb

@@ -73,11 +73,24 @@ describe Arbre::HTML::Element do
       let(:child){ "Hello World" }
       it "should add as a TextNode" do
         element.children.first.should be_instance_of(Arbre::HTML::TextNode)
-        element.children.first.to_html.should == "Hello World"
+        element.children.first.to_s.should == "Hello World"
       end
     end
   end
 
+  describe "#children?" do
+
+    it "returns true when a child has been added" do
+      element.add_child Arbre::HTML::Element.new
+      element.children?.should == true
+    end
+
+    it "returns false when no children" do
+      element.children?.should == false
+    end
+
+  end
+
   describe "setting the content" do
 
     context "when a string" do
@@ -90,13 +103,13 @@ describe Arbre::HTML::Element do
       end
 
       it "should add the string as a child" do
-        element.children.first.to_html.should == "Goodbye"
+        element.children.first.to_s.should == "Goodbye"
       end
 
       it "should html escape the string" do
         string = "Goodbye <br />"
         element.content = string
-        element.content.to_html.should == "Goodbye &lt;br /&gt;"
+        element.to_s.should == "Goodbye &lt;br /&gt;"
       end
     end
 
@@ -146,8 +159,8 @@ describe Arbre::HTML::Element do
 
   describe "rendering to html" do
     it "should render the children collection" do
-      element.children.should_receive(:to_html).and_return("content")
-      element.to_html.should == "content"
+      element.children.should_receive(:to_s).and_return("content")
+      element.to_s.should == "content"
     end
   end
 

data/spec/unit/arbre/html/tag_attributes_spec.rb

@@ -14,7 +14,7 @@ describe Arbre::HTML::Tag, "Attributes" do
     end
 
     it "should render the attributes to html" do
-      tag.to_html.should == <<-HTML
+      tag.to_s.should == <<-HTML
 <tag id="my_id"></tag>
 HTML
     end
@@ -47,13 +47,13 @@ HTML
   describe "rendering attributes" do
     it "should html safe the attribute values" do
       tag.set_attribute(:class, "\">bad things!")
-      tag.to_html.should == <<-HTML
+      tag.to_s.should == <<-HTML
 <tag class="&quot;&gt;bad things!"></tag>
 HTML
     end
     it "should should escape the attribute names" do
       tag.set_attribute(">bad", "things")
-      tag.to_html.should == <<-HTML
+      tag.to_s.should == <<-HTML
 <tag &gt;bad="things"></tag>
 HTML
     end