activeadmin 0.1.1 → 0.2.0

data/spec/spec_helper.rb

@@ -1,21 +1,34 @@
 $LOAD_PATH.unshift(File.dirname(__FILE__))
+$LOAD_PATH << File.expand_path('../support', __FILE__)
+
 ENV['BUNDLE_GEMFILE'] = File.expand_path('../../Gemfile', __FILE__)
 
 require 'rubygems'
 require "bundler"
 Bundler.setup
 
+require 'shoulda/active_record'
+include Shoulda::ActiveRecord::Macros
+
 # Setup autoloading of ActiveAdmin and the load path
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 autoload :ActiveAdmin, 'active_admin'
 
-module ActiveAdminIntegrationSpecHelper
 
+module ActiveAdminIntegrationSpecHelper
   extend self
 
   def load_defaults!
-    ActiveAdmin.register Post
-    ActiveAdmin.register Category
+    ActiveAdmin.unload!
+    ActiveAdmin.load!
+    ActiveAdmin.register(Category)
+    ActiveAdmin.register(User)
+    ActiveAdmin.register(Post){ belongs_to :user, :optional => true }
+    reload_menus!
+  end
+
+  def reload_menus!
+    ActiveAdmin.namespaces.values.each{|n| n.load_menu! }
   end
 
   # Sometimes we need to reload the routes within
@@ -24,6 +37,21 @@ module ActiveAdminIntegrationSpecHelper
     Rails.application.reload_routes!
   end
 
+  # Helper method to load resources and ensure that Active Admin is
+  # setup with the new configurations.
+  #
+  # Eg:
+  #   load_resources do
+  #     ActiveAdmin.regiser(Post)
+  #   end
+  #
+  def load_resources
+    ActiveAdmin.unload!
+    yield
+    reload_menus!
+    reload_routes!
+  end
+
   # Sets up a describe block where you can render controller 
   # actions. Uses the Admin::PostsController as the subject
   # for the describe block
@@ -31,7 +59,17 @@ module ActiveAdminIntegrationSpecHelper
     describe *args do
       include RSpec::Rails::ControllerExampleGroup
       render_views  
-      metadata[:behaviour][:describes] = Admin::PostsController
+      metadata[:behaviour][:describes] = ActiveAdmin.namespaces[:admin].resources['Post'].controller
+      module_eval &block
+    end
+  end
+
+  # Setup a describe block which uses capybara and rails integration
+  # test methods.
+  def describe_with_capybara(*args, &block)
+    require 'integration_example_group'
+    describe *args do
+      include RSpec::Rails::IntegrationExampleGroup
       module_eval &block
     end
   end
@@ -41,6 +79,7 @@ module ActiveAdminIntegrationSpecHelper
     controller = ActionView::TestCase::TestController.new
     ActionView::Base.send :include, ActionView::Helpers
     ActionView::Base.send :include, ActiveAdmin::ViewHelpers
+    ActionView::Base.send :include, Rails.application.routes.url_helpers
     ActionView::Base.new(ActionController::Base.view_paths, assigns, controller)
   end  
   alias_method :action_view, :mock_action_view
@@ -50,63 +89,63 @@ end
 ENV['RAILS'] ||= '3.0.0'
 ENV['RAILS_ENV'] = 'test'
 
-if ENV['RAILS'] == '3.0.0'
-  ENV['RAILS_ROOT'] = File.expand_path('../rails/rails-3.0.0', __FILE__)
+ENV['RAILS_ROOT'] = File.expand_path("../rails/rails-#{ENV["RAILS"]}", __FILE__)
 
-  # Create the test app if it doesn't exists
-  unless File.exists?(ENV['RAILS_ROOT'])
-    system 'rake setup'  
-  end
+# Create the test app if it doesn't exists
+unless File.exists?(ENV['RAILS_ROOT'])
+  system 'rake setup'
+end
 
-  require ENV['RAILS_ROOT'] + '/config/environment'
-  require 'rspec/rails'
+require ENV['RAILS_ROOT'] + '/config/environment'
+require 'rspec/rails'
 
