aa-rails4 0.6.0

data/spec/support/jslint.yml

@@ -0,0 +1,80 @@
+# ------------ rake task options ------------
+
+# JS files to check by default, if no parameters are passed to rake jslint
+# (you may want to limit this only to your own scripts and exclude any external scripts and frameworks)
+
+# this can be overridden by adding 'paths' and 'exclude_paths' parameter to rake command:
+#   rake jslint paths=path1,path2,... exclude_paths=library1,library2,...
+
+paths:
+  - app/assets/javascripts/active_admin/**/*.js
+
+exclude_paths:
+  - app/assets/javascripts/active_admin/vendor.js
+
+# ------------ jslint options ------------
+# see http://www.jslint.com/lint.html#options for more detailed explanations
+
+# "enforce" type options (true means potentially more warnings)
+
+adsafe:   false     # true if ADsafe rules should be enforced. See http://www.ADsafe.org/
+bitwise:  true      # true if bitwise operators should not be allowed
+newcap:   true      # true if Initial Caps must be used with constructor functions
+eqeqeq:   false     # true if === should be required (for ALL equality comparisons)
+immed:    false     # true if immediate function invocations must be wrapped in parens
+nomen:    false     # true if initial or trailing underscore in identifiers should be forbidden
+onevar:   false     # true if only one var statement per function should be allowed
+plusplus: false     # true if ++ and -- should not be allowed
+regexp:   false     # true if . and [^...] should not be allowed in RegExp literals
+safe:     false     # true if the safe subset rules are enforced (used by ADsafe)
+strict:   false     # true if the ES5 "use strict"; pragma is required
+undef:    false     # true if variables must be declared before used
+white:    false     # true if strict whitespace rules apply (see also 'indent' option)
+
+# "allow" type options (false means potentially more warnings)
+
+cap:      false     # true if upper case HTML should be allowed
+css:      true      # true if CSS workarounds should be tolerated
+debug:    false     # true if debugger statements should be allowed (set to false before going into production)
+es5:      false     # true if ECMAScript 5 syntax should be allowed
+evil:     false     # true if eval should be allowed
+forin:    true      # true if unfiltered 'for in' statements should be allowed
+fragment: true      # true if HTML fragments should be allowed
+laxbreak: false     # true if statement breaks should not be checked
+on:       false     # true if HTML event handlers (e.g. onclick="...") should be allowed
+sub:      false     # true if subscript notation may be used for expressions better expressed in dot notation
+
+# other options
+
+maxlen:   300       # Maximum line length
+indent:   2         # Number of spaces that should be used for indentation - used only if 'white' option is set
+maxerr:   50        # The maximum number of warnings reported (per file)
+passfail: false     # true if the scan should stop on first error (per file)
+# following are relevant only if undef = true
+predef:   ''        # Names of predefined global variables - comma-separated string or a YAML array
+browser:  true      # true if the standard browser globals should be predefined
+rhino:    false     # true if the Rhino environment globals should be predefined
+windows:  false     # true if Windows-specific globals should be predefined
+widget:   false     # true if the Yahoo Widgets globals should be predefined
+devel:    true      # true if functions like alert, confirm, console, prompt etc. are predefined
+
+
+# ------------ jslint_on_rails custom lint options (switch to true to disable some annoying warnings) ------------
+
+# ignores "missing semicolon" warning at the end of a function; this lets you write one-liners
+# like: x.map(function(i) { return i + 1 }); without having to put a second semicolon inside the function
+lastsemic: false
+
+# allows you to use the 'new' expression as a statement (without assignment)
+# so you can call e.g. new Ajax.Request(...), new Effect.Highlight(...) without assigning to a dummy variable
+newstat: false
+
+# ignores the "Expected an assignment or function call and instead saw an expression" warning,
+# if the expression contains a proper statement and makes sense; this lets you write things like:
+#    element && element.show();
+#    valid || other || lastChance || alert('OMG!');
+#    selected ? show() : hide();
+# although these will still cause a warning:
+#    element && link;
+#    selected ? 5 : 10;
+statinexp: false

data/spec/support/rails_template.rb

@@ -0,0 +1,104 @@
+# Rails template to build the sample app for specs
+
+run "rm Gemfile"
+run "rm -r test"
+run "rm -r spec"
+
+# 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 :model, "post title:string body:text published_at:datetime author_id:integer category_id:integer starred:boolean"
+inject_into_file 'app/models/post.rb', %q{
+  belongs_to :category
+  belongs_to :author, :class_name => 'User'
+  accepts_nested_attributes_for :author
+  attr_accessible :author unless Rails::VERSION::MAJOR > 3 && !defined? ProtectedAttributes
+}, :after => 'class Post < ActiveRecord::Base'
+copy_file File.expand_path('../templates/post_decorator.rb', __FILE__), "app/models/post_decorator.rb"
+
+generate :model, "user type:string first_name:string last_name:string username:string age:integer"
+inject_into_file 'app/models/user.rb', %q{
+  has_many :posts, :foreign_key => 'author_id'
+  def display_name
+    "#{first_name} #{last_name}"
+  end
+}, :after => 'class User < ActiveRecord::Base'
+
+generate :model, 'publisher --migration=false --parent=User'
+generate :model, 'category name:string description:text'
+inject_into_file 'app/models/category.rb', %q{
+  has_many :posts
+  accepts_nested_attributes_for :posts
+}, :after => 'class Category < ActiveRecord::Base'
+generate :model, 'store name:string'
+
+# Generate a model with string ids
+generate :model, "tag name:string"
+gsub_file(Dir['db/migrate/*_create_tags.rb'][0], /\:tags\sdo\s.*/, ":tags, :id => false, :primary_key => :id do |t|\n\t\t\tt.string :id\n")
+inject_into_file 'app/models/tag.rb', %q{
+  self.primary_key = :id
+  before_create :set_id
+
+  private
+  def set_id
+    self.id = 8.times.inject("") { |s,e| s << (i = Kernel.rand(62); i += ((i < 10) ? 48 : ((i < 36) ? 55 : 61 ))).chr }
+  end
+}, :after => 'class Tag < ActiveRecord::Base'
+
+# Configure default_url_options in test environment
+inject_into_file "config/environments/test.rb", "  config.action_mailer.default_url_options = { :host => 'example.com' }\n", :after => "config.cache_classes = true\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__)"
+
+# Add some translations
+append_file "config/locales/en.yml", File.read(File.expand_path('../templates/en.yml', __FILE__))
+
+# Add predefined admin resources
+directory File.expand_path('../templates/admin', __FILE__), "app/admin"
+
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+
+generate :'active_admin:install'
+
+rake "db:migrate"
+rake "db:test:prepare"
+run "/usr/bin/env RAILS_ENV=cucumber rake db:migrate"
+
+# Setup parallel_tests
+def setup_parallel_tests_database(after, force_insert_same_content = false)
+  inject_into_file 'config/database.yml', "<%= ENV['TEST_ENV_NUMBER'] %>", :after => after, :force => force_insert_same_content
+end
+
+setup_parallel_tests_database "test.sqlite3"
+setup_parallel_tests_database "cucumber.sqlite3", true
+
+# Note: this is hack!
+# Somehow, calling parallel_tests tasks from Rails generator using Thor does not work ...
+# RAILS_ENV variable never makes it to parallel_tests tasks.
+# We need to call these tasks in the after set up hook in order to creates cucumber DBs + run migrations on test & cucumber DBs
+create_file 'lib/tasks/parallel.rake', %q{
+namespace :parallel do
+  def run_in_parallel(cmd, options)
+    count = "-n #{options[:count]}" if options[:count]
+    executable = 'parallel_test'
+    command = "#{executable} --exec '#{cmd}' #{count} #{'--non-parallel' if options[:non_parallel]}"
+    abort unless system(command)
+  end
+
+  desc "create cucumber databases via db:create --> parallel:create_cucumber_db[num_cpus]"
+  task :create_cucumber_db, :count do |t, args|
+    run_in_parallel("rake db:create RAILS_ENV=cucumber", args)
+  end
+
+  desc "load dumped schema for cucumber databases"
+  task :load_schema_cucumber_db, :count do |t,args|
+    run_in_parallel("rake db:schema:load RAILS_ENV=cucumber", args)
+  end
+end
+}

data/spec/support/rails_template_with_data.rb

@@ -0,0 +1,59 @@
+# 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
+
+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|
+    first, last = name.split(" ")
+    User.create!  :first_name => first,
+                  :last_name => last,
+                  :username => [first,last].join('-').downcase,
+                  :age => rand(80)
+  end
+
+  categories = ["Rock", "Pop Rock", "Alt-Country", "Blues", "Dub-Step"].collect do |name|
+    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,
+                :starred => true
+  end
+EOF
+
+rake 'db:seed'

data/spec/support/templates/admin/stores.rb

@@ -0,0 +1,11 @@
+ActiveAdmin.register Store do
+
+  controller do
+    def permitted_params
+      params.permit store: [:name]
+    end if Rails::VERSION::MAJOR == 4
+  end
+
+  index :pagination_total => false
+
+end

data/spec/support/templates/cucumber.rb

@@ -0,0 +1,24 @@
+require File.expand_path('config/environments/test', Rails.root)
+
+# 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/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/support/templates/en.yml