-  # Setup Some Admin stuff for us to play with
-  include ActiveAdminIntegrationSpecHelper
-  load_defaults!
-  reload_routes!
+# Setup Some Admin stuff for us to play with
+include ActiveAdminIntegrationSpecHelper
+load_defaults!
+reload_routes!
 
-  # Don't add asset cache timestamps. Makes it easy to integration
-  # test for the presence of an asset file
-  ENV["RAILS_ASSET_ID"] = ''
+# Disabling authentication in specs so that we don't have to worry about
+# it allover the place
+ActiveAdmin.authentication_method = false
+ActiveAdmin.current_user_method = false
 
-  Rspec.configure do |config|
-    config.use_transactional_fixtures = true
-    config.use_instantiated_fixtures = false
-  end
+# Don't add asset cache timestamps. Makes it easy to integration
+# test for the presence of an asset file
+ENV["RAILS_ASSET_ID"] = ''
 
-  # Ensure this is defined for Ruby 1.8
-  module MiniTest; class Assertion < Exception; end; end
+Rspec.configure do |config|
+  config.use_transactional_fixtures = true
+  config.use_instantiated_fixtures = false
+end
+
+# Ensure this is defined for Ruby 1.8
+module MiniTest; class Assertion < Exception; end; end
 
-  Rspec::Matchers.define :have_tag do |*args|
+Rspec::Matchers.define :have_tag do |*args|
 
-    match_unless_raises Test::Unit::AssertionFailedError do |response|
-      tag = args.shift
-      content = args.first.is_a?(Hash) ? nil : args.shift
-      
-      options = {
-        :tag => tag
-      }.merge(args[0] || {})
-      
-      options[:content] = content if content
+  match_unless_raises Test::Unit::AssertionFailedError do |response|
+    tag = args.shift
+    content = args.first.is_a?(Hash) ? nil : args.shift
+    
+    options = {
+      :tag => tag.to_s
+    }.merge(args[0] || {})
+    
+    options[:content] = content if content
 
+    begin
       begin
-        begin
-          assert_tag(options)
-        rescue NoMethodError
-          # We are not in a controller, so let's do the checking ourselves
-          doc = HTML::Document.new(response, false, false)
-          tag = doc.find(options)
-          assert tag, "expected tag, but no tag found matching #{options.inspect} in:\n#{response.inspect}"
-        end
-      # In Ruby 1.9, MiniTest::Assertion get's raised, so we'll
-      # handle raising a Test::Unit::AssertionFailedError
-      rescue MiniTest::Assertion => e
-        raise Test::Unit::AssertionFailedError, e.message
+        assert_tag(options)
+      rescue NoMethodError
+        # We are not in a controller, so let's do the checking ourselves
+        doc = HTML::Document.new(response, false, false)
+        tag = doc.find(options)
+        assert tag, "expected tag, but no tag found matching #{options.inspect} in:\n#{response.inspect}"
       end
+    # In Ruby 1.9, MiniTest::Assertion get's raised, so we'll
+    # handle raising a Test::Unit::AssertionFailedError
+    rescue MiniTest::Assertion => e
+      raise Test::Unit::AssertionFailedError, e.message
     end
   end
-
 end
-
-

data/spec/support/integration_example_group.rb