@@ -0,0 +1,8 @@
+# Sample translations used to test ActiveAdmin's I18n integration.
+  activerecord:
+    models:
+      store:
+        one: Bookstore
+        other: Bookstores
+  active_admin:
+    download: "Download this:"

data/spec/support/templates/post_decorator.rb

@@ -0,0 +1,53 @@
+# Minimal example of a decorator
+require_dependency 'post'
+
+# DelegateClass(Post) does not delegate attribute methods
+# in some environments:
+# http://travis-ci.org/#!/gregbell/active_admin/jobs/2021466
+#
+class PostDecorator < SimpleDelegator
+  delegate :id, :to => :__getobj__
+
+  def self.decorate(object)
+    new(object)
+  end
+
+  def self.decorate_collection(collection)
+    DecoratedEnumerableProxy.new(collection)
+  end
+
+  def self.model_name
+    ActiveModel::Name.new Post
+  end
+
+  def decorator_method
+    'A method only available on the decorator'
+  end
+
+  # Minimal example of decorating a collection.
+  # A full example can be found in the draper project:
+  # https://github.com/jcasimir/draper/blob/master/lib/draper/decorated_enumerable_proxy.rb
+  class DecoratedEnumerableProxy < DelegateClass(ActiveRecord::Relation)
+    include Enumerable
+
+    delegate :as_json, :collect, :map, :each, :[], :all?, :include?, :first, :last, :shift, :to => :decorated_collection
+
+    def klass
+      PostDecorator
+    end
+
+    def wrapped_collection
+      __getobj__
+    end
+
+    def decorated_collection
+      @decorated_collection ||= wrapped_collection.collect { |member| klass.decorate(member) }
+    end
+    alias_method :to_ary, :decorated_collection
+
+    def each(&blk)
+      to_ary.each(&blk)
+    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

@@ -0,0 +1,126 @@
+require 'spec_helper'
+
+describe 'defining new actions from registration blocks' do
+
+  let(:controller){ Admin::PostsController }
+
+  describe "generating a new member action" do
+    before do
+      action!
+      reload_routes!
+    end
+
+    after(:each) do
+      controller.clear_member_actions!
+    end
+
+    context "with a block" do
+      let(:action!) do
+        ActiveAdmin.register Post do
+          member_action :comment do
+            # Do nothing
+          end
+        end
+      end
+
+      it "should create a new public instance method" do
+        controller.public_instance_methods.collect(&:to_s).should include("comment")
+      end
+      it "should add itself to the member actions config" do
+        controller.active_admin_config.member_actions.size.should == 1
+      end
+      it "should create a new named route" do
+        Rails.application.routes.url_helpers.methods.collect(&:to_s).should include("comment_admin_post_path")
+      end
+    end
+
+    context "without a block" do
+      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
+    end
+
+    context "with :title" do
+      let(:action!) do
+        ActiveAdmin.register Post do
+          member_action :comment, :title => "My Awesome Comment"
+        end
+      end
+
+      subject { find_before_filter controller, :comment }
+
+      it { should set_page_title_to "My Awesome Comment" }
+    end
+  end
+
+  describe "generate a new collection action" do
+    before do
+      action!
+      reload_routes!
+    end
+    after(:each) do
+      controller.clear_collection_actions!
+    end
+
+    context "with a block" do
+      let(:action!) do
+        ActiveAdmin.register Post do
+          collection_action :comments do
+            # Do nothing
+          end
+        end
+      end
+      it "should create a new public instance method" do
+        controller.public_instance_methods.collect(&:to_s).should include("comments")
+      end
+      it "should add itself to the member actions config" do
+        controller.active_admin_config.collection_actions.size.should == 1
+      end
+      it "should create a new named route" do
+        Rails.application.routes.url_helpers.methods.collect(&:to_s).should include("comments_admin_posts_path")
+      end
+    end
+    context "without a block" do
+      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
+    end
+    context "with :title" do
+      let(:action!) do
+        ActiveAdmin.register Post do
+          collection_action :comments, :title => "My Awesome Comments"
+        end
+      end
+
+      subject { find_before_filter controller, :comments }
+
+      it { should set_page_title_to "My Awesome Comments" }
+    end
+  end
+
+  def find_before_filter(controller, action)
+    controller._process_action_callbacks.detect { |f| f.kind == :before && f.options[:only] == [action] }
+  end
+
+  RSpec::Matchers.define :set_page_title_to do |expected|
+    match do |filter|
+      filter.raw_filter.call
+      @actual = filter.klass.instance_variable_get(:@page_title)
+      @actual == expected
+    end
+
+    failure_message_for_should do |filter|
+      message = "expected before_filter to set the @page_title to '#{expected}', but was '#{@actual}'"
+    end
+  end
+end