@@ -0,0 +1,37 @@
+require 'action_dispatch'
+require 'capybara/rails'
+require 'capybara/dsl'
+
+module RSpec
+  module Rails
+    module IntegrationExampleGroup
+      extend ActiveSupport::Concern
+
+      include ActionDispatch::Integration::Runner
+      include RSpec::Rails::TestUnitAssertionAdapter
+      include ActionDispatch::Assertions
+      include Capybara
+      include RSpec::Matchers
+
+      module InstanceMethods
+        def app
+          ::Rails.application
+        end
+
+        def last_response
+          page
+        end
+      end
+
+      included do
+        before do
+          @router = ::Rails.application.routes
+        end
+      end
+
+      RSpec.configure do |c|
+        c.include self, :example_group => { :file_path => /\bspec\/integration\// }
+      end
+    end
+  end
+end

data/spec/support/rails_template.rb

@@ -1,19 +1,31 @@
 # Rails template to build the sample app for specs
-generate :model, "post title:string body:text published_at:datetime author_id:integer"
-inject_into_file 'app/models/post.rb', "  belongs_to :author, :class_name => 'User'\n  accepts_nested_attributes_for :author\n", :after => "class Post < ActiveRecord::Base\n"
+
+# Create a cucumber database and environment
+copy_file File.expand_path('../templates/cucumber.rb', __FILE__), "config/environments/cucumber.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"
+
+# Generate some test models
+generate :model, "post title:string body:text published_at:datetime author_id:integer category_id:integer"
+inject_into_file 'app/models/post.rb', "  belongs_to :author, :class_name => 'User'\n  belongs_to :category\n  accepts_nested_attributes_for :author\n", :after => "class Post < ActiveRecord::Base\n"
 generate :model, "user first_name:string last_name:string username:string"
 inject_into_file 'app/models/user.rb', "  has_many :posts, :foreign_key => 'author_id'\n", :after => "class User < ActiveRecord::Base\n"
-generate :model, 'category name:string'
+generate :model, 'category name:string description:text'
+inject_into_file 'app/models/category.rb', "  has_many :posts\n", :after => "class Category < ActiveRecord::Base\n"
+
+# Add our local Active Admin to the load path
+inject_into_file "config/environment.rb", "\n$LOAD_PATH.unshift('#{File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'lib'))}')\nrequire \"active_admin\"\n", :after => "require File.expand_path('../application', __FILE__)"
 
 run "rm Gemfile"
 run "rm -r test"
 run "rm -r spec"
 
-rake "db:migrate"
-rake "db:test:prepare"
-
-# Install ActiveAdmin
-# This should happen last so that we don't have to worry about the
-# rake's blowing up due to not having active admin in their load paths
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 generate :'active_admin:install'
+
+# Setup a root path for devise
+route "root :to => 'admin/dashboard#index'"
+
+rake "db:migrate"
+rake "db:test:prepare"
+run "/usr/bin/env RAILS_ENV=cucumber rake db:migrate"

data/spec/support/rails_template_with_data.rb

@@ -0,0 +1,32 @@
+# Use the default
+apply File.expand_path("../rails_template.rb", __FILE__)
+
+# Register Active Admin controllers
+%w{ Post User Category }.each do |type|
+  generate :'active_admin:resource', type
+end
+
+# Setup some default data
+append_file "db/seeds.rb", <<-EOF
+  users = ["Jimi Hendrix", "Jimmy Page", "Yngwie Malmsteen", "Eric Clapton", "Kirk Hammett"].collect do |name|
+    first, last = name.split(" ")
+    User.create!  :first_name => first,
+                  :last_name => last,
+                  :username => [first,last].join('-').downcase
+  end
+
+  categories = ["Rock", "Pop Rock", "Alt-Country", "Blues", "Dub-Step"].collect do |name|
+    Category.create! :name => name
+  end
+
+  1_000.times do |i|
+    user = users[i % users.size]
+    cat = categories[i % categories.size]
+    Post.create :title => "Blog Post \#{i}",
+                :body => "Blog post \#{i} is written by \#{user.username} about \#{cat.name}",
+                :category => cat,
+                :author => user
+  end
+EOF
+
+rake 'db:seed'

data/spec/support/templates/cucumber.rb

@@ -0,0 +1,28 @@
+require File.expand_path('config/environments/test', Rails.root)
+
+Rails.application.class.configure do
+  config.cache_classes = false
+end
+
+# rails/railties/lib/rails/test_help.rb aborts if the environment is not 'test'. (Rails 3.0.0.beta3)
+# We can't run Cucumber/RSpec/Test_Unit tests in different environments then.
+#
+# For now, I patch StringInquirer so that Rails.env.test? returns true when Rails.env is 'test' or 'cucumber'
+#
+# https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/4458-rails-should-allow-test-to-run-in-cucumber-environment
+module ActiveSupport
+  class StringInquirer < String
+    def method_missing(method_name, *arguments)
+      if method_name.to_s[-1,1] == "?"
+        test_string = method_name.to_s[0..-2]
+        if test_string == 'test'
+          self == 'test' or self == 'cucumber'
+        else
+          self == test_string
+        end
+      else
+        super
+      end
+    end
+  end
+end

data/spec/unit/abstract_view_factory_spec.rb

@@ -0,0 +1,79 @@
+require 'spec_helper'
+
+require 'active_admin/abstract_view_factory'
+
+describe ActiveAdmin::AbstractViewFactory do
+  
+  let(:view_factory){ ActiveAdmin::AbstractViewFactory.new }
+  let(:mock_view){ Class.new }
+
+  describe "registering a new view key" do
+    before do
+      view_factory.register :my_new_view_class => mock_view
+    end
+
+    it "should respond to :my_new_view_class" do
+      view_factory.respond_to?(:my_new_view_class)
+    end
+
+    it "should respond to :my_new_view_class=" do
+      view_factory.respond_to?(:my_new_view_class=)
+    end
+
+    it "should generate a getter method" do
+      view_factory.my_new_view_class.should == mock_view
+    end
+
+    it "should be settable view a setter method" do
+      view_factory.my_new_view_class = "Some Obj"
+      view_factory.my_new_view_class.should == "Some Obj"
+    end
+  end
+
+  describe "array syntax access" do
+    before do
+      view_factory.register :my_new_view_class => mock_view
+    end
+
+    it "should be available through array syntax" do
+      view_factory[:my_new_view_class].should == mock_view
+    end
+
+    it "should be settable through array syntax" do
+      view_factory[:my_new_view_class] = "My New View Class"
+      view_factory[:my_new_view_class].should == "My New View Class"
+    end
+  end
+
+  describe "registering default views" do
+    before do
+      ActiveAdmin::AbstractViewFactory.register :my_default_view_class => mock_view
+    end
+    it "should generate a getter method" do
+      view_factory.my_default_view_class.should == mock_view
+    end
+    it "should be settable view a setter method and not change default" do
+      view_factory.my_default_view_class = "Some Obj"
+      view_factory.my_default_view_class.should == "Some Obj"
+      view_factory.default_for(:my_default_view_class).should == mock_view
+    end
+  end
+
+  describe "subclassing the ViewFactory" do
+    let(:subclass) do
+      ActiveAdmin::AbstractViewFactory.register :my_subclassed_view => "From Parent"
+      Class.new(ActiveAdmin::AbstractViewFactory) do
+        def my_subclassed_view
+          "From Subclass"
+        end
+      end
+    end
+
+    it "should use the subclass implementation" do
+      factory = subclass.new
+      factory.my_subclassed_view.should == "From Subclass"
+    end
+  end
+
+
+end

data/spec/unit/action_builder_spec.rb

@@ -1,6 +1,6 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper') 
 
-describe ActiveAdmin::ActionBuilder do
+describe 'defining new actions from registration blocks' do
 
   let(:controller){ Admin::PostsController }
 
@@ -16,8 +16,10 @@ describe ActiveAdmin::ActionBuilder do
     
     context "with a block" do
       let(:action!) do
-        controller.member_action :comment do
-          # Do nothing
+        ActiveAdmin.register Post do
+          member_action :comment do
+            # Do nothing
+          end
         end
       end
         
@@ -33,7 +35,11 @@ describe ActiveAdmin::ActionBuilder do
     end
 
     context "without a block" do
-      let(:action!){ controller.member_action :comment }
+      let(:action!) do 
+        ActiveAdmin.register Post do
+          member_action :comment
+        end
+      end
       it "should still generate a new empty action" do
         controller.public_instance_methods.collect(&:to_s).should include("comment")
       end
@@ -51,8 +57,10 @@ describe ActiveAdmin::ActionBuilder do
 
     context "with a block" do
       let(:action!) do
-        controller.collection_action :comments do
-          # Do nothing
+        ActiveAdmin.register Post do
+          collection_action :comments do
+            # Do nothing
+          end
         end
       end
       it "should create a new public instance method" do
@@ -66,7 +74,11 @@ describe ActiveAdmin::ActionBuilder do
       end
     end
     context "without a block" do
-      let(:action!){ controller.collection_action :comments }
+      let(:action!) do 
+        ActiveAdmin.register Post do
+          collection_action :comments
+        end
+      end
       it "should still generate a new empty action" do
         controller.public_instance_methods.collect(&:to_s).should include("comments")
       